Phân tích luồng sau chuyển — warehouse/update — vòng 1

F-01 Sửa kho

• Điểm vào: cũ easyposbackend1:src/main/java/vn/softdreams/easypos/web/rest/WarehouseResource.java:59-67 → mới src/main/java/com/softdreams/easypos/warehouse/presentation/web/warehouse/WarehouseController.java:69-80
• Quy tắc: R-01, R-02, R-03, R-04, R-05, R-06, R-07, R-08, R-09, R-10, R-11, R-12, R-13, R-14

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

sequenceDiagram
    participant C as Client
    participant WC as WarehouseController.updateWarehouse
    participant G as Guard.validateInput
    participant WS as WarehouseManagementService.saveWarehouse (@Transactional)
    participant CC as CurrentCompanyService.requireCompanyOfCurrentUser
    participant WR as WarehouseRelationalRepository
    participant W as Warehouse
    participant DP as WarehouseDuplicatePolicy.validate
    participant BP as BillRelationalAdapter.countProcessingBills
    C->>WC: PUT /update (WarehouseSaveRequest)
    WC->>WC: request.id()==null → BadRequestException(ID_NOT_NULL)
    WC->>G: validateInput(validator, request) → INVALID_REQUEST (mọi vi phạm)
    WC->>WS: saveWarehouse(mapper.toSaveCommand(request))
    WS->>CC: requireCompanyOfCurrentUser(comId) → INVALID_COMPANY / COMPANY_NOT_OF_CURRENT_USER / 401
    WS->>WR: lockCompanyWarehouses(comId) (SELECT TOP 1 + sp_getapplock)
    WS->>WS: WarehouseAudit(findUserId() or 0, now(clock))
    WS->>WR: findById(comId,id) → không có: SystemException(WAREHOUSE_NOT_FOUND)
    WS->>W: update(command, audit): copyFrom, status=activeStatus, so businessValues; đổi → cleanText + updater/updateTime
    WS->>WR: findNameSkus(comId) (native, không ORDER BY)
    WS->>WS: duplicateRows: thay dòng id==warehouse.id bằng warehouse.nameSku()
    WS->>DP: validate(rows, command) (SKU trước tên, chạm 2 → lỗi)
    WS->>W: validateDefaultWarehouse() (code null/sellable null → SystemException EXCEPTION_ERROR)
    opt !warehouse.isActive()
        WS->>BP: countProcessingBills(comId, command.id) > 0 → BadRequestException(WAREHOUSE_STATUS_FALSE_CHECK_BILL, name)
    end
    WS->>WR: save(warehouse): findByComIdAndId → mapper.apply(entity, warehouse) → UPDATE lúc commit (@DynamicUpdate, chỉ khi dirty)
    WR-->>WS: WarehouseResult (toResult(entity))
    WS-->>WC: WarehouseResult
    WC-->>C: 200 BaseResponseDTO.ok(WarehouseResponse, SUCCESS_UPDATE)

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

Dữ liệu và side effect

Transaction, khoá, idempotency, tenant

• Ranh giới transaction: cũ dùng @Transactional cấp class trên BaseWarehouseServiceImpl (easyposbase:…/BaseWarehouseServiceImpl.java:66), kiểm id và validate nằm ngoài, ở controller. Mới dùng @Transactional trên WarehouseManagementService.saveWarehouse (WarehouseManagementService.java:75), gọi từ controller qua proxy, không có self-invocation. Kiểm id và validate cũng ở controller, ngoài transaction. Khớp.
• Khoá / cập nhật nguyên tử: cũ không khoá; khi entity dirty thì khoá X dòng của chính kho từ lúc flush (trước câu kiểm trùng). Mới khoá applock theo công ty trước khi tải kho; khoá X dòng chỉ có lúc commit. Request của service chạy lần lượt; giữa monolith và service không có bảo vệ (D-05). Đã xét thứ tự khoá monolith ↔ service, không thấy vòng chờ mới.
• Idempotency, thời điểm phát event: không có event. PUT lặp lại cùng body thì businessValues không đổi → không làm sạch, không gán audit, Hibernate không UPDATE.
• Lọc comId: có ở findByComIdAndId, findNameSkusByComId, câu COUNT đơn, câu SELECT trong khoá và tên tài nguyên khoá. comId dùng là giá trị đã kiểm theo token.

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

Không. Code chỉ có thêm WarehouseAudit dựng sau khoá và save tải lại entity trước mapper.apply, cả hai là chi tiết hiện thực của phần tái sử dụng, không đổi thứ tự nghiệp vụ.

Dev cần chú ý khi review