From a47dfbc27e15ad3746d0716dc6550f722b389892 Mon Sep 17 00:00:00 2001 From: Adam Caouette Date: Wed, 11 Oct 2023 14:01:50 -0400 Subject: [PATCH] CSR-1384: more cleanup, remove deep cloning --- .../add-vaps-modal-buttons.vue | 13 +++++++---- src/layouts/payment-method/payment-method.vue | 22 +++++-------------- 2 files changed, 15 insertions(+), 20 deletions(-) diff --git a/src/fmg-components/add-vaps-modal-buttons/add-vaps-modal-buttons.vue b/src/fmg-components/add-vaps-modal-buttons/add-vaps-modal-buttons.vue index 0a6faa72f..d894f2ee8 100644 --- a/src/fmg-components/add-vaps-modal-buttons/add-vaps-modal-buttons.vue +++ b/src/fmg-components/add-vaps-modal-buttons/add-vaps-modal-buttons.vue @@ -21,9 +21,8 @@ export default { name: "add-vaps-modal-buttons", props: { cartItems: Object, - rainDefenseItem: Object, - wiperItems: Object, vapsTilesCmsName: String, + availableLineItems: Object, }, data() { return { @@ -55,13 +54,19 @@ export default { }); }, rainDefensePrice() { - const lineItem = this.rainDefenseItem; + // const lineItem = this.rainDefenseItem; + const lineItem = this.availableLineItems.find((item) => { + return item.partType === "RAIN DEFENSE"; + }); + const price = lineItem.laborAmount + lineItem.sellingPrice + lineItem.kitPrice; return parseFloat(price).toFixed(2); }, wipersPrice() { let price = 0; - const lineItems = this.wiperItems; + const lineItems = this.availableLineItems.filter((item) => { + return item.partType.includes(" WIPER"); + }); lineItems?.forEach((item) => { price += item.laborAmount + item.sellingPrice + item.kitPrice; }); diff --git a/src/layouts/payment-method/payment-method.vue b/src/layouts/payment-method/payment-method.vue index e451958e4..eddaa6a20 100644 --- a/src/layouts/payment-method/payment-method.vue +++ b/src/layouts/payment-method/payment-method.vue @@ -34,8 +34,7 @@ @@ -105,7 +104,6 @@ import alert from "@/ux-components/alert/alert"; import baseMixin from "@/mixins/base-mixin.js"; import { settleAllPromises } from "@/helpers/layout-helper"; -import { deepClone } from "@/helpers/object-helper"; import { storeActions } from "@/constants/store-actions"; import store from "@/store"; import { fetchCmsContentForPage } from "@/helpers/cms-content-helper"; @@ -128,6 +126,8 @@ export default { null, "payment-method" ); + console.log("rainDefensePromise: ", rainDefensePromise); + const supportingItemsPromise = baseMixin.methods.dispatchStoreActionWithLogging( storeActions.GET_SUPPORTING_ITEMS, null, @@ -155,15 +155,13 @@ export default { const resultMap = await settleAllPromises(promiseResultMap); - const resultMapCopy = deepClone(resultMap) ?? []; - - const glassPartsCopy = deepClone(store.getters.order.lineItems.glassParts) ?? []; + const glassParts = (await store.getters.order.lineItems.glassParts) ?? []; const availableLineItems = [ resultMap.rainDefense, ...resultMap.supportingItems, ...resultMap.wipers, - ...glassPartsCopy, + ...glassParts, ]; const pricedAvailableLineItems = await baseMixin.methods.dispatchStoreActionWithLogging( @@ -176,12 +174,10 @@ export default { ); const lineItems = store.getters.lineItems; - const cartItems = [...glassPartsCopy, ...lineItems.supportingItems, ...lineItems.vaps]; + const cartItems = [...glassParts, ...lineItems.supportingItems, ...lineItems.vaps]; // TOBEREMOVED BY AJC IN CSR-1384 // Call the "next" function to complete the transition to this page. next((vm) => { - vm.availableWipers = resultMap.wipers; // TOBEREMOVED BY AJC IN CSR-1384 - vm.availableRainDefense = resultMap.rainDefense; // TOBEREMOVED BY AJC IN CSR-1384 vm.cartItems = cartItems; // TOBEREMOVED BY AJC IN CSR-1384 vm.setCmsContent(resultMap.cmsContent); @@ -192,8 +188,6 @@ export default { data() { return { cartItems: [], // TOBEREMOVED BY AJC IN CSR-1384 - availableWipers: [], // TOBEREMOVED BY AJC IN CSR-1384 - availableRainDefense: {}, // TOBEREMOVED BY AJC IN CSR-1384 lineItems: [], availableLineItems: [], @@ -326,10 +320,6 @@ export default { // console.log("this.$store.getters.damage ", this.$store.getters.damage) return this.$store.getters.damage; }, - storeLineItems() { - // console.log("storeLineItems, aka lineItems from store: ", this.$store.getters.lineItems) - return this.$store.getters.lineItems; - }, }, components: { funnelHeader,