Changeset 7000


Ignore:
Timestamp:
Nov 5, 2015, 8:12:50 AM (7 years ago)
Author:
Nicklas Nordborg
Message:

References #1962: Trying to load item with ID=1 as parent item when inheriting annotations

Need to check for 'NaN' instead of '0' in all list handling code before deciding if a new option should be inserted or if the existing option should be replaced.

Location:
trunk/www
Files:
26 edited

Legend:

Unmodified
Added
Removed
  • trunk/www/admin/itemsubtypes/subtypes.js

    r6389 r7000  
    193193    var frm = document.forms['subtype'];
    194194    var list = frm[event.currentTarget.id];
    195     if (list.length < 2 || list[1].value == '0')
     195    if (list.length < 2 || list[1].value == 'NaN')
    196196    {
    197197      Forms.addListOption(list, 1, new Option());
  • trunk/www/admin/pluginconfigurations/configurations.js

    r6944 r7000  
    196196    var frm = document.forms['configuration'];
    197197    var list = frm.plugindefinition_id;
    198     if (list.length < 1 || list[0].value == '0')
     198    if (list.length < 1 || list[0].value == 'NaN')
    199199    {
    200200      Forms.addListOption(list, 0, new Option());
  • trunk/www/admin/protocols/protocols.js

    r6400 r7000  
    166166    var frm = document.forms['protocol'];
    167167    var list = frm.file_id;
    168     if (list.length < 2 || list[1].value == '0') // >
     168    if (list.length < 2 || list[1].value == 'NaN')
    169169    {
    170170      Forms.addListOption(list, 1, new Option());
  • trunk/www/biomaterials/bioplates/events/events.js

    r6997 r7000  
    149149    var frm = document.forms['event'];
    150150    var list = frm.hardware_id;
    151     if (list.length < 2 || list[1].value == '0')
     151    if (list.length < 2 || list[1].value == 'NaN')
    152152    {
    153153      Forms.addListOption(list, 1, new Option());
     
    174174    var frm = document.forms['event'];
    175175    var list = frm.protocol_id;
    176     if (list.length < 2 || list[1].value == '0')
     176    if (list.length < 2 || list[1].value == 'NaN')
    177177    {
    178178      Forms.addListOption(list, 1, new Option());
     
    200200    var frm = document.forms['event'];
    201201    var list = frm.kit_id;
    202     if (list.length < 2 || list[1].value == '0')
     202    if (list.length < 2 || list[1].value == 'NaN')
    203203    {
    204204      Forms.addListOption(list, 1, new Option());
  • trunk/www/biomaterials/bioplates/wells/biowells.js

    r6306 r7000  
    135135    var frm = document.forms['biowell'];
    136136    var list = frm.biomaterial_id;
    137     if (list.length < 2 || list[1].value == '0')
     137    if (list.length < 2 || list[1].value == 'NaN')
    138138    {
    139139      Forms.addListOption(list, 1, new Option());
  • trunk/www/biomaterials/events/events.js

    r6993 r7000  
    129129    var frm = document.forms['event'];
    130130    var list = frm.protocol_id;
    131     if (list.length < 2 || list[1].value == '0')
     131    if (list.length < 2 || list[1].value == 'NaN')
    132132    {
    133133      Forms.addListOption(list, 1, new Option());
     
    157157    var frm = document.forms['event'];
    158158    var list = frm.kit_id;
    159     if (list.length < 2 || list[1].value == '0')
     159    if (list.length < 2 || list[1].value == 'NaN')
    160160    {
    161161      Forms.addListOption(list, 1, new Option());
  • trunk/www/biomaterials/extracts/extracts.js

    r6996 r7000  
    305305    var frm = document.forms['extract'];
    306306    var list = frm.tag_id;
    307     if (list.length < 2 || list[1].value == '0')
     307    if (list.length < 2 || list[1].value == 'NaN')
    308308    {
    309309      Forms.addListOption(list, 1, new Option());
     
    332332    var frm = document.forms['extract'];
    333333    var list = frm.protocol_id;
    334     if (list.length < 2 || list[1].value == '0')
     334    if (list.length < 2 || list[1].value == 'NaN')
    335335    {
    336336      Forms.addListOption(list, 1, new Option());
     
    359359    var frm = document.forms['extract'];
    360360    var list = frm.kit_id;
    361     if (list.length < 2 || list[1].value == '0')
     361    if (list.length < 2 || list[1].value == 'NaN')
    362362    {
    363363      Forms.addListOption(list, 1, new Option());
     
    385385    var frm = document.forms['extract'];
    386386    var list = frm.bioplate_id;
    387     if (list.length < 2 || list[1].value == '0')
     387    if (list.length < 2 || list[1].value == 'NaN')
    388388    {
    389389      Forms.addListOption(list, 1, new Option());
     
    436436    var frm = document.forms['extract'];
    437437    var list = frm.biowell_id;
    438     if (list.length < 2 || list[1].value == '0')
     438    if (list.length < 2 || list[1].value == 'NaN')
    439439    {
    440440      Forms.addListOption(list, 1, new Option());
     
    472472    var frm = document.forms['extract'];
    473473    var list = frm.sample_id;
    474     if (list.length < 2 || list[1].value == '0')
     474    if (list.length < 2 || list[1].value == 'NaN')
    475475    {
    476476      Forms.addListOption(list, 1, new Option());
  • trunk/www/biomaterials/samples/samples.js

    r6995 r7000  
    228228    var frm = document.forms['sample'];
    229229    var list = frm.bioplate_id;
    230     if (list.length < 2 || list[1].value == '0')
     230    if (list.length < 2 || list[1].value == 'NaN')
    231231    {
    232232      Forms.addListOption(list, 1, new Option());
     
    278278    var frm = document.forms['sample'];
    279279    var list = frm.biowell_id;
    280     if (list.length < 2 || list[1].value == '0')
     280    if (list.length < 2 || list[1].value == 'NaN')
    281281    {
    282282      Forms.addListOption(list, 1, new Option());
     
    335335    var frm = document.forms['sample'];
    336336    var list = frm.protocol_id;
    337     if (list.length < 2 || list[1].value == '0')
     337    if (list.length < 2 || list[1].value == 'NaN')
    338338    {
    339339      Forms.addListOption(list, 1, new Option());
     
    362362    var frm = document.forms['sample'];
    363363    var list = frm.kit_id;
    364     if (list.length < 2 || list[1].value == '0')
     364    if (list.length < 2 || list[1].value == 'NaN')
    365365    {
    366366      Forms.addListOption(list, 1, new Option());
     
    407407    var frm = document.forms['sample'];
    408408    var list = frm.biosource_id;
    409     if (list.length < 2 || list[1].value == '0')
     409    if (list.length < 2 || list[1].value == 'NaN')
    410410    {
    411411      Forms.addListOption(list, 1, new Option());
  • trunk/www/biomaterials/wizards/create_child_bioplate.js

    r6997 r7000  
    220220    var frm = document.forms['main'];
    221221    var list = frm.tag_id;
    222     if (list.length < 2 || list[1].value == '0')
     222    if (list.length < 2 || list[1].value == 'NaN')
    223223    {
    224224      Forms.addListOption(list, 1, new Option());
     
    246246    var frm = document.forms['main'];
    247247    var list = frm.freezer_id;
    248     if (list.length < 2 || list[1].value == '0')
     248    if (list.length < 2 || list[1].value == 'NaN')
    249249    {
    250250      Forms.addListOption(list, 1, new Option());
     
    267267    var frm = document.forms['main'];
    268268    var list = frm.plategeometry_id;
    269     if (list.length < 1 || list[0].value == '0')
     269    if (list.length < 1 || list[0].value == 'NaN')
    270270    {
    271271      Forms.addListOption(list, 0, new Option());
  • trunk/www/biomaterials/wizards/wizards.js

    r6997 r7000  
    5050    var frm = document.forms['main'];
    5151    var list = frm.hardware_id;
    52     if (list.length < 2 || list[1].value == '0')
     52    if (list.length < 2 || list[1].value == 'NaN')
    5353    {
    5454      Forms.addListOption(list, 1, new Option());
     
    8181    var frm = document.forms['main'];
    8282    var list = frm.protocol_id;
    83     if (list.length < 2 || list[1].value == '0')
     83    if (list.length < 2 || list[1].value == 'NaN')
    8484    {
    8585      Forms.addListOption(list, 1, new Option());
     
    112112    var frm = document.forms['main'];
    113113    var list = frm.kit_id;
    114     if (list.length < 2 || list[1].value == '0')
     114    if (list.length < 2 || list[1].value == 'NaN')
    115115    {
    116116      Forms.addListOption(list, 1, new Option());
  • trunk/www/common/annotations/inherit.jsp

    r6981 r7000  
    221221  // Parent items may have been submitted by caller
    222222  String[] parents = request.getParameterValues("parents");
     223  parents = null;
    223224  // Each parameter is ITEMTYPE:ID:ID:...
    224225  if (parents != null)
  • trunk/www/common/ownership/ownership.js

    r6306 r7000  
    6262    var frm = document.forms['ownership'];
    6363    var list = frm.owner_id;
    64     if (list.length < 2 || list[1].value == '0')
     64    if (list.length < 2 || list[1].value == 'NaN')
    6565    {
    6666      Forms.addListOption(list, 1, new Option());
  • trunk/www/filemanager/files/files.js

    r6576 r7000  
    418418    var frm = document.forms['file'];
    419419    var list = frm.server_id;
    420     if (list.length < 2 || list[1].value == '0') // >
     420    if (list.length < 2 || list[1].value == 'NaN')
    421421    {
    422422      Forms.addListOption(list, 1, new Option());
  • trunk/www/lims/arraybatches/batches.js

    r6947 r7000  
    195195    var frm = document.forms['batch'];
    196196    var list = frm.arraydesign_id;
    197     if (list.length < 1 || list[0].value == '0')
     197    if (list.length < 1 || list[0].value == 'NaN')
    198198    {
    199199      Forms.addListOption(list, 0, new Option());
     
    221221    var frm = document.forms['batch'];
    222222    var list = frm.protocol_id;
    223     if (list.length < 2 || list[1].value == '0')
     223    if (list.length < 2 || list[1].value == 'NaN')
    224224    {
    225225      Forms.addListOption(list, 1, new Option());
     
    247247    var frm = document.forms['batch'];
    248248    var list = frm.printrobot_id;
    249     if (list.length < 2 || list[1].value == '0')
     249    if (list.length < 2 || list[1].value == 'NaN')
    250250    {
    251251      Forms.addListOption(list, 1, new Option());
  • trunk/www/lims/arrayslides/slides.js

    r6947 r7000  
    155155    var frm = document.forms['slide'];
    156156    var list = frm.arraybatch_id;
    157     if (list.length < 1 || list[0].value == '0')
     157    if (list.length < 1 || list[0].value == 'NaN')
    158158    {
    159159      Forms.addListOption(list, 0, new Option());
  • trunk/www/lims/arrayslides/wizard.js

    r6412 r7000  
    144144    var frm = document.forms['WizardStep1'];
    145145    var list = frm.arraybatch_id;
    146     if (list.length < 1 || list[0].value == '0')
     146    if (list.length < 1 || list[0].value == 'NaN')
    147147    {
    148148      Forms.addListOption(list, 0, new Option());
  • trunk/www/lims/plates/events/events.js

    r6312 r7000  
    133133    var frm = document.forms['event'];
    134134    var list = frm.hardware_id;
    135     if (list.length < 2 || list[1].value == '0')
     135    if (list.length < 2 || list[1].value == 'NaN')
    136136    {
    137137      Forms.addListOption(list, 1, new Option());
     
    173173    var frm = document.forms['event'];
    174174    var list = frm.protocol_id;
    175     if (list.length < 2 || list[1].value == '0')
     175    if (list.length < 2 || list[1].value == 'NaN')
    176176    {
    177177      Forms.addListOption(list, 1, new Option());
  • trunk/www/lims/plates/merge_plates.js

    r6389 r7000  
    9797    var frm = document.forms['merge'];
    9898    var list = frm.platemapping_id;
    99     if (list.length < 1 || list[0].value == '0')
     99    if (list.length < 1 || list[0].value == 'NaN')
    100100    {
    101101      Forms.addListOption(list, 0, new Option());
     
    115115    var frm = document.forms['merge'];
    116116    var list = frm.platetype_id;
    117     if (list.length < 1 || list[0].value == '0')
     117    if (list.length < 1 || list[0].value == 'NaN')
    118118    {
    119119      Forms.addListOption(list, 0, new Option());
  • trunk/www/login.js

    r6591 r7000  
    272272    var frm = document.forms[pForm];
    273273    var list = frm.user_id;
    274     if (list.length < 1 || list[0].value == '0')
     274    if (list.length < 1 || list[0].value == 'NaN')
    275275    {
    276276      Forms.addListOption(list, 0, new Option());
  • trunk/www/my_base/projects/projects.js

    r6412 r7000  
    332332    var frm = document.forms['project'];
    333333    var list = frm.permissiontemplate_id;
    334     if (list.length < 2 || list[1].value == '0')
     334    if (list.length < 2 || list[1].value == 'NaN')
    335335    {
    336336      Forms.addListOption(list, 1, new Option());
  • trunk/www/plugins/net/sf/basedb/plugins/jep_extra_value_calculator.js

    r6389 r7000  
    8484    var frm = document.forms['extraValue'];
    8585    var list = frm['parameter:extraValueType'];
    86     if (list.length < 1 || list[0].value == '0')
     86    if (list.length < 1 || list[0].value == 'NaN')
    8787    {
    8888      Forms.addListOption(list, 0, new Option());
  • trunk/www/views/derivedbioassays/bioassays.js

    r6996 r7000  
    280280    var frm = document.forms['bioAssay'];
    281281    var list = frm.protocol_id;
    282     if (list.length < 2 || list[1].value == '0')
     282    if (list.length < 2 || list[1].value == 'NaN')
    283283    {
    284284      Forms.addListOption(list, 1, new Option());
     
    307307    var frm = document.forms['bioAssay'];
    308308    var list = frm.kit_id;
    309     if (list.length < 2 || list[1].value == '0')
     309    if (list.length < 2 || list[1].value == 'NaN')
    310310    {
    311311      Forms.addListOption(list, 1, new Option());
     
    333333    var frm = document.forms['bioAssay'];
    334334    var list = frm.hardware_id;
    335     if (list.length < 2 || list[1].value == '0')
     335    if (list.length < 2 || list[1].value == 'NaN')
    336336    {
    337337      Forms.addListOption(list, 1, new Option());
     
    359359    var frm = document.forms['bioAssay'];
    360360    var list = frm.software_id;
    361     if (list.length < 2 || list[1].value == '0')
     361    if (list.length < 2 || list[1].value == 'NaN')
    362362    {
    363363      Forms.addListOption(list, 1, new Option());
     
    456456    var frm = document.forms['bioAssay'];
    457457    var list = frm.extract_id;
    458     if (list.length < 2 || list[1].value == '0') // >
     458    if (list.length < 2 || list[1].value == 'NaN')
    459459    {
    460460      Forms.addListOption(list, 1, new Option());
  • trunk/www/views/experiments/clone_reporters.js

    r6315 r7000  
    8181    var frm = document.forms['clone'];
    8282    var list = frm.reporterclonetemplate_id;
    83     if (list.length < 2 || list[1].value == '0')
     83    if (list.length < 2 || list[1].value == 'NaN')
    8484    {
    8585      Forms.addListOption(list, 1, new Option());
  • trunk/www/views/experiments/experiments.js

    r6944 r7000  
    271271    var frm = document.forms['experiment'];
    272272    var list = frm.directory_id;
    273     if (list.length < 2 || list[1].value == '0')
     273    if (list.length < 2 || list[1].value == 'NaN')
    274274    {
    275275      Forms.addListOption(list, 1, new Option());
  • trunk/www/views/physicalbioassays/bioassays.js

    r6995 r7000  
    233233    var frm = document.forms['bioassay'];
    234234    var list = frm.protocol_id;
    235     if (list.length < 2 || list[1].value == '0')
     235    if (list.length < 2 || list[1].value == 'NaN')
    236236    {
    237237      Forms.addListOption(list, 1, new Option());
     
    260260    var frm = document.forms['bioassay'];
    261261    var list = frm.kit_id;
    262     if (list.length < 2 || list[1].value == '0')
     262    if (list.length < 2 || list[1].value == 'NaN')
    263263    {
    264264      Forms.addListOption(list, 1, new Option());
     
    297297    var frm = document.forms['bioassay'];
    298298    var list = frm.hardware_id;
    299     if (list.length < 2 || list[1].value == '0')
     299    if (list.length < 2 || list[1].value == 'NaN')
    300300    {
    301301      Forms.addListOption(list, 1, new Option());
  • trunk/www/views/rawbioassays/bioassays.js

    r6947 r7000  
    263263    var frm = document.forms['rawbioassay'];
    264264    var list = frm.protocol_id;
    265     if (list.length < 2 || list[1].value == '0')
     265    if (list.length < 2 || list[1].value == 'NaN')
    266266    {
    267267      Forms.addListOption(list, 1, new Option());
     
    289289    var frm = document.forms['rawbioassay'];
    290290    var list = frm.software_id;
    291     if (list.length < 2 || list[1].value == '0')
     291    if (list.length < 2 || list[1].value == 'NaN')
    292292    {
    293293      Forms.addListOption(list, 1, new Option());
     
    314314    var frm = document.forms['rawbioassay'];
    315315    var list = frm.bioassay_id;
    316     if (list.length < 2 || list[1].value == '0')
     316    if (list.length < 2 || list[1].value == 'NaN')
    317317    {
    318318      Forms.addListOption(list, 1, new Option());
     
    351351    {
    352352      var designList = frm.arraydesign_id;
    353       if (designList.length < 2 || designList[1].value == '0')
     353      if (designList.length < 2 || designList[1].value == 'NaN')
    354354      {
    355355        Forms.addListOption(designList, 1, new Option());
     
    384384    var frm = document.forms['rawbioassay'];
    385385    var list = frm.arraydesign_id;
    386     if (list.length < 2 || list[1].value == '0')
     386    if (list.length < 2 || list[1].value == 'NaN')
    387387    {
    388388      Forms.addListOption(list, 1, new Option());
     
    453453    var frm = document.forms['rawbioassay'];
    454454    var list = frm.extract_id;
    455     if (list.length < 2 || list[1].value == '0')
     455    if (list.length < 2 || list[1].value == 'NaN')
    456456    {
    457457      Forms.addListOption(list, 1, new Option());
Note: See TracChangeset for help on using the changeset viewer.