Changeset 6244


Ignore:
Timestamp:
Feb 25, 2013, 1:10:42 PM (9 years ago)
Author:
Nicklas Nordborg
Message:

References #1729 and #1730. Implement lazy loading of 'Change history' as a tab activation event instead of a page reload. The change history list page now uses separate script file.

Location:
trunk/www
Files:
1 added
23 edited

Legend:

Unmodified
Added
Removed
  • trunk/www/admin/annotationtypes/view_annotationtype.jsp

    r6136 r6244  
    142142      Main.openPopup('index.jsp?ID=<%=ID%>&cmd='+cmd+'&item_id=<%=itemId%>', 'RunPlugin'+cmd, 750, 500);
    143143    }
    144     function switchTab(tabControlId, tabId)
    145     {
    146       if (TabControl.isActive(tabControlId, tabId)) return;
    147       if (tabId == 'history' && tabId != '<%=tab%>')
    148       {
    149         location.href = 'index.jsp?ID=<%=ID%>&cmd=ViewItem&item_id=<%=itemId%>&tab='+tabId;
    150       }
    151       else
    152       {
    153         TabControl.setActiveTab(tabControlId, tabId);
    154       }
    155     }
    156144    </script>
    157145  </base:head>
     
    165153      id="main"
    166154      subclass="content mastertabcontrol"
    167       active="<%=tab%>" switch="switchTab" remember="false">
     155      active="<%=tab%>">
    168156    <t:tab id="properties" title="Properties">
    169157      <div>
     
    416404      <t:tab id="history" title="Change history"
    417405        tooltip="Displays a log of all modifications made to this item"
    418         visible="<%=ChangeHistoryUtil.showChangeHistoryTab(sc)%>">
    419         <%
    420         if ("history".equals(tab))
    421         {
    422           %>
    423           <jsp:include page="../../common/history/frameset.jsp">
    424             <jsp:param name="source_type" value="<%=itemType.name()%>" />
    425             <jsp:param name="source_id" value="<%=itemId%>" />
    426             <jsp:param name="ID" value="<%=ID%>" />
    427           </jsp:include>
    428           <%
    429         }
    430         %>
     406        visible="<%=ChangeHistoryUtil.showChangeHistoryTab(sc)%>"
     407        activate="History.loadOnce()">
     408        <jsp:include page="../../common/history/frameset.jsp">
     409          <jsp:param name="item_type" value="<%=itemType.name()%>" />
     410          <jsp:param name="item_id" value="<%=itemId%>" />
     411          <jsp:param name="ID" value="<%=ID%>" />
     412        </jsp:include>
    431413      </t:tab>
    432414      </t:tabcontrol>
  • trunk/www/admin/pluginconfigurations/view_configuration.jsp

    r6136 r6244  
    172172      Main.openPopup('../../common/plugin/index.jsp?ID=<%=ID%>&cmd=ConfigurePlugin&pluginconfiguration_id=<%=itemId%>', 'ConfigurePlugin', 750, 500);
    173173    }
    174     function switchTab(tabControlId, tabId)
    175     {
    176       if (TabControl.isActive(tabControlId, tabId)) return;
    177       if (tabId == 'history' && tabId != '<%=tab%>')
    178       {
    179         location.href = 'index.jsp?ID=<%=ID%>&cmd=ViewItem&item_id=<%=itemId%>&tab='+tabId;
    180       }
    181       else
    182       {
    183         TabControl.setActiveTab(tabControlId, tabId);
    184       }
    185     }
    186174    </script>
    187175  </base:head>
     
    195183      id="main"
    196184      subclass="content mastertabcontrol"
    197       active="<%=tab%>" switch="switchTab" remember="false">
     185      active="<%=tab%>">
    198186    <t:tab id="properties" title="Properties">
    199187      <div>
     
    464452      <t:tab id="history" title="Change history"
    465453        tooltip="Displays a log of all modifications made to this item"
    466         visible="<%=ChangeHistoryUtil.showChangeHistoryTab(sc)%>">
    467         <%
    468         if ("history".equals(tab))
    469         {
    470           %>
    471           <jsp:include page="../../common/history/frameset.jsp">
    472             <jsp:param name="source_type" value="<%=itemType.name()%>" />
    473             <jsp:param name="source_id" value="<%=itemId%>" />
    474             <jsp:param name="ID" value="<%=ID%>" />
    475           </jsp:include>
    476           <%
    477         }
    478         %>
     454        visible="<%=ChangeHistoryUtil.showChangeHistoryTab(sc)%>"
     455        activate="History.loadOnce()">
     456        <jsp:include page="../../common/history/frameset.jsp">
     457          <jsp:param name="item_type" value="<%=itemType.name()%>" />
     458          <jsp:param name="item_id" value="<%=itemId%>" />
     459          <jsp:param name="ID" value="<%=ID%>" />
     460        </jsp:include>
    479461      </t:tab>
    480462      </t:tabcontrol>
  • trunk/www/admin/protocols/view_protocol.jsp

    r6147 r6244  
    141141      Main.openPopup('index.jsp?ID=<%=ID%>&cmd='+cmd+'&item_id=<%=itemId%>', 'RunPlugin'+cmd, 750, 500);
    142142    }
    143     function switchTab(tabControlId, tabId)
    144     {
    145       if (TabControl.isActive(tabControlId, tabId)) return;
    146       if (tabId == 'history' && tabId != '<%=tab%>')
    147       {
    148         location.href = 'index.jsp?ID=<%=ID%>&cmd=ViewItem&item_id=<%=itemId%>&tab='+tabId;
    149       }
    150       else
    151       {
    152         TabControl.setActiveTab(tabControlId, tabId);
    153       }
    154     }
    155143    </script>
    156144  </base:head>
     
    164152      id="main"
    165153      subclass="content mastertabcontrol"
    166       active="<%=tab%>" switch="switchTab" remember="false">
     154      active="<%=tab%>">
    167155    <t:tab id="properties" title="Properties">
    168156      <div>
     
    390378    <t:tab id="history" title="Change history"
    391379      tooltip="Displays a log of all modifications made to this item"
    392       visible="<%=ChangeHistoryUtil.showChangeHistoryTab(sc)%>">
    393       <%
    394       if ("history".equals(tab))
    395       {
    396         %>
    397         <jsp:include page="../../common/history/frameset.jsp">
    398           <jsp:param name="source_type" value="<%=itemType.name()%>" />
    399           <jsp:param name="source_id" value="<%=itemId%>" />
    400           <jsp:param name="ID" value="<%=ID%>" />
    401         </jsp:include>
    402         <%
    403       }
    404       %>
     380      visible="<%=ChangeHistoryUtil.showChangeHistoryTab(sc)%>"
     381      activate="History.loadOnce()">
     382      <jsp:include page="../../common/history/frameset.jsp">
     383        <jsp:param name="item_type" value="<%=itemType.name()%>" />
     384        <jsp:param name="item_id" value="<%=itemId%>" />
     385        <jsp:param name="ID" value="<%=ID%>" />
     386      </jsp:include>
    405387    </t:tab>
    406388    </t:tabcontrol>
  • trunk/www/admin/reporterclonetemplates/view_template.jsp

    r6147 r6244  
    138138      Main.openPopup('index.jsp?ID=<%=ID%>&cmd='+cmd+'&item_id=<%=itemId%>', 'RunPlugin'+cmd, 750, 500);
    139139    }
    140     function switchTab(tabControlId, tabId)
    141     {
    142       if (TabControl.isActive(tabControlId, tabId)) return;
    143       if (tabId == 'history' && tabId != '<%=tab%>')
    144       {
    145         location.href = 'index.jsp?ID=<%=ID%>&cmd=ViewItem&item_id=<%=itemId%>&tab='+tabId;
    146       }
    147       else
    148       {
    149         TabControl.setActiveTab(tabControlId, tabId);
    150       }
    151     }
    152140    </script>
    153141  </base:head>
     
    161149      id="main"
    162150      subclass="content mastertabcontrol"
    163       active="<%=tab%>" switch="switchTab" remember="false">
     151      active="<%=tab%>">
    164152    <t:tab id="properties" title="Properties">
    165153      <div>
     
    372360    <t:tab id="history" title="Change history"
    373361      tooltip="Displays a log of all modifications made to this item"
    374       visible="<%=ChangeHistoryUtil.showChangeHistoryTab(sc)%>">
    375       <%
    376       if ("history".equals(tab))
    377       {
    378         %>
    379         <jsp:include page="../../common/history/frameset.jsp">
    380           <jsp:param name="source_type" value="<%=itemType.name()%>" />
    381           <jsp:param name="source_id" value="<%=itemId%>" />
    382           <jsp:param name="ID" value="<%=ID%>" />
    383         </jsp:include>
    384         <%
    385       }
    386       %>
     362      visible="<%=ChangeHistoryUtil.showChangeHistoryTab(sc)%>"
     363      activate="History.loadOnce()">
     364      <jsp:include page="../../common/history/frameset.jsp">
     365        <jsp:param name="item_type" value="<%=itemType.name()%>" />
     366        <jsp:param name="item_id" value="<%=itemId%>" />
     367        <jsp:param name="ID" value="<%=ID%>" />
     368      </jsp:include>
    387369    </t:tab>
    388370    </t:tabcontrol>
  • trunk/www/biomaterials/bioplates/view_bioplate.jsp

    r6242 r6244  
    159159    function switchTab(tabControlId, tabId)
    160160    {
    161       if ((tabId == 'history') && tabId != '<%=tab%>')
    162       {
    163         location.href = 'index.jsp?ID=<%=ID%>&cmd=ViewItem&item_id=<%=itemId%>&tab='+tabId;
    164       }
    165       else if (tabId == 'wells')
     161      if (tabId == 'wells')
    166162      {
    167163        viewWells();
     
    592588      <t:tab id="history" title="Change history"
    593589        tooltip="Displays a log of all modifications made to this item"
    594         visible="<%=ChangeHistoryUtil.showChangeHistoryTab(sc)%>">
    595         <%
    596         if ("history".equals(tab))
    597         {
    598           %>
    599           <jsp:include page="../../common/history/frameset.jsp">
    600             <jsp:param name="source_type" value="<%=itemType.name()%>" />
    601             <jsp:param name="source_id" value="<%=itemId%>" />
    602             <jsp:param name="ID" value="<%=ID%>" />
    603           </jsp:include>
    604           <%
    605         }
    606         %>
     590        visible="<%=ChangeHistoryUtil.showChangeHistoryTab(sc)%>"
     591        activate="History.loadOnce()">
     592        <jsp:include page="../../common/history/frameset.jsp">
     593          <jsp:param name="item_type" value="<%=itemType.name()%>" />
     594          <jsp:param name="item_id" value="<%=itemId%>" />
     595          <jsp:param name="ID" value="<%=ID%>" />
     596        </jsp:include>
    607597      </t:tab>
    608598    </t:tabcontrol>
  • trunk/www/biomaterials/biosources/view_biosource.jsp

    r6242 r6244  
    105105  ExtensionsInvoker invoker = ToolbarUtil.useExtensions(jspContext);
    106106  %>
    107   <base:page title="<%=title%>">
     107  <base:page title="<%=title%>" id="view-page">
    108108  <base:head scripts="table.js,tabcontrol.js" styles="table.css,toolbar.css,headertabcontrol.css,path.css">
    109109    <ext:scripts context="<%=jspContext%>" />
     
    149149    {
    150150      Main.viewOrEditItem('<%=ID%>', 'SAMPLE', 0, true, '&biosource_id=<%=itemId%>');
    151     }
    152     function switchTab(tabControlId, tabId)
    153     {
    154       if (TabControl.isActive(tabControlId, tabId)) return;
    155       if ((tabId == 'history') && tabId != '<%=tab%>')
    156       {
    157         location.href = 'index.jsp?ID=<%=ID%>&cmd=ViewItem&item_id=<%=itemId%>&tab='+tabId;
    158       }
    159       else
    160       {
    161         TabControl.setActiveTab(tabControlId, tabId);
    162       }
    163151    }
    164152    </script>
     
    169157      title="<%=HTML.encodeTags(bioSource.getName())%>" /></p:path>
    170158   
    171     <t:tabcontrol id="main" subclass="content mastertabcontrol" active="<%=tab%>" switch="switchTab" remember="false">
     159    <t:tabcontrol id="main" subclass="content mastertabcontrol" active="<%=tab%>">
    172160    <t:tab id="properties" title="Properties">
    173161
     
    406394      <t:tab id="history" title="Change history"
    407395        tooltip="Displays a log of all modifications made to this item"
    408         visible="<%=ChangeHistoryUtil.showChangeHistoryTab(sc)%>">
    409         <%
    410         if ("history".equals(tab))
    411         {
    412           %>
    413           <jsp:include page="../../common/history/frameset.jsp">
    414             <jsp:param name="source_type" value="<%=itemType.name()%>" />
    415             <jsp:param name="source_id" value="<%=itemId%>" />
    416             <jsp:param name="ID" value="<%=ID%>" />
    417           </jsp:include>
    418           <%
    419         }
    420         %>
     396        visible="<%=ChangeHistoryUtil.showChangeHistoryTab(sc)%>"
     397        activate="History.loadOnce()">
     398        <jsp:include page="../../common/history/frameset.jsp">
     399          <jsp:param name="item_type" value="<%=itemType.name()%>" />
     400          <jsp:param name="item_id" value="<%=itemId%>" />
     401          <jsp:param name="ID" value="<%=ID%>" />
     402        </jsp:include>
    421403      </t:tab>     
    422404      </t:tabcontrol>
  • trunk/www/biomaterials/extracts/view_extract.jsp

    r6242 r6244  
    173173    {
    174174      if (TabControl.isActive(tabControlId, tabId)) return;
    175       if ((tabId == 'history') && tabId != '<%=tab%>')
    176       {
    177         location.href = 'index.jsp?ID=<%=ID%>&cmd=ViewItem&item_id=<%=itemId%>&tab='+tabId;
    178       }
    179       else if (tabId == 'events')
     175      if (tabId == 'events')
    180176      {
    181177        viewEvents();
     
    635631      <t:tab id="history" title="Change history"
    636632        tooltip="Displays a log of all modifications made to this item"
    637         visible="<%=ChangeHistoryUtil.showChangeHistoryTab(sc)%>">
    638         <%
    639         if ("history".equals(tab))
    640         {
    641           %>
    642           <jsp:include page="../../common/history/frameset.jsp">
    643             <jsp:param name="source_type" value="<%=itemType.name()%>" />
    644             <jsp:param name="source_id" value="<%=itemId%>" />
    645             <jsp:param name="ID" value="<%=ID%>" />
    646           </jsp:include>
    647           <%
    648         }
    649         %>
     633        visible="<%=ChangeHistoryUtil.showChangeHistoryTab(sc)%>"
     634        activate="History.loadOnce()">
     635        <jsp:include page="../../common/history/frameset.jsp">
     636          <jsp:param name="item_type" value="<%=itemType.name()%>" />
     637          <jsp:param name="item_id" value="<%=itemId%>" />
     638          <jsp:param name="ID" value="<%=ID%>" />
     639        </jsp:include>
    650640      </t:tab>     
    651641      </t:tabcontrol>
  • trunk/www/biomaterials/samples/view_sample.jsp

    r6242 r6244  
    173173    {
    174174      if (TabControl.isActive(tabControlId, tabId)) return;
    175       if ((tabId == 'history') && tabId != '<%=tab%>')
    176       {
    177         location.href = 'index.jsp?ID=<%=ID%>&cmd=ViewItem&item_id=<%=itemId%>&tab='+tabId;
    178       }
    179       else if (tabId == 'events')
     175      if (tabId == 'events')
    180176      {
    181177        viewEvents();
     
    608604      <t:tab id="history" title="Change history"
    609605        tooltip="Displays a log of all modifications made to this item"
    610         visible="<%=ChangeHistoryUtil.showChangeHistoryTab(sc)%>">
    611         <%
    612         if ("history".equals(tab))
    613         {
    614           %>
    615           <jsp:include page="../../common/history/frameset.jsp">
    616             <jsp:param name="source_type" value="<%=itemType.name()%>" />
    617             <jsp:param name="source_id" value="<%=itemId%>" />
    618             <jsp:param name="ID" value="<%=ID%>" />
    619           </jsp:include>
    620           <%
    621         }
    622         %>
     606        visible="<%=ChangeHistoryUtil.showChangeHistoryTab(sc)%>"
     607        activate="History.loadOnce()">
     608        <jsp:include page="../../common/history/frameset.jsp">
     609          <jsp:param name="item_type" value="<%=itemType.name()%>" />
     610          <jsp:param name="item_id" value="<%=itemId%>" />
     611          <jsp:param name="ID" value="<%=ID%>" />
     612        </jsp:include>
    623613      </t:tab>     
    624614      </t:tabcontrol>
  • trunk/www/common/history/frameset.jsp

    r6139 r6244  
    3636final String ID = sc.getId();
    3737final String root = request.getContextPath();
    38 Item sourceType = Item.valueOf(request.getParameter("source_type"));
    39 int sourceId = Values.getInt(request.getParameter("source_id"));
     38Item sourceType = Item.valueOf(request.getParameter("item_type"));
     39int sourceId = Values.getInt(request.getParameter("item_id"));
    4040%>
    4141  <base:page type="include">
    4242  <base:body>
     43    <div id="history-data" class="datacontainer"
     44      data-source-type="<%=sourceType.name() %>"
     45      data-source-id="<%=sourceId %>"
     46    ></div>
     47    <script type="text/javascript" charset="UTF-8" src="<%=root%>/common/history/history.js"></script>
    4348    <div class="absolutefull"><iframe name="history" id="idHistory"
    44         src="<%=root%>/common/history/index.jsp?ID=<%=ID%>&amp;cmd=List&amp;source_type=<%=sourceType.name()%>&amp;source_id=<%=sourceId%>"
    45         class="absolutefull" style="width: 100%; height: 100%;"></iframe></div>
     49      src="<%=root%>/common/history/wait.jsp"
     50      class="absolutefull" style="width: 100%; height: 100%;"></iframe></div>
    4651  </base:body>
    4752  </base:page>
  • trunk/www/common/history/history.js

    r6181 r6244  
    2626{
    2727  var history = {};
     28  var loaded = false;
    2829 
    2930  /**
     
    3233  history.initPage = function()
    3334  {
    34     // Buttons
    35     Buttons.addClickHandler('close', App.closeWindow);
     35    var pageId = Doc.getPageId();
     36    if (pageId == 'view-history')
     37    {
     38      // Buttons
     39      Buttons.addClickHandler('close', App.closeWindow);
     40    }
     41    else if (pageId == 'list-history')
     42    {
     43      var attributes = {'item-type': 'CHANGEHISTORY'};
     44      var tableAttributes = {'table-id': 'history'};
     45      Buttons.addClickHandler('btnColumns', Buttons.configureColumns, tableAttributes);
     46      Buttons.addClickHandler('btnExport', Buttons.runListPlugin, tableAttributes);
     47
     48    }
     49    else
     50    {
     51      // Iframe included on single-item view page of other items
     52    }
    3653  }
    3754 
     55  history.loadOnce = function()
     56  {
     57    if (loaded) return;
     58    var url = App.getRoot() + 'common/history/index.jsp?ID='+App.getSessionId();
     59    url += '&cmd=List';
     60    url += '&source_type='+Data.get('history-data', 'source-type');
     61    url += '&source_id='+Data.get('history-data', 'source-id');
     62    frames['history'].location.href = url;
     63    loaded = true;
     64  }
     65
    3866  return history;
    3967}();
  • trunk/www/common/history/index.jsp

    r6192 r6244  
    7474<%
    7575final int sourceId = Values.getInt(request.getParameter("source_id"));
    76 final Item sourceType = Item.valueOf(request.getParameter("source_type"));
     76final Item sourceType = request.getParameter("source_type") != null ?
     77    Item.valueOf(request.getParameter("source_type")) : null;
    7778
    7879final SessionControl sc = Base.getExistingSessionControl(pageContext, true);
     
    8586final String listPage = "list_history.jsp?ID="+ID
    8687  +"&source_id="+sourceId
    87   +"&source_type="+sourceType.name()
     88  +(sourceType == null ? "" : "&source_type="+sourceType.name())
    8889  +(mode == null ? "" : "&mode="+mode)
    8990  +(callback == null ? "" : "&callback="+callback)
  • trunk/www/common/history/list_history.jsp

    r6221 r6244  
    103103  int numListed = 0;
    104104  %>
    105   <base:page type="iframe">
    106   <base:head scripts="table.js" styles="table.css,toolbar.css">
     105  <base:page type="iframe" id="list-history">
     106  <base:head scripts="table.js,~history.js" styles="table.css,toolbar.css">
     107    <ext:scripts context="<%=jspContext%>" />
     108    <ext:stylesheets context="<%=jspContext%>" />
    107109  </base:head>
    108   <base:body >
    109     <script>
    110     var submitPage = 'index.jsp';
    111     var formId = 'history';
    112     function viewItem(itemId)
    113     {
    114       Main.viewOrEditItem('<%=ID%>', '<%=itemType.name()%>', itemId, false, '&source_id=<%=sourceId%>&source_type=<%=sourceType.name()%>');
    115     }
    116     function itemOnClick(evt, itemId)
    117     {
    118       viewItem(itemId);
    119     }
    120 
    121     function configureColumns()
    122     {
    123       Table.configureColumns(formId);
    124     }
    125     function returnSelected()
    126     {
    127       Table.returnSelected(formId, <%=callback != null ? "window.opener."+callback : "null" %>);
    128       window.close();
    129     }
    130     function runPlugin(cmd)
    131     {
    132       Table.submitToPopup(formId, cmd, 750, 500);
    133     }
    134     </script>
     110  <base:body>
    135111    <tbl:table
    136112      id="history"
     
    269245          >
    270246          <tbl:button
     247            id="btnColumns"
    271248            image="columns.png"
    272             onclick="configureColumns()"
    273249            title="Columns&hellip;"
    274250            tooltip="Show, hide and re-order columns"
    275251          />
    276252          <tbl:button
     253            id="btnExport"
     254            data-plugin-type="EXPORT"
    277255            image="export.png"
    278             onclick="runPlugin('ExportItems')"
    279256            title="Export&hellip;"
    280257            tooltip="Export data"
     
    372349                  visible="<%=mode.hasIcons()%>"
    373350                  >&nbsp;</tbl:header>
    374                 <tbl:cell column="time"><div class="link"
    375                   onclick="itemOnClick(null, <%=itemId%>)"
    376                   title="<%=tooltip%>"><%=dateFormatter.format(item.getTime())%></div></tbl:cell>
     351                <tbl:cell column="time"><div
     352                    class="link table-item"
     353                    data-item-id="<%=itemId%>"
     354                    data-no-edit="1" tabindex="0"
     355                    title="<%=tooltip%>"><%=dateFormatter.format(item.getTime())%></div>
     356                  </tbl:cell>
    377357                <tbl:cell column="id"><%=item.getId()%></tbl:cell>
    378358                <tbl:cell column="changeType"><%=item.getChangeType()%></tbl:cell>
  • trunk/www/common/history/view_history.jsp

    r6181 r6244  
    7373  %>
    7474
    75   <base:page type="popup" title="<%=title%>">
     75  <base:page type="popup" title="<%=title%>" id="view-history">
    7676  <base:head scripts="~history.js" />
    7777  <base:body>
  • trunk/www/filemanager/files/view_file.jsp

    r6194 r6244  
    170170      Main.openPopup('index.jsp?ID=<%=ID%>&cmd=EditFileData&item_id=<%=itemId%>', 'EditFileData', 800, 600);
    171171    }
    172     function switchTab(tabControlId, tabId)
    173     {
    174       if (TabControl.isActive(tabControlId, tabId)) return;
    175       if (tabId == 'history' && tabId != '<%=tab%>')
    176       {
    177         location.href = 'index.jsp?ID=<%=ID%>&cmd=ViewItem&item_id=<%=itemId%>&tab='+tabId;
    178       }
    179       else
    180       {
    181         TabControl.setActiveTab(tabControlId, tabId);
    182       }
    183     }
    184172    </script>
    185173  </base:head>
     
    190178      subclass="content mastertabcontrol"
    191179      style="top: 0px;"
    192       id="main" active="<%=tab%>" switch="switchTab" remember="false">
     180      id="main" active="<%=tab%>">
    193181    <t:tab id="properties" title="Properties">
    194182      <div>
     
    476464    <t:tab id="history" title="Change history"
    477465      tooltip="Displays a log of all modifications made to this item"
    478       visible="<%=ChangeHistoryUtil.showChangeHistoryTab(sc)%>">
    479       <%
    480       if ("history".equals(tab))
    481       {
    482         %>
    483         <jsp:include page="../../common/history/frameset.jsp">
    484           <jsp:param name="source_type" value="<%=itemType.name()%>" />
    485           <jsp:param name="source_id" value="<%=itemId%>" />
    486           <jsp:param name="ID" value="<%=ID%>" />
    487         </jsp:include>
    488         <%
    489       }
    490       %>
     466      visible="<%=ChangeHistoryUtil.showChangeHistoryTab(sc)%>"
     467      activate="History.loadOnce()">
     468      <jsp:include page="../../common/history/frameset.jsp">
     469        <jsp:param name="item_type" value="<%=itemType.name()%>" />
     470        <jsp:param name="item_id" value="<%=itemId%>" />
     471        <jsp:param name="ID" value="<%=ID%>" />
     472      </jsp:include>
    491473    </t:tab>
    492474    </t:tabcontrol>
  • trunk/www/lims/arraybatches/view_batch.jsp

    r6242 r6244  
    156156      Main.openPopup('../arrayslides/index.jsp?ID=<%=ID%>&cmd=NewItems&arraybatch_id=<%=itemId%>', 'NewArraySlides', 750, 500);
    157157    }
    158     function switchTab(tabControlId, tabId)
    159     {
    160       if (TabControl.isActive(tabControlId, tabId)) return;
    161       if ((tabId == 'history') && tabId != '<%=tab%>')
    162       {
    163         location.href = 'index.jsp?ID=<%=ID%>&cmd=ViewItem&item_id=<%=itemId%>&tab='+tabId;
    164       }
    165       else
    166       {
    167         TabControl.setActiveTab(tabControlId, tabId);
    168       }
    169     }
    170158    </script>
    171159  </base:head>
     
    179167      id="main"
    180168      subclass="content mastertabcontrol"
    181       active="<%=tab%>" switch="switchTab" remember="false">
     169      active="<%=tab%>">
    182170    <t:tab id="properties" title="Properties">
    183171      <div>
     
    425413      <t:tab id="history" title="Change history"
    426414        tooltip="Displays a log of all modifications made to this item"
    427         visible="<%=ChangeHistoryUtil.showChangeHistoryTab(sc)%>">
    428         <%
    429         if ("history".equals(tab))
    430         {
    431           %>
    432           <jsp:include page="../../common/history/frameset.jsp">
    433             <jsp:param name="source_type" value="<%=itemType.name()%>" />
    434             <jsp:param name="source_id" value="<%=itemId%>" />
    435             <jsp:param name="ID" value="<%=ID%>" />
    436           </jsp:include>
    437           <%
    438         }
    439         %>
     415        visible="<%=ChangeHistoryUtil.showChangeHistoryTab(sc)%>"
     416        activate="History.loadOnce()">
     417        <jsp:include page="../../common/history/frameset.jsp">
     418          <jsp:param name="item_type" value="<%=itemType.name()%>" />
     419          <jsp:param name="item_id" value="<%=itemId%>" />
     420          <jsp:param name="ID" value="<%=ID%>" />
     421        </jsp:include>
    440422      </t:tab>
    441423    </t:tabcontrol>
  • trunk/www/lims/arraydesigns/view_design.jsp

    r6242 r6244  
    176176    {
    177177      if (TabControl.isActive(tabControlId, tabId)) return;
    178       if ((tabId == 'history') && tabId != '<%=tab%>')
    179       {
    180         location.href = 'index.jsp?ID=<%=ID%>&cmd=ViewItem&item_id=<%=itemId%>&tab='+tabId;
    181       }
    182       else if (tabId == 'features')
     178      if (tabId == 'features')
    183179      {
    184180        viewFeatures();
     
    579575    <t:tab id="history" title="Change history"
    580576      tooltip="Displays a log of all modifications made to this item"
    581       visible="<%=ChangeHistoryUtil.showChangeHistoryTab(sc)%>">
    582       <%
    583       if ("history".equals(tab))
    584       {
    585         %>
    586         <jsp:include page="../../common/history/frameset.jsp">
    587           <jsp:param name="source_type" value="<%=itemType.name()%>" />
    588           <jsp:param name="source_id" value="<%=itemId%>" />
    589           <jsp:param name="ID" value="<%=ID%>" />
    590         </jsp:include>
    591         <%
    592       }
    593       %>
     577      visible="<%=ChangeHistoryUtil.showChangeHistoryTab(sc)%>"
     578      activate="History.loadOnce()">
     579      <jsp:include page="../../common/history/frameset.jsp">
     580        <jsp:param name="item_type" value="<%=itemType.name()%>" />
     581        <jsp:param name="item_id" value="<%=itemId%>" />
     582        <jsp:param name="ID" value="<%=ID%>" />
     583      </jsp:include>
    594584    </t:tab>
    595585    </t:tabcontrol>
  • trunk/www/lims/arrayslides/view_slide.jsp

    r6242 r6244  
    146146      Main.openPopup('index.jsp?ID=<%=ID%>&cmd='+cmd+'&item_id=<%=itemId%>', 'RunPlugin'+cmd, 750, 500);
    147147    }
    148     function switchTab(tabControlId, tabId)
    149     {
    150       if (TabControl.isActive(tabControlId, tabId)) return;
    151       if ((tabId == 'history') && tabId != '<%=tab%>')
    152       {
    153         location.href = 'index.jsp?ID=<%=ID%>&cmd=ViewItem&item_id=<%=itemId%>&tab='+tabId;
    154       }
    155       else
    156       {
    157         TabControl.setActiveTab(tabControlId, tabId);
    158       }
    159     }
    160148    </script>
    161149  </base:head>
     
    169157      id="main"
    170158      subclass="content mastertabcontrol"
    171       active="<%=tab%>" switch="switchTab" remember="false">
     159      active="<%=tab%>">
    172160    <t:tab id="properties" title="Properties">
    173161      <div>
     
    344332      <t:tab id="history" title="Change history"
    345333        tooltip="Displays a log of all modifications made to this item"
    346         visible="<%=ChangeHistoryUtil.showChangeHistoryTab(sc)%>">
    347         <%
    348         if ("history".equals(tab))
    349         {
    350           %>
    351           <jsp:include page="../../common/history/frameset.jsp">
    352             <jsp:param name="source_type" value="<%=itemType.name()%>" />
    353             <jsp:param name="source_id" value="<%=itemId%>" />
    354             <jsp:param name="ID" value="<%=ID%>" />
    355           </jsp:include>
    356           <%
    357         }
    358         %>
     334        visible="<%=ChangeHistoryUtil.showChangeHistoryTab(sc)%>"
     335        activate="History.loadOnce()">
     336        <jsp:include page="../../common/history/frameset.jsp">
     337          <jsp:param name="item_type" value="<%=itemType.name()%>" />
     338          <jsp:param name="item_id" value="<%=itemId%>" />
     339          <jsp:param name="ID" value="<%=ID%>" />
     340        </jsp:include>
    359341      </t:tab>
    360342    </t:tabcontrol>
  • trunk/www/lims/plates/view_plate.jsp

    r6136 r6244  
    164164    function switchTab(tabControlId, tabId)
    165165    {
    166       if (tabId == 'history' && tabId != '<%=tab%>')
    167       {
    168         location.href = 'index.jsp?ID=<%=ID%>&cmd=ViewItem&item_id=<%=itemId%>&tab='+tabId;
    169       }
    170       else if (tabId == 'events')
     166      if (tabId == 'events')
    171167      {
    172168        viewEvents();
     
    507503      <t:tab id="history" title="Change history"
    508504        tooltip="Displays a log of all modifications made to this item"
    509         visible="<%=ChangeHistoryUtil.showChangeHistoryTab(sc)%>">
    510         <%
    511         if ("history".equals(tab))
    512         {
    513           %>
    514           <jsp:include page="../../common/history/frameset.jsp">
    515             <jsp:param name="source_type" value="<%=itemType.name()%>" />
    516             <jsp:param name="source_id" value="<%=itemId%>" />
    517             <jsp:param name="ID" value="<%=ID%>" />
    518           </jsp:include>
    519           <%
    520         }
    521         %>
     505        visible="<%=ChangeHistoryUtil.showChangeHistoryTab(sc)%>"
     506        activate="History.loadOnce()">
     507        <jsp:include page="../../common/history/frameset.jsp">
     508          <jsp:param name="item_type" value="<%=itemType.name()%>" />
     509          <jsp:param name="item_id" value="<%=itemId%>" />
     510          <jsp:param name="ID" value="<%=ID%>" />
     511        </jsp:include>
    522512      </t:tab>
    523513    </t:tabcontrol>
  • trunk/www/views/derivedbioassays/view_bioassay.jsp

    r6242 r6244  
    171171      Main.openPopup('index.jsp?ID=<%=ID%>&cmd=SetOwnerOfItem&item_id=<%=itemId%>', 'SetOwnerOfItem', 450, 300);
    172172    }
    173     function switchTab(tabControlId, tabId)
    174     {
    175       if (TabControl.isActive(tabControlId, tabId)) return;
    176       if ((tabId == 'history') && tabId != '<%=tab%>')
    177       {
    178         location.href = 'index.jsp?ID=<%=ID%>&cmd=ViewItem&item_id=<%=itemId%>&tab='+tabId;
    179       }
    180       else
    181       {
    182         TabControl.setActiveTab(tabControlId, tabId);
    183       }
    184     }
    185173    function runItemPlugin(cmd)
    186174    {
     
    206194      id="main"
    207195      subclass="content mastertabcontrol"
    208       active="<%=tab%>" switch="switchTab" remember="false">
     196      active="<%=tab%>">
    209197      <t:tab id="properties" title="Properties">
    210198      <div>
     
    788776      <t:tab id="history" title="Change history"
    789777        tooltip="Displays a log of all modifications made to this item"
    790         visible="<%=ChangeHistoryUtil.showChangeHistoryTab(sc)%>">
    791         <%
    792         if ("history".equals(tab))
    793         {
    794           %>
    795           <jsp:include page="../../common/history/frameset.jsp">
    796             <jsp:param name="source_type" value="<%=itemType.name()%>" />
    797             <jsp:param name="source_id" value="<%=itemId%>" />
    798             <jsp:param name="ID" value="<%=ID%>" />
    799           </jsp:include>
    800           <%
    801         }
    802         %>
     778        visible="<%=ChangeHistoryUtil.showChangeHistoryTab(sc)%>"
     779        activate="History.loadOnce()">
     780        <jsp:include page="../../common/history/frameset.jsp">
     781          <jsp:param name="item_type" value="<%=itemType.name()%>" />
     782          <jsp:param name="item_id" value="<%=itemId%>" />
     783          <jsp:param name="ID" value="<%=ID%>" />
     784        </jsp:include>
    803785      </t:tab>
    804786      </t:tabcontrol>
  • trunk/www/views/experiments/view_experiment.jsp

    r6242 r6244  
    178178    {
    179179      if (TabControl.isActive(tabControlId, tabId)) return;
    180       if ((tabId == 'history') && tabId != '<%=tab%>')
    181       {
    182         location.href = 'index.jsp?ID=<%=ID%>&cmd=ViewItem&item_id=<%=itemId%>&tab='+tabId;
    183       }
    184       else if (tabId == 'clonedreporters')
     180      if (tabId == 'clonedreporters')
    185181      {
    186182        viewClonedReporters();
     
    824820      <t:tab id="history" title="Change history"
    825821        tooltip="Displays a log of all modifications made to this item"
    826         visible="<%=ChangeHistoryUtil.showChangeHistoryTab(sc)%>">
    827         <%
    828         if ("history".equals(tab))
    829         {
    830           %>
    831           <jsp:include page="../../common/history/frameset.jsp">
    832             <jsp:param name="source_type" value="<%=itemType.name()%>" />
    833             <jsp:param name="source_id" value="<%=itemId%>" />
    834             <jsp:param name="ID" value="<%=ID%>" />
    835           </jsp:include>
    836           <%
    837         }
    838         %>
     822        visible="<%=ChangeHistoryUtil.showChangeHistoryTab(sc)%>"
     823        activate="History.loadOnce()">
     824        <jsp:include page="../../common/history/frameset.jsp">
     825          <jsp:param name="item_type" value="<%=itemType.name()%>" />
     826          <jsp:param name="item_id" value="<%=itemId%>" />
     827          <jsp:param name="ID" value="<%=ID%>" />
     828        </jsp:include>
    839829      </t:tab>
    840830      </t:tabcontrol>
  • trunk/www/views/formulas/view_formula.jsp

    r6136 r6244  
    141141      Main.openPopup('index.jsp?ID=<%=ID%>&cmd='+cmd+'&item_id=<%=itemId%>', 'RunPlugin'+cmd, 750, 500);
    142142    }
    143     function switchTab(tabControlId, tabId)
    144     {
    145       if (TabControl.isActive(tabControlId, tabId)) return;
    146       if (tabId == 'history' && tabId != '<%=tab%>')
    147       {
    148         location.href = 'index.jsp?ID=<%=ID%>&cmd=ViewItem&item_id=<%=itemId%>&tab='+tabId;
    149       }
    150       else
    151       {
    152         TabControl.setActiveTab(tabControlId, tabId);
    153       }
    154     }
    155143    </script>
    156144  </base:head>
     
    164152      id="main"
    165153      subclass="content mastertabcontrol"
    166       active="<%=tab%>" switch="switchTab" remember="false">
     154      active="<%=tab%>">
    167155    <t:tab id="properties" title="Properties">
    168156      <div>
     
    359347    <t:tab id="history" title="Change history"
    360348      tooltip="Displays a log of all modifications made to this item"
    361       visible="<%=ChangeHistoryUtil.showChangeHistoryTab(sc)%>">
    362       <%
    363       if ("history".equals(tab))
    364       {
    365         %>
    366         <jsp:include page="../../common/history/frameset.jsp">
    367           <jsp:param name="source_type" value="<%=itemType.name()%>" />
    368           <jsp:param name="source_id" value="<%=itemId%>" />
    369           <jsp:param name="ID" value="<%=ID%>" />
    370         </jsp:include>
    371         <%
    372       }
    373       %>
     349      visible="<%=ChangeHistoryUtil.showChangeHistoryTab(sc)%>"
     350      activate="History.loadOnce()">
     351      <jsp:include page="../../common/history/frameset.jsp">
     352        <jsp:param name="item_type" value="<%=itemType.name()%>" />
     353        <jsp:param name="item_id" value="<%=itemId%>" />
     354        <jsp:param name="ID" value="<%=ID%>" />
     355      </jsp:include>
    374356    </t:tab>
    375357    </t:tabcontrol>
  • trunk/www/views/physicalbioassays/view_bioassay.jsp

    r6242 r6244  
    156156      Main.openPopup('index.jsp?ID=<%=ID%>&cmd='+cmd+'&item_id=<%=itemId%>', 'RunPlugin'+cmd, 750, 500);
    157157    }
    158     function switchTab(tabControlId, tabId)
    159     {
    160       if (TabControl.isActive(tabControlId, tabId)) return;
    161       if ((tabId == 'history') && tabId != '<%=tab%>')
    162       {
    163         location.href = 'index.jsp?ID=<%=ID%>&cmd=ViewItem&item_id=<%=itemId%>&tab='+tabId;
    164       }
    165       else
    166       {
    167         TabControl.setActiveTab(tabControlId, tabId);
    168       }
    169     }
    170158    function newDerivedBioAssay()
    171159    {
     
    182170      id="main"
    183171      subclass="content mastertabcontrol"
    184       active="<%=tab%>" switch="switchTab" remember="false">
     172      active="<%=tab%>">
    185173    <t:tab id="properties" title="Properties">
    186174      <div>
     
    544532      <t:tab id="history" title="Change history"
    545533        tooltip="Displays a log of all modifications made to this item"
    546         visible="<%=ChangeHistoryUtil.showChangeHistoryTab(sc)%>">
    547         <%
    548         if ("history".equals(tab))
    549         {
    550           %>
    551           <jsp:include page="../../common/history/frameset.jsp">
    552             <jsp:param name="source_type" value="<%=itemType.name()%>" />
    553             <jsp:param name="source_id" value="<%=itemId%>" />
    554             <jsp:param name="ID" value="<%=ID%>" />
    555           </jsp:include>
    556           <%
    557         }
    558         %>
     534        visible="<%=ChangeHistoryUtil.showChangeHistoryTab(sc)%>"
     535        activate="History.loadOnce()">
     536        <jsp:include page="../../common/history/frameset.jsp">
     537          <jsp:param name="item_type" value="<%=itemType.name()%>" />
     538          <jsp:param name="item_id" value="<%=itemId%>" />
     539          <jsp:param name="ID" value="<%=ID%>" />
     540        </jsp:include>
    559541      </t:tab>
    560542      </t:tabcontrol>
  • trunk/www/views/rawbioassays/view_rawbioassay.jsp

    r6242 r6244  
    186186    {
    187187      if (TabControl.isActive(tabControlId, tabId)) return;
    188       if ((tabId == 'history') && tabId != '<%=tab%>')
    189       {
    190         location.href = 'index.jsp?ID=<%=ID%>&cmd=ViewItem&item_id=<%=itemId%>&tab='+tabId;
    191       }
    192       else if (tabId == 'rawdata')
     188      if (tabId == 'rawdata')
    193189      {
    194190        viewRawData();
     
    616612      <t:tab id="history" title="Change history"
    617613        tooltip="Displays a log of all modifications made to this item"
    618         visible="<%=ChangeHistoryUtil.showChangeHistoryTab(sc)%>">
    619         <%
    620         if ("history".equals(tab))
    621         {
    622           %>
    623           <jsp:include page="../../common/history/frameset.jsp">
    624             <jsp:param name="source_type" value="<%=itemType.name()%>" />
    625             <jsp:param name="source_id" value="<%=itemId%>" />
    626             <jsp:param name="ID" value="<%=ID%>" />
    627           </jsp:include>
    628           <%
    629         }
    630         %>
     614        visible="<%=ChangeHistoryUtil.showChangeHistoryTab(sc)%>"
     615        activate="History.loadOnce()">
     616        <jsp:include page="../../common/history/frameset.jsp">
     617          <jsp:param name="item_type" value="<%=itemType.name()%>" />
     618          <jsp:param name="item_id" value="<%=itemId%>" />
     619          <jsp:param name="ID" value="<%=ID%>" />
     620        </jsp:include>
    631621      </t:tab>
    632622      </t:tabcontrol>
Note: See TracChangeset for help on using the changeset viewer.