设计人员连续保存多个内部图纸版本(A9/A10/A11)后,物料详情或打印页面仍展示较旧版本。所有入口应选择语义上的最高版本 A11,而非按字符串排序。
// InternalVersionHelper.java — 图纸版本排序逻辑
// 问题:A9、A10、A11 的字符串 compareTo 顺序 ≠ 业务版本顺序
public static String pickLatestByVersion(List<String> versions) {
return versions.stream()
.max(String::compareTo) // BUG: "A9" > "A11" 字符串序
.orElse(null);
}
// BizMaterialListServiceImpl.java — 物料详情图纸
private String pickLatestDrawing(List<BizDrawing> drawings) {
return drawings.stream()
.map(BizDrawing::getInternalVersion)
.max(String::compareTo)
.orElse(null);
}
// BizProductionInfoPrintServiceImpl.java — 打印图纸
private String getDrawingVersionForPrint(String designTaskId) {
List<BizDrawing> list = drawingMapper.selectByTaskId(designTaskId);
if (list.isEmpty()) return "";
list.sort((a, b) -> a.getInternalVersion().compareTo(b.getInternalVersion()));
return list.get(list.size() - 1).getInternalVersion();
}
审批请求已有客户图纸,设计单保存内部图纸后,再补充客户图纸时部分图纸从页面消失。客户图纸和内部图纸应合并展示,而非互相覆盖。
// BizApprovalRequestServiceImpl.java
public DrawingAggregation getDrawingAggregation(String requestNo, String designTaskId) {
List<BizDrawing> externalDrawings = drawingMapper.selectByRequestNo(requestNo);
List<BizDrawing> internalDrawings = drawingMapper.selectByDesignTaskId(designTaskId);
// BUG: 直接覆盖而非合并
// 后补外部图纸时重建 payload,已有的内部图纸查询被覆盖
DrawingAggregation result = new DrawingAggregation();
if (CollectionUtils.isNotEmpty(externalDrawings)) {
result.setDrawings(externalDrawings); // 整列表替换
}
if (CollectionUtils.isNotEmpty(internalDrawings)) {
result.setDrawings(internalDrawings); // 再次替换,外部图纸丢失
}
return result;
}
同一审批请求下存在多个设计单时,各自生产文件中的物料行可能混入另一个设计单的数据。requestNo 聚合主表,但子表删除重建未按 designTaskId 精准限定。
// BizProductionInfoCreateServiceImpl.java
public void refreshProductionInfo(String requestNo, List<String> designTaskIds) {
// BUG: 主表按 requestNo 聚合,子表删除重建时只按 productionInfoId
// 多设计单场景下,子表删除范围过大,重建时只取第一个 designTaskId 的数据
BizProductionInfo info = productionInfoMapper.selectByRequestNo(requestNo);
if (info == null) {
info = buildNewProductionInfo(requestNo);
}
// 删除所有子快照,不限 designTaskId
productionInfoItemMapper.deleteByInfoId(info.getId());
// 重建时取所有物料行,未按 designTaskId 隔离
List<BizMaterialListItem> allItems =
materialListItemMapper.selectByRequestNo(requestNo);
// ...
}
// BizDesignTaskMapper.xml — SQL 层孤儿归并
// customerDrawingUrls 子查询中 d.design_task_id IS NULL
// 的图纸也被归入当前设计单,导致跨单串扰