690329:1250 Fixing bugs uuid by Kimi K2.5 #04

backend/src/modules/master/dto/create-tag.dto.ts

@@ -5,7 +5,7 @@ export class CreateTagDto {
   @ApiProperty({ example: 'URGENT', description: 'ชื่อ Tag' })
   @IsString()
   @IsNotEmpty()
-  tag_name!: string; // เพิ่ม !
+  tagName!: string;
 
   @ApiProperty({ example: 'คำอธิบาย', description: 'คำอธิบาย' })
   @IsString()
@@ -19,7 +19,7 @@ export class CreateTagDto {
   })
   @IsString()
   @IsOptional()
-  color_code?: string;
+  colorCode?: string;
 
   @ApiProperty({
     example: 1,
@@ -27,5 +27,5 @@ export class CreateTagDto {
     required: false,
   })
   @IsOptional()
-  project_id?: number | string;
+  projectId?: number | string;
 }

backend/src/modules/master/dto/search-tag.dto.ts

@@ -2,16 +2,17 @@
 
 import { IsString, IsOptional, IsInt, Min } from 'class-validator';
 import { Type } from 'class-transformer';
-import { ApiPropertyOptional } from '@nestjs/swagger';
+import { ApiPropertyOptional, ApiProperty } from '@nestjs/swagger';
 
 export class SearchTagDto {
-  @ApiPropertyOptional({
-    description: 'ID โครงการ (ใช้กรอง Tag ของแต่ละโปรเจกต์)',
+  @ApiProperty({
+    example: 1,
+    description:
+      'Project ID or UUID (ใช้กรอง Tag ของแต่ละโปรเจกต์) - ADR-019: Accept UUID',
+    required: false,
   })
   @IsOptional()
-  @Type(() => Number)
-  @IsInt()
-  project_id?: number;
+  projectId?: number | string;
 
   @ApiPropertyOptional({ description: 'คำค้นหา (ชื่อ Tag หรือ คำอธิบาย)' })
   @IsOptional()

backend/src/modules/master/master.service.ts

@@ -303,25 +303,24 @@ export class MasterService {
       .createQueryBuilder('tag')
       .leftJoinAndSelect('tag.project', 'project');
 
-    if (query?.project_id) {
-      // In Tags, we use project_id (INT) directly or resolve if UUID passed via query
+    if (query?.projectId) {
       const internalId = await this.uuidResolver.resolveProjectId(
-        query.project_id
+        query.projectId
       );
-      qb.andWhere('tag.project_id = :projectId', {
+      qb.andWhere('tag.projectId = :projectId', {
         projectId: internalId,
       });
     }
 
     if (query?.search) {
       qb.andWhere(
-        '(tag.tag_name LIKE :search OR tag.description LIKE :search)',
+        '(tag.tagName LIKE :search OR tag.description LIKE :search)',
         {
           search: `%${query.search}%`,
         }
       );
     }
-    qb.orderBy('tag.tag_name', 'ASC');
+    qb.orderBy('tag.tagName', 'ASC');
     if (query?.page && query?.limit) {
       const page = query.page;
       const limit = query.limit;
@@ -342,11 +341,13 @@ export class MasterService {
   }
 
   async createTag(dto: CreateTagDto, userId: number) {
-    const internalProjectId = dto.project_id
-      ? await this.uuidResolver.resolveProjectId(dto.project_id)
+    const internalProjectId = dto.projectId
+      ? await this.uuidResolver.resolveProjectId(dto.projectId)
       : null;
     const tag = this.tagRepo.create({
-      ...dto,
+      tagName: dto.tagName,
+      colorCode: dto.colorCode,
+      description: dto.description,
       projectId: internalProjectId,
       createdBy: userId,
     });
@@ -355,10 +356,18 @@ export class MasterService {
 
   async updateTag(id: number, dto: UpdateTagDto) {
     const tag = await this.findOneTag(id);
-    if (dto.project_id) {
-      dto.project_id = await this.uuidResolver.resolveProjectId(dto.project_id);
+    let internalProjectId = dto.projectId;
+    if (dto.projectId) {
+      internalProjectId = await this.uuidResolver.resolveProjectId(
+        dto.projectId
+      );
     }
-    Object.assign(tag, dto);
+    Object.assign(tag, {
+      tagName: dto.tagName,
+      colorCode: dto.colorCode,
+      description: dto.description,
+      projectId: internalProjectId,
+    });
     return this.tagRepo.save(tag);
   }