Phân tích luồng sau chuyển — warehouse/get-with-paging — vòng 2

F-01 Danh sách kho theo trang

• Điểm vào: cũ easyposbackend1:src/main/java/vn/softdreams/easypos/web/rest/WarehouseResource.java:69-78 → mới src/main/java/com/softdreams/easypos/warehouse/presentation/web/warehouse/WarehouseController.java:119-139
• Quy tắc: R-01 … R-11

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

sequenceDiagram
    participant C as Client
    participant WC as WarehouseController.getWithPaging
    participant QS as WarehouseQueryService.getWarehousePage (@Transactional, không readOnly)
    participant CC as CurrentCompanyService.requireCompanyOfCurrentUser
    participant WR as WarehouseRelationalRepository.findPage
    participant IQ as InventoryQueryService.countActiveProductsByWarehouses
    participant IR as InventoryRelationalRepository.countActiveProductsByWarehouses
    C->>WC: GET ?page&size&comId&keyword&status&countProduct (Pageable + @RequestParam comId + record filter)
    WC->>QS: WarehousePageQuery(comId, keyword, status, countProduct, pageable.getOffset(), pageSize)
    QS->>CC: requireCompanyOfCurrentUser(comId) (null/<=0 → INVALID_COMPANY; khác token → COMPANY_NOT_OF_CURRENT_USER)
    QS->>QS: keywordPattern: null/"" → null, còn lại "%"+WarehouseText.normalizedName([kw])+"%"
    QS->>WR: findPage(WarehousePageFilter)
    WR->>WR: native SELECT COUNT(id) FROM warehouse WHERE com_id [+normalized_name LIKE][+status =]
    alt count > 0
        WR->>WR: (int) offset; âm → SystemException(EXCEPTION_ERROR)
        WR->>WR: native SELECT id,…,is_sellable … setFirstResult/setMaxResults, không ORDER BY
    end
    WR->>WR: pageTotal theo quy tắc PageImpl (long); PersistenceException → SystemException(EXCEPTION_ERROR)
    WR-->>QS: WarehousePage(items countProduct=null, total)
    opt items không rỗng và Boolean.TRUE.equals(countProduct)
        QS->>IQ: countActiveProductsByWarehouses(comId, ids)
        IQ->>IR: native COUNT(DISTINCT product_id) … GROUP BY warehouse_id → Map
        QS->>QS: item.withCountProduct(counts.get(id)) (null → 0)
    end
    WC-->>C: 200 BaseResponseDTO.count(total).ok(items, SUCCESS_GET_LIST) (serialize bằng ObjectMapper NON_EMPTY của starter)

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

Dữ liệu và side effect

Transaction, khoá, idempotency, tenant

• Ranh giới transaction: cũ là @Transactional cấp class của BaseWarehouseServiceImpl (LF-33), đọc DB chính. Mới là @Transactional không readOnly ở WarehouseQueryService.getWarehousePage (gọi từ controller qua proxy, không tự gọi nội bộ), định tuyến về primary (D-06).
• Khoá / cập nhật nguyên tử: không có ở cả hai (chỉ đọc).
• Idempotency, thời điểm phát event: không có event, không ghi.
• Lọc comId: mọi câu (COUNT, SELECT trang, đếm SP ở cả inventory và product) đều có comId. comId dùng trong truy vấn là giá trị đã kiểm với token (D-01).
• N+1: không có. Mỗi request tối đa 3 câu, giống code cũ.

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

Không lệch về thứ tự bước. Chỉ lệch ở cách bind tham số lọc (record thay cho @RequestParam, F-logic-4) và cách serialize response (F-logic-1, do hạ tầng starter).

Dev cần chú ý khi review

Vòng 2 (thu hẹp): chỉ dòng 12 viết lại theo logic-reviewer vòng 2; các bước khác, dữ liệu, transaction không đổi so với vòng 1. Dev chú ý thêm: WarehouseListItemResponse.java:13 — ALWAYS ở mức class là chỗ duy nhất giữ key null; gỡ đi thì mất key (TC-01-EXTRA-1 khoá).