PM-1504 - create/edit scorecard

Author: vas3a

src/api/scorecard/scorecard.controller.ts

@@ -32,6 +32,8 @@ import { ChallengeTrack } from 'src/shared/enums/challengeTrack.enum';
 import { ScoreCardService } from './scorecard.service';
 import { PaginationHeaderInterceptor } from 'src/interceptors/PaginationHeaderInterceptor';
 import { $Enums } from '@prisma/client';
+import { User } from 'src/shared/decorators/user.decorator';
+import { JwtUser } from 'src/shared/modules/global/jwt.service';
 
 @ApiTags('Scorecard')
 @ApiBearerAuth()
@@ -55,8 +57,9 @@ export class ScorecardController {
   @ApiResponse({ status: 403, description: 'Forbidden.' })
   async addScorecard(
     @Body() body: ScorecardRequestDto,
+    @User() user: JwtUser,
   ): Promise<ScorecardWithGroupResponseDto> {
-    return await this.scorecardService.addScorecard(body);
+    return await this.scorecardService.addScorecard(body, user);
   }
 
   @Put('/:id')
@@ -81,9 +84,10 @@ export class ScorecardController {
   @ApiResponse({ status: 404, description: 'Scorecard not found.' })
   async editScorecard(
     @Param('id') id: string,
-    @Body() body: ScorecardWithGroupResponseDto,
+    @Body() body: ScorecardRequestDto,
+    @User() user: JwtUser,
   ): Promise<ScorecardWithGroupResponseDto> {
-    return await this.scorecardService.editScorecard(id, body);
+    return await this.scorecardService.editScorecard(id, body, user);
   }
 
   @Delete(':id')
@@ -249,7 +253,10 @@ export class ScorecardController {
   })
   @ApiResponse({ status: 403, description: 'Forbidden.' })
   @ApiResponse({ status: 404, description: 'Scorecard not found.' })
-  async cloneScorecard(@Param('id') id: string): Promise<ScorecardResponseDto> {
-    return this.scorecardService.cloneScorecard(id);
+  async cloneScorecard(
+    @Param('id') id: string,
+    @User() user: JwtUser,
+  ): Promise<ScorecardResponseDto> {
+    return this.scorecardService.cloneScorecard(id, user);
   }
 }

src/api/scorecard/scorecard.service.ts

@@ -15,6 +15,7 @@ import {
   ScorecardSectionBaseDto,
   ScorecardWithGroupResponseDto,
 } from 'src/dto/scorecard.dto';
+import { JwtUser } from 'src/shared/modules/global/jwt.service';
 import { PrismaService } from 'src/shared/modules/global/prisma.service';
 
 @Injectable()
@@ -28,9 +29,16 @@ export class ScoreCardService {
    */
   async addScorecard(
     body: ScorecardRequestDto,
+    user: JwtUser,
   ): Promise<ScorecardWithGroupResponseDto> {
     const data = await this.prisma.scorecard.create({
-      data: mapScorecardRequestToDto(body),
+      data: {
+        ...(mapScorecardRequestToDto({
+          ...body,
+          createdBy: user.isMachine ? 'System' : (user.userId as string),
+          updatedBy: user.isMachine ? 'System' : (user.userId as string),
+        }) as any),
+      },
       include: {
         scorecardGroups: {
           include: {
@@ -44,7 +52,7 @@ export class ScoreCardService {
       },
     });
 
-    return data as ScorecardWithGroupResponseDto;
+    return data as unknown as ScorecardWithGroupResponseDto;
   }
 
   /**
@@ -54,12 +62,17 @@ export class ScoreCardService {
    */
   async editScorecard(
     id: string,
-    body: ScorecardWithGroupResponseDto,
+    body: ScorecardRequestDto,
+    user: JwtUser,
   ): Promise<ScorecardWithGroupResponseDto> {
     const data = await this.prisma.scorecard
       .update({
         where: { id },
-        data: mapScorecardRequestToDto(body),
+        data: mapScorecardRequestToDto({
+          ...body,
+          createdBy: user.isMachine ? 'System' : (user.userId as string),
+          updatedBy: user.isMachine ? 'System' : (user.userId as string),
+        }) as any,
         include: {
           scorecardGroups: {
             include: {
@@ -81,7 +94,7 @@ export class ScoreCardService {
         });
       });
 
-    return data as ScorecardWithGroupResponseDto;
+    return data as unknown as ScorecardWithGroupResponseDto;
   }
 
   /**
@@ -203,59 +216,70 @@ export class ScoreCardService {
   }
 
   async cloneScorecard(
-    id: string
+    id: string,
+    user: { userId?: string; isMachine: boolean },
   ): Promise<ScorecardResponseDto> {
-    const original = await this.prisma.scorecard
-      .findUnique({
-        where: { id },
-        include: {
-          scorecardGroups: {
-            include: {
-              sections: {
-                include: {
-                  questions: true,
-                },
+    const original = await this.prisma.scorecard.findUnique({
+      where: { id },
+      include: {
+        scorecardGroups: {
+          include: {
+            sections: {
+              include: {
+                questions: true,
               },
             },
           },
         },
-      });
+      },
+    });
 
     if (!original) {
       throw new NotFoundException({ message: `Scorecard not found.` });
     }
 
-    // Remove id fields from nested objects for cloning
-    const cloneGroups = original.scorecardGroups.map((group: ScorecardGroupBaseDto) => ({
-      ...group,
-      id: undefined,
+    const auditFields = {
+      createdBy: user.isMachine ? 'System' : (user.userId as string),
+      updatedBy: user.isMachine ? 'System' : (user.userId as string),
       createdAt: undefined,
       updatedAt: undefined,
-      scorecardId: undefined,
-      sections: group.sections.map((section: ScorecardSectionBaseDto) => ({
-        ...section,
+    };
+
+    // Remove id fields from nested objects for cloning
+    const cloneGroups = original.scorecardGroups.map(
+      (group: ScorecardGroupBaseDto) => ({
+        ...group,
         id: undefined,
-        createdAt: undefined,
-        updatedAt: undefined,
-        scorecardGroupId: undefined,
-        questions: section.questions.map((question: ScorecardQuestionBaseDto) => ({
-          ...question,
-          id: undefined,
-          createdAt: undefined,
-          updatedAt: undefined,
-          sectionId: undefined,
-          scorecardSectionId: undefined,
-        })),
-      })),
-    }));
+        ...auditFields,
+        scorecardId: undefined,
+        sections: {
+          create: group.sections.map((section: ScorecardSectionBaseDto) => ({
+            ...section,
+            id: undefined,
+            ...auditFields,
+            scorecardGroupId: undefined,
+            questions: {
+              create: section.questions.map(
+                (question: ScorecardQuestionBaseDto) => ({
+                  ...question,
+                  id: undefined,
+                  ...auditFields,
+                  sectionId: undefined,
+                  scorecardSectionId: undefined,
+                }),
+              ),
+            },
+          })),
+        },
+      }),
+    ) as any;
 
     const clonedScorecard = await this.prisma.scorecard.create({
       data: {
         ...original,
         id: undefined,
         name: `${original.name} (Clone)`,
-        createdAt: undefined,
-        updatedAt: undefined,
+        ...auditFields,
         scorecardGroups: {
           create: cloneGroups,
         },