Phân tích luồng sau chuyển — rsinoutward/get-all-transactions — vòng 2 (thu hẹp)

logic-reviewer viết dựa trên code đã implement, đối chiếu với code cũ. Vòng 2: fix.diff chỉ sửa test (RsInoutwardTransactionQueryServiceTest — phần count R-08/R-14 thuộc F-01; RsInoutwardTransactionRelationalRepositoryTest — câu tổng R-11 thuộc F-02). Code src/main không đổi; đã mở lại các dòng trích dẫn bên dưới và xác nhận số dòng như vòng 1. Đường dẫn mới tính từ C:/Projects/warehouse-service-wt/ws1/src/main/java/com/softdreams/easypos/warehouse/. backend: = easyposbackend1:src/main/java/vn/softdreams/easypos/.

F-01 Kiểm đầu vào, dựng điều kiện, đếm + đọc trang

• Điểm vào: cũ backend:web/rest/RsInoutWardManagementResource.java:66-93 → mới presentation/web/rsinoutward/RsInoutwardQueryController.java:47-72
• Quy tắc: R-01..R-09, R-14..R-17 (vòng này đối chiếu lại R-08, R-14)

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

sequenceDiagram
    participant Ctl as RsInoutwardQueryController
    participant S as RsInoutwardTransactionQueryService.getAllTransactions (readOnly tx)
    participant R as RsInoutwardTransactionRelationalRepository.findPage
    participant B as RsInoutwardTransactionSqlBuilder.pageSql
    Ctl->>S: RsInoutwardTransactionQuery(offset, pageSize)
    S->>S: checkCompany, checkFormat x2, reasonOf, findPpuIds, TransactionDateRange.of
    S->>R: findPage(filter, offset, pageSize)
    R->>B: pageSql (COUNT + SELECT, ORDER BY r.norm_date DESC, r.id DESC)
    R->>R: COUNT(DISTINCT r.id); 0 → trang rỗng, total 0
    R->>R: setFirstResult((int) offset), setMaxResults(size); total = PageImpl(rows, PageRequest(offset/size,size), total)
    S->>S: count = (int) total
    alt trang rỗng
        S-->>Ctl: Result(reason, count, null, null, null, ZERO, null)
    else có dòng
        S->>S: F-02 (countAll = (long) count), F-03
        S-->>Ctl: Result(reason, count, comId, fromDate, toDate, totals|ZERO, items)
    end

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

Dữ liệu và side effect

Transaction, khoá, idempotency, tenant

• Không đổi so với vòng 1: một transaction readOnly ở service (RsInoutwardTransactionQueryService.java:75), chỉ đọc, 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

F-02 Bổ sung dòng và tổng (trang có dòng)

• Điểm vào: cũ backend:service/impl/RsInoutWardManagementServiceImpl.java:488-543 → mới application/rsinoutward/RsInoutwardTransactionQueryService.java:109-124
• Quy tắc: R-10, R-11 (vòng này đối chiếu lại R-11)

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

sequenceDiagram
    participant S as RsInoutwardTransactionQueryService
    participant R as RsInoutwardTransactionRelationalRepository
    participant B as RsInoutwardTransactionSqlBuilder.totalsSql
    S->>S: loadLookups (business type, customer code2, đếm SP; countAll = (long) count)
    S->>R: sumTotals(filter)
    R->>B: TOTALS_HEAD + " AND " + quý/ngày + type/keyword/paymentStatus + " ) r "
    B->>B: nếu cần chi tiết: JOIN r2 ON r2.rs_inoutward_id = r.id + ppu + r2.from_warehouse_id + " and r2.batch_id"
    R-->>S: totals | null (không có dòng)
    S->>S: totals == null → ZERO

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

Dữ liệu và side effect

Transaction, khoá, idempotency, tenant

• Không đổi so với vòng 1: cùng transaction readOnly, chỉ đọc; câu tổng lọc com_id trong subquery.

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

Không.

Dev cần chú ý khi review

F-03 Liên thông dược (Mongo + Redis) — không đổi so với vòng 1: xem verify/round-1/flow-analysis.md