Feature/optimize summary calculation (#3088)

* Optimize calculation

* Update changelog

CHANGELOG.md

@@ -5,6 +5,12 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## Unreleased
+
+### Changed
+
+- Optimized the calculation of the portfolio summary
+
 ## 2.60.0 - 2024-03-02
 
 ### Added

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

@@ -62,6 +62,7 @@ import {
   Tag
 } from '@prisma/client';
 import Big from 'big.js';
+import { isUUID } from 'class-validator';
 import {
   differenceInDays,
   format,
@@ -617,6 +618,7 @@ export class PortfolioService {
     }
 
     const summary = await this.getSummary({
+      holdings,
       impersonationId,
       userCurrency,
       userId,
@@ -1708,12 +1710,14 @@ export class PortfolioService {
   private async getSummary({
     balanceInBaseCurrency,
     emergencyFundPositionsValueInBaseCurrency,
+    holdings,
     impersonationId,
     userCurrency,
     userId
   }: {
     balanceInBaseCurrency: number;
     emergencyFundPositionsValueInBaseCurrency: number;
+    holdings: PortfolioDetails['holdings'];
     impersonationId: string;
     userCurrency: string;
     userId: string;
@@ -1727,43 +1731,57 @@ export class PortfolioService {
     });
 
     const { activities } = await this.orderService.getOrders({
-      userCurrency,
-      userId
-    });
-
-    let { activities: excludedActivities } = await this.orderService.getOrders({
       userCurrency,
       userId,
       withExcludedAccounts: true
     });
 
-    excludedActivities = excludedActivities.filter(({ Account: account }) => {
+    const excludedActivities: Activity[] = [];
-      return account?.isExcluded ?? false;
+    const nonExcludedActivities: Activity[] = [];
-    });
+
+    for (const activity of activities) {
+      if (activity.Account?.isExcluded) {
+        excludedActivities.push(activity);
+      } else {
+        nonExcludedActivities.push(activity);
+      }
+    }
 
     const dividend = this.getSumOfActivityType({
       activities,
       userCurrency,
       activityType: 'DIVIDEND'
     }).toNumber();
+
     const emergencyFund = new Big(
       Math.max(
         emergencyFundPositionsValueInBaseCurrency,
         (user.Settings?.settings as UserSettings)?.emergencyFund ?? 0
       )
     );
+
     const fees = this.getFees({ activities, userCurrency }).toNumber();
     const firstOrderDate = activities[0]?.date;
+
     const interest = this.getSumOfActivityType({
       activities,
       userCurrency,
       activityType: 'INTEREST'
     }).toNumber();
-    const items = this.getSumOfActivityType({
+
-      activities,
+    const items = Object.keys(holdings)
-      userCurrency,
+      .filter((symbol) => {
-      activityType: 'ITEM'
+        return isUUID(symbol) && holdings[symbol].dataSource === 'MANUAL';
-    }).toNumber();
+      })
+      .map((symbol) => {
+        return holdings[symbol].valueInBaseCurrency;
+      })
+      .reduce(
+        (previous, current) => new Big(previous).plus(current),
+        new Big(0)
+      )
+      .toNumber();
+
     const liabilities = this.getSumOfActivityType({
       activities,
       userCurrency,
@@ -1771,13 +1789,14 @@ export class PortfolioService {
     }).toNumber();
 
     const totalBuy = this.getSumOfActivityType({
-      activities,
       userCurrency,
+      activities: nonExcludedActivities,
       activityType: 'BUY'
     }).toNumber();
+
     const totalSell = this.getSumOfActivityType({
-      activities,
       userCurrency,
+      activities: nonExcludedActivities,
       activityType: 'SELL'
     }).toNumber();
 
@@ -1785,7 +1804,9 @@ export class PortfolioService {
       .minus(emergencyFund)
       .plus(emergencyFundPositionsValueInBaseCurrency)
       .toNumber();
+
     const committedFunds = new Big(totalBuy).minus(totalSell);
+
     const totalOfExcludedActivities = this.getSumOfActivityType({
       userCurrency,
       activities: excludedActivities,