251125:0000 Phase 6 wait start dev Check

backend/src/modules/notification/notification.service.ts

@@ -1,4 +1,5 @@
 // File: src/modules/notification/notification.service.ts
+
 import { Injectable, Logger, NotFoundException } from '@nestjs/common';
 import { InjectQueue } from '@nestjs/bullmq';
 import { Queue } from 'bullmq';
@@ -22,9 +23,9 @@ export interface NotificationJobData {
   title: string;
   message: string;
   type: 'EMAIL' | 'LINE' | 'SYSTEM'; // ช่องทางหลักที่ต้องการส่ง (Trigger Type)
-  entityType?: string; // e.g., 'rfa', 'correspondence'
-  entityId?: number; // e.g., rfa_id
-  link?: string; // Deep link to frontend page
+  entityType?: string;
+  entityId?: number;
+  link?: string;
 }
 
 @Injectable()
@@ -37,109 +38,57 @@ export class NotificationService {
     private notificationRepo: Repository<Notification>,
     @InjectRepository(User)
     private userRepo: Repository<User>,
-    @InjectRepository(UserPreference)
-    private userPrefRepo: Repository<UserPreference>,
+    // ไม่ต้อง Inject UserPrefRepo แล้ว เพราะ Processor จะจัดการเอง
     private notificationGateway: NotificationGateway,
   ) {}
 
   /**
    * ส่งการแจ้งเตือน (Centralized Notification Sender)
-   * 1. บันทึก DB (System Log)
-   * 2. ส่ง Real-time (WebSocket)
-   * 3. ส่ง External (Email/Line) ผ่าน Queue ตาม User Preference
    */
   async send(data: NotificationJobData): Promise<void> {
     try {
       // ---------------------------------------------------------
-      // 1. สร้าง Entity และบันทึกลง DB (เพื่อให้มี History ในระบบ)
+      // 1. สร้าง Entity และบันทึกลง DB (System Log)
       // ---------------------------------------------------------
       const notification = this.notificationRepo.create({
         userId: data.userId,
         title: data.title,
         message: data.message,
-        notificationType: NotificationType.SYSTEM, // ใน DB เก็บเป็น SYSTEM เสมอเพื่อแสดงใน App
+        notificationType: NotificationType.SYSTEM,
         entityType: data.entityType,
         entityId: data.entityId,
         isRead: false,
-        // link: data.link // ถ้า Entity มี field link ให้ใส่ด้วย
       });
 
       const savedNotification = await this.notificationRepo.save(notification);
 
       // ---------------------------------------------------------
-      // 2. Real-time Push (WebSocket) -> ส่งให้ User ทันทีถ้า Online
+      // 2. Real-time Push (WebSocket)
       // ---------------------------------------------------------
       this.notificationGateway.sendToUser(data.userId, savedNotification);
 
       // ---------------------------------------------------------
-      // 3. ตรวจสอบ User Preferences เพื่อส่งช่องทางอื่น (Email/Line)
+      // 3. Push Job ลง Redis BullMQ (Dispatch Logic)
+      // เปลี่ยนชื่อ Job เป็น 'dispatch-notification' ตาม Processor
       // ---------------------------------------------------------
-      const userPref = await this.userPrefRepo.findOne({
-        where: { userId: data.userId },
-      });
-
-      // ใช้ Nullish Coalescing Operator (??)
-      // ถ้าไม่มีค่า (undefined/null) ให้ Default เป็น true
-      const shouldSendEmail = userPref?.notifyEmail ?? true;
-      const shouldSendLine = userPref?.notifyLine ?? true;
-
-      const jobs = [];
-
-      // ---------------------------------------------------------
-      // 4. เตรียม Job สำหรับ Email Queue
-      // เงื่อนไข: User เปิดรับ Email และ Noti นี้ไม่ได้บังคับส่งแค่ LINE
-      // ---------------------------------------------------------
-      if (shouldSendEmail && data.type !== 'LINE') {
-        jobs.push({
-          name: 'send-email',
-          data: {
-            ...data,
-            notificationId: savedNotification.id,
-            target: 'EMAIL',
+      await this.notificationQueue.add(
+        'dispatch-notification',
+        {
+          ...data,
+          notificationId: savedNotification.id, // ส่ง ID ไปด้วยเผื่อใช้ Tracking
+        },
+        {
+          attempts: 3,
+          backoff: {
+            type: 'exponential',
+            delay: 5000,
           },
-          opts: {
-            attempts: 3, // ลองใหม่ 3 ครั้งถ้าล่ม (Resilience)
-            backoff: {
-              type: 'exponential',
-              delay: 5000, // รอ 5s, 10s, 20s...
-            },
-            removeOnComplete: true, // ลบ Job เมื่อเสร็จ (ประหยัด Redis Memory)
-          },
-        });
-      }
+          removeOnComplete: true,
+        },
+      );
 
-      // ---------------------------------------------------------
-      // 5. เตรียม Job สำหรับ Line Queue
-      // เงื่อนไข: User เปิดรับ Line และ Noti นี้ไม่ได้บังคับส่งแค่ EMAIL
-      // ---------------------------------------------------------
-      if (shouldSendLine && data.type !== 'EMAIL') {
-        jobs.push({
-          name: 'send-line',
-          data: {
-            ...data,
-            notificationId: savedNotification.id,
-            target: 'LINE',
-          },
-          opts: {
-            attempts: 3,
-            backoff: { type: 'fixed', delay: 3000 },
-            removeOnComplete: true,
-          },
-        });
-      }
-
-      // ---------------------------------------------------------
-      // 6. Push Jobs ลง Redis BullMQ
-      // ---------------------------------------------------------
-      if (jobs.length > 0) {
-        await this.notificationQueue.addBulk(jobs);
-        this.logger.debug(
-          `Queued ${jobs.length} external notifications for user ${data.userId}`,
-        );
-      }
+      this.logger.debug(`Dispatched notification job for user ${data.userId}`);
     } catch (error) {
-      // Error Handling: ไม่ Throw เพื่อไม่ให้ Flow หลัก (เช่น การสร้างเอกสาร) พัง
-      // แต่บันทึก Error ไว้ตรวจสอบ
       this.logger.error(
         `Failed to process notification for user ${data.userId}`,
         (error as Error).stack,
@@ -147,9 +96,8 @@ export class NotificationService {
     }
   }
 
-  /**
-   * ดึงรายการแจ้งเตือนของ User (สำหรับ Controller)
-   */
+  // ... (ส่วน findAll, markAsRead, cleanupOldNotifications เหมือนเดิม ไม่ต้องแก้) ...
+
   async findAll(userId: number, searchDto: SearchNotificationDto) {
     const { page = 1, limit = 20, isRead } = searchDto;
     const skip = (page - 1) * limit;
@@ -161,14 +109,11 @@ export class NotificationService {
       .take(limit)
       .skip(skip);
 
-    // Filter by Read Status (ถ้ามีการส่งมา)
     if (isRead !== undefined) {
       queryBuilder.andWhere('notification.isRead = :isRead', { isRead });
     }
 
     const [items, total] = await queryBuilder.getManyAndCount();
-
-    // นับจำนวนที่ยังไม่ได้อ่านทั้งหมด (เพื่อแสดง Badge ที่กระดิ่ง)
     const unreadCount = await this.notificationRepo.count({
       where: { userId, isRead: false },
     });
@@ -185,9 +130,6 @@ export class NotificationService {
     };
   }
 
-  /**
-   * อ่านแจ้งเตือน (Mark as Read)
-   */
   async markAsRead(id: number, userId: number): Promise<void> {
     const notification = await this.notificationRepo.findOne({
       where: { id, userId },
@@ -200,15 +142,9 @@ export class NotificationService {
     if (!notification.isRead) {
       notification.isRead = true;
       await this.notificationRepo.save(notification);
-
-      // Update Unread Count via WebSocket (Optional)
-      // this.notificationGateway.sendUnreadCount(userId, ...);
     }
   }
 
-  /**
-   * อ่านทั้งหมด (Mark All as Read)
-   */
   async markAllAsRead(userId: number): Promise<void> {
     await this.notificationRepo.update(
       { userId, isRead: false },
@@ -216,10 +152,6 @@ export class NotificationService {
     );
   }
 
-  /**
-   * ลบการแจ้งเตือนที่เก่าเกินกำหนด (ใช้กับ Cron Job Cleanup)
-   * เก็บไว้ 90 วัน
-   */
   async cleanupOldNotifications(days: number = 90): Promise<number> {
     const dateLimit = new Date();
     dateLimit.setDate(dateLimit.getDate() - days);