Merge pull request #234 from SFTtech/milo/fix-position-sum-order

fix(web): correct position sum order #226
SFTtech · Sep 4, 2024 · c00c4e5 · c00c4e5
2 parents b51f9b4 + 909e928
commit c00c4e5
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/frontend/apps/web/src/pages/transactions/TransactionDetail/purchase/TransactionPositions.tsx b/frontend/apps/web/src/pages/transactions/TransactionDetail/purchase/TransactionPositions.tsx
@@ -92,10 +92,11 @@ export const TransactionPositions: React.FC<TransactionPositionsProps> = ({
                 )
             );
         }
+        const accs = accountIDsToShow.map((id) => accountIDMap[id]).sort(getAccountSortFunc("name"));
 
         return {
-            shownAccounts: accountIDsToShow.map((id) => accountIDMap[id]).sort(getAccountSortFunc("name")),
-            shownAccountIDs: accountIDsToShow,
+            shownAccounts: accs,
+            shownAccountIDs: accs.map((a) => a.id),
         };
     }, [transaction, positions, additionalPurchaseItemAccounts, accountIDMap]);