Fix category map using lowest weight suggestion

yong-space · Nov 22, 2023 · e74e1c1 · e74e1c1
1 parent 7d7841e
commit e74e1c1
Show file tree

Hide file tree

Showing 4 changed files with 28 additions and 12 deletions.
diff --git a/src/settings/templates.jsx b/src/settings/templates.jsx
@@ -150,9 +150,13 @@ const Templates = ({ isMobile }) => {
     useEffect(() => {
         setCategoryOptions(prepareOptions(categories, 'category'));
         setSubCategoryOptions(prepareOptions(categories, 'subCategory'));
-        setCategoryMap(
-            categories.reduce((o, c) => ({ ...o, [c.subCategory]: c.category }), {})
-        );
+        const tempMap = {};
+        categories.forEach(({ category, subCategory }) => {
+            if (!tempMap[subCategory]) {
+                tempMap[subCategory] = category;
+            }
+        });
+        setCategoryMap(tempMap);
     }, [ categories ]);
 
     const postProcess = (row, newRows, verb) => {

diff --git a/src/transactions/add-transaction-dialog.jsx b/src/transactions/add-transaction-dialog.jsx
@@ -80,9 +80,13 @@ const AddTransactionDialog = ({
     useEffect(() => {
         setCategoryOptions(prepareOptions(categories, 'category'));
         setSubCategoryOptions(prepareOptions(categories, 'subCategory'));
-        setCategoryMap(
-            categories.reduce((o, c) => ({ ...o, [c.subCategory]: c.category }), {})
-        );
+        const tempMap = {};
+        categories.forEach(({ category, subCategory }) => {
+            if (!tempMap[subCategory]) {
+                tempMap[subCategory] = category;
+            }
+        });
+        setCategoryMap(tempMap);
     }, [ categories ]);
 
     useEffect(() => {

diff --git a/src/transactions/bulk-transaction-dialog.jsx b/src/transactions/bulk-transaction-dialog.jsx
@@ -53,9 +53,13 @@ const BulkTransactionDialog = ({
     useEffect(() => {
         setCategoryOptions(prepareOptions(categories, 'category'));
         setSubCategoryOptions(prepareOptions(categories, 'subCategory'));
-        setCategoryMap(
-            categories.reduce((o, c) => ({ ...o, [c.subCategory]: c.category }), {})
-        );
+        const tempMap = {};
+        categories.forEach(({ category, subCategory }) => {
+            if (!tempMap[subCategory]) {
+                tempMap[subCategory] = category;
+            }
+        });
+        setCategoryMap(tempMap);
     }, [ categories ]);
 
     const submit = (event) => {

diff --git a/src/transactions/transactions-import.jsx b/src/transactions/transactions-import.jsx
@@ -70,9 +70,13 @@ const TransactionsImport = ({ setImportMode, selectedAccount }) => {
     useEffect(() => {
         setCategoryOptions(prepareOptions(categories, 'category'));
         setSubCategoryOptions(prepareOptions(categories, 'subCategory'));
-        setCategoryMap(
-            categories.reduce((o, c) => ({ ...o, [c.subCategory]: c.category }), {})
-        );
+        const tempMap = {};
+        categories.forEach(({ category, subCategory }) => {
+            if (!tempMap[subCategory]) {
+                tempMap[subCategory] = category;
+            }
+        });
+        setCategoryMap(tempMap);
     }, [ categories ]);
 
     const maxGridSize = {