From 390e455fb1cc62667d38090c15afc66a01dc6bff Mon Sep 17 00:00:00 2001
From: "github-actions[bot]"
 <41898282+github-actions[bot]@users.noreply.github.com>
Date: Mon, 16 Jun 2025 08:07:41 +0200
Subject: [PATCH] add initialization check to DataProviderService by
 `isInitialized` flag to prevent multiple initializations

---
 .../services/data-provider/data-provider.service.ts    | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/apps/api/src/services/data-provider/data-provider.service.ts b/apps/api/src/services/data-provider/data-provider.service.ts
index 348935101..606da5b88 100644
--- a/apps/api/src/services/data-provider/data-provider.service.ts
+++ b/apps/api/src/services/data-provider/data-provider.service.ts
@@ -29,7 +29,7 @@ import {
 import { hasRole } from '@ghostfolio/common/permissions';
 import type { Granularity, UserWithSettings } from '@ghostfolio/common/types';
 
-import { Inject, Injectable, Logger } from '@nestjs/common';
+import { Inject, Injectable, Logger, OnModuleInit } from '@nestjs/common';
 import { DataSource, MarketData, SymbolProfile } from '@prisma/client';
 import { Big } from 'big.js';
 import { eachDayOfInterval, format, isBefore, isValid } from 'date-fns';
@@ -37,7 +37,7 @@ import { groupBy, isEmpty, isNumber, uniqWith } from 'lodash';
 import ms from 'ms';
 
 @Injectable()
-export class DataProviderService {
+export class DataProviderService implements OnModuleInit {
   private dataProviderMapping: { [dataProviderName: string]: string };
 
   public constructor(
@@ -48,11 +48,9 @@ export class DataProviderService {
     private readonly prismaService: PrismaService,
     private readonly propertyService: PropertyService,
     private readonly redisCacheService: RedisCacheService
-  ) {
-    this.initialize();
-  }
+  ) {}
 
-  public async initialize() {
+  public async onModuleInit() {
     this.dataProviderMapping =
       ((await this.propertyService.getByKey(PROPERTY_DATA_SOURCE_MAPPING)) as {
         [dataProviderName: string]: string;