Compare commits

..

4 Commits

Author SHA1 Message Date
关羽
0197f5509c Fix Bug #461: [系统管理-执行科室配置] 保存项目配置后,项目名称回显为ID码,未显示正确名称
**后端开发重点**:优先搜索 Java/Spring 后端代码。
关键词:Controller, Service, Mapper, API, 接口, 数据查询
搜索目录:openhis-server-new/src/, his-repo/src/

在 getOrgLocPage 方法中手动填充 activityDefinitionId_dictText,
确保前端能正确回显项目名称而非 ID 码。

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
2026-05-16 14:16:07 +08:00
赵云
a75063430c Fix Bug #463: 根因+修复方案摘要 2026-05-16 14:16:07 +08:00
关羽
300d53bdc6 Fix Bug #454: 门诊医生站-医嘱页签:删除"待签发"状态的检验项目时,错误触发"执行科室"校验导致删除失败
前端补充:删除医嘱前添加确认弹窗,对诊疗类项目提示"删除此医嘱将同时删除关联的检验申请单",
满足Bug期望中"触发级联删除前应有明确提示"的要求。

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
2026-05-16 13:28:15 +08:00
关羽
284818bd8f Fix Bug #445: 门诊手术安排-临时医嘱生成界面:引用计费时过滤已生成医嘱项目
根因: handleQuoteBilling 从后端拉取计费数据后,只用 requestId 过滤已生成项目,
但已生成医嘱的计费项在后端可能没有 requestId(从 adm_charge_item 关联来的项目 requestId 为空),
导致已提交项目重新出现在"待生成"列表中。

修复: 在 handleQuoteBilling 中,用 chargeItemId/requestId/id 三重匹配,
与已有的 temporaryAdvices 做比对,排除已生成项目。

**后端开发重点**:优先搜索 Java/Spring 后端代码。
关键词:Controller, Service, Mapper, API, 接口, 数据查询
搜索目录:openhis-server-new/src/, his-repo/src/
2026-05-16 13:18:36 +08:00
8 changed files with 33 additions and 183 deletions

View File

@@ -1,72 +0,0 @@
# 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. 缓存未命中时查询数据库后写入 Redis24小时过期
4. 有搜索关键字时不缓存避免缓存爆炸
**改动量:** 20
### 前端:修复 `getList()` 缓存命中时的 loading 状态
**改动文件:** `surgery.vue`
1. `getList()` 方法中当命中内存缓存时显式设置 `loading.value = false`
**改动量:** 1
## 验证计划
1. 编译验证 Java 代码
2. 语法验证 Vue 文件`node --check surgery.vue`
3. 手动验证登录医生工作站打开手术申请单观察加载速度首次应有loading二次打开应秒开

View File

@@ -1,65 +0,0 @@
# 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/-4DISTINCT ON + ORDER BY 调整)
- `surgery.vue`+4/-2初始化空数组 + 空值过滤)
**修复结果:✅ 成功8行改动**

View File

@@ -2463,34 +2463,12 @@ 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;
}

View File

@@ -872,7 +872,7 @@
<!-- 手术项目专用分页查询:仅查手术 + 定价,无库存/草稿库存/取药科室等无关逻辑 -->
<select id="getSurgeryPage" resultType="com.openhis.web.doctorstation.dto.SurgeryItemDto">
SELECT DISTINCT ON (t1.ID)
SELECT
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.ID, t1.name ASC, t2.ID ASC
ORDER BY t1.name ASC
</select>
<!-- 检查项目专用分页查询:仅查检查(23) + 定价,无库存/草稿库存/取药科室等无关逻辑 -->
<select id="getExaminationPage" resultType="com.openhis.web.doctorstation.dto.SurgeryItemDto">
SELECT DISTINCT ON (t1.ID)
SELECT
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.ID, t1.name ASC, t2.ID ASC
ORDER BY t1.name ASC
</select>
</mapper>

View File

@@ -19,7 +19,16 @@
drf.requester_id,
drf.create_time,
ap.NAME AS patient_name,
drf.status
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
FROM doc_request_form AS drf
LEFT JOIN adm_encounter AS ae ON ae.ID = drf.encounter_id
AND ae.delete_flag = '0'
@@ -37,7 +46,16 @@
AND drf.create_time &lt;= (#{endDate}::date + INTERVAL '1 day' - INTERVAL '1 second')
</if>
<if test="status != null and status != ''">
AND drf.status = #{status}::integer
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
</if>
<if test="keyword != null and keyword != ''">
AND (drf.prescription_no ILIKE '%' || #{keyword} || '%'
@@ -53,7 +71,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.status
drf.requester_id, drf.create_time, ap.name
</select>
<select id="getRequestFormDetail" resultType="com.openhis.web.regdoctorstation.dto.RequestFormDetailQueryDto">

View File

@@ -93,6 +93,7 @@
</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>
@@ -100,7 +101,6 @@
</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">
<!-- 详情 - 所有状态都显示 -->

View File

@@ -41,8 +41,8 @@
<el-option label="全部" value="" />
<el-option label="待签发" value="0" />
<el-option label="已签发" value="1" />
<el-option label="已出报告" value="6" />
<el-option label="已作废" value="7" />
<el-option label="报告已出" value="4" />
<el-option label="已作废" value="5" />
</el-select>
</el-form-item>
<el-form-item label="关键字">
@@ -105,18 +105,15 @@
</template>
</el-table-column>
<el-table-column prop="requesterId_dictText" label="申请者" width="120" />
<el-table-column label="操作" align="center" fixed="right" width="180">
<el-table-column label="操作" align="center" fixed="right" width="160">
<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>
@@ -331,11 +328,8 @@ const parseBillStatus = (status) => {
const statusMap = {
'0': '待签发',
'1': '已签发',
'2': '已校对',
'3': '待接收',
'4': '已收样',
'6': '已出报告',
'7': '已作废',
'4': '报告已出',
'5': '已作废',
};
return statusMap[String(status)] || '-';
};

View File

@@ -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,7 +150,6 @@ const getList = () => {
if (surgeryMappedCache && surgeryMappedCache.length > 0) {
applicationList.value = surgeryMappedCache;
applicationListAll.value = surgeryRecordsCache;
loading.value = false;
return;
}
loadPage('');
@@ -174,9 +173,7 @@ const loadPage = (key) => {
dbTotal.value = res.data.total || 0;
const records = res.data.records;
applicationListAll.value = records;
applicationList.value = records
.filter(item => item.adviceDefinitionId != null)
.map(mapToTransferItem);
applicationList.value = records.map(mapToTransferItem);
// 仅在无搜索时缓存
if (!key) {
surgeryRecordsCache = records;