fix: remove number conversion in sumMoney utility

leather-io · Dec 9, 2024 · 2193733 · 2193733
1 parent af7f990
commit 2193733
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/packages/utils/src/money/calculate-money.ts b/packages/utils/src/money/calculate-money.ts
@@ -3,7 +3,7 @@ import { BigNumber } from 'bignumber.js';
 import { type MarketData, type Money, type NumType, formatMarketPair } from '@leather.io/models';
 
 import { isNumber } from '..';
-import { initBigNumber, sumNumbers } from '../math/helpers';
+import { initBigNumber } from '../math/helpers';
 import { createMoney, formatMoney } from './format-money';
 import { isMoney } from './is-money';
 
@@ -53,6 +53,6 @@ export function sumMoney(moneysArr: Money[]) {
   if (moneysArr.some(item => item.symbol !== moneysArr[0].symbol))
     throw new Error('Cannot sum different currencies');
 
-  const sum = sumNumbers(moneysArr.map(item => item.amount.toNumber()));
+  const sum = moneysArr.reduce((acc, item) => acc.plus(item.amount), new BigNumber(0));
   return createMoney(sum, moneysArr[0].symbol, moneysArr[0].decimals);
 }