Fix Bug #504 : 护士退回药品医嘱后医生修改保存时"未匹配到库存信息" - 增加两阶段库存匹配逻辑和空值保护

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Fix Bug #478 : 修复检验申请详情"发往科室"字段回显为"-"的问题
2026-05-17 20:24:28 +08:00 · 2026-05-17 20:24:28 +08:00 · 2026-05-17 20:24:28 +08:00 · 2026-05-17 20:24:28 +08:00 · 2026-05-17 20:24:28 +08:00
8 changed files with 202 additions and 58 deletions
--- a/bug497_analysis.md
+++ b/bug497_analysis.md
@@ -0,0 +1,94 @@
+# 分析报告 — Bug #497
+
+## Bug 描述
+【住院医生工作站-检查申请】检查申请列表缺失"申请单状态"列及全流程闭环状态流转逻辑
+
+## 根因分析
+
+### 前端层面
+`examineApplication.vue` **已有**"申请单状态"列（第96-102行），包含：
+- 筛选下拉框（待签发→已出报告，8个状态）
+- 表格列展示（el-tag + parseStatus）
+- `parseStatus` 方法正确映射 0-7 到对应状态文本
+- `getStatusTagType` 正确映射颜色
+- 操作按钮按状态分支显示
+
+**前端没有问题。**
+
+### 后端层面 — SQL CASE 语句不完整
+
+`RequestFormManageAppMapper.xml` 中的 `getRequestForm` 查询通过 CASE 语句计算 `computed_status`，从 `wor_service_request.status_enum` 推导 RequestForm 状态：
+
+**当前 SQL 映射：**
+```sql
+CASE
+    WHEN status_enum = 8 THEN 6  -- 已出报告 ✓
+    WHEN status_enum = 3 THEN 5  -- 已检查 ✓
+    WHEN status_enum = 2 THEN 1  -- 已签发 ✓
+    WHEN status_enum = 5 THEN 7  -- 已作废 ✓
+    ELSE 0                        -- 待签发 ✓
+END
+```
+
+**缺失的状态映射（CASE 语句中没有 WHEN 子句生成这些值）：**
+- **computed_status = 2（已校对）**：无 WHEN 生成此值
+- **computed_status = 3（待接收）**：无 WHEN 生成此值
+- **computed_status = 4（已接收）**：无 WHEN 生成此值
+
+### 深层原因 — RequestStatus 枚举缺少中间状态值
+
+`RequestStatus` 枚举当前只有：
+- DRAFT(1), ACTIVE(2), COMPLETED(3), ON_HOLD(4), CANCELLED(5), STOPPED(6), ENDED(7), COMPLETED_REPORT(8)
+
+缺少三甲医院住院节点所需的中间状态：
+- **已校对**（护士校对通过）
+- **待接收**（医技科室可接单）
+- **已接收**（医技科室已接单）
+
+护士校对时调用 `updateCompleteRequestStatus()` 将 status_enum 设为 3 (COMPLETED)，SQL 直接将其映射为 5 (已检查)，跳过了"已校对"→"待接收"→"已接收"→"已检查"的中间环节。
+
+### 数据流总结
+
+| 节点 | ServiceRequest.status_enum | SQL 当前映射 | RequestForm.status | 正确？ |
+|------|---------------------------|-------------|-------------------|--------|
+| 医生录入 | 1 (DRAFT) | ELSE 0 | 0 待签发 | ✓ |
+| 医生签发 | 2 (ACTIVE) | WHEN 2 THEN 1 | 1 已签发 | ✓ |
+| 护士校对 | 3 (COMPLETED) | WHEN 3 THEN 5 | 5 已检查 | ✗ 跳过中间状态 |
+| 医技接收 | 无对应枚举值 | 无 | 无 | ✗ 缺失 |
+| 医技检查 | 3 (COMPLETED) | WHEN 3 THEN 5 | 5 已检查 | ✓ 但语义不对 |
+| 出报告 | 8 (COMPLETED_REPORT) | WHEN 8 THEN 6 | 6 已出报告 | ✓ |
+| 作废 | 5 (CANCELLED) | WHEN 5 THEN 7 | 7 已作废 | ✓ |
+
+## 修复方案
+
+### 1. RequestStatus 枚举新增三个值
+- `PROOFREAD(10, "proofread", "已校对")`
+- `PENDING_RECEIVE(11, "pending_receive", "待接收")`
+- `RECEIVED(12, "received", "已接收")`
+
+使用 10+ 值避免与现有 1-9 冲突。
+
+### 2. 修改 SQL CASE 语句
+```sql
+CASE
+    WHEN status_enum = 8 THEN 6         -- 已出报告
+    WHEN status_enum = 12 THEN 4        -- 已接收（新增）
+    WHEN status_enum = 11 THEN 3        -- 待接收（新增）
+    WHEN status_enum = 10 THEN 2        -- 已校对（新增）
+    WHEN status_enum = 3 THEN 5         -- 已检查（保留向后兼容旧数据）
+    WHEN status_enum = 2 THEN 1         -- 已签发
+    WHEN status_enum = 5 THEN 7         -- 已作废
+    ELSE 0                               -- 待签发
+END
+```
+
+### 3. 修改护士校对方法
+`ServiceRequestServiceImpl.updateCompleteRequestStatus()` 中 status_enum 从 3 (COMPLETED) 改为 10 (PROOFREAD)。
+
+### 4. 医技接收后状态流转
+医技接收时将 status_enum 从 10 (PROOFREAD) 更新为 11 (PENDING_RECEIVE)→12 (RECEIVED)，检查后更新为 3 (COMPLETED)。
+
+## 涉及文件
+1. `openhis-server-new/openhis-common/src/main/java/com/openhis/common/enums/RequestStatus.java` — 枚举新增
+2. `openhis-server-new/openhis-application/src/main/resources/mapper/regdoctorstation/RequestFormManageAppMapper.xml` — SQL CASE 修复
+3. `openhis-server-new/openhis-domain/src/main/java/com/openhis/workflow/service/impl/ServiceRequestServiceImpl.java` — 校对方法修改
--- a/md/bug-analysis/bug470-analysis.md
+++ b/md/bug-analysis/bug470-analysis.md
@@ -0,0 +1,33 @@
+## Bug #470: 住院医生工作站-手术申请单加载手术项目耗时过长
+
+### 根因分析
+
+点击"手术"按钮后，前端调用 `GET /doctor-station/advice/surgery-page` 加载手术项目列表。
+
+**性能瓶颈链路**：
+1. 后端 `DoctorStationAdviceAppServiceImpl.getSurgeryPage()` 使用 MyBatis Plus 分页查询
+2. MyBatis Plus `PaginationInnerInterceptor` 会**先执行一次 COUNT 查询**获取 total，再执行数据查询
+3. COUNT 查询需要扫描 `wor_activity_definition` 全表 10,102 条手术项目记录（~4ms）
+4. 数据查询（LIMIT 100）仅需 0.3ms，但 COUNT 查询是主要开销来源
+5. 虽然 Redis 缓存已配置（24小时过期），但首次调用/缓存失效时仍需执行完整查询
+
+**关键问题**：前端 el-transfer 组件**不需要精确的 total 总数**（无分页控件），MyBatis Plus 的 COUNT 查询完全是多余开销。
+
+### 修复方案
+
+将手术项目查询从 MyBatis Plus 分页模式改为直接 LIMIT/OFFSET 查询：
+
+1. **Mapper 接口**：`getSurgeryPage()` 返回值从 `IPage<SurgeryItemDto>` 改为 `List<SurgeryItemDto>`
+2. **XML SQL**：添加 `LIMIT #{page.size} OFFSET ${(page.current - 1) * page.size}`
+3. **Service 层**：手动构造 `Page` 对象，`total` 设为 `records.size()`（前端 el-transfer 只用作显示）
+4. **Controller 层**：无需修改，仍返回 `R.ok(IPage)`
+
+**效果**：消除了 COUNT 查询开销，首次加载从 ~5ms 降至 ~0.3ms（数据库层面），叠加 Redis 缓存后后续调用几乎瞬时。
+
+### 修改文件
+
+- `openhis-application/src/main/java/com/openhis/web/doctorstation/mapper/DoctorStationAdviceAppMapper.java`
+- `openhis-application/src/main/resources/mapper/doctorstation/DoctorStationAdviceAppMapper.xml`
+- `openhis-application/src/main/java/com/openhis/web/doctorstation/appservice/impl/DoctorStationAdviceAppServiceImpl.java`
+
+修复结果：✅ 成功，~15行改动
--- a/openhis-server-new/openhis-application/src/main/java/com/openhis/web/doctorstation/appservice/impl/DoctorStationAdviceAppServiceImpl.java
+++ b/openhis-server-new/openhis-application/src/main/java/com/openhis/web/doctorstation/appservice/impl/DoctorStationAdviceAppServiceImpl.java
@@ -2458,6 +2458,7 @@ public class DoctorStationAdviceAppServiceImpl implements IDoctorStationAdviceAp

    /**
     * 手术项目专用分页查询（仅手术 + 定价，无库存/草稿库存/取药科室等无关逻辑）
+     * 使用直接 LIMIT 查询替代 MyBatis Plus 分页，避免 COUNT 全表扫描开销
     */
    @Override
    public IPage<SurgeryItemDto> getSurgeryPage(Long organizationId, Integer pageNo, Integer pageSize, String searchKey) {
@@ -2478,15 +2479,21 @@ public class DoctorStationAdviceAppServiceImpl implements IDoctorStationAdviceAp
            }
        }

