Ignore future-dated account balances in the portfolio calculation

apps/api/src/app/account-balance/account-balance.service.ts

@@ -120,7 +120,7 @@ export class AccountBalanceService {
     const endOfTodayDate = endOfToday();
 
     for (const { accountId, date, valueInBaseCurrency } of balances) {
-      if (isAccountBalanceInFuture(date, endOfTodayDate)) {
+      if (isAccountBalanceInFuture({ date, endOfTodayDate })) {
         continue;
       }
 

apps/api/src/app/account/account.service.ts

@@ -139,7 +139,7 @@ export class AccountService {
           // The balances are ordered by date descending, hence the first account
           // balance which is not in the future reflects the current balance
           account.balances.find(({ date }) => {
-            return !isAccountBalanceInFuture(date, endOfTodayDate);
+            return !isAccountBalanceInFuture({ date, endOfTodayDate });
           })?.value ?? 0,
         tags: isTagsIncluded
           ? (account.tags as unknown as { tag: Tag }[]).map(({ tag }) => {

apps/api/src/app/activities/activities.service.ts

@@ -472,7 +472,12 @@ export class ActivitiesService {
       let currentBalanceInBaseCurrency = 0;
 
       for (const balanceItem of balances) {
-        if (isAccountBalanceInFuture(balanceItem.date, endOfTodayDate)) {
+        if (
+          isAccountBalanceInFuture({
+            endOfTodayDate,
+            date: balanceItem.date
+          })
+        ) {
           continue;
         }
 

apps/api/src/helper/account.helper.ts

@@ -18,9 +18,12 @@ export function getWhereAccountBalanceNotInFuture(): Prisma.AccountBalanceWhereI
   };
 }
 
-export function isAccountBalanceInFuture(
-  aDate: Date,
-  aEndOfToday = endOfToday()
-) {
-  return isAfter(aDate, aEndOfToday);
+export function isAccountBalanceInFuture({
+  date,
+  endOfTodayDate = endOfToday()
+}: {
+  date: Date;
+  endOfTodayDate?: Date;
+}) {
+  return isAfter(date, endOfTodayDate);
 }