npm-pkg-hook 1.7.1 → 1.7.3

package/package.json

@@ -44,5 +44,5 @@
     "rm": "rm -rf node_modules package-lock.json && npm i",
     "test": "echo \"Error: no test specified\" && exit 1"
   },
-  "version": "1.7.1"
+  "version": "1.7.3"
 }

package/src/hooks/useDeleteExtraProductFoods/index.js

@@ -0,0 +1,12 @@
+import { useMutation } from '@apollo/client'
+import { DELETE_EXTRA_PRODUCTS } from '../useDessertWithPrice/queries'
+
+export const useDeleteExtraProductFoods = () => {
+  const [deleteExtraProductFoods, { loading, error }] = useMutation(DELETE_EXTRA_PRODUCTS)
+
+  return {
+    deleteExtraProductFoods,
+    loading,
+    error
+  }
+}

package/src/hooks/useDessertWithPrice/helpers/index.js

@@ -0,0 +1,76 @@
+export const transformData = (dataExtra) => {
+  const transformedData = dataExtra?.map(item => ({
+    extraName: item.extraName || '',
+    extraPrice: item?.extraPrice?.toString() || '',
+    exState: !!item.exState,
+    forEdit: true,
+    ...item
+  }))
+
+  return transformedData
+}
+
+export const MAX_INTEGER = 2147483647
+/**
+ * Validate if a number is within a specified range.
+ * @param {number} num - The number to validate.
+ * @returns {boolean} - True if the number is within the range, false otherwise.
+ */
+export const isWithinRange = (num) => {
+  // Verificar si el número está dentro del rango permitido.
+  return num >= MAX_INTEGER
+}
+
+/**
+ * Find objects in the array where the value of 'extraPrice' exceeds the specified range.
+ * @param {array} arr - The array to search.
+ * @returns {array} - An array containing the indices and objects of the items exceeding the range.
+ */
+export const findNumbersExceedingRange = (arr) => {
+  return arr.reduce((acc, item, index) => {
+    const extraPrice = typeof item.extraPrice === 'number' ? item.extraPrice : parseFloat(item.extraPrice.replace(/\./g, ''))
+    if (isWithinRange(extraPrice)) {
+      acc.push({ index, item })
+    }
+    return acc
+  }, [])
+}
+
+export const updateErrorFieldByIndex = ({
+  setLine = (array) => {
+    return array
+  },
+  checkNumberRange = []
+} = {
+  setLine: (array) => {
+    return array
+  },
+  checkNumberRange: []
+}) => {
+  setLine(prevLineItems => {
+    // Crea una copia del estado anterior de LineItems
+    const updatedLineItems = { ...prevLineItems }
+
+    // Utiliza map para iterar sobre cada elemento en checkNumberRange
+    const updatedLines = updatedLineItems.Lines.map((line, index) => {
+      // Verifica si el índice está dentro del rango de LineItems.Lines
+      if (checkNumberRange.some(item => item.index === index)) {
+        // Crea una copia del elemento actual
+        const updatedLine = { ...line }
+
+        // Actualiza el campo 'error' del elemento a true
+        updatedLine.error = true
+        updatedLine.messageError = 'El precio no puede ser mayor a 2147483647.00'
+
+        // Devuelve el elemento actualizado
+        return updatedLine
+      }
+
+      // Si el índice no está en checkNumberRange, devuelve el elemento sin cambios
+      return line
+    })
+
+    // Actualiza el array Lines en el estado de LineItems con los elementos actualizados
+    return { ...updatedLineItems, Lines: updatedLines }
+  })
+}

package/src/hooks/useDessertWithPrice/index.js

@@ -1,12 +1,31 @@
-import { useCallback, useEffect, useMemo, useRef, useState, createRef } from 'react'
+import {
+  useCallback,
+  useEffect,
+  useMemo,
+  useRef,
+  useState,
+  createRef
+} from 'react'
 import { useUpdateMultipleExtProductFoods } from '../useUpdateMultipleExtProductFoods'
 import { useMutation } from '@apollo/client'
