From 0cd70fdc359fc51ed1f9fe8bda38b554cdeb65e3 Mon Sep 17 00:00:00 2001
From: Thomas Kaul <4159106+dtslvr@users.noreply.github.com>
Date: Sun, 6 Sep 2026 09:18:56 +0200
Subject: [PATCH] Improve handling of excluded holdings

---
 .../calculator/portfolio-calculator.factory.ts     | 14 +++++++-------
 .../portfolio/calculator/portfolio-calculator.ts   |  8 ++++----
 .../roai/portfolio-calculator-valuable.spec.ts     |  4 ++--
 3 files changed, 13 insertions(+), 13 deletions(-)

diff --git a/apps/api/src/app/portfolio/calculator/portfolio-calculator.factory.ts b/apps/api/src/app/portfolio/calculator/portfolio-calculator.factory.ts
index effb185c3..7d95988c1 100644
--- a/apps/api/src/app/portfolio/calculator/portfolio-calculator.factory.ts
+++ b/apps/api/src/app/portfolio/calculator/portfolio-calculator.factory.ts
@@ -34,16 +34,16 @@ export class PortfolioCalculatorFactory {
     calculationType,
     currency,
     filters = [],
-    userId,
-    usePortfolioSnapshotCache = true
+    usePortfolioSnapshotCache = true,
+    userId
   }: {
     accountBalanceItems?: HistoricalDataItem[];
     activities: Activity[];
     calculationType: PerformanceCalculationType;
     currency: string;
     filters?: Filter[];
-    userId: string;
     usePortfolioSnapshotCache?: boolean;
+    userId: string;
   }): PortfolioCalculator {
     switch (calculationType) {
       case PerformanceCalculationType.MWR:
@@ -52,8 +52,8 @@ export class PortfolioCalculatorFactory {
           activities,
           currency,
           filters,
-          userId,
           usePortfolioSnapshotCache,
+          userId,
           configurationService: this.configurationService,
           currentRateService: this.currentRateService,
           exchangeRateDataService: this.exchangeRateDataService,
@@ -67,8 +67,8 @@ export class PortfolioCalculatorFactory {
           activities,
           currency,
           filters,
-          userId,
           usePortfolioSnapshotCache,
+          userId,
           configurationService: this.configurationService,
           currentRateService: this.currentRateService,
           exchangeRateDataService: this.exchangeRateDataService,
@@ -82,8 +82,8 @@ export class PortfolioCalculatorFactory {
           activities,
           currency,
           filters,
-          userId,
           usePortfolioSnapshotCache,
+          userId,
           configurationService: this.configurationService,
           currentRateService: this.currentRateService,
           exchangeRateDataService: this.exchangeRateDataService,
@@ -97,8 +97,8 @@ export class PortfolioCalculatorFactory {
           activities,
           currency,
           filters,
-          userId,
           usePortfolioSnapshotCache,
+          userId,
           configurationService: this.configurationService,
           currentRateService: this.currentRateService,
           exchangeRateDataService: this.exchangeRateDataService,
diff --git a/apps/api/src/app/portfolio/calculator/portfolio-calculator.ts b/apps/api/src/app/portfolio/calculator/portfolio-calculator.ts
index 13c2d64fb..37d45d191 100644
--- a/apps/api/src/app/portfolio/calculator/portfolio-calculator.ts
+++ b/apps/api/src/app/portfolio/calculator/portfolio-calculator.ts
@@ -90,8 +90,8 @@ export abstract class PortfolioCalculator {
   private snapshotPromise: Promise<void>;
   private startDate: Date;
   private transactionPoints: TransactionPoint[];
-  private userId: string;
   private usePortfolioSnapshotCache: boolean;
+  private userId: string;
 
   public constructor({
     accountBalanceItems,
@@ -103,8 +103,8 @@ export abstract class PortfolioCalculator {
     filters,
     portfolioSnapshotService,
     redisCacheService,
-    userId,
-    usePortfolioSnapshotCache = true
+    usePortfolioSnapshotCache = true,
+    userId
   }: {
     accountBalanceItems: HistoricalDataItem[];
     activities: Activity[];
@@ -115,8 +115,8 @@ export abstract class PortfolioCalculator {
     filters: Filter[];
     portfolioSnapshotService: PortfolioSnapshotService;
     redisCacheService: RedisCacheService;
-    userId: string;
     usePortfolioSnapshotCache?: boolean;
+    userId: string;
   }) {
     this.accountBalanceItems = accountBalanceItems;
     this.configurationService = configurationService;
diff --git a/apps/api/src/app/portfolio/calculator/roai/portfolio-calculator-valuable.spec.ts b/apps/api/src/app/portfolio/calculator/roai/portfolio-calculator-valuable.spec.ts
index 23d3d2a8c..3681bc754 100644
--- a/apps/api/src/app/portfolio/calculator/roai/portfolio-calculator-valuable.spec.ts
+++ b/apps/api/src/app/portfolio/calculator/roai/portfolio-calculator-valuable.spec.ts
@@ -108,8 +108,8 @@ describe('PortfolioCalculator', () => {
         activities,
         calculationType: PerformanceCalculationType.ROAI,
         currency: 'USD',
-        userId: userDummyData.id,
-        usePortfolioSnapshotCache: false
+        usePortfolioSnapshotCache: false,
+        userId: userDummyData.id
       });
 
       const portfolioSnapshot = await portfolioCalculator.getSnapshot();