Browse Source

Fix missing first data point in portfolio evolution chart

pull/8089/head
Thomas Kaul 1 week ago
parent
commit
8f9a0cd8b7
  1. 48
      apps/client/src/app/pages/portfolio/analysis/analysis-page.component.ts

48
apps/client/src/app/pages/portfolio/analysis/analysis-page.component.ts

@ -329,36 +329,30 @@ export class GfAnalysisPageComponent implements OnInit {
this.performanceDataItems = []; this.performanceDataItems = [];
this.performanceDataItemsInPercentage = []; this.performanceDataItemsInPercentage = [];
for (const [ for (const {
index, date,
{ dividendInPercentageWithCurrencyEffect,
date, netPerformanceInPercentageWithCurrencyEffect,
dividendInPercentageWithCurrencyEffect, totalInvestmentValueWithCurrencyEffect,
netPerformanceInPercentageWithCurrencyEffect, valueInPercentage,
totalInvestmentValueWithCurrencyEffect, valueWithCurrencyEffect
valueInPercentage, } of chart ?? []) {
valueWithCurrencyEffect if (totalInvestmentValueWithCurrencyEffect !== undefined) {
this.investments.push({
date,
investment: totalInvestmentValueWithCurrencyEffect
});
} }
] of (chart ?? []).entries()) {
// Ignore first item where value is 0
if (index > 0 || this.user?.settings?.dateRange === 'max') {
if (totalInvestmentValueWithCurrencyEffect !== undefined) {
this.investments.push({
date,
investment: totalInvestmentValueWithCurrencyEffect
});
}
const value = isNumber(valueWithCurrencyEffect) const value = isNumber(valueWithCurrencyEffect)
? valueWithCurrencyEffect ? valueWithCurrencyEffect
: valueInPercentage; : valueInPercentage;
if (isNumber(value)) { if (isNumber(value)) {
this.performanceDataItems.push({ this.performanceDataItems.push({
date, date,
value value
}); });
}
} }
if (isNumber(netPerformanceInPercentageWithCurrencyEffect)) { if (isNumber(netPerformanceInPercentageWithCurrencyEffect)) {

Loading…
Cancel
Save