-import { DELETE_EXTRA_PRODUCTS, EDIT_EXTRA_PRODUCT_FOODS } from './queries'
+import { EDIT_EXTRA_PRODUCT_FOODS } from './queries'
+import { findNumbersExceedingRange, transformData, updateErrorFieldByIndex } from './helpers'
+import { useDeleteExtraProductFoods } from '../useDeleteExtraProductFoods'
 
 export const useDessertWithPrice = ({
   dataExtra = [],
-  sendNotification = () => { },
-  setAlertBox = () => { }
+  sendNotification = ({
+    title,
+    description,
+    backgroundColor
+  }) => {
+    return {
+      title,
+      description,
+      backgroundColor
+    }
+  },
+  setAlertBox = ({ message, duration = 10000, success = true }) => { return { message, duration, success } }
 } = {}) => {
   const [selected, setSelected] = useState({
     loading: false,
@@ -34,19 +53,11 @@ export const useDessertWithPrice = ({
       ]
     }
   }, [initialLine])
-  const transformedData = dataExtra?.map(item => ({
-    extraName: item.extraName || '',
-    extraPrice: item?.extraPrice?.toString() || '', // Convierte a string si es necesario
-    exState: !!item.exState,
-    forEdit: true,
-    ...item
-  }))
+  const transformedData = transformData(dataExtra)
+
   const [LineItems, setLine] = useState(
     Array.isArray(dataExtra) && dataExtra.length > 0 ? { Lines: transformedData } : initialLineItems
   )
-  useEffect(() => {
-    setLine(Array.isArray(dataExtra) && dataExtra.length > 0 ? { Lines: transformedData } : initialLineItems)
-  }, [dataExtra.length])
 
   const inputRefs = useRef(LineItems.Lines.map(() => createRef()))
 
@@ -58,7 +69,7 @@ export const useDessertWithPrice = ({
         inputRefs.current[index].current.focus()
       }
     } catch (error) {
-      console.log(error)
+      return null
     }
   }
 
@@ -66,11 +77,12 @@ export const useDessertWithPrice = ({
     // Asegurándote de que las referencias se actualicen si LineItems cambia
     inputRefs.current = LineItems.Lines.map((_, i) => inputRefs.current[i] || createRef())
   }, [LineItems])
+
   const handleCleanLines = useCallback(() => {
-    setLine(initialLineItems)
+    setLine(Array.isArray(dataExtra) && dataExtra.length > 0 ? { Lines: transformedData } : initialLineItems)
   }, [initialLineItems])
 
-  const [updateMultipleExtProductFoods, { loading }] = useUpdateMultipleExtProductFoods({ handleCleanLines: () => { } })
+  const [updateMultipleExtProductFoods, { loading }] = useUpdateMultipleExtProductFoods({ handleCleanLines: () => { return } })
   /**
    * Handles the addition of two new lines to the Lines array in LineItems state.
    */
@@ -132,11 +144,16 @@ export const useDessertWithPrice = ({
 
     setLine({ ...LineItems, Lines: newLines })
   }
-  const [deleteExtraProductFoods] = useMutation(DELETE_EXTRA_PRODUCTS)
+  const { deleteExtraProductFoods } = useDeleteExtraProductFoods()
 
+  /**
+ * Filter out a specific line from the LineItems array.
+ * @param {number} index - Index of the line to be filtered out.
+ */
   const filterOneLine = (index) => {
-    console.log(index)
+  // Use optional chaining to safely access nested properties.
     const Lines = LineItems?.Lines?.filter((_, i) => { return i !== index })
+    // Use spread operator to create a new object with the filtered Lines array.
     return setLine({ ...LineItems, Lines })
   }
   const handleRemove = async (i, exPid) => {
@@ -148,13 +165,24 @@ export const useDessertWithPrice = ({
             variables: {
               state: 1,
               id: exPid
+            },
+            update: (cache) => {
+              cache.modify({
+                fields: {
+                  ExtProductFoodsAll: (dataOld = []) => {
+                    const { success } = data?.data?.deleteextraproductfoods || {}
+                    if (success && Array.isArray(dataOld)) {
+                      const transformedData = transformData(dataOld)
+                      const Lines = transformedData.filter((_, index) => { return index !== i })
+                      const newCache = dataOld.filter((_, index) => { return index !== i })
+                      setLine({ ...LineItems, Lines })
+                      return newCache
+                    }
+                  }
+                }
+              })
             }
           })
-          const { success } = data?.data?.deleteextraproductfoods || {}
-          if (success) {
-            console.log(i, exPid)
-            return filterOneLine(i)
-          }
         }
       }
       if (!exPid) {
@@ -168,8 +196,10 @@ export const useDessertWithPrice = ({
       })
     }
   }
