From 412053e9291e6c91eee925f840425202843d5eec Mon Sep 17 00:00:00 2001
From: Thomas Kaul <4159106+dtslvr@users.noreply.github.com>
Date: Sat, 11 May 2024 09:30:08 +0200
Subject: [PATCH] Refactoring

---
 .../app/portfolio/calculator/portfolio-calculator.ts   | 10 +++++-----
 apps/api/src/app/portfolio/portfolio.service.ts        |  6 +++---
 2 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/apps/api/src/app/portfolio/calculator/portfolio-calculator.ts b/apps/api/src/app/portfolio/calculator/portfolio-calculator.ts
index 4428799b4..8e4049692 100644
--- a/apps/api/src/app/portfolio/calculator/portfolio-calculator.ts
+++ b/apps/api/src/app/portfolio/calculator/portfolio-calculator.ts
@@ -1099,10 +1099,10 @@ export abstract class PortfolioCalculator {
 
     const newChartData: HistoricalDataItem[] = [];
 
-    let netPerformanceAtStartDate;
-    let netPerformanceWithCurrencyEffectAtStartDate;
-    let netPerformanceInPercentageWithCurrencyEffectAtStartDate;
-    let totalInvestmentValuesWithCurrencyEffect = [];
+    let netPerformanceAtStartDate: number;
+    let netPerformanceWithCurrencyEffectAtStartDate: number;
+    let netPerformanceInPercentageWithCurrencyEffectAtStartDate: number;
+    let totalInvestmentValuesWithCurrencyEffect: number[] = [];
 
     for (let historicalDataItem of chartData) {
       if (
@@ -1158,7 +1158,7 @@ export abstract class PortfolioCalculator {
       }
     }
 
-    return newChartData;
+    return { chart: newChartData };
   }
 
   public getStartDate() {
diff --git a/apps/api/src/app/portfolio/portfolio.service.ts b/apps/api/src/app/portfolio/portfolio.service.ts
index 688ea9cf6..ad83dba88 100644
--- a/apps/api/src/app/portfolio/portfolio.service.ts
+++ b/apps/api/src/app/portfolio/portfolio.service.ts
@@ -70,7 +70,7 @@ import {
   parseISO,
   set
 } from 'date-fns';
-import { isEmpty, isNumber, last, uniq, uniqBy } from 'lodash';
+import { isEmpty, isNumber, uniq, uniqBy } from 'lodash';
 
 import { PortfolioCalculator } from './calculator/portfolio-calculator';
 import {
@@ -1230,15 +1230,15 @@ export class PortfolioService {
 
     console.timeEnd('------ PortfolioService.getPerformance');
 
-    const newChartData = await portfolioCalculator.getPerformance({
+    const { chart } = await portfolioCalculator.getPerformance({
       end: endDate,
       start: startDate
     });
 
     return {
+      chart,
       errors,
       hasErrors,
-      chart: newChartData,
       firstOrderDate: parseDate(chartData[0]?.date),
       performance: {
         currentNetWorth,