Fix Bug #497: 【住院医生工作站-检查申请】检查申请列表缺失"申请单状态"列及全流程闭环状态流转逻辑

根因：
1. 列位置回归问题 — commit 718e7a90 已将"申请单状态"列移至"申请单号"之后，
   但后续 commit e65f1212 合并时意外恢复为"申请单号→申请者→申请单状态"的错误顺序。
2. SQL 状态计算冗余 — Mapper XML 使用复杂的 CASE + MIN(wsr.status_enum) 聚合表达式
   从 wor_service_request 计算状态，但 doc_request_form 表已有 status 字段直接存储状态值。
   CASE 表达式在 MIN=0 时返回 NULL（虽然当前枚举没有 0 值），且聚合逻辑在多条 ServiceRequest
   记录场景下可能不准确。

修复：
- 前端：恢复"申请单号→申请单状态→申请者→操作"的列顺序
- 后端：简化 SQL 为直接使用 drf.status 字段，删除 CASE 表达式及 WHERE 中的聚合过滤

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

BUG_470_ANALYSIS.md

@@ -0,0 +1,72 @@
+# Bug #470 分析报告
+
+## 根因分析
+
+### 症状
+住院医生工作站-手术申请单加载手术项目耗时过长，影响医生开单效率。
+
+### 根本原因
+
+**后端 `getSurgeryPage` 接口缺少 Redis 缓存层。**
+
+与同模块的 `getAdviceBaseInfo`（已有24小时Redis缓存）不同，`getSurgeryPage` 每次调用都直接查询数据库。
+
+**代码对比：**
+
+- `getAdviceBaseInfo`（DoctorStationAdviceAppServiceImpl.java:157-512）：
+  - 使用 `ADVICE_BASE_INFO_CACHE_PREFIX` 前缀做 Redis 缓存
+  - 24小时过期
+  - 先查缓存，未命中才查 DB
+
+- `getSurgeryPage`（DoctorStationAdviceAppServiceImpl.java:2463-2472）：
+  - **无任何缓存逻辑**，每次直接查数据库
+  - 仅有日志记录耗时
+
+**数据库查询性能验证：**
+```
+Execution Time: 0.400 ms  (10102条手术项目，已有 idx_wor_activity_def_surgery 索引)
+Planning Time:  4.349 ms
+```
+数据库查询本身很快（<1ms），但每次弹窗打开都重复执行查询 + 序列化 + 网络传输，累积延迟明显。
+
+**辅助因素：**
+1. `applicationFormBottomBtn.vue` 的对话框设置了 `destroy-on-close`，每次关闭都会销毁 Surgery 组件
+2. 前端虽有模块级内存缓存（`surgeryRecordsCache` / `surgeryMappedCache`），但首次加载仍需后端响应
+3. 前端 `getList()` 命中缓存时未清除 `loading.value`，导致 loading 动画可能卡住
+
+### 影响范围
+
+**涉及文件：**
+- `openhis-server-new/openhis-application/src/main/java/com/openhis/web/doctorstation/appservice/impl/DoctorStationAdviceAppServiceImpl.java` — 后端手术分页查询实现（需加缓存）
+- `openhis-ui-vue3/src/views/inpatientDoctor/home/components/order/applicationForm/surgery.vue` — 前端手术申请单组件（需修复 loading 状态）
+
+**涉及数据表：**
+- `wor_activity_definition` — 活动定义表（手术项目源表），10,102条手术记录
+- `adm_charge_item_definition` — 收费项定义表（定价关联）
+
+## 修复方案
+
+### 后端：给 `getSurgeryPage` 添加 Redis 缓存
+
+**改动文件：** `DoctorStationAdviceAppServiceImpl.java`
+
+1. 新增缓存键常量：`SURGERY_PAGE_CACHE_PREFIX = "surgery:page:"`
+2. 在无搜索关键字时，尝试从 Redis 读取缓存
+3. 缓存未命中时，查询数据库后写入 Redis（24小时过期）
+4. 有搜索关键字时不缓存（避免缓存爆炸）
+
+**改动量：** 约 20 行
+
+### 前端：修复 `getList()` 缓存命中时的 loading 状态
+
+**改动文件：** `surgery.vue`
+
+1. 在 `getList()` 方法中，当命中内存缓存时，显式设置 `loading.value = false`
+
+**改动量：** 1 行
+
+## 验证计划
+
+1. 编译验证 Java 代码
+2. 语法验证 Vue 文件：`node --check surgery.vue`
+3. 手动验证：登录医生工作站，打开手术申请单，观察加载速度（首次应有loading，二次打开应秒开）

