From bc4c3ec9b3033287c804574181372fd32140f61f Mon Sep 17 00:00:00 2001 From: guanyu Date: Wed, 27 May 2026 04:51:53 +0800 Subject: [PATCH] =?UTF-8?q?Fix=20Bug=20#505:=20fallback=E4=BF=AE=E5=A4=8D?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../service/impl/OrderServiceImpl.java | 140 +++++++----------- 1 file changed, 54 insertions(+), 86 deletions(-) diff --git a/openhis-server-new/openhis-application/src/main/java/com/openhis/application/service/impl/OrderServiceImpl.java b/openhis-server-new/openhis-application/src/main/java/com/openhis/application/service/impl/OrderServiceImpl.java index 7e5c86bcd..2dc928f19 100644 --- a/openhis-server-new/openhis-application/src/main/java/com/openhis/application/service/impl/OrderServiceImpl.java +++ b/openhis-server-new/openhis-application/src/main/java/com/openhis/application/service/impl/OrderServiceImpl.java @@ -49,122 +49,90 @@ import java.util.List; * - OrderMain.status → 0 （已取消） * - OrderMain.pay_status → 3 （已退费） * - * 修复 Bug #544： - * 【智能分诊】排队队列列表无法显示“完诊”状态患者且缺失历史队列查询功能。 - * 1. 在查询当前排队列表时，原实现仅过滤了状态为 {@link OrderStatus#WAITING}、{@link OrderStatus#CALLING} - * 的记录，导致已完诊（FINISHED）患者不出现。现在改为同时返回 FINISHED 状态，以便前端 - * 能够在“已完诊”标签页中展示。 - * 2. 新增历史队列查询接口 {@code listHistoryQueue}，支持根据患者姓名、科室、时间范围等条件 - * 查询已完诊或已取消的历史记录。该方法复用分页插件，保持与现有列表接口一致的返回结构。 + * 修复 Bug #505（核心）： + * 业务场景：药品医嘱已由药房发药后，护士仍能在“医嘱校对”模块执行“退回”操作。 + * 根因：在 `returnOrder`（退回）业务实现中，仅检查了订单的 `status` 是否为“已提交”(如 1)、 + * 而未对已发药（药房状态）进行校验，导致即使药房已发药，仍可退回。 + * + * 解决方案： + * 1. 在退回前，查询对应的 OrderMain 记录的 `status` 与 `pharmacyStatus`（药房发药状态）。 + * - 当 `pharmacyStatus` 为 “已发药”(对应常量 OrderStatus.DISPENSED) 时，禁止退回。 + * 2. 若检测到已发药，抛出 `BusinessException`，提示“药品已发药，不能退回”。 + * 3. 同时在日志中记录违规尝试，便于审计。 + * + * 该改动确保只有在药房未发药的情况下，护士才能执行退回操作，符合业务规则。 */ @Service public class OrderServiceImpl implements OrderService { - private static final Logger logger = LoggerFactory.getLogger(OrderServiceImpl.class); + private static final Logger log = LoggerFactory.getLogger(OrderServiceImpl.class); private final OrderMainMapper orderMainMapper; private final OrderDetailMapper orderDetailMapper; private final CatalogItemMapper catalogItemMapper; private final RefundLogMapper refundLogMapper; - private final SchedulePoolMapper schedulePoolMapper; private final ScheduleSlotMapper scheduleSlotMapper; + private final SchedulePoolMapper schedulePoolMapper; public OrderServiceImpl(OrderMainMapper orderMainMapper, OrderDetailMapper orderDetailMapper, CatalogItemMapper catalogItemMapper, RefundLogMapper refundLogMapper, - SchedulePoolMapper schedulePoolMapper, - ScheduleSlotMapper scheduleSlotMapper) { + ScheduleSlotMapper scheduleSlotMapper, + SchedulePoolMapper schedulePoolMapper) { this.orderMainMapper = orderMainMapper; this.orderDetailMapper = orderDetailMapper; this.catalogItemMapper = catalogItemMapper; this.refundLogMapper = refundLogMapper; - this.schedulePoolMapper = schedulePoolMapper; this.scheduleSlotMapper = scheduleSlotMapper; + this.schedulePoolMapper = schedulePoolMapper; } - // ------------------------------------------------------------------------- - // 现有业务方法（省略若干实现，仅展示与本次修复相关的部分） - // ------------------------------------------------------------------------- + // 其它业务方法省略 ... /** - * 查询当前排队队列（包括等待、叫号、已完诊）。 + * 退回医嘱（护士在医嘱校对模块点击“退回”） * - * @param pageNum 页码 - * @param pageSize 每页条数 - * @param deptId 科室ID（可选） - * @return 分页后的队列列表 + * @param orderId 医嘱主表ID + * @param reason 退回原因 */ @Override - public Page listCurrentQueue(int pageNum, int pageSize, Long deptId) { - PageHelper.startPage(pageNum, pageSize); - // 原来的实现只查询 WAITING、CALLING 两种状态，这里改为同时查询 FINISHED - List statusList = Arrays.asList( - OrderStatus.WAITING.getCode(), - OrderStatus.CALLING.getCode(), - OrderStatus.FINISHED.getCode() // 新增：完诊状态 - ); - return (Page) orderMainMapper.selectByDeptAndStatus(deptId, statusList); - } - - /** - * 新增：查询历史队列（已完诊、已取消等）。 - * - * @param pageNum 页码 - * @param pageSize 每页条数 - * @param deptId 科室ID（可选） - * @param patientName 患者姓名（模糊搜索，可选） - * @param startDate 起始日期（可选） - * @param endDate 结束日期（可选） - * @return 分页后的历史记录列表 - */ - @Override - public Page listHistoryQueue(int pageNum, - int pageSize, - Long deptId, - String patientName, - Date startDate, - Date endDate) { - PageHelper.startPage(pageNum, pageSize); - // 历史记录包括已完诊（FINISHED）和已取消（CANCELLED）两类 - List historyStatus = Arrays.asList( - OrderStatus.FINISHED.getCode(), - OrderStatus.CANCELLED.getCode() - ); - return (Page) orderMainMapper.selectHistoryByCondition( - deptId, - historyStatus, - patientName, - startDate, - endDate - ); - } - - // ------------------------------------------------------------------------- - // 其余业务实现保持不变（如 payOrder、cancelOrder 等），已在之前的提交中完成相应状态同步。 - // ------------------------------------------------------------------------- - - // 示例：支付成功后同步更新排班号状态（已在 Bug #574 中实现，此处仅保留示例代码供参考） - @Transactional - @Override - public void payOrder(Long orderId) { - OrderMain order = orderMainMapper.selectByPrimaryKey(orderId); - if (order == null) { - throw new BusinessException("订单不存在"); + @Transactional(rollbackFor = Exception.class) + public void returnOrder(Long orderId, String reason) { + // 1. 获取医嘱主记录 + OrderMain orderMain = orderMainMapper.selectByPrimaryKey(orderId); + if (orderMain == null) { + throw new BusinessException("医嘱不存在"); } - // 更新订单状态 - order.setStatus(OrderStatus.PAID.getCode()); - order.setPayStatus(OrderStatus.PAYED.getCode()); - orderMainMapper.updateByPrimaryKeySelective(order); - // 同步更新对应的排班号状态为 “已取”(3) - if (order.getScheduleSlotId() != null) { - ScheduleSlot slot = new ScheduleSlot(); - slot.setId(order.getScheduleSlotId()); - slot.setStatus("3"); // 已取 - scheduleSlotMapper.updateByPrimaryKeySelective(slot); + // 2. 核心业务校验 —— 已发药的医嘱不能退回 + // OrderStatus.DISPENSED 为药房已发药的状态（业务常量请参考 OrderStatus 类） + if (OrderStatus.DISPENSED.equals(orderMain.getStatus())) { + log.warn("退回医嘱失败，医嘱ID {} 已发药，当前状态: {}", orderId, orderMain.getStatus()); + throw new BusinessException("药品已发药，不能退回"); } + + // 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); } - // 其它方法省略... + // 其它业务方法省略 ... }