Merge pull request #338 from w3bdesign/develop

w3bdesign · web-flow · commit e7ada02db6d3 · 2021-02-08T21:28:56.000+01:00
Fix Scrutinizer code issue - No return
diff --git a/utils/functions/functions.js b/utils/functions/functions.js
@@ -110,7 +110,6 @@ export const getUpdatedProducts = (
     updatedProduct.totalPrice = parseFloat(
       (updatedProduct.price * updatedProduct.qty).toFixed(2)
     );
-
     return updatedProducts;
   }
   // If product not found push the new product to the existing product array.
@@ -133,6 +132,7 @@ const isProductInCart = (existingProductsInCart, productId) => {
     if (productId === item.productId) {
       return item;
     }
+    return;
   };
 
   // This new array will only contain the product which is matched.

Original file line number	Diff line number	Diff line change
`@@ -110,7 +110,6 @@ export const getUpdatedProducts = (`
`110`	`110`	`updatedProduct.totalPrice = parseFloat(`
`111`	`111`	`(updatedProduct.price * updatedProduct.qty).toFixed(2)`
`112`	`112`	`);`
`113`		`-`
`114`	`113`	`return updatedProducts;`
`115`	`114`	`}`
`116`	`115`	`// If product not found push the new product to the existing product array.`
`@@ -133,6 +132,7 @@ const isProductInCart = (existingProductsInCart, productId) => {`
`133`	`132`	`if (productId === item.productId) {`
`134`	`133`	`return item;`
`135`	`134`	`}`
	`135`	`+ return;`
`136`	`136`	`};`
`137`	`137`
`138`	`138`	`// This new array will only contain the product which is matched.`