Fix Bug #571: fallback修复

com/openhis/web/outpatient/controller/LabApplyController.java

@@ -0,0 +1,48 @@
+package com.openhis.web.outpatient.controller;
+
+import com.openhis.web.outpatient.service.impl.LabApplyServiceImpl;
+import org.springframework.web.bind.annotation.*;
+
+import java.util.HashMap;
+import java.util.Map;
+
+/**
+ * 检验申请（实验室）控制层
+ *
+ * 修复 Bug #571：为撤回接口返回统一的成功/错误结构，并捕获业务异常以返回前端可读的错误信息。
+ */
+@RestController
+@RequestMapping("/api/lab-apply")
+public class LabApplyController {
+
+    private final LabApplyServiceImpl labApplyService;
+
+    public LabApplyController(LabApplyServiceImpl labApplyService) {
+        this.labApplyService = labApplyService;
+    }
+
+    /**
+     * 撤回检验申请
+     *
+     * @param applyId 检验申请 ID
+     * @return {code:0, msg:"撤回成功"} 或 {code:1, msg:"错误信息"}
+     */
+    @PostMapping("/withdraw")
+    public Map<String, Object> withdraw(@RequestParam Long applyId) {
+        Map<String, Object> resp = new HashMap<>();
+        try {
+            labApplyService.withdrawApply(applyId);
+            resp.put("code", 0);
+            resp.put("msg", "撤回成功");
+        } catch (RuntimeException ex) {
+            resp.put("code", 1);
+            resp.put("msg", ex.getMessage());
+        } catch (Exception ex) {
+            resp.put("code", 1);
+            resp.put("msg", "系统异常，请联系管理员");
+        }
+        return resp;
+    }
+
+    // 其他接口保持不变
+}

com/openhis/web/outpatient/mapper/LabApplyMapper.java

@@ -0,0 +1,84 @@
+package com.openhis.web.outpatient.mapper;
+
+import org.apache.ibatis.annotations.Mapper;
+import org.apache.ibatis.annotations.Param;
+import org.apache.ibatis.annotations.Select;
+import org.apache.ibatis.annotations.Update;
+
+import java.util.List;
+import java.util.Map;
+
+/**
+ * 检验申请（实验室）数据访问层
+ *
+ * 修复 Bug #571：
+ *   在检验申请执行“撤回”操作时，原实现直接调用 {@link #updateStatus(Long, String)} 并硬编码
+ *   为 'RETURNED'，导致前端提示 “撤回失败，请检查状态”。实际业务中撤回应将状态改为
+ *   PRD 中统一定义的 “CANCELLED”。同时需要在撤回前校验当前状态只能是 “APPLIED”(已申请) 或
+ *   “PENDING”(待处理)，否则抛出明确异常，前端可捕获并展示友好提示。
+ *
+ * 为此做了以下改动：
+ *   1. 新增常量 {@link #STATUS_CANCELLED}，统一使用 PRD 中的取消状态码。
+ *   2. 新增方法 {@link #withdrawLabApply(Long)}，在内部完成状态合法性校验并将状态更新为
+ *      {@link #STATUS_CANCELLED}。
+ *   3. 将原有的 {@code updateStatus} 方法的 Javadoc 说明为通用状态更新，供内部使用。
+ */
+@Mapper
+public interface LabApplyMapper {
+
+    /** 检验申请已撤回（取消）状态 */
+    String STATUS_CANCELLED = "CANCELLED";
+
+    /** 检验申请已申请状态（可撤回） */
+    String STATUS_APPLIED = "APPLIED";
+
+    /** 检验申请待处理状态（可撤回） */
+    String STATUS_PENDING = "PENDING";
+
+    /**
+     * 根据 ID 查询检验申请的完整信息（用于状态校验）。
+     *
+     * @param applyId 检验申请主键
+     * @return 包含所有字段的 Map，若不存在返回 null
+     */
+    @Select("SELECT * FROM his_lab_apply WHERE id = #{applyId}")
+    Map<String, Object> selectApplyById(@Param("applyId") Long applyId);
+
+    /**
+     * 通用状态更新（内部使用）。
+     *
+     * @param applyId 检验申请主键
+     * @param status  新状态码
+     */
+    @Update("UPDATE his_lab_apply SET status = #{status}, update_time = NOW() WHERE id = #{applyId}")
+    void updateStatus(@Param("applyId") Long applyId, @Param("status") String status);
+
+    /**
+     * 撤回检验申请。
+     *
+     * <p>业务规则：
+     * <ul>
+     *   <li>仅当当前状态为 {@link #STATUS_APPLIED} 或 {@link #STATUS_PENDING} 时允许撤回。</li>
+     *   <li>撤回后状态统一设为 {@link #STATUS_CANCELLED}。</li>
+     *   <li>若状态不符合要求，抛出 RuntimeException，前端可捕获并展示错误提示。</li>
+     * </ul>
+     *
+     * @param applyId 检验申请主键
+     */
+    default void withdrawLabApply(Long applyId) {
+        Map<String, Object> apply = selectApplyById(applyId);
+        if (apply == null) {
+            throw new RuntimeException("检验申请不存在");
+        }
+        String currentStatus = (String) apply.get("status");
+        if (!STATUS_APPLIED.equals(currentStatus) && !STATUS_PENDING.equals(currentStatus)) {
+            throw new RuntimeException("仅在已申请或待处理状态下才能撤回，当前状态为 " + currentStatus);
+        }
+        // 更新为取消状态
+        updateStatus(applyId, STATUS_CANCELLED);
+    }
+
+    // 其他已有查询方法保持不变
+    @Select("SELECT id, patient_id, item_name, status, apply_time FROM his_lab_apply WHERE patient_id = #{patientId}")
+    List<Map<String, Object>> selectByPatientId(@Param("patientId") Long patientId);
+}

com/openhis/web/outpatient/service/impl/LabApplyServiceImpl.java

@@ -0,0 +1,42 @@
+package com.openhis.web.outpatient.service.impl;
+
+import com.openhis.web.outpatient.mapper.LabApplyMapper;
+import org.springframework.stereotype.Service;
+import org.springframework.transaction.annotation.Transactional;
+
+/**
+ * 检验申请业务实现
+ *
+ * 修复 Bug #571：
+ *   原来的撤回实现直接调用 {@code updateStatus(applyId, "RETURNED")}，状态码与 PRD 不匹配，
+ *   并且缺少对当前状态的校验，导致在已执行、已报告等状态下仍能撤回，引发系统异常。
+ *
+ *   现在通过调用 {@link LabApplyMapper#withdrawLabApply(Long)} 完成撤回，确保：
+ *   <ul>
+ *     <li>仅在可撤回的状态（APPLIED、PENDING）下执行。</li>
+ *     <li>撤回后统一使用 PRD 定义的 CANCELLED 状态。</li>
+ *     <li>异常信息更加友好，前端可直接展示。</li>
+ *   </ul>
+ */
+@Service
+public class LabApplyServiceImpl {
+
+    private final LabApplyMapper labApplyMapper;
+
+    public LabApplyServiceImpl(LabApplyMapper labApplyMapper) {
+        this.labApplyMapper = labApplyMapper;
+    }
+
+    /**
+     * 撤回检验申请。
+     *
+     * @param applyId 检验申请主键
+     */
+    @Transactional(rollbackFor = Exception.class)
+    public void withdrawApply(Long applyId) {
+        // LabApplyMapper 已经在内部完成状态校验并抛出异常
+        labApplyMapper.withdrawLabApply(applyId);
+    }
+
+    // 其余业务方法保持不变
+}