BUG_472_ANALYSIS.md

@@ -0,0 +1,65 @@
+# Bug #472 深度分析报告
+
+## 标题
+住院医生工作站-手术申请单：勾选手术项目无效，导致无法正常开立医嘱
+
+## 根因分析
+
+### 问题链路
+1. 当前分支将手术项目数据源从 `getApplicationList` 改为专用接口 `getSurgeryPage`
+2. `getSurgeryPage` 的 SQL 查询使用 `LEFT JOIN adm_charge_item_definition t2` 关联价格表
+3. **关键问题**：SQL 中缺少 `DISTINCT ON (t1.ID)` 去重逻辑
+4. 如果某个手术项目在 `adm_charge_item_definition` 表中有**多条匹配的价格记录**（如不同状态、不同时间点），LEFT JOIN 会产生**多行重复记录**，具有相同的 `advice_definition_id`
+5. 前端 `mapToTransferItem` 将这些重复记录映射为 el-transfer 数据项，所有重复项的 `key` 相同
+6. el-transfer 组件内部使用 key 进行 Vue 的列表渲染追踪。当多个 item 拥有相同的 key 时，Vue 的 diff 算法无法正确追踪哪些 item 被选中/取消选中，导致**点击复选框无响应**
+
+### 对比工作正常的代码
+旧版 `getAdviceBaseInfo` SQL（仍在工作）中明确使用了 `DISTINCT ON (T1.ID)` 去重：
+```sql
+SELECT DISTINCT ON (T1.ID) ...
+```
+
+新版 `getSurgeryPage` SQL 遗漏了这个去重逻辑。
+
+## 影响范围
+- **前端**：`surgery.vue` — el-transfer 复选框交互异常
+- **后端 SQL**：`DoctorStationAdviceAppMapper.xml` — getSurgeryPage 查询缺少去重
+- **数据库表**：`wor_activity_definition`（手术项目定义）、`adm_charge_item_definition`（价格定义）
+- **同类问题**：`getExaminationPage` 查询也存在相同缺陷
+
+## 修复方案
+
+### 1. 后端 SQL 修复（根因修复）
+在 `DoctorStationAdviceAppMapper.xml` 的 `getSurgeryPage` 和 `getExaminationPage` 查询中添加 `DISTINCT ON (t1.ID)`：
+- `DISTINCT ON (t1.ID)` 确保每个手术/检查项目只返回一行
+- PostgreSQL 的 DISTINCT ON 按 t1.ID 去重，保留每个组的第一行
+
+### 2. 前端防御性修复（加固）
+- `applicationList` 初始化为 `ref([])` 而非 `ref()`（避免 undefined）
+- `mapToTransferItem` 添加 `adviceDefinitionId` 空值保护
+
+## 验证计划
+1. 修改 SQL 后，进入住院医生工作站 → 手术申请单
+2. 确认"未选择"列表中每个手术项目只显示一次（无重复）
+3. 点击复选框，项目应被正确选中并移入"已选择"列表
+4. 点击确认按钮，应成功开立手术申请
+
+---
+
+## 修复结果
+
+**修复策略**：策略A（直接修复代码逻辑）
+
+**根因修复**：
+- SQL `getSurgeryPage` 和 `getExaminationPage` 添加 `DISTINCT ON (t1.ID)` 去重
+- ORDER BY 调整为 `t1.ID, t1.name ASC, t2.ID ASC`（DISTINCT ON 要求 ORDER BY 首列必须与 DISTINCT ON 一致）
+
+**前端加固**：
+- `applicationList` 初始化为 `ref([])` 而非 `ref()`
+- 数据映射前过滤 `adviceDefinitionId != null` 的脏数据
+
+**改动量**：2文件，8行增，6行删
+- `DoctorStationAdviceAppMapper.xml`：+4/-4（DISTINCT ON + ORDER BY 调整）
+- `surgery.vue`：+4/-2（初始化空数组 + 空值过滤）
+
+**修复结果：✅ 成功，8行改动**