+  useEffect(() => {
+    setLine(Array.isArray(dataExtra) && dataExtra.length > 0 ? { Lines: transformedData } : initialLineItems)
+  }, [dataExtra.length])
 
-  // Prepares and validates data for submission.
   const prepareAndValidateData = useCallback((pId) => {
     const dataArr = LineItems?.Lines?.map(({ extraPrice, exState, extraName }) => ({
       extraPrice: parseFloat(extraPrice),
@@ -179,8 +209,8 @@ export const useDessertWithPrice = ({
     }))
 
     const message = 'Complete los campos vacíos'
-    const findInputEmpty = dataArr.find(({ extraName }) => extraName === '')
-    const findInputEmptyPrice = dataArr.find(({ extraPrice }) => isNaN(extraPrice) || extraPrice === '')
+    const findInputEmpty = dataArr?.find(({ extraName }) => extraName === '')
+    const findInputEmptyPrice = dataArr?.find(({ extraPrice }) => isNaN(extraPrice) || extraPrice === '')
 
     if (findInputEmpty || findInputEmptyPrice) {
       setAlertBox({ message })
@@ -270,6 +300,12 @@ export const useDessertWithPrice = ({
 
   const handleSubmit = ({ pId }) => {
     try {
+      const checkNumberRange = findNumbersExceedingRange(LineItems?.Lines)
+      updateErrorFieldByIndex({ checkNumberRange, setLine })
+      if (checkNumberRange?.length > 0) {
+        return setAlertBox({ message: 'El precio no puede ser tan alto', duration: 10000 })
+      }
+
       if (!prepareAndValidateData(pId)) return
       const dataArr = LineItems?.Lines?.map(x => {
         const extraPrice = stringToInt(x.extraPrice)

package/src/hooks/useProductsFood/index.js

@@ -72,7 +72,13 @@ export const useProductsFood = ({
   ]
 }
 
-export const useDeleteProductsFood = ({ sendNotification = () => { } } = {}) => {
+export const useDeleteProductsFood = ({
+  sendNotification = (asrg) => { return asrg },
+  onSuccess = (asrg) => { return asrg }
+} = {
+  sendNotification: (asrg) => { return asrg },
+  onSuccess: (asrg) => { return asrg }
+}) => {
   const [updateProductFoods, { data, loading, error }] = useMutation(UPDATE_PRODUCT_FOOD)
 
   const handleDelete = async product => {
@@ -130,8 +136,9 @@ export const useDeleteProductsFood = ({ sendNotification = () => { } } = {}) =>
         })
       }
     }).then(() => {
+      onSuccess()
       return sendNotification({
-        title: 'Success',
+        title: 'Exito',
         description: 'El producto se ha eliminado correctamente',
         backgroundColor: 'success'
       })

package/src/hooks/useSales/index.js

@@ -904,6 +904,11 @@ export const useSales = ({
         setLoadingSale(false)
         setErrorSale(true)
         setPrint(false)
+        sendNotification({
+          title: 'error',
+          backgroundColor: 'error',
+          description: 'Lo sentimos, ocurrió un error'
+        })
       })
       .finally(() => {
         setPrint(false)

package/src/hooks/useUpdateMultipleExtProductFoods/index.js

@@ -4,9 +4,13 @@ import { UPDATE_MULTI_EXTRAS_PRODUCT_FOOD } from './queries'
 /**
  * Custom hook para manejar la actualización de múltiples extras de productos alimenticios.
  * @param {Function} cleanLines - Función para limpiar líneas después de completar la mutación.
+ * @param {Function} handleCleanLines - Función.
  * @returns {Array} Retorna un array con la función de mutación y el estado de carga.
  */
-export const useUpdateMultipleExtProductFoods = ({ cleanLines = () => { } } = {}) => {
+export const useUpdateMultipleExtProductFoods = ({ cleanLines = () => { }, handleCleanLines = () => { } } = {
+  cleanLines: () => { },
+  handleCleanLines: () => { }
+}) => {
   const [updateMultipleExtProductFoods, { loading }] = useMutation(UPDATE_MULTI_EXTRAS_PRODUCT_FOOD, {
     onCompleted: () => {
       cleanLines()