ticket.basicData Update zone on changes + fixes #1668

modules/ticket/back/methods/ticket/priceDifference.js

@@ -94,14 +94,16 @@ module.exports = Self => {
         salesObj.items.forEach(sale => {
             const difComponent = map.get(sale.id);
 
-            if (difComponent)
+            if (difComponent) {
                 sale.component = difComponent;
 
-            salesObj.totalUnitPrice += sale.price;
                 salesObj.totalNewPrice += sale.component.newPrice;
                 salesObj.totalDifference += sale.component.difference;
                 salesObj.totalUnitPrice = Math.round(salesObj.totalUnitPrice * 100) / 100;
                 salesObj.totalNewPrice = Math.round(salesObj.totalNewPrice * 100) / 100;
+            }
+
+            salesObj.totalUnitPrice += sale.price;
             salesObj.totalDifference = Math.round(salesObj.totalDifference * 100) / 100;
         });
 

modules/ticket/front/basic-data/step-one/index.html

@@ -36,8 +36,7 @@
                 label="Warehouse"
                 show-field="name"
                 value-field="id"
-                field="$ctrl.ticket.warehouseFk"
+                field="$ctrl.warehouseId">
-                initial-data="$ctrl.ticket.warehouseFk">
             </vn-autocomplete>
         </vn-horizontal>
         <vn-horizontal>

modules/ticket/front/basic-data/step-one/index.js

@@ -39,11 +39,21 @@ class Controller {
         this.onChangeClient(value);
     }
 
-    set shipped(value) {
+    get warehouseId() {
-        this.ticket.shipped = value;
+        if (this.ticket)
-        this.onChangeShipped(value);
+            return this.ticket.warehouseFk;
+
+        return null;
     }
 
+    set warehouseId(id) {
+        if (id != this.ticket.warehouseFk) {
+            this.ticket.warehouseFk = id;
+            this.onChangeWarehouse(id);
+        }
+    }
+
+
     get shipped() {
         if (this.ticket)
             return this.ticket.shipped;
@@ -51,6 +61,11 @@ class Controller {
         return null;
     }
 
+    set shipped(value) {
+        this.ticket.shipped = value;
+        this.onChangeShipped(value);
+    }
+
     get landed() {
         if (this.ticket)
             return this.ticket.landed;
@@ -120,23 +135,6 @@ class Controller {
         });
     }
 
-    /*
-     * Returns a landing date
-     */
-    getLanded(params) {
-        let query = `/api/Agencies/getLanded`;
-        return this.$http.get(query, {params});
-    }
-
-    /*
-     * Returns a shipment date
-     */
-    getShipped(params) {
-        let query = `/api/Agencies/getShipped`;
-        return this.$http.get(query, {params});
-    }
-
-
     onChangeShipped(shipped) {
         let params = {
             shipped: shipped,
@@ -144,6 +142,8 @@ class Controller {
             agencyModeFk: this.ticket.agencyModeFk,
             warehouseFk: this.ticket.warehouseFk
         };
+        this.ticket.zoneFk = null;
+
         let query = `/api/Agencies/getLanded`;
         this.$http.get(query, {params}).then(res => {
             if (res.data && res.data.landed) {
@@ -164,6 +164,8 @@ class Controller {
             agencyModeFk: this.ticket.agencyModeFk,
             warehouseFk: this.ticket.warehouseFk
         };
+        this.ticket.zoneFk = null;
+
         let query = `/api/Agencies/getShipped`;
         this.$http.get(query, {params}).then(res => {
             if (res.data) {
@@ -183,8 +185,10 @@ class Controller {
     onChangeZone(zoneId) {
         const query = `/api/Zones/${zoneId}`;
         this.$http.get(query).then(res => {
-            if (res.data)
+            if (res.data) {
                 this.ticket.agencyModeFk = res.data.agencyModeFk;
+                this.ticket.warehouseFk = res.data.warehouseFk;
+            }
         });
     }
 
@@ -212,6 +216,31 @@ class Controller {
         });
     }
 
+    /*
+     * Gets a zone from an agency
+     */
+    onChangeWarehouse(warehouseId) {
+        let params = {
+            landed: this.ticket.landed,
+            addressFk: this.ticket.addressFk,
+            agencyModeFk: this.ticket.agencyModeFk,
+            warehouseFk: warehouseId
+        };
+
+        this.ticket.zoneFk = null;
+        let query = `/api/Agencies/getShipped`;
+        this.$http.get(query, {params}).then(res => {
+            if (res.data)
+                this.ticket.zoneFk = res.data.id;
+
+            if (!res.data) {
+                this.vnApp.showMessage(
+                    this.$translate.instant('No delivery zone available for this parameters')
+                );
+            }
+        });
+    }
+
     async onStepChange() {
         if (this.isFormInvalid()) {
             return this.vnApp.showError(
@@ -240,6 +269,22 @@ class Controller {
         });
     }
 
+    /*
+     * Returns a landing date
+     */
+    getLanded(params) {
+        let query = `/api/Agencies/getLanded`;
+        return this.$http.get(query, {params});
+    }
+
+    /*
+     * Returns a shipment date
+     */
+    getShipped(params) {
+        let query = `/api/Agencies/getShipped`;
+        return this.$http.get(query, {params});
+    }
+
     isFormInvalid() {
         return !this.ticket.clientFk || !this.ticket.addressFk || !this.ticket.agencyModeFk
             || !this.ticket.companyFk || !this.ticket.shipped || !this.ticket.landed

modules/ticket/front/basic-data/step-two/index.js

@@ -28,8 +28,8 @@ class Controller {
     getTotalNewPrice() {
         let totalNewPrice = 0;
         this.ticket.sale.items.forEach(item => {
-            let itemTotalNewPrice = item.quantity * item.component.newPrice;
+            if (item.component)
-            totalNewPrice += itemTotalNewPrice;
+                totalNewPrice += item.quantity * item.component.newPrice;
         });
         this.totalNewPrice = totalNewPrice;
     }
@@ -37,6 +37,7 @@ class Controller {
     getTotalDifferenceOfPrice() {
         let totalPriceDifference = 0;
         this.ticket.sale.items.forEach(item => {
+            if (item.component)
                 totalPriceDifference += item.component.difference;
         });
         this.totalPriceDifference = totalPriceDifference;