From 089c27a4a879a447c8ac75d19e4f775a728a72e4 Mon Sep 17 00:00:00 2001 From: Thomas Kaul <4159106+dtslvr@users.noreply.github.com> Date: Fri, 1 Aug 2025 19:00:26 +0200 Subject: [PATCH] Refactoring --- apps/api/src/app/export/export.service.ts | 2 +- apps/api/src/app/import/import.service.ts | 2 +- apps/api/src/app/order/order.controller.ts | 4 ++-- apps/api/src/app/order/order.service.ts | 10 +++++----- apps/api/src/app/portfolio/portfolio.service.ts | 2 +- 5 files changed, 10 insertions(+), 10 deletions(-) diff --git a/apps/api/src/app/export/export.service.ts b/apps/api/src/app/export/export.service.ts index 54fd0763d..09168b505 100644 --- a/apps/api/src/app/export/export.service.ts +++ b/apps/api/src/app/export/export.service.ts @@ -41,7 +41,7 @@ export class ExportService { includeDrafts: true, sortColumn: 'date', sortDirection: 'asc', - withExcludedAccounts: true + withExcludedAccountsAndActivities: true }); if (activityIds?.length > 0) { diff --git a/apps/api/src/app/import/import.service.ts b/apps/api/src/app/import/import.service.ts index d23427616..ef9d25f53 100644 --- a/apps/api/src/app/import/import.service.ts +++ b/apps/api/src/app/import/import.service.ts @@ -533,7 +533,7 @@ export class ImportService { userCurrency, userId, includeDrafts: true, - withExcludedAccounts: true + withExcludedAccountsAndActivities: true }); return activitiesDto.map( diff --git a/apps/api/src/app/order/order.controller.ts b/apps/api/src/app/order/order.controller.ts index cd8012cec..9bd45050e 100644 --- a/apps/api/src/app/order/order.controller.ts +++ b/apps/api/src/app/order/order.controller.ts @@ -144,7 +144,7 @@ export class OrderController { skip: isNaN(skip) ? undefined : skip, take: isNaN(take) ? undefined : take, userId: impersonationUserId || this.request.user.id, - withExcludedAccounts: true + withExcludedAccountsAndActivities: true }); return { activities, count }; @@ -165,7 +165,7 @@ export class OrderController { const { activities } = await this.orderService.getOrders({ userCurrency, userId: impersonationUserId || this.request.user.id, - withExcludedAccounts: true + withExcludedAccountsAndActivities: true }); const activity = activities.find((activity) => { diff --git a/apps/api/src/app/order/order.service.ts b/apps/api/src/app/order/order.service.ts index 4151487fe..e9d72233e 100644 --- a/apps/api/src/app/order/order.service.ts +++ b/apps/api/src/app/order/order.service.ts @@ -276,7 +276,7 @@ export class OrderService { userId, includeDrafts: true, userCurrency: undefined, - withExcludedAccounts: true + withExcludedAccountsAndActivities: true }); const { count } = await this.prismaService.order.deleteMany({ @@ -333,7 +333,7 @@ export class OrderService { types, userCurrency, userId, - withExcludedAccounts = false + withExcludedAccountsAndActivities = false }: { endDate?: Date; filters?: Filter[]; @@ -346,7 +346,7 @@ export class OrderService { types?: ActivityType[]; userCurrency: string; userId: string; - withExcludedAccounts?: boolean; + withExcludedAccountsAndActivities?: boolean; }): Promise { let orderBy: Prisma.Enumerable = [ { date: 'asc' }, @@ -492,7 +492,7 @@ export class OrderService { where.type = { in: types }; } - if (withExcludedAccounts === false) { + if (withExcludedAccountsAndActivities === false) { where.OR = [ { account: null }, { account: { NOT: { isExcluded: true } } } @@ -617,7 +617,7 @@ export class OrderService { filters, userCurrency, userId, - withExcludedAccounts: false // TODO + withExcludedAccountsAndActivities: false // TODO }); } diff --git a/apps/api/src/app/portfolio/portfolio.service.ts b/apps/api/src/app/portfolio/portfolio.service.ts index fbf93b543..d1b9af892 100644 --- a/apps/api/src/app/portfolio/portfolio.service.ts +++ b/apps/api/src/app/portfolio/portfolio.service.ts @@ -1800,7 +1800,7 @@ export class PortfolioService { const { activities } = await this.orderService.getOrders({ userCurrency, userId, - withExcludedAccounts: true + withExcludedAccountsAndActivities: true }); const excludedActivities: Activity[] = [];