else is not necessary here

w3bdesign · w3bdesign · commit 4e7c890f6ba0 · 2021-02-08T21:05:56.000+01:00
else is not necessary here since all if branches return, consider removing it to reduce nesting and make code more readable.
diff --git a/utils/functions/functions.js b/utils/functions/functions.js
@@ -176,9 +176,8 @@ export const removeItemFromCart = (productId) => {
 
     localStorage.setItem('woo-next-cart', JSON.stringify(updatedCart));
     return updatedCart;
-  } else {
-    return existingCart;
   }
+  return existingCart;
 };
 
 /**

Original file line number	Diff line number	Diff line change
`@@ -176,9 +176,8 @@ export const removeItemFromCart = (productId) => {`
`176`	`176`
`177`	`177`	`localStorage.setItem('woo-next-cart', JSON.stringify(updatedCart));`
`178`	`178`	`return updatedCart;`
`179`		`- } else {`
`180`		`- return existingCart;`
`181`	`179`	`}`
	`180`	`+ return existingCart;`
`182`	`181`	`};`
`183`	`182`
`184`	`183`	`/**`