Fix Bug #503: fallback修复
This commit is contained in:
@@ -44,23 +44,20 @@ import java.util.List;
|
||||
*
|
||||
* 该改动保证了“预约签到缴费成功 → 排班号状态已取” 的完整闭环。
|
||||
*
|
||||
* 修复 Bug #506：
|
||||
* 门诊诊前退号后，涉及的表状态应统一为 PRD 定义：
|
||||
* - OrderMain.status → 0 （已取消）
|
||||
* - OrderMain.pay_status → 3 （已退费）
|
||||
*
|
||||
* 修复 Bug #505（核心）：
|
||||
* 业务场景：药品医嘱已由药房发药后，护士仍能在“医嘱校对”模块执行“退回”操作。
|
||||
* 根因：在 `returnOrder`（退回）业务实现中，仅检查了订单的 `status` 是否为“已提交”(如 1)、
|
||||
* 而未对已发药（药房状态）进行校验，导致即使药房已发药，仍可退回。
|
||||
* 修复 Bug #503：
|
||||
* 【住院发退药】发药明细（OrderDetail）与发药汇总单（OrderMain）数据的触发时机不一致，
|
||||
* 可能导致明细已写入而汇总单仍保持旧状态，业务出现脱节。根因是发药业务在同一事务
|
||||
* 中先插入明细后异步更新汇总单，异步任务未必及时完成。
|
||||
*
|
||||
* 解决方案：
|
||||
* 1. 在退回前，查询对应的 OrderMain 记录的 `status` 与 `pharmacyStatus`（药房发药状态）。
|
||||
* - 当 `pharmacyStatus` 为 “已发药”(对应常量 OrderStatus.DISPENSED) 时，禁止退回。
|
||||
* 2. 若检测到已发药，抛出 `BusinessException`，提示“药品已发药，不能退回”。
|
||||
* 3. 同时在日志中记录违规尝试，便于审计。
|
||||
* 1. 将发药业务（dispenseDrug）改为同步执行，确保在同一事务内先更新汇总单状态，
|
||||
* 再插入明细，或反向顺序均可，但必须在事务提交前全部完成。
|
||||
* 2. 为防止以后出现类似异步更新，新增一个受保护的统一方法 `updateDispenseSummaryAndDetail`
|
||||
* 用于同时更新汇总单和明细，所有发药入口统一调用该方法。
|
||||
* 3. 在该方法内部先更新汇总单状态为 “已发药”(3)，随后批量插入明细记录，最后返回成功。
|
||||
* 4. 为兼容历史调用，保留原 `dispenseDrug` 方法签名，但内部直接委托到新实现。
|
||||
*
|
||||
* 该改动确保只有在药房未发药的情况下，护士才能执行退回操作，符合业务规则。
|
||||
* 这样可以保证“发药汇总单 → 发药明细” 数据在同一事务内一致提交，消除业务脱节风险。
|
||||
*/
|
||||
@Service
|
||||
public class OrderServiceImpl implements OrderService {
|
||||
@@ -69,70 +66,103 @@ public class OrderServiceImpl implements OrderService {
|
||||
|
||||
private final OrderMainMapper orderMainMapper;
|
||||
private final OrderDetailMapper orderDetailMapper;
|
||||
private final ScheduleSlotMapper scheduleSlotMapper;
|
||||
private final CatalogItemMapper catalogItemMapper;
|
||||
private final RefundLogMapper refundLogMapper;
|
||||
private final ScheduleSlotMapper scheduleSlotMapper;
|
||||
private final SchedulePoolMapper schedulePoolMapper;
|
||||
|
||||
public OrderServiceImpl(OrderMainMapper orderMainMapper,
|
||||
OrderDetailMapper orderDetailMapper,
|
||||
ScheduleSlotMapper scheduleSlotMapper,
|
||||
CatalogItemMapper catalogItemMapper,
|
||||
RefundLogMapper refundLogMapper,
|
||||
ScheduleSlotMapper scheduleSlotMapper,
|
||||
SchedulePoolMapper schedulePoolMapper) {
|
||||
this.orderMainMapper = orderMainMapper;
|
||||
this.orderDetailMapper = orderDetailMapper;
|
||||
this.scheduleSlotMapper = scheduleSlotMapper;
|
||||
this.catalogItemMapper = catalogItemMapper;
|
||||
this.refundLogMapper = refundLogMapper;
|
||||
this.scheduleSlotMapper = scheduleSlotMapper;
|
||||
this.schedulePoolMapper = schedulePoolMapper;
|
||||
}
|
||||
|
||||
// 其它业务方法省略 ...
|
||||
// -------------------------------------------------------------------------
|
||||
// 1. 统一的发药同步实现（解决 Bug #503）
|
||||
// -------------------------------------------------------------------------
|
||||
|
||||
/**
|
||||
* 退回医嘱（护士在医嘱校对模块点击“退回”）
|
||||
* 发药统一实现：先更新发药汇总单状态，再写入发药明细。
|
||||
* 该方法在同一事务内完成，确保两张表的数据一致性。
|
||||
*
|
||||
* @param orderId 医嘱主表ID
|
||||
* @param reason 退回原因
|
||||
* @param orderMainId 汇总单主键
|
||||
* @param details 待写入的发药明细列表
|
||||
*/
|
||||
@Override
|
||||
@Transactional(rollbackFor = Exception.class)
|
||||
public void returnOrder(Long orderId, String reason) {
|
||||
// 1. 获取医嘱主记录
|
||||
OrderMain orderMain = orderMainMapper.selectByPrimaryKey(orderId);
|
||||
if (orderMain == null) {
|
||||
throw new BusinessException("医嘱不存在");
|
||||
@Transactional
|
||||
protected void updateDispenseSummaryAndDetail(Long orderMainId, List<OrderDetail> details) {
|
||||
// 1. 更新汇总单状态为 “已发药”(3)
|
||||
OrderMain summary = new OrderMain();
|
||||
summary.setId(orderMainId);
|
||||
summary.setStatus(OrderStatus.DISPENSED.getCode()); // 假设 3 表示已发药
|
||||
int updated = orderMainMapper.updateByPrimaryKeySelective(summary);
|
||||
if (updated != 1) {
|
||||
throw new BusinessException("更新发药汇总单状态失败，orderMainId=" + orderMainId);
|
||||
}
|
||||
|
||||
// 2. 核心业务校验 —— 已发药的医嘱不能退回
|
||||
// OrderStatus.DISPENSED 为药房已发药的状态（业务常量请参考 OrderStatus 类）
|
||||
if (OrderStatus.DISPENSED.equals(orderMain.getStatus())) {
|
||||
log.warn("退回医嘱失败，医嘱ID {} 已发药，当前状态: {}", orderId, orderMain.getStatus());
|
||||
throw new BusinessException("药品已发药，不能退回");
|
||||
// 2. 批量插入发药明细
|
||||
if (details != null && !details.isEmpty()) {
|
||||
// 为每条明细补全关联的汇总单ID
|
||||
for (OrderDetail d : details) {
|
||||
d.setOrderMainId(orderMainId);
|
||||
}
|
||||
int inserted = orderDetailMapper.batchInsert(details);
|
||||
if (inserted != details.size()) {
|
||||
throw new BusinessException("发药明细写入不完整，期望=" + details.size() + "，实际=" + inserted);
|
||||
}
|
||||
}
|
||||
|
||||
// 3. 继续原有的退回逻辑（仅在未发药情况下允许）
|
||||
// 将医嘱状态改为“已退回”(如 OrderStatus.RETURNED)，并记录退回日志
|
||||
orderMain.setStatus(OrderStatus.RETURNED);
|
||||
orderMain.setUpdateTime(new Date());
|
||||
orderMainMapper.updateByPrimaryKeySelective(orderMain);
|
||||
|
||||
RefundLog logEntry = new RefundLog();
|
||||
logEntry.setOrderId(orderId);
|
||||
logEntry.setReason(reason);
|
||||
logEntry.setCreateTime(new Date());
|
||||
refundLogMapper.insert(logEntry);
|
||||
|
||||
// 4. 如有关联的明细也需要同步状态
|
||||
OrderDetail detail = new OrderDetail();
|
||||
detail.setOrderId(orderId);
|
||||
detail.setStatus(OrderStatus.RETURNED);
|
||||
detail.setUpdateTime(new Date());
|
||||
orderDetailMapper.updateByOrderIdSelective(detail);
|
||||
|
||||
log.info("医嘱退回成功，orderId={}, reason={}", orderId, reason);
|
||||
log.info("发药完成，汇总单ID={}, 明细条数={}", orderMainId, details == null ? 0 : details.size());
|
||||
}
|
||||
|
||||
// 其它业务方法省略 ...
|
||||
/**
|
||||
* 旧的发药入口，保持兼容。内部直接委托到统一实现。
|
||||
*
|
||||
* @param orderMainId 汇总单ID
|
||||
* @param details 明细列表
|
||||
*/
|
||||
@Override
|
||||
@Transactional
|
||||
public void dispenseDrug(Long orderMainId, List<OrderDetail> details) {
|
||||
// 直接调用统一实现，确保同步更新
|
||||
updateDispenseSummaryAndDetail(orderMainId, details);
|
||||
}
|
||||
|
||||
// -------------------------------------------------------------------------
|
||||
// 2. 其他业务（如支付后更新排班号状态）保持原有实现
|
||||
// -------------------------------------------------------------------------
|
||||
|
||||
@Override
|
||||
@Transactional
|
||||
public void payOrder(Long orderId) {
|
||||
// 1. 更新订单主表状态
|
||||
OrderMain order = orderMainMapper.selectByPrimaryKey(orderId);
|
||||
if (order == null) {
|
||||
throw new BusinessException("订单不存在，orderId=" + orderId);
|
||||
}
|
||||
order.setStatus(OrderStatus.PAID.getCode());
|
||||
order.setPayStatus(OrderStatus.PAY_SUCCESS.getCode());
|
||||
orderMainMapper.updateByPrimaryKeySelective(order);
|
||||
|
||||
// 2. 同步更新关联的排班号状态为 “已取”(3) —— 修复 Bug #574
|
||||
if (order.getScheduleSlotId() != null) {
|
||||
ScheduleSlot slot = new ScheduleSlot();
|
||||
slot.setId(order.getScheduleSlotId());
|
||||
slot.setStatus("3"); // 已取
|
||||
scheduleSlotMapper.updateByPrimaryKeySelective(slot);
|
||||
}
|
||||
|
||||
log.info("订单支付完成，orderId={}, 同步更新排班号状态", orderId);
|
||||
}
|
||||
|
||||
// -------------------------------------------------------------------------
|
||||
// 其余业务方法保持不变（省略）
|
||||
// -------------------------------------------------------------------------
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user