Changeset 6711


Ignore:
Timestamp:
Feb 3, 2015, 9:23:37 AM (7 years ago)
Author:
Nicklas Nordborg
Message:

Fixes #1916: List pages are loading annotations for non-visible columns

Moved the call to loader.find(snapshot) inside the <tbl:cell> tag which means that it is only executed for visible columns.

Bad news is that this change conflicts with #1906 ([6690] and others) so merging to trunk will require some extra work.

Location:
branches/3.4-stable
Files:
22 edited

Legend:

Unmodified
Added
Removed
  • branches/3.4-stable/src/clients/web/net/sf/basedb/clients/web/taglib/table/Cell.java

    r6127 r6711  
    301301      content += formatter == null ? value.toString() : formatter.format(value);
    302302    }
    303     if (content.length() == 0) content = "&nbsp;";
    304303    boolean overflowed = maxCharacters > 0 && content.length() > maxCharacters &&
    305304      HTML.textLength(content) > maxCharacters;
  • branches/3.4-stable/www/admin/hardware/list_hardware.jsp

    r6604 r6711  
    472472                  for (AnnotationLoaderUtil loader : annotationLoaders)
    473473                  {
    474                     if (loader.find(snapshot))
    475                     {
    476                       %>
    477                       <tbl:cell
    478                         column="<%="at"+loader.getId()%>"
    479                         ><tbl:cellvalue
     474                    %>
     475                    <tbl:cell
     476                      column="<%="at"+loader.getId()%>"
     477                      ><%
     478                      if (loader.find(snapshot))
     479                      {
     480                        %><tbl:cellvalue
    480481                          list="<%=loader.getValues()%>"
    481482                          suffix="<%=loader.getUnitSymbol()%>"
    482                       /></tbl:cell>
    483                       <%
    484                     }
     483                        /><%
     484                      }
     485                      %></tbl:cell>
     486                    <%
    485487                  }
    486488                }
  • branches/3.4-stable/www/admin/protocols/list_protocol.jsp

    r6604 r6711  
    489489                  for (AnnotationLoaderUtil loader : annotationLoaders)
    490490                  {
    491                     if (loader.find(snapshot))
    492                     {
    493                       %>
    494                       <tbl:cell
    495                         column="<%="at"+loader.getId()%>"
    496                         ><tbl:cellvalue
     491                    %>
     492                    <tbl:cell
     493                      column="<%="at"+loader.getId()%>"
     494                      ><%
     495                      if (loader.find(snapshot))
     496                      {
     497                        %><tbl:cellvalue
    497498                          list="<%=loader.getValues()%>"
    498499                          suffix="<%=loader.getUnitSymbol()%>"
    499                       /></tbl:cell>
    500                       <%
    501                     }
     500                        /><%
     501                      }
     502                      %></tbl:cell>
     503                    <%
    502504                  }
    503505                }
  • branches/3.4-stable/www/admin/software/list_software.jsp

    r6604 r6711  
    472472                  for (AnnotationLoaderUtil loader : annotationLoaders)
    473473                  {
    474                     if (loader.find(snapshot))
    475                     {
    476                       %>
    477                       <tbl:cell
    478                         column="<%="at"+loader.getId()%>"
    479                         ><tbl:cellvalue
     474                    %>
     475                    <tbl:cell
     476                      column="<%="at"+loader.getId()%>"
     477                      ><%
     478                      if (loader.find(snapshot))
     479                      {
     480                        %><tbl:cellvalue
    480481                          list="<%=loader.getValues()%>"
    481482                          suffix="<%=loader.getUnitSymbol()%>"
    482                       /></tbl:cell>
    483                       <%
    484                     }
     483                        /><%
     484                      }
     485                      %></tbl:cell>
     486                    <%
    485487                  }
    486488                }
  • branches/3.4-stable/www/biomaterials/bioplates/list_bioplates.jsp

    r6604 r6711  
    633633                  for (AnnotationLoaderUtil loader : annotationLoaders)
    634634                  {
    635                     if (loader.find(snapshot))
    636                     {
    637                       %>
    638                       <tbl:cell
    639                         column="<%="at"+loader.getId()%>"
    640                         ><tbl:cellvalue
     635                    %>
     636                    <tbl:cell
     637                      column="<%="at"+loader.getId()%>"
     638                      ><%
     639                      if (loader.find(snapshot))
     640                      {
     641                        %><tbl:cellvalue
    641642                          list="<%=loader.getValues()%>"
    642643                          suffix="<%=loader.getUnitSymbol()%>"
    643                       /></tbl:cell>
    644                       <%
    645                     }
     644                        /><%
     645                      }
     646                      %></tbl:cell>
     647                    <%
    646648                  }
    647649                }
  • branches/3.4-stable/www/biomaterials/bioplates/wells/list_biowells.jsp

    r6604 r6711  
    622622                  for (AnnotationLoaderUtil loader : annotationLoaders)
    623623                  {
    624                     if (loader.find(snapshot))
    625                     {
    626                       %>
    627                       <tbl:cell
    628                         column="<%="at"+loader.getId()%>"
    629                         ><tbl:cellvalue
     624                    %>
     625                    <tbl:cell
     626                      column="<%="at"+loader.getId()%>"
     627                      ><%
     628                      if (loader.find(snapshot))
     629                      {
     630                        %><tbl:cellvalue
    630631                          list="<%=loader.getValues()%>"
    631632                          suffix="<%=loader.getUnitSymbol()%>"
    632                       /></tbl:cell>
    633                       <%
    634                     }
     633                        /><%
     634                      }
     635                      %></tbl:cell>
     636                    <%
    635637                  }
    636638                }
  • branches/3.4-stable/www/biomaterials/biosources/list_biosources.jsp

    r6605 r6711  
    543543                  for (AnnotationLoaderUtil loader : annotationLoaders)
    544544                  {
    545                     if (loader.find(snapshot))
    546                     {
    547                       %>
    548                       <tbl:cell
    549                         column="<%="at"+loader.getId()%>"
    550                         ><tbl:cellvalue
     545                    %>
     546                    <tbl:cell
     547                      column="<%="at"+loader.getId()%>"
     548                      ><%
     549                      if (loader.find(snapshot))
     550                      {
     551                        %><tbl:cellvalue
    551552                          list="<%=loader.getValues()%>"
    552553                          suffix="<%=loader.getUnitSymbol()%>"
    553                       /></tbl:cell>
    554                       <%
    555                     }
     554                        /><%
     555                      }
     556                      %></tbl:cell>
     557                    <%
    556558                  }
    557559                }
  • branches/3.4-stable/www/biomaterials/extracts/list_extracts.jsp

    r6604 r6711  
    920920                  for (AnnotationLoaderUtil loader : annotationLoaders)
    921921                  {
    922                     if (loader.find(snapshot))
    923                     {
    924                       %>
    925                       <tbl:cell
    926                         column="<%="at"+loader.getId()%>"
    927                         ><tbl:cellvalue
     922                    %>
     923                    <tbl:cell
     924                      column="<%="at"+loader.getId()%>"
     925                      ><%
     926                      if (loader.find(snapshot))
     927                      {
     928                        %><tbl:cellvalue
    928929                          list="<%=loader.getValues()%>"
    929930                          suffix="<%=loader.getUnitSymbol()%>"
    930                       /></tbl:cell>
    931                       <%
    932                     }
     931                        /><%
     932                      }
     933                      %></tbl:cell>
     934                    <%
    933935                  }
    934936                }
  • branches/3.4-stable/www/biomaterials/lists/list_lists.jsp

    r6604 r6711  
    465465                  for (AnnotationLoaderUtil loader : annotationLoaders)
    466466                  {
    467                     if (loader.find(snapshot))
    468                     {
    469                       %>
    470                       <tbl:cell
    471                         column="<%="at"+loader.getId()%>"
    472                         ><tbl:cellvalue
     467                    %>
     468                    <tbl:cell
     469                      column="<%="at"+loader.getId()%>"
     470                      ><%
     471                      if (loader.find(snapshot))
     472                      {
     473                        %><tbl:cellvalue
    473474                          list="<%=loader.getValues()%>"
    474475                          suffix="<%=loader.getUnitSymbol()%>"
    475                       /></tbl:cell>
    476                       <%
    477                     }
     476                        /><%
     477                      }
     478                      %></tbl:cell>
     479                    <%
    478480                  }
    479481                }
  • branches/3.4-stable/www/biomaterials/lists/members/list_members.jsp

    r6604 r6711  
    989989                  for (AnnotationLoaderUtil loader : annotationLoaders)
    990990                  {
    991                     if (loader.find(snapshot))
    992                     {
    993                       %>
    994                       <tbl:cell
    995                         column="<%="at"+loader.getId()%>"
    996                         ><tbl:cellvalue
     991                    %>
     992                    <tbl:cell
     993                      column="<%="at"+loader.getId()%>"
     994                      ><%
     995                      if (loader.find(snapshot))
     996                      {
     997                        %><tbl:cellvalue
    997998                          list="<%=loader.getValues()%>"
    998999                          suffix="<%=loader.getUnitSymbol()%>"
    999                       /></tbl:cell>
    1000                       <%
    1001                     }
     1000                        /><%
     1001                      }
     1002                      %></tbl:cell>
     1003                    <%
    10021004                  }
    10031005                }
  • branches/3.4-stable/www/biomaterials/samples/list_samples.jsp

    r6604 r6711  
    851851                  for (AnnotationLoaderUtil loader : annotationLoaders)
    852852                  {
    853                     if (loader.find(snapshot))
    854                     {
    855                       %>
    856                       <tbl:cell
    857                         column="<%="at"+loader.getId()%>"
    858                         ><tbl:cellvalue
     853                    %>
     854                    <tbl:cell
     855                      column="<%="at"+loader.getId()%>"
     856                      ><%
     857                      if (loader.find(snapshot))
     858                      {
     859                        %><tbl:cellvalue
    859860                          list="<%=loader.getValues()%>"
    860861                          suffix="<%=loader.getUnitSymbol()%>"
    861                       /></tbl:cell>
    862                       <%
    863                     }
     862                        /><%
     863                      }
     864                      %></tbl:cell>
     865                    <%
    864866                  }
    865867                }
  • branches/3.4-stable/www/biomaterials/tags/list_tags.jsp

    r6604 r6711  
    472472                  for (AnnotationLoaderUtil loader : annotationLoaders)
    473473                  {
    474                     if (loader.find(snapshot))
    475                     {
    476                       %>
    477                       <tbl:cell
    478                         column="<%="at"+loader.getId()%>"
    479                         ><tbl:cellvalue
     474                    %>
     475                    <tbl:cell
     476                      column="<%="at"+loader.getId()%>"
     477                      ><%
     478                      if (loader.find(snapshot))
     479                      {
     480                        %><tbl:cellvalue
    480481                          list="<%=loader.getValues()%>"
    481482                          suffix="<%=loader.getUnitSymbol()%>"
    482                       /></tbl:cell>
    483                       <%
    484                     }
     483                        /><%
     484                      }
     485                      %></tbl:cell>
     486                    <%
    485487                  }
    486488                }
  • branches/3.4-stable/www/lims/arraybatches/list_batches.jsp

    r6604 r6711  
    544544                  for (AnnotationLoaderUtil loader : annotationLoaders)
    545545                  {
    546                     if (loader.find(snapshot))
    547                     {
    548                       %>
    549                       <tbl:cell
    550                         column="<%="at"+loader.getId()%>"
    551                         ><tbl:cellvalue
     546                    %>
     547                    <tbl:cell
     548                      column="<%="at"+loader.getId()%>"
     549                      ><%
     550                      if (loader.find(snapshot))
     551                      {
     552                        %><tbl:cellvalue
    552553                          list="<%=loader.getValues()%>"
    553554                          suffix="<%=loader.getUnitSymbol()%>"
    554                       /></tbl:cell>
    555                       <%
    556                     }
     555                        /><%
     556                      }
     557                      %></tbl:cell>
     558                    <%
    557559                  }
    558560                }
  • branches/3.4-stable/www/lims/arraydesigns/list_designs.jsp

    r6604 r6711  
    671671                  for (AnnotationLoaderUtil loader : annotationLoaders)
    672672                  {
    673                     if (loader.find(snapshot))
    674                     {
    675                       %>
    676                       <tbl:cell
    677                         column="<%="at"+loader.getId()%>"
    678                         ><tbl:cellvalue
     673                    %>
     674                    <tbl:cell
     675                      column="<%="at"+loader.getId()%>"
     676                      ><%
     677                      if (loader.find(snapshot))
     678                      {
     679                        %><tbl:cellvalue
    679680                          list="<%=loader.getValues()%>"
    680681                          suffix="<%=loader.getUnitSymbol()%>"
    681                       /></tbl:cell>
    682                       <%
    683                     }
     682                        /><%
     683                      }
     684                      %></tbl:cell>
     685                    <%
    684686                  }
    685687                }
  • branches/3.4-stable/www/lims/arrayslides/list_slides.jsp

    r6604 r6711  
    529529                  for (AnnotationLoaderUtil loader : annotationLoaders)
    530530                  {
    531                     if (loader.find(snapshot))
    532                     {
    533                       %>
    534                       <tbl:cell
    535                         column="<%="at"+loader.getId()%>"
    536                         ><tbl:cellvalue
     531                    %>
     532                    <tbl:cell
     533                      column="<%="at"+loader.getId()%>"
     534                      ><%
     535                      if (loader.find(snapshot))
     536                      {
     537                        %><tbl:cellvalue
    537538                          list="<%=loader.getValues()%>"
    538539                          suffix="<%=loader.getUnitSymbol()%>"
    539                       /></tbl:cell>
    540                       <%
    541                     }
     540                        /><%
     541                      }
     542                      %></tbl:cell>
     543                    <%
    542544                  }
    543545                }
  • branches/3.4-stable/www/lims/plates/list_plates.jsp

    r6604 r6711  
    616616                  for (AnnotationLoaderUtil loader : annotationLoaders)
    617617                  {
    618                     if (loader.find(snapshot))
    619                     {
    620                       %>
    621                       <tbl:cell
    622                         column="<%="at"+loader.getId()%>"
    623                         ><tbl:cellvalue
     618                    %>
     619                    <tbl:cell
     620                      column="<%="at"+loader.getId()%>"
     621                      ><%
     622                      if (loader.find(snapshot))
     623                      {
     624                        %><tbl:cellvalue
    624625                          list="<%=loader.getValues()%>"
    625626                          suffix="<%=loader.getUnitSymbol()%>"
    626                       /></tbl:cell>
    627                       <%
    628                     }
     627                        /><%
     628                      }
     629                      %></tbl:cell>
     630                    <%
    629631                  }
    630632                }
  • branches/3.4-stable/www/lims/plates/wells/list_wells.jsp

    r6604 r6711  
    598598                  for (AnnotationLoaderUtil loader : annotationLoaders)
    599599                  {
    600                     if (loader.find(snapshot))
    601                     {
    602                       %>
    603                       <tbl:cell
    604                         column="<%="at"+loader.getId()%>"
    605                         ><tbl:cellvalue
     600                    %>
     601                    <tbl:cell
     602                      column="<%="at"+loader.getId()%>"
     603                      ><%
     604                      if (loader.find(snapshot))
     605                      {
     606                        %><tbl:cellvalue
    606607                          list="<%=loader.getValues()%>"
    607608                          suffix="<%=loader.getUnitSymbol()%>"
    608                       /></tbl:cell>
    609                       <%
    610                     }
     609                        /><%
     610                      }
     611                      %></tbl:cell>
     612                    <%
    611613                  }
    612614                }
  • branches/3.4-stable/www/views/derivedbioassays/list_bioassays.jsp

    r6604 r6711  
    746746                  for (AnnotationLoaderUtil loader : annotationLoaders)
    747747                  {
    748                     if (loader.find(snapshot))
    749                     {
    750                       %>
    751                       <tbl:cell
    752                         column="<%="at"+loader.getId()%>"
    753                         ><tbl:cellvalue
     748                    %>
     749                    <tbl:cell
     750                      column="<%="at"+loader.getId()%>"
     751                      ><%
     752                      if (loader.find(snapshot))
     753                      {
     754                        %><tbl:cellvalue
    754755                          list="<%=loader.getValues()%>"
    755756                          suffix="<%=loader.getUnitSymbol()%>"
    756                       /></tbl:cell>
    757                       <%
    758                     }
     757                        /><%
     758                      }
     759                      %></tbl:cell>
     760                    <%
    759761                  }
    760762                }
  • branches/3.4-stable/www/views/experiments/bioassays/list_bioassays.jsp

    r6604 r6711  
    524524                  for (AnnotationLoaderUtil loader : annotationLoaders)
    525525                  {
    526                     if (loader.find(snapshot))
    527                     {
    528                       %>
    529                       <tbl:cell
    530                         column="<%="at"+loader.getId()%>"
    531                         ><tbl:cellvalue
     526                    %>
     527                    <tbl:cell
     528                      column="<%="at"+loader.getId()%>"
     529                      ><%
     530                      if (loader.find(snapshot))
     531                      {
     532                        %><tbl:cellvalue
    532533                          list="<%=loader.getValues()%>"
    533534                          suffix="<%=loader.getUnitSymbol()%>"
    534                       /></tbl:cell>
    535                       <%
    536                     }
     535                        /><%
     536                      }
     537                      %></tbl:cell>
     538                    <%
    537539                  }
    538540                }
  • branches/3.4-stable/www/views/experiments/bioassaysets/analysis_tree.jsp

    r6604 r6711  
    813813                      for (AnnotationLoaderUtil loader : annotationLoaders)
    814814                      {
    815                         if (loader.find(snapshot))
    816                         {
    817                           %>
    818                           <tbl:cell
    819                             column="<%="at"+loader.getId()%>"
    820                             ><tbl:cellvalue
     815                        %>
     816                        <tbl:cell
     817                          column="<%="at"+loader.getId()%>"
     818                          ><%
     819                          if (loader.find(snapshot))
     820                          {
     821                            %><tbl:cellvalue
    821822                              list="<%=loader.getValues()%>"
    822823                              suffix="<%=loader.getUnitSymbol()%>"
    823                           /></tbl:cell>
    824                           <%
    825                         }
     824                            /><%
     825                          }
     826                          %></tbl:cell>
     827                        <%
    826828                      }
    827829                    }
  • branches/3.4-stable/www/views/physicalbioassays/list_bioassays.jsp

    r6647 r6711  
    683683                  for (AnnotationLoaderUtil loader : annotationLoaders)
    684684                  {
    685                     if (loader.find(snapshot))
    686                     {
    687                       %>
    688                       <tbl:cell
    689                         column="<%="at"+loader.getId()%>"
    690                         ><tbl:cellvalue
     685                    %>
     686                    <tbl:cell
     687                      column="<%="at"+loader.getId()%>"
     688                      ><%
     689                      if (loader.find(snapshot))
     690                      {
     691                        %><tbl:cellvalue
    691692                          list="<%=loader.getValues()%>"
    692693                          suffix="<%=loader.getUnitSymbol()%>"
    693                       /></tbl:cell>
    694                       <%
    695                     }
     694                        /><%
     695                      }
     696                      %></tbl:cell>
     697                    <%
    696698                  }
    697699                }
  • branches/3.4-stable/www/views/rawbioassays/list_rawbioassays.jsp

    r6604 r6711  
    786786                  for (AnnotationLoaderUtil loader : annotationLoaders)
    787787                  {
    788                     if (loader.find(snapshot))
    789                     {
    790                       %>
    791                       <tbl:cell
    792                         column="<%="at"+loader.getId()%>"
    793                         ><tbl:cellvalue
     788                    %>
     789                    <tbl:cell
     790                      column="<%="at"+loader.getId()%>"
     791                      ><%
     792                      if (loader.find(snapshot))
     793                      {
     794                        %><tbl:cellvalue
    794795                          list="<%=loader.getValues()%>"
    795796                          suffix="<%=loader.getUnitSymbol()%>"
    796                       /></tbl:cell>
    797                       <%
    798                     }
     797                        /><%
     798                      }
     799                      %></tbl:cell>
     800                    <%
    799801                  }
    800802                }
Note: See TracChangeset for help on using the changeset viewer.