feat: add option to add customerNumber Id on customer create.

SparkoSol · Jan 14, 2025 · 674a30b · 674a30b
1 parent fe7a003
commit 674a30b
Show file tree

Hide file tree

Showing 4 changed files with 32 additions and 14 deletions.
diff --git a/src/modules/auth/auth.service.ts b/src/modules/auth/auth.service.ts
@@ -1,4 +1,5 @@
 import {
+    ConflictException,
     forwardRef,
     HttpStatus,
     Inject,
@@ -146,7 +147,17 @@ export class AuthService {
 
         data.role = role._id.toString();
 
-        data.odooCustomerId = await this.personService.getLastOdooCustomerId();
+        if (data.odooCustomerId) {
+            const customer = await this.personService.findOneByQuery({
+                odooCustomerId: data.odooCustomerId,
+            });
+            if (customer)
+                throw new ConflictException(
+                  'Customer with the same customer number already exist in system!'
+                );
+        } else {
+            data.odooCustomerId = await this.personService.getLastOdooCustomerId();
+        }
 
         data.session = new Date();
 

diff --git a/src/modules/auth/dto/sign-up-request.dto.ts b/src/modules/auth/dto/sign-up-request.dto.ts
@@ -1,5 +1,5 @@
 import { ApiProperty } from '@nestjs/swagger';
-import { IsISO8601, IsMongoId, IsNotEmpty, IsOptional, IsString } from 'class-validator';
+import { IsISO8601, IsMongoId, IsNotEmpty, IsNumber, IsOptional, IsString } from 'class-validator';
 
 export class AdminCreateUserRequest {
     @ApiProperty()
@@ -67,6 +67,9 @@ export class MobileSignUpRequest {
 
     role?: string;
 
+    @ApiProperty()
+    @IsOptional()
+    @IsNumber()
     odooCustomerId?: number;
 
     @ApiProperty()

diff --git a/src/modules/person/dto/person.dto.ts b/src/modules/person/dto/person.dto.ts
@@ -78,6 +78,9 @@ export class PersonCreateDto {
     @IsNumber()
     customerNumber?: number;
 
+    @ApiProperty()
+    @IsOptional()
+    @IsNumber()
     odooCustomerId?: number;
 
     @ApiProperty()
@@ -346,7 +349,6 @@ export class PersonFiltersDto {
     @ApiProperty({ description: 'Filter object' })
     @IsOptional()
     @IsObject()
-    // filters?: Record<string, any>;
     filters?: filterPayload;
 
     @ApiProperty({ description: 'PopulatedFilter object' })

diff --git a/src/modules/person/person.controller.ts b/src/modules/person/person.controller.ts
@@ -1,22 +1,18 @@
 import {
     Body,
+    ConflictException,
     Controller,
     Delete,
     Get,
     Param,
     Patch,
     Post,
     Query,
-    Res,
-    UploadedFile,
-    UseGuards,
-    UseInterceptors,
 } from '@nestjs/common';
 import {
     ApiBadRequestResponse,
     ApiBearerAuth,
     ApiBody,
-    ApiConsumes,
     ApiInternalServerErrorResponse,
     ApiNotAcceptableResponse,
     ApiNotFoundResponse,
@@ -27,8 +23,6 @@ import {
     ApiUnauthorizedResponse,
 } from '@nestjs/swagger';
 import {
-    BulkUploadDto,
-    BulkUploadResponseDto,
     PaginatedPersonResponseDto,
     PasswordUpdateRequestDto,
     PersonCreateDto,
@@ -39,9 +33,6 @@ import {
     UpdateFcmTokenRequestDto,
 } from './dto/person.dto';
 import { PersonService } from './person.service';
-import { FileInterceptor } from '@nestjs/platform-express';
-import { AuthGuard } from '@nestjs/passport';
-import { Response } from 'express';
 
 @ApiBearerAuth('access-token')
 @ApiTags('Person')
@@ -63,7 +54,18 @@ export class PersonController {
     })
     @Post()
     async create(@Body() data: PersonCreateDto): Promise<any> {
-        data.odooCustomerId = await this.service.getLastOdooCustomerId();
+        if (data.odooCustomerId) {
+            const customer = await this.service.findOneByQuery({
+                odooCustomerId: data.odooCustomerId,
+            });
+            if (customer)
+                throw new ConflictException(
+                    'Customer with the same customer number already exist in system!'
+                );
+        } else {
+            data.odooCustomerId = await this.service.getLastOdooCustomerId();
+        }
+
         return await this.service.create(data);
     }