Fix unused custom asset profiles created by activities import

apps/api/src/app/import/import.helper.ts

@@ -1,34 +0,0 @@
-import { getAssetProfileIdentifier } from '@ghostfolio/common/helper';
-import { Activity } from '@ghostfolio/common/interfaces';
-
-import { AssetProfileToCreate } from './interfaces/asset-profile-to-create.interface';
-
-/**
- * Returns the asset profiles which at least one activity of the import uses.
- * The asset profiles are created after the validation of the activities,
- * thus an asset profile of an activity which is not imported must not be
- * created.
- */
-export function getAssetProfilesToCreate({
-  activities,
-  assetProfiles
-}: {
-  activities: Pick<Activity, 'assetProfile' | 'error'>[];
-  assetProfiles: AssetProfileToCreate[];
-}) {
-  const assetProfileIdentifiersToImport = new Set(
-    activities
-      .filter(({ error }) => {
-        return !error;
-      })
-      .map(({ assetProfile }) => {
-        return getAssetProfileIdentifier(assetProfile);
-      })
-  );
-
-  return assetProfiles.filter(({ assetProfile }) => {
-    return assetProfileIdentifiersToImport.has(
-      getAssetProfileIdentifier(assetProfile)
-    );
-  });
-}

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

@@ -48,7 +48,6 @@ import { omit, uniqBy } from 'lodash';
 import { randomUUID } from 'node:crypto';
 
 import { ImportDataDto } from './import-data.dto';
-import { getAssetProfilesToCreate } from './import.helper';
 import { AssetProfileToCreate } from './interfaces/asset-profile-to-create.interface';
 
 @Injectable()
@@ -741,7 +740,7 @@ export class ImportService {
       for (const {
         assetProfile,
         marketDataObjects
-      } of getAssetProfilesToCreate({
+      } of this.getAssetProfilesToCreate({
         activities: activitiesExtendedWithErrors,
         assetProfiles: assetProfilesToCreate
       })) {
@@ -1087,6 +1086,30 @@ export class ImportService {
     return matchingAccountsOfUser[0];
   }
 
+  private getAssetProfilesToCreate({
+    activities,
+    assetProfiles
+  }: {
+    activities: Pick<Activity, 'assetProfile' | 'error'>[];
+    assetProfiles: AssetProfileToCreate[];
+  }) {
+    const assetProfileIdentifiersToImport = new Set(
+      activities
+        .filter(({ error }) => {
+          return !error;
+        })
+        .map(({ assetProfile }) => {
+          return getAssetProfileIdentifier(assetProfile);
+        })
+    );
+
+    return assetProfiles.filter(({ assetProfile }) => {
+      return assetProfileIdentifiersToImport.has(
+        getAssetProfileIdentifier(assetProfile)
+      );
+    });
+  }
+
   private isUniqueAccount(accounts: AccountWithValue[]) {
     const uniqueAccountIds = new Set<string>();