X-Git-Url: http://wagnertech.de/git?a=blobdiff_plain;f=templates%2Fwebpages%2Fdbupgrade%2Fdefault_bin_parts.html;h=2c7413d6c73e12ca4e746d4724b9c05280cfef8d;hb=3b9c2119792ae678e9368a0141d7f8646fd6ef42;hp=07b397e8b5534b60e8114b184a4d6cd96b05e6ed;hpb=e5dd32ea97088b89764dbf45f460a2898aceacaf;p=kivitendo-erp.git

diff --git a/templates/webpages/dbupgrade/default_bin_parts.html b/templates/webpages/dbupgrade/default_bin_parts.html
index 07b397e8b..2c7413d6c 100644
--- a/templates/webpages/dbupgrade/default_bin_parts.html
+++ b/templates/webpages/dbupgrade/default_bin_parts.html
@@ -50,22 +50,25 @@
 
         control.options[bin_index].selected = true;
       }
+
       function bin_match(rowcount) {
-        for (i = 1; i < rowcount + 1; i++) {              // über alle parts_id
+        for (i = 1; i <= rowcount + 1; i++) {              // über alle parts_id
           var lagerplatz = document.getElementById("bin_" + i).value;
           var control = document.getElementById("bin_id_" + i);
           var bin_index = 0;
-          //alert(lagerplatz);
           for (j = 0; j < warehouses.length; j++) {       // über alle lager
             var warehouse = warehouses[j];
 
             for (k = 0; k < warehouse['bins'].length; k++) {       // über alle lagerplätze
 
               if (lagerplatz == warehouse['bins'][k]['description']) {
-                //alert('ware ' + warehouse['bins'][k]['description']);
                 var lager = document.getElementById("warehouse_id_" + i);
                 lager.selectedIndex = j;
                 bin_index = k;
+                for (bin_i = 0; bin_i < warehouse['bins'].length; bin_i++) {
+                  control.options[bin_i] = new Option(warehouse['bins'][bin_i]['description'], warehouse['bins'][bin_i]['id']);
+                }
+                control.options[bin_index].selected = true;
                 break;
                 /*var lagerplatz = document.getElementById("bin_id_" + i);
                 alert('lagerplatz ' + lagerplatz.value);
@@ -75,11 +78,6 @@
             }
           }
         }
-        for (i = 0; i < warehouse['bins'].length; i++) {
-                control.options[i] = new Option(warehouse['bins'][i]['description'], warehouse['bins'][i]['id']);
-              }
-              control.options[bin_index].selected = true;
-
 
       }
 
@@ -104,6 +102,7 @@
 <p>[% 'This has been changed in this version, therefore please change the "old" bins to some real warehouse bins.' | $T8 %]</p>
 <p>[% 'If your old bins match exactly Bins in the Warehouse CLICK on <b>AUTOMATICALLY MATCH BINS</b>.' | $T8 %]</p>
 <p>[% 'Otherwise you can simply check create warehouse and bins and define a name for the warehouse (Bins will be created automatically) and then continue'  | $T8 %]</p>
+<p>[% 'Empty selection for warehouse will not be added, even if the old bin is still visible (use back and forth to edit again).'  | $T8 %]</p>
 
 <table>
   <tr>