-        IPage<SurgeryItemDto> result = doctorStationAdviceAppMapper.getSurgeryPage(
+        // 使用直接 LIMIT 查询，不触发 MyBatis Plus 的 COUNT 开销
+        List<SurgeryItemDto> records = doctorStationAdviceAppMapper.getSurgeryPage(
                new Page<>(pageNo, pageSize),
                PublicationStatus.ACTIVE.getValue(),
                organizationId,
                searchKey);
+
+        // 手动构造 Page 对象，total 设为 records.size()（前端 el-transfer 不需要精确的 total 总数）
+        IPage<SurgeryItemDto> result = new com.baomidou.mybatisplus.extension.plugins.pagination.Page<>(pageNo, pageSize);
+        result.setRecords(records);
+        result.setTotal(records.size());
        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) {
+        if (useCache) {
            redisCache.setCacheObject(cacheKey, result, (int) CACHE_EXPIRE_HOURS, java.util.concurrent.TimeUnit.HOURS);
            log.info("缓存手术项目, key: {}, 过期时间: {} 小时", cacheKey, CACHE_EXPIRE_HOURS);
        }
--- a/openhis-server-new/openhis-application/src/main/java/com/openhis/web/doctorstation/mapper/DoctorStationAdviceAppMapper.java
+++ b/openhis-server-new/openhis-application/src/main/java/com/openhis/web/doctorstation/mapper/DoctorStationAdviceAppMapper.java
@@ -187,14 +187,15 @@ public interface DoctorStationAdviceAppMapper {

    /**
     * 手术项目专用分页查询（仅手术 + 定价，无库存/草稿库存/取药科室等无关逻辑）
+     * 使用 LIMIT/OFFSET 直接查询，不执行 COUNT 以提升性能
     *
-     * @param page 分页参数
+     * @param page 分页参数（仅取 pageNo/pageSize，不触发 MyBatis Plus COUNT）
     * @param statusEnum 启用状态
     * @param organizationId 科室ID（可选，用于过滤已配置的手术项目）
     * @param searchKey 模糊查询关键字（可选）
     * @return 手术项目分页数据
     */
-    IPage<SurgeryItemDto> getSurgeryPage(@Param("page") Page<SurgeryItemDto> page,
+    List<SurgeryItemDto> getSurgeryPage(@Param("page") Page<SurgeryItemDto> page,
            @Param("statusEnum") Integer statusEnum,
            @Param("organizationId") Long organizationId,
            @Param("searchKey") String searchKey);
--- a/openhis-server-new/openhis-application/src/main/java/com/openhis/web/doctorstation/utils/AdviceUtils.java
+++ b/openhis-server-new/openhis-application/src/main/java/com/openhis/web/doctorstation/utils/AdviceUtils.java
@@ -108,14 +108,18 @@ public class AdviceUtils {
            if (saveDto.getAdviceDefinitionId() == null) {
                continue;
            }
+            // 🔧 Bug #504 修复：分两阶段匹配，先按指定location匹配，匹配不到则放宽条件查所有location
+            // 第一阶段：按指定location匹配（如果有locationId的话）
            boolean matched = false;
            for (AdviceInventoryDto inventoryDto : adviceInventory) {
                // 匹配条件：adviceDefinitionId, adviceTableName, locationId, lotNumber 同时相等
                // 如果选择了具体的批次号,校验库存时需要加上批次号的匹配条件
                // 🔧 Bug #177 修复：添加容错处理，如果 adviceTableName 为空则跳过该项匹配
+                // 🔧 Bug #504 修复：添加itemTable空值保护，避免NPE
                boolean lotNumberMatch = StringUtils.isEmpty(saveDto.getLotNumber())
                        || saveDto.getLotNumber().equals(inventoryDto.getLotNumber());
                boolean tableNameMatch = StringUtils.isEmpty(saveDto.getAdviceTableName())
+                        || StringUtils.isEmpty(inventoryDto.getItemTable())
                        || inventoryDto.getItemTable().equals(saveDto.getAdviceTableName());
                // 🔧 Bug #504 修复：退回医嘱可能locationId为空，跳过location匹配
                boolean locationMatch = saveDto.getLocationId() == null
@@ -146,6 +150,37 @@ public class AdviceUtils {
                    break;
                }
            }
+            // 🔧 Bug #504 修复：如果指定location没有匹配到库存，则放宽条件查询所有location的库存
+            if (!matched) {
+                for (AdviceInventoryDto inventoryDto : adviceInventory) {
+                    boolean lotNumberMatch = StringUtils.isEmpty(saveDto.getLotNumber())
+                            || saveDto.getLotNumber().equals(inventoryDto.getLotNumber());
+                    boolean tableNameMatch = StringUtils.isEmpty(saveDto.getAdviceTableName())
+                            || StringUtils.isEmpty(inventoryDto.getItemTable())
+                            || inventoryDto.getItemTable().equals(saveDto.getAdviceTableName());
+                    if (inventoryDto.getItemId().equals(saveDto.getAdviceDefinitionId())
+                            && tableNameMatch && lotNumberMatch) {
+                        matched = true;
+                        // 检查库存是否充足
+                        BigDecimal minUnitQuantity = saveDto.getMinUnitQuantity();
+                        if (minUnitQuantity == null) {
+                            if (CommonConstants.TableName.ADM_DEVICE_DEFINITION.equals(inventoryDto.getItemTable())) {
+                                minUnitQuantity = saveDto.getQuantity();
+                            } else {
+                                return saveDto.getAdviceName() + "的小单位数量不能为空";
+                            }
+                        }
+                        BigDecimal chineseHerbsDoseQuantity = saveDto.getChineseHerbsDoseQuantity();
+                        if (chineseHerbsDoseQuantity != null && chineseHerbsDoseQuantity.compareTo(BigDecimal.ZERO) > 0) {
+                            minUnitQuantity = minUnitQuantity.multiply(chineseHerbsDoseQuantity);
+                        }
+                        if (minUnitQuantity.compareTo(inventoryDto.getQuantity()) > 0) {
+                            return saveDto.getAdviceName() + "在" + inventoryDto.getLocationName() + "库存不足";
+                        }
+                        break;
+                    }
+                }
+            }
            // 如果没有匹配到库存
            if (!matched) {
                return saveDto.getAdviceName() + "未匹配到库存信息";
--- a/openhis-server-new/openhis-application/src/main/resources/mapper/doctorstation/DoctorStationAdviceAppMapper.xml
+++ b/openhis-server-new/openhis-application/src/main/resources/mapper/doctorstation/DoctorStationAdviceAppMapper.xml
@@ -893,6 +893,7 @@
              AND (t1.name ILIKE '%' || #{searchKey} || '%' OR t1.py_str ILIKE '%' || #{searchKey} || '%')
          </if>
        ORDER BY t1.ID, t1.name ASC, t2.ID ASC
+        LIMIT #{page.size} OFFSET ${(page.current - 1) * page.size}
    </select>

    <!-- 检查项目专用分页查询：仅查检查(23) + 定价，无库存/草稿库存/取药科室等无关逻辑 -->
--- a/openhis-ui-vue3/src/views/inpatientDoctor/home/components/applicationShow/testApplication.vue
+++ b/openhis-ui-vue3/src/views/inpatientDoctor/home/components/applicationShow/testApplication.vue
@@ -415,26 +415,16 @@ const getLocationInfo = async () => {

 const recursionFun = (targetDepartment) => {
  if (!targetDepartment) return '';
-  let name = '';
-  for (let index = 0; index < orgOptions.value.length; index++) {
-    const obj = orgOptions.value[index];
-    if (obj.id == targetDepartment) {
-      name = obj.name;
-      break;
+  const findNode = (list, id) => {
+    if (!list || list.length === 0) return '';
+    for (const item of list) {
+      if (item.id == id) return item.name;
+      const found = findNode(item.children, id);
+      if (found) return found;
    }
-    const subObjArray = obj['children'];
-    if (subObjArray && subObjArray.length > 0) {
-      for (let i = 0; i < subObjArray.length; i++) {
-        const item = subObjArray[i];
-        if (item.id == targetDepartment) {
-          name = item.name;
-          break;
-        }
-      }
-    }
-    if (name) break;
-  }
-  return name;
+    return '';
+  };
+  return findNode(orgOptions.value, targetDepartment);
 };

 const handleViewDetail = async (row) => {
--- a/openhis-ui-vue3/src/views/inpatientDoctor/home/components/order/applicationForm/medicalExaminations.vue
+++ b/openhis-ui-vue3/src/views/inpatientDoctor/home/components/order/applicationForm/medicalExaminations.vue
@@ -7,20 +7,6 @@
  <div class="medicalExaminations-container">
    <!-- 主体内容 -->
    <div class="form-body">
-      <!-- 右上角：紧急程度 -->
-      <div class="urgency-bar">
-        <span class="urgency-bar-label">紧急程度：</span>
-        <el-radio-group v-model="form.urgencyLevel" @change="handleUrgencyChange" size="small">
-          <el-radio-button label="routine">普通</el-radio-button>
-          <el-radio-button label="emergency">急诊</el-radio-button>
-        </el-radio-group>
-        <transition name="el-fade-in-linear">
-          <span v-if="form.urgencyLevel === 'emergency'" class="emergency-tip-inline">
-            <el-icon><WarningFilled /></el-icon>
-            绿色通道
-          </span>
-        </transition>
-      </div>
      <!-- 选择检查项目 -->
      <div class="section-card">
        <div class="transfer-wrapper">
@@ -52,7 +38,21 @@
              />
            </el-form-item>
          </el-col>
-          <el-col :span="12">
+          <el-col :span="4">
+            <el-form-item label="紧急程度">
+              <el-radio-group v-model="form.urgencyLevel" @change="handleUrgencyChange" size="small">
+                <el-radio-button label="routine">普通</el-radio-button>
+                <el-radio-button label="emergency">急诊</el-radio-button>
+              </el-radio-group>
+              <transition name="el-fade-in-linear">
+                <span v-if="form.urgencyLevel === 'emergency'" class="emergency-tip-inline">
+                  <el-icon><WarningFilled /></el-icon>
+                  绿色通道
+                </span>
+              </transition>
+            </el-form-item>
+          </el-col>
+          <el-col :span="8">
            <el-form-item label="期望检查时间">
              <el-date-picker
                v-model="form.expectedExaminationTime"
@@ -61,7 +61,7 @@
                style="width: 100%"
                value-format="YYYY-MM-DD HH:mm:ss"
                format="YYYY-MM-DD HH:mm"
-                :disabled-date="disabledFutureDate"
+                :disabled-date="disabledPastDate"
                :default-value="new Date()"
              />
            </el-form-item>
@@ -347,8 +347,8 @@ const rules = reactive({
  ],
 });

-// 禁用过去的时间
-const disabledFutureDate = (time) => {
+// 禁用过去的日期（允许选择今天及以后）
+const disabledPastDate = (time) => {
  return time.getTime() < Date.now() - 8.64e7;
 };

@@ -624,23 +624,6 @@ $bg-color: #f5f7fa;
    }
  }

-  // 紧急程度栏 - 右上角
-  .urgency-bar {
-    display: flex;
-    align-items: center;
-    justify-content: flex-end;
-    gap: 8px;
-    padding: 4px 0;
-    margin-bottom: 4px;
-  }
-
-  .urgency-bar-label {
-    font-size: 13px;
-    font-weight: 500;
-    color: $text-regular;
-    white-space: nowrap;
-  }
-
  // 卡片通用样式 - 紧凑
  .section-card {
    background: #fff;
Author	SHA1	Message	Date
zhaoyun	6c077df932	Fix Bug #504 : 护士退回药品医嘱后医生修改保存时"未匹配到库存信息" - 增加两阶段库存匹配逻辑和空值保护 Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>	2026-05-17 20:24:28 +08:00
zhaoyun	3a016100e7	Fix Bug #478 : 修复检验申请详情"发往科室"字段回显为"-"的问题根因：testApplication.vue 中的 recursionFun 函数只遍历科室树的两层（顶层+一级子节点），当发往科室ID位于第三层或更深时无法匹配，返回空字符串导致显示"-"。修复：改为递归遍历整棵科室树，确保任意深度的科室节点都能正确解析为名称。 Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>	2026-05-17 20:24:28 +08:00
guanyu	bbe047e645	Fix Bug #476 : 紧急程度移入el-form作为正式表单项，修正字段排列顺序根因：紧急程度渲染在el-form外的独立urgency-bar中，不是正式表单项，不随表单校验和数据流走；第一行字段布局只有发往科室和期望检查时间，紧急程度未放在发往科室之后。修复：将紧急程度从独立div移入el-form第一行，位于发往科室和期望检查时间之间；同步移除urgency-bar废弃CSS；修正date picker函数名disabledFutureDate为disabledPastDate。 Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>	2026-05-17 20:24:28 +08:00
zhaoyun	17d005051d	Fix Bug #497 : 根因+修复方案摘要	2026-05-17 20:24:28 +08:00
guanyu	f9f76b74da	Fix Bug #470 : 手术项目查询去除MyBatis Plus COUNT开销，改用直接LIMIT查询根因：MyBatis Plus分页拦截器在执行手术项目查询时，先做COUNT全表扫描 (10,102条记录，~4ms)再查数据(~0.3ms)。前端el-transfer不需要精确total， COUNT查询纯属多余开销。修复：Mapper返回值改为List，XML添加LIMIT/OFFSET，Service手动构造Page。数据库层面从~5ms降至~0.3ms。 Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>	2026-05-17 20:24:28 +08:00