fix bug de type pour les poids et refactor de callback de confirmation

reception/static/js/reception_produits.js

@@ -31,9 +31,7 @@ var reception_status = null,
     validProducts = [], // Keep record of directly validated products
     updateType = "", // step 1: qty_valid; step2: br_valid
     barcodes = null, // Barcodes stored locally
-    priceToWeightIsCorrect = true,
-    found = {data: null, place: null},
-    p = null;
+    priceToWeightIsCorrect = true;
 
 var dbc = null,
     sync = null,
@@ -70,11 +68,10 @@ function searchUpdatedProduct() {
 function select_product_from_bc(barcode) {
     try {
         if (editing_product == null) {
-            p = barcodes.get_corresponding_odoo_product(barcode);
+            var p = barcodes.get_corresponding_odoo_product(barcode);
             var qtyIsWeight = false;
             var qtyIsPriceToWeight = false;
             priceToWeightIsCorrect = true;
-            found = {data: null, place: null};
 
             console.log(p);
 
@@ -84,6 +81,8 @@ function select_product_from_bc(barcode) {
                 return -1;
             }
             
+            var found = {data: null, place: null};
+
             console.log(p.rule);
             if(p.rule == 'weight') 
                 qtyIsWeight = true;
@@ -112,7 +111,7 @@ function select_product_from_bc(barcode) {
                 found.data.product_qty = p.qty;
 
             else if(qtyIsPriceToWeight){
-                openModal($('#templates #modal_confirm_price_to_weight').html(), price_to_weight_is_wrong, 'Non', false, true, price_to_weight_confirmed);
+                openModal($('#templates #modal_confirm_price_to_weight').html(), price_to_weight_is_wrong, 'Non', false, true, price_to_weight_confirmed_callback(found, p));
 
                 //On inverse en quelque sorte les boutons succes et d'annulation en mettant "Oui" sur le btn d'annulation
                 // et "Non" sur le bouton de reussite. 
@@ -197,23 +196,29 @@ function update_distant_orders() {
 
 //TODO not the cleanest way to manage the callback, needs improvement
 //TODO too much repeating code, needs refactor 
-function price_to_weight_confirmed(){
-    console.log("confirming")
-    if(priceToWeightIsCorrect)
-        found.data.product_qty = p.qty;
-    else
-        found.data.product_qty = (p.value/document.getElementById("new_price_to_weight").value).toFixed(2); 
+function price_to_weight_confirmed_callback (found, p){
+
+    return function price_to_weight_confirmed(){
+        console.log("confirming")
+        if(priceToWeightIsCorrect)
+            found.data.product_qty = p.qty;
+        else
+            found.data.product_qty = Number((p.value/document.getElementById("new_price_to_weight").value).toFixed(2)); 
 
-    if (found.data !== null) {
-        setLineEdition(found.data);
-        if (found.place === 'to_process') {
-            let row = table_to_process.row($('#'+found.data.product_id[0]));
+        if (found.data !== null) {
+            setLineEdition(found.data);
+            if (found.place === 'to_process') {
+                let row = table_to_process.row($('#'+found.data.product_id[0]));
 
-            remove_from_toProcess(row, found.data);
+                remove_from_toProcess(row, found.data);
+            }
         }
-    }
 }
 
+}
+
+
+
 function price_to_weight_is_wrong(){
     document.getElementById("new_price_to_weight").style.display = "";
     document.getElementsByClassName("btn--success")[0].style.display = "none";
@@ -937,7 +942,7 @@ function editProductInfo (productToEdit, value = null, batch = false) {
             if (e.product_id[0] == productToEdit.product_id[0]) {
                 addition = true;
                 productToEdit = e;
-                newValue = (newValue + productToEdit.product_qty).toFixed(2);
+                newValue = Number((newValue + productToEdit.product_qty).toFixed(2));
             }
         });
         // If qty edition & Check if qty changed