fix: infinite loop in calculate button

Signed-off-by: rare-magma <[email protected]>
rare-magma · Aug 10, 2024 · 6a2455b · 6a2455b
1 parent dd5544b
commit 6a2455b
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 12 deletions.
diff --git a/src/components/CalculateButton/CalculateButton.tsx b/src/components/CalculateButton/CalculateButton.tsx
@@ -253,6 +253,7 @@ export function CalculateButton({
             setTimeout(() => {
               inputRef.current?.focus();
             }, 0);
+            getHistory();
           }}
         >
           <BsPlusSlashMinus aria-hidden />

diff --git a/src/hooks/useDB.ts b/src/hooks/useDB.ts
@@ -386,6 +386,7 @@ export function useDB() {
 
     return { filteredIncomes, filteredExpenses };
   }
+
   function renameBudget(event: React.ChangeEvent<HTMLInputElement>) {
     if (budget && event.target.value) {
       const newState = produce((draft) => {
@@ -395,18 +396,21 @@ export function useDB() {
     }
   }
 
-  async function getCalcHist(id: string): Promise<CalculationHistoryItem[]> {
-    let item;
-    await calcHistDB
-      .getItem(id)
-      .then((i) => {
-        item = i;
-      })
-      .catch((e: unknown) => {
-        throw e;
-      });
-    return item ?? [];
-  }
+  const getCalcHist = useCallback(
+    async (id: string): Promise<CalculationHistoryItem[]> => {
+      let item;
+      await calcHistDB
+        .getItem(id)
+        .then((i) => {
+          item = i;
+        })
+        .catch((e: unknown) => {
+          throw e;
+        });
+      return item ?? [];
+    },
+    [],
+  );
 
   async function saveCalcHist(id: string, item: CalculationHistoryItem) {
     const calcHist = await getCalcHist(id);