Phân tích luồng sau chuyển — batch/delete-multiple — vòng 1

logic-reviewer viết dựa trên code đã implement (repo C:/Projects/warehouse-service-wt/ws1), đối chiếu với code cũ. Viết tắt: backend: = easyposbackend1:src/main/java/vn/softdreams/easypos/; đường dẫn mới tính từ src/main/java/com/softdreams/easypos/warehouse/.

F-01 Xoá nhiều lô theo danh sách

• Điểm vào: cũ backend:web/rest/BatchesManagementResource.java:60-64 → mới presentation/web/batch/BatchController.java:96-104
• Quy tắc: R-01, R-03, R-04, R-05, R-06, R-07

Sơ đồ luồng mới (theo code thực tế)

sequenceDiagram
    participant C as Client
    participant BC as BatchController.deleteMultiple
    participant M as BatchWebMapper.toDeleteMultipleCommand
    participant S as BatchManagementService.deleteMultiple (@Transactional)
    participant R as BatchRelationalRepository.deleteAllWithoutDetails (translate)
    participant W as BatchWriter.deleteAllWithoutDetails
    C->>BC: POST BatchDeleteMultipleRequest (không validate)
    BC->>M: request
    M-->>BC: Command(ids, paramCheckAll, BatchFilter)
    BC->>S: command
    S->>S: paramCheckAll khác TRUE → ids = command.ids()
    S->>S: ids null/rỗng → return 0
    S->>R: deleteAllWithoutDetails(ids)
    R->>W: DELETE b FROM batches b WHERE b.id IN (:ids) AND NOT EXISTS(batches_detail)
    W-->>S: n (PersistenceException → SystemException EXCEPTION_ERROR)
    S->>S: n != ids.size() → BadRequestException LOT_HAS_TRANSACTIONS_CANNOT_DELETE (rollback)
    BC-->>C: 200 ok(n, DELETE_BATCHES_SUCCESS)

Đối chiếu từng bước

Dữ liệu và side effect

Transaction, khoá, idempotency, tenant

• Ranh giới transaction: cũ @Transactional cấp class BatchesManagementServiceImpl (:55-57) → mới @Transactional trên BatchManagementService.deleteMultiple (:105), gọi từ controller qua proxy; ngoại lệ unchecked → rollback cả câu xoá.
• Khoá / cập nhật nguyên tử: một câu DELETE duy nhất ở cả hai bên; không đọc-sửa-ghi.
• Idempotency, event: không áp dụng (API, không phát event). Gọi lại cùng ids sau khi đã xoá → 0 ≠ size → 400 như cũ.
• Lọc comId: không có ở câu xoá ở cả hai bên (D-01).

Lệch so với luồng mới đã duyệt (spec mục 6)

Không (port tên deleteAllWithoutDetails thay cho deleteWithoutDetails(List) — chỉ đổi tên).

Dev cần chú ý khi review

F-02 Xoá mọi lô khớp bộ lọc

• Điểm vào: cũ backend:web/rest/BatchesManagementResource.java:60-64 → mới presentation/web/batch/BatchController.java:96-104
• Quy tắc: R-01, R-02, R-03, R-04, R-05, R-06, R-07

Sơ đồ luồng mới (theo code thực tế)

sequenceDiagram
    participant C as Client
    participant BC as BatchController.deleteMultiple
    participant S as BatchManagementService.deleteMultiple (@Transactional)
    participant R as BatchRelationalRepository
    participant F as BatchFilterReader.findIds
    participant K as BatchKeyword.likePattern
    participant W as BatchWriter.deleteAllWithoutDetails
    C->>BC: POST {comId, paramCheckAll:true, keyword, status, inventoryQuantityStatus, ...}
    BC->>S: Command(ids bị bỏ qua, TRUE, BatchFilter(comId, keyword, status, invQtyStatus))
    S->>R: findIdsByFilter(filter)
    R->>F: findIds(filter)
    F->>K: likePattern(keyword) (null/"" → không lọc)
    F->>F: SELECT b.id FROM batches b ⟕ batches_detail ⟕ ppu WHERE com_id=:comId(null→"") [+LIKE][+status] GROUP BY [+HAVING]
    F-->>S: ids (PersistenceException → SystemException EXCEPTION_ERROR)
    S->>S: rỗng → return 0
    S->>R: deleteAllWithoutDetails(ids)
    R->>W: DELETE ... IN (:ids) AND NOT EXISTS(...)
    S->>S: n != ids.size() → 400 LOT_HAS_TRANSACTIONS_CANNOT_DELETE (rollback)
    BC-->>C: 200 ok(n, DELETE_BATCHES_SUCCESS)

Đối chiếu từng bước

Dữ liệu và side effect

Transaction, khoá, idempotency, tenant

• Ranh giới transaction: cũ cả lọc + xoá trong transaction cấp class (nguồn đọc lọc chưa chắc — D-08) → mới cả hai trong BatchManagementService.deleteMultiple @Transactional (readOnly=false).
• Khoá / cập nhật nguyên tử: SELECT id rồi DELETE; lô được thêm chi tiết giữa hai câu → NOT EXISTS loại ra → 400 rollback, như cũ.
• Idempotency, event: không áp dụng.
• Lọc comId: theo comId body, không đối chiếu token (D-01); câu xoá không lọc com_id.

Lệch so với luồng mới đã duyệt (spec mục 6)

Không.

Dev cần chú ý khi review