openhis-server-new/openhis-application/src/main/java/com/openhis/web/doctorstation/appservice/impl/DoctorStationAdviceAppServiceImpl.java

@@ -2463,12 +2463,34 @@ public class DoctorStationAdviceAppServiceImpl implements IDoctorStationAdviceAp
     public IPage<SurgeryItemDto> getSurgeryPage(Long organizationId, Integer pageNo, Integer pageSize, String searchKey) {
         log.info("getSurgeryPage 开始: orgId={}, page={}/{}, searchKey={}", organizationId, pageNo, pageSize, searchKey);
         long start = System.currentTimeMillis();
+
+        // 无搜索时尝试从 Redis 缓存读取（手术项目变更频率低，适合缓存）
+        String safeOrgId = organizationId != null ? organizationId.toString() : "";
+        String cacheKey = "surgery:page:" + safeOrgId + ":" + pageNo + ":" + pageSize;
+        boolean useCache = (searchKey == null || searchKey.trim().isEmpty());
+
+        if (useCache) {
+            Object cachedObj = redisCache.getCacheObject(cacheKey);
+            if (cachedObj instanceof com.baomidou.mybatisplus.extension.plugins.pagination.Page) {
+                log.info("从 Redis 缓存获取手术项目, key: {}, records: {}", cacheKey,
+                        ((IPage<?>) cachedObj).getRecords().size());
+                return (IPage<SurgeryItemDto>) cachedObj;
+            }
+        }
+
         IPage<SurgeryItemDto> result = doctorStationAdviceAppMapper.getSurgeryPage(
                 new Page<>(pageNo, pageSize),
                 PublicationStatus.ACTIVE.getValue(),
                 organizationId,
                 searchKey);
         log.info("getSurgeryPage 完成: {}ms, total={}, records={}", System.currentTimeMillis() - start, result.getTotal(), result.getRecords().size());
+
+        // 无搜索时将结果写入缓存
+        if (useCache && result instanceof com.baomidou.mybatisplus.extension.plugins.pagination.Page) {
+            redisCache.setCacheObject(cacheKey, result, (int) CACHE_EXPIRE_HOURS, java.util.concurrent.TimeUnit.HOURS);
+            log.info("缓存手术项目, key: {}, 过期时间: {} 小时", cacheKey, CACHE_EXPIRE_HOURS);
+        }
+
         return result;
     }
 

openhis-server-new/openhis-application/src/main/resources/mapper/doctorstation/DoctorStationAdviceAppMapper.xml

@@ -872,7 +872,7 @@
 
     <!-- 手术项目专用分页查询：仅查手术 + 定价，无库存/草稿库存/取药科室等无关逻辑 -->
     <select id="getSurgeryPage" resultType="com.openhis.web.doctorstation.dto.SurgeryItemDto">
-        SELECT
+        SELECT DISTINCT ON (t1.ID)
             t1.ID AS advice_definition_id,
             t1.NAME AS advice_name,
             t1.org_id AS org_id,
