Bugfix/restore incorrect fee currency conversion (#4645)

* Restore incorrect fee currency conversion

* Update changelog

CHANGELOG.md

@@ -20,6 +20,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Renamed `Account` to `accounts` in the `Platform` database schema
 - Upgraded `prisma` from version `6.6.0` to `6.7.0`
 
+### Fixed
+
+- Fixed an issue with the fee calculations related to activities in a custom currency
+
 ## 2.159.0 - 2025-05-02
 
 ### Added

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

@@ -118,6 +118,7 @@ export class ImportService {
             createdAt: undefined,
             fee: 0,
             feeInAssetProfileCurrency: 0,
+            feeInBaseCurrency: 0,
             id: assetProfile.id,
             isDraft: false,
             SymbolProfile: assetProfile,
@@ -126,7 +127,8 @@ export class ImportService {
             unitPrice: marketPrice,
             unitPriceInAssetProfileCurrency: marketPrice,
             updatedAt: undefined,
-            userId: Account?.userId
+            userId: Account?.userId,
+            valueInBaseCurrency: value
           };
         })
       );

apps/api/src/app/order/interfaces/activities.interface.ts

@@ -12,11 +12,13 @@ export interface Activity extends Order {
   Account?: AccountWithPlatform;
   error?: ActivityError;
   feeInAssetProfileCurrency: number;
+  feeInBaseCurrency: number;
   SymbolProfile?: EnhancedSymbolProfile;
   tags?: Tag[];
   unitPriceInAssetProfileCurrency: number;
   updateAccountBalance?: boolean;
   value: number;
+  valueInBaseCurrency: number;
 }
 
 export interface ActivityError {

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

@@ -531,31 +531,46 @@ export class OrderService {
 
         const value = new Big(order.quantity).mul(order.unitPrice).toNumber();
 
-        return {
+        const [
-          ...order,
+          feeInAssetProfileCurrency,
-          value,
+          feeInBaseCurrency,
-          feeInAssetProfileCurrency:
+          unitPriceInAssetProfileCurrency,
-            await this.exchangeRateDataService.toCurrencyAtDate(
+          valueInBaseCurrency
+        ] = await Promise.all([
+          this.exchangeRateDataService.toCurrencyAtDate(
             order.fee,
             order.currency ?? order.SymbolProfile.currency,
             order.SymbolProfile.currency,
             order.date
           ),
-          SymbolProfile: assetProfile,
+          this.exchangeRateDataService.toCurrencyAtDate(
-          unitPriceInAssetProfileCurrency:
+            order.fee,
-            await this.exchangeRateDataService.toCurrencyAtDate(
+            order.currency ?? order.SymbolProfile.currency,
+            userCurrency,
+            order.date
+          ),
+          this.exchangeRateDataService.toCurrencyAtDate(
             order.unitPrice,
             order.currency ?? order.SymbolProfile.currency,
             order.SymbolProfile.currency,
             order.date
           ),
-          valueInBaseCurrency:
+          this.exchangeRateDataService.toCurrencyAtDate(
-            await this.exchangeRateDataService.toCurrencyAtDate(
             value,
             order.currency ?? order.SymbolProfile.currency,
             userCurrency,
             order.date
           )
+        ]);
+
+        return {
+          ...order,
+          feeInAssetProfileCurrency,
+          feeInBaseCurrency,
+          unitPriceInAssetProfileCurrency,
+          value,
+          valueInBaseCurrency,
+          SymbolProfile: assetProfile
         };
       })
     );

apps/api/src/app/portfolio/calculator/portfolio-calculator-test-utils.ts

@@ -8,6 +8,7 @@ export const activityDummyData = {
   currency: undefined,
   fee: undefined,
   feeInAssetProfileCurrency: undefined,
+  feeInBaseCurrency: undefined,
   id: undefined,
   isDraft: false,
   symbolProfileId: undefined,