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>

BUG461_ANALYSIS.md

@@ -1,89 +0,0 @@
-# Bug #461 分析报告
-
-## Bug描述
-[系统管理-执行科室配置] 保存项目配置后，项目名称回显为ID码，未显示正确名称
-
-## 根因分析
-
-### 数据流
-1. 前端调用 `getDiagnosisTreatmentList()` → GET `/base-data-manage/org-loc/org-loc`
-2. 后端 `OrganizationLocationController.getOrgLocPage()` 返回 `Page<OrgLocQueryDto>`
-3. `OrgLocQueryDto.activityDefinitionId` 标注了 `@Dict(dictTable="wor_activity_definition", dictCode="id", dictText="name")`
-4. `DictAspect` AOP 拦截 GET/POST 请求，对带 `@Dict` 注解的字段执行 SQL 翻译：`SELECT name FROM wor_activity_definition WHERE id::varchar = ? LIMIT 1`
-5. 翻译结果写入 `activityDefinitionId_dictText` 字段
-6. 前端使用 `record.activityDefinitionId_dictText` 作为项目名称显示
-
-### 根本原因
-**`DictAspect.queryDictLabel` 方法在 SQL 查询失败时返回空字符串 `""`，导致 `_dictText` 字段被设置为空值。**
-
-具体代码 (`DictAspect.java:123-149`)：
-```java
-private String queryDictLabel(String dictTable, String dictCode, String dictText, String deleteFlag, String dictValue) {
-    if (!StringUtils.hasText(dictTable)) {
-        return DictUtils.getDictLabel(dictCode, dictValue);
-    } else {
-        if (!StringUtils.hasText(dictText)) {
-            return DictUtils.getDictLabel(dictCode, dictValue);
-        }
-        String sql = String.format("SELECT %s FROM %s WHERE %s::varchar = ?", dictText, dictTable, dictCode);
-        // ...
-        try {
-            return jdbcTemplate.queryForObject(sql, String.class, dictValue);
-        } catch (DataAccessException e) {
-            return "";  // ← 关键问题：查询失败返回空字符串
-        }
-    }
-}
-```
-
-当 `jdbcTemplate.queryForObject` 查询失败（无结果或异常）时，返回空字符串 `""`。而在 `processDict` 中：
-```java
-String dictLabel = queryDictLabel(...);
-if (dictLabel != null) {  // ← 空字符串 "" 不等于 null，条件为 true
-    textField.set(dto, dictLabel);  // ← 设置为空字符串
-}
-```
-
-空字符串 `""` 不是 `null`，所以 `_dictText` 被设为空字符串，而不是保持 `null`（未翻译）。前端收到 `activityDefinitionId_dictText: ""`，当作有效值处理，显示为空或回退到 ID。
-
-### 前端 fallback 的不足
-前端代码在 `getList()` 中尝试用 `activityDefinitionId_dictText` 补充选项，但：
-```javascript
-if (record.activityDefinitionId && !filteredOptions.some(o => o.value === record.activityDefinitionId)) {
-    filteredOptions.push({
-        value: record.activityDefinitionId,
-        label: record.activityDefinitionId_dictText || record.activityDefinitionId  // ← 空字符串时显示ID
-    });
-}
-```
-空字符串是 falsy 值，所以 `"" || record.activityDefinitionId` 回退到 ID，仍然显示 ID。
-
-## 影响范围
-- **前端**: `openhis-ui-vue3/src/views/basicmanage/implementDepartment/index.vue`
-- **后端**: `openhis-server-new/.../basedatamanage/appservice/impl/OrganizationLocationAppServiceImpl.java`
-- **AOP**: `openhis-server-new/.../common/aspectj/DictAspect.java`
-- **DTO**: `openhis-server-new/.../basedatamanage/dto/OrgLocQueryDto.java`
-- **数据库表**: `adm_organization_location`, `wor_activity_definition`
-
-## 修复方案
-
-### 方案：在 service 层直接 JOIN 查询项目名称
-
-修改 `OrganizationLocationAppServiceImpl.getOrgLocPage()` 方法，在返回结果前手动填充 `activityDefinitionId_dictText`，不依赖 DictAspect 的行为。这样更可靠，避免了 AOP 执行顺序、SQL异常处理等不确定因素。
-
-具体改动：在 `OrganizationLocationAppServiceImpl` 中，利用已有的 `activityDefinitionMapper` 对每条记录补充 `activityDefinitionId_dictText`。
-
-## 验证计划
-1. 修改代码后编译通过
-2. 验证 `activityDefinitionId_dictText` 在 API 响应中正确填充
-3. 前端 el-select 能正确显示项目名称而非 ID
-
----
-
-## 修复结果：✅ 成功，12行改动
-
-**修改文件**: `openhis-server-new/openhis-application/src/main/java/com/openhis/web/basedatamanage/appservice/impl/OrganizationLocationAppServiceImpl.java`
-
-**改动内容**: 在 `getOrgLocPage()` 方法中，对分页返回的每条记录，使用已注入的 `activityDefinitionMapper` 查询对应的 `ActivityDefinition` 实体，手动填充 `activityDefinitionId_dictText` 字段。
-
-**策略**: 不依赖 DictAspect 的 AOP 翻译机制，在 service 层直接填充字典翻译值，确保前端能接收到正确的项目名称。

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/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;