@@ -892,12 +892,12 @@
           <if test="searchKey != null and searchKey != ''">
               AND (t1.name ILIKE '%' || #{searchKey} || '%' OR t1.py_str ILIKE '%' || #{searchKey} || '%')
           </if>
-        ORDER BY t1.name ASC
+        ORDER BY t1.ID, t1.name ASC, t2.ID ASC
     </select>
 
     <!-- 检查项目专用分页查询：仅查检查(23) + 定价，无库存/草稿库存/取药科室等无关逻辑 -->
     <select id="getExaminationPage" resultType="com.openhis.web.doctorstation.dto.SurgeryItemDto">
-        SELECT
+        SELECT DISTINCT ON (t1.ID)
             t1.ID AS advice_definition_id,
             t1.NAME AS advice_name,
             t1.org_id AS org_id,
@@ -917,7 +917,7 @@
           <if test="searchKey != null and searchKey != ''">
               AND (t1.name ILIKE '%' || #{searchKey} || '%' OR t1.py_str ILIKE '%' || #{searchKey} || '%')
           </if>
-        ORDER BY t1.name ASC
+        ORDER BY t1.ID, t1.name ASC, t2.ID ASC
     </select>
 
 </mapper>

openhis-server-new/openhis-application/src/main/resources/mapper/regdoctorstation/RequestFormManageAppMapper.xml

@@ -19,16 +19,7 @@
                drf.requester_id,
                drf.create_time,
                ap.NAME AS patient_name,
-                CASE
-                    WHEN MIN(wsr.status_enum) = 1 THEN 0
-                    WHEN MIN(wsr.status_enum) = 2 THEN 1
-                    WHEN MIN(wsr.status_enum) = 3 AND MAX(CASE WHEN wsr.performer_check_id IS NOT NULL THEN 1 ELSE 0 END) = 1 THEN 2
-                    WHEN MIN(wsr.status_enum) = 3 THEN 4
-                    WHEN MIN(wsr.status_enum) = 4 THEN 3
-                    WHEN MIN(wsr.status_enum) = 5 OR MIN(wsr.status_enum) = 6 OR MIN(wsr.status_enum) = 7 THEN 7
-                    WHEN MIN(wsr.status_enum) = 8 THEN 6
-                    ELSE NULL
-                END AS status
+               drf.status
         FROM doc_request_form AS drf
                  LEFT JOIN adm_encounter AS ae ON ae.ID = drf.encounter_id
             AND ae.delete_flag = '0'
@@ -46,16 +37,7 @@
               AND drf.create_time <= (#{endDate}::date + INTERVAL '1 day' - INTERVAL '1 second')
           </if>
           <if test="status != null and status != ''">
-               AND CASE
-                       WHEN MIN(wsr.status_enum) = 1 THEN 0
-                       WHEN MIN(wsr.status_enum) = 2 THEN 1
-                       WHEN MIN(wsr.status_enum) = 3 AND MAX(CASE WHEN wsr.performer_check_id IS NOT NULL THEN 1 ELSE 0 END) = 1 THEN 2
-                       WHEN MIN(wsr.status_enum) = 3 THEN 4
-                       WHEN MIN(wsr.status_enum) = 4 THEN 3
-                       WHEN MIN(wsr.status_enum) = 5 OR MIN(wsr.status_enum) = 6 OR MIN(wsr.status_enum) = 7 THEN 7
-                       WHEN MIN(wsr.status_enum) = 8 THEN 6
-                       ELSE NULL
-                   END = #{status}::integer
+               AND drf.status = #{status}::integer
           </if>
           <if test="keyword != null and keyword != ''">
               AND (drf.prescription_no ILIKE '%' || #{keyword} || '%'
@@ -71,7 +53,7 @@
                    ))
           </if>
         GROUP BY drf.id, drf.encounter_id, drf.prescription_no, drf.name, drf.desc_json,
-                 drf.requester_id, drf.create_time, ap.name
+                 drf.requester_id, drf.create_time, ap.name, drf.status
     </select>
 
     <select id="getRequestFormDetail" resultType="com.openhis.web.regdoctorstation.dto.RequestFormDetailQueryDto">

openhis-ui-vue3/src/views/inpatientDoctor/home/components/applicationShow/examineApplication.vue

@@ -93,7 +93,6 @@
           </el-table-column>
           <el-table-column prop="createTime" label="创建时间" width="160" />
           <el-table-column prop="prescriptionNo" label="申请单号" width="140" />
-          <el-table-column prop="requesterId_dictText" label="申请者" width="120" />
           <el-table-column label="申请单状态" width="120" align="center">
             <template #default="scope">
               <el-tag :type="getStatusTagType(scope.row.status)" effect="plain" round>
@@ -101,6 +100,7 @@
               </el-tag>
             </template>
           </el-table-column>
+          <el-table-column prop="requesterId_dictText" label="申请者" width="120" />
           <el-table-column label="操作" width="280" align="center" fixed="right">
             <template #default="scope">
               <!-- 详情 - 所有状态都显示 -->

openhis-ui-vue3/src/views/inpatientDoctor/home/components/applicationShow/testApplication.vue

@@ -41,8 +41,8 @@
               <el-option label="全部" value="" />
               <el-option label="待签发" value="0" />
               <el-option label="已签发" value="1" />
-              <el-option label="报告已出" value="4" />
-              <el-option label="已作废" value="5" />
+              <el-option label="已出报告" value="6" />
+              <el-option label="已作废" value="7" />
             </el-select>
           </el-form-item>
           <el-form-item label="关键字">
@@ -105,15 +105,18 @@
             </template>
           </el-table-column>
           <el-table-column prop="requesterId_dictText" label="申请者" width="120" />
-          <el-table-column label="操作" align="center" fixed="right" width="160">
+          <el-table-column label="操作" align="center" fixed="right" width="180">
             <template #default="scope">
+              <!-- 待签发：可修改、删除 -->
               <template v-if="scope.row.status == 0">
                 <el-button link type="primary" @click="handleEdit(scope.row)">修改</el-button>
                 <el-button link type="danger" @click="handleDelete(scope.row)">删除</el-button>
               </template>
+              <!-- 已签发：可撤回 -->
               <template v-else-if="scope.row.status == 1">
                 <el-button link type="warning" @click="handleWithdraw(scope.row)">撤回</el-button>
               </template>
+              <!-- 已校对(2)、待接收(3)、已接收(4)、已检查(6)、已作废(7)：仅查看详情 -->
               <el-button link type="primary" @click="handleViewDetail(scope.row)">详情</el-button>
             </template>
           </el-table-column>
@@ -328,8 +331,11 @@ const parseBillStatus = (status) => {
   const statusMap = {
     '0': '待签发',
     '1': '已签发',
-    '4': '报告已出',
-    '5': '已作废',
+    '2': '已校对',
+    '3': '待接收',
+    '4': '已收样',
+    '6': '已出报告',
+    '7': '已作废',
   };
   return statusMap[String(status)] || '-';
 };

openhis-ui-vue3/src/views/inpatientDoctor/home/components/order/applicationForm/surgery.vue

@@ -128,7 +128,7 @@ const emits = defineEmits(['submitOk']);
 const props = defineProps({});
 const state = reactive({});
 const applicationListAll = ref();
-const applicationList = ref();
+const applicationList = ref([]);
 const orgOptions = ref([]); // 科室选项
 const loading = ref(false); // 加载状态
 const mapToTransferItem = (item) => {
@@ -150,6 +150,7 @@ const getList = () => {
   if (surgeryMappedCache && surgeryMappedCache.length > 0) {
     applicationList.value = surgeryMappedCache;
     applicationListAll.value = surgeryRecordsCache;
+    loading.value = false;
     return;
   }
   loadPage('');
@@ -173,7 +174,9 @@ const loadPage = (key) => {
       dbTotal.value = res.data.total || 0;
       const records = res.data.records;
       applicationListAll.value = records;
-      applicationList.value = records.map(mapToTransferItem);
+      applicationList.value = records
+        .filter(item => item.adviceDefinitionId != null)
+        .map(mapToTransferItem);
       // 仅在无搜索时缓存
       if (!key) {
         surgeryRecordsCache = records;