1 files changed, 39 insertions, 26 deletions

diff --git a/src-migrate/modules/cart/components/ItemAction.tsx b/src-migrate/modules/cart/components/ItemAction.tsx
index 4dcebd9e..b06e8e75 100644
--- a/src-migrate/modules/cart/components/ItemAction.tsx
+++ b/src-migrate/modules/cart/components/ItemAction.tsx
@@ -22,11 +22,11 @@ import {
 
 import { toast } from 'react-hot-toast';
 
-type Props = {
+interface Props {
   item: CartItem;
-};
+}
 
-const CartItemAction = ({ item }: Props) => {
+const CartItemAction: React.FC<Props> = ({ item }) => {
   const auth = getAuth();
   const { setRefreshCart } = useProductCartContext();
   const [isLoadDelete, setIsLoadDelete] = useState<boolean>(false);
@@ -36,9 +36,9 @@ const CartItemAction = ({ item }: Props) => {
 
   const { loadCart, cart, updateCartItem } = useCartStore();
 
-  const limitQty = item.limit_qty?.transaction || 0;
+  const limitQty: number = item.limit_qty?.transaction || 0;
 
-  const handleDelete = async () => {
+  const handleDelete = async (): Promise<void> => {
     if (typeof auth !== 'object') return;
 
     setIsLoadDelete(true);
@@ -80,25 +80,29 @@ const CartItemAction = ({ item }: Props) => {
     }
   };
 
-  const updateQuantityInCookie = (productId, cartId, newQuantity) => {
+  const updateQuantityInCookie = (
+    productId: number,
+    cartId: string | number,
+    newQuantity: number
+  ): boolean => {
     try {
-      const cartData = getCartDataFromCookie();
+      const cartData = getCartDataFromCookie() as Record<string, any>;
       let itemFound = false;
+      const cartIdString = String(cartId);
 
       // Find item by cart_id key or search within objects
-      if (cartData[cartId]) {
-        cartData[cartId].quantity = newQuantity;
+      if (cartData[cartIdString]) {
+        cartData[cartIdString].quantity = newQuantity;
         itemFound = true;
       } else {
         // Search by product id or cart_id within objects
-        for (const key in cartData) {
+        Object.keys(cartData).forEach((key) => {
           const item = cartData[key];
-          if (item.id === productId || item.cart_id === cartId) {
+          if (item.id === productId || String(item.cart_id) === cartIdString) {
             item.quantity = newQuantity;
             itemFound = true;
-            break;
           }
-        }
+        });
       }
 
       if (itemFound) {
@@ -113,12 +117,12 @@ const CartItemAction = ({ item }: Props) => {
     }
   };
 
-  const decreaseQty = () => {
-    setQuantity((quantity) => (quantity -= 1));
+  const decreaseQty = (): void => {
+    setQuantity((prevQuantity) => prevQuantity - 1);
   };
 
-  const increaseQty = () => {
-    setQuantity((quantity) => (quantity += 1));
+  const increaseQty = (): void => {
+    setQuantity((prevQuantity) => prevQuantity + 1);
   };
 
   const debounceQty = useDebounce(quantity, 1000);
@@ -129,7 +133,7 @@ const CartItemAction = ({ item }: Props) => {
   }, [debounceQty, limitQty]);
 
   useEffect(() => {
-    const updateCart = async () => {
+    const updateCart = async (): Promise<void> => {
       if (typeof auth !== 'object' || isNaN(debounceQty)) return;
       if (debounceQty === item.quantity) return;
 
@@ -167,20 +171,22 @@ const CartItemAction = ({ item }: Props) => {
         await loadCart(auth.id);
 
         // Re-update cookie if server reload overwrote it
-        const currentCookieData = getCartDataFromCookie();
+        const currentCookieData = getCartDataFromCookie() as Record<
+          string,
+          any
+        >;
         let needsReUpdate = false;
 
-        for (const key in currentCookieData) {
+        Object.keys(currentCookieData).forEach((key) => {
           const cookieItem = currentCookieData[key];
           if (
             (cookieItem.id === item.id ||
-              cookieItem.cart_id === item.cart_id) &&
+              String(cookieItem.cart_id) === String(item.cart_id)) &&
             cookieItem.quantity !== debounceQty
           ) {
             needsReUpdate = true;
-            break;
           }
-        }
+        });
 
         if (needsReUpdate) {
           updateQuantityInCookie(item.id, item.cart_id, debounceQty);
@@ -202,6 +208,13 @@ const CartItemAction = ({ item }: Props) => {
     //eslint-disable-next-line react-hooks/exhaustive-deps
   }, [debounceQty]);
 
+  const handleQuantityInputChange = (
+    e: React.ChangeEvent<HTMLInputElement>
+  ): void => {
+    const value = parseInt(e.target.value);
+    setQuantity(isNaN(value) ? 1 : value);
+  };
+
   return (
     <div className={style.actionSection}>
       <button
@@ -230,8 +243,8 @@ const CartItemAction = ({ item }: Props) => {
 
         <input
           type='number'
-          className={style.quantity.toString()}
-          onChange={(e) => setQuantity(parseInt(e.target.value))}
+          className={style.quantity}
+          onChange={handleQuantityInputChange}
           value={quantity}
         />
 
@@ -249,4 +262,4 @@ const CartItemAction = ({ item }: Props) => {
   );
 };
 
-export default CartItemAction;
\ No newline at end of file
+export default CartItemAction;