Browse Source

Refactoring

pull/1635/head
Thomas 3 years ago
parent
commit
0770b9923d
  1. 2
      apps/api/src/app/account/create-account.dto.ts
  2. 6
      apps/api/src/app/import/import.controller.ts
  3. 16
      apps/api/src/app/import/import.service.ts

2
apps/api/src/app/account/create-account.dto.ts

@ -19,7 +19,7 @@ export class CreateAccountDto {
@IsOptional()
@IsString()
id: string;
id?: string;
@IsBoolean()
@IsOptional()

6
apps/api/src/app/import/import.controller.ts

@ -41,11 +41,7 @@ export class ImportController {
): Promise<ImportResponse> {
if (
!this.configurationService.get('ENABLE_FEATURE_IMPORT') ||
(importData.accounts?.length > 0 &&
!hasPermission(
this.request.user.permissions,
permissions.createAccount
))
!hasPermission(this.request.user.permissions, permissions.createAccount)
) {
throw new HttpException(
getReasonPhrase(StatusCodes.FORBIDDEN),

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

@ -200,9 +200,9 @@ export class ImportService {
);
if (isDryRun) {
accountsDto.forEach((accountsDto) =>
accounts.push({ id: accountsDto.id, name: accountsDto.name })
);
accountsDto.forEach(({ id, name }) => {
accounts.push({ id, name });
});
}
const activities: Activity[] = [];
@ -220,9 +220,9 @@ export class ImportService {
unitPrice
} of activitiesDto) {
const date = parseISO(<string>(<unknown>dateString));
const validatedAccount = accounts.find(
(account) => account.id === accountId
);
const validatedAccount = accounts.find(({ id }) => {
return id === accountId;
});
let order:
| OrderWithAccount
@ -239,7 +239,7 @@ export class ImportService {
type,
unitPrice,
userId,
accountId: validatedAccount.id,
accountId: validatedAccount?.id,
accountUserId: undefined,
createdAt: new Date(),
id: uuidv4(),
@ -275,7 +275,7 @@ export class ImportService {
type,
unitPrice,
userId,
accountId: validatedAccount.id,
accountId: validatedAccount?.id,
SymbolProfile: {
connectOrCreate: {
create: {

Loading…
Cancel
Save