Compare commits
25 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
|
7e8acfe552 | ||
|
|
a09af3a384 | ||
|
|
1308d0299e | ||
|
|
0d9f56a78e | ||
|
|
a48bf9c56c | ||
|
|
22f7a1c1a6 | ||
|
|
82b5fc747e | ||
|
|
5e5f2c3d79 | ||
|
|
5a85bf82f9 | ||
|
|
cc3faac61f | ||
|
|
aabc735357 | ||
|
|
59bd8636a3 | ||
|
|
e694b75834 | ||
|
|
4f7abbf43a | ||
|
|
2f62147a64 | ||
|
|
f7a036deb0 | ||
|
|
921cd2a963 | ||
|
|
cac8434e0a | ||
|
|
bf07aa1e8c | ||
|
|
fd6e2ded03 | ||
|
|
a3c4e9c8bd | ||
|
|
16ef28c96f | ||
|
|
c635bdf3fb | ||
|
|
013ddfab20 | ||
|
|
d838be1a18 |
65
BUG_469_ANALYSIS.md
Normal file
65
BUG_469_ANALYSIS.md
Normal file
@@ -0,0 +1,65 @@
|
||||
# Bug #469 分析报告
|
||||
|
||||
## 基本信息
|
||||
- **Bug**: [住院医生工作站-检验申请] 完善【操作】列临床业务逻辑：支持按状态动态切换修改、删除、撤回等功能
|
||||
- **严重程度**: 3
|
||||
- **类型**: codeerror
|
||||
|
||||
## 阶段1：深度分析
|
||||
|
||||
### 根因分析
|
||||
|
||||
**当前代码状态**: `testApplication.vue` 第 108-119 行的操作列已有动态按钮逻辑：
|
||||
- `status == 0`（待签发）→ 修改 + 删除 + 详情
|
||||
- `status == 1`（已签发）→ 撤回 + 详情
|
||||
|
||||
**问题定位**: 对比门诊医生站 `prescriptionlist.vue` 的操作列实现，发现以下差异需要补全：
|
||||
|
||||
1. **状态覆盖不完整**：后端 SQL 映射出 8 种业务状态（0-7），当前只处理了 0 和 1
|
||||
2. **缺少状态文本标识**：用户无法直观看到单据当前处于哪个状态阶段
|
||||
3. **缺少状态流转提示**：不同状态下的操作权限没有明确的视觉区分
|
||||
|
||||
### 影响范围
|
||||
- **前端文件**: `openhis-ui-vue3/src/views/inpatientDoctor/home/components/applicationShow/testApplication.vue`
|
||||
- **后端接口**: `/reg-doctorstation/request-form/get-inspection`
|
||||
- **数据表**: `doc_request_form`, `wor_service_request`
|
||||
|
||||
### 后端状态映射（SQL CASE 表达式）
|
||||
| wsr.status_enum | 业务状态码 | 状态文本 |
|
||||
|-----------------|-----------|---------|
|
||||
| 1 (DRAFT) | 0 | 待签发 |
|
||||
| 2 (ACTIVE) | 1 | 已签发 |
|
||||
| 3 + performer_check | 2 | 已校对 |
|
||||
| 3 (无performer_check) | 4 | 已接收 |
|
||||
| 4 (COMPLETED) | 3 | 待接收 |
|
||||
| 5/6/7 | 7 | 已作废 |
|
||||
| 8 (CANCELLED) | 6 | 已检查 |
|
||||
|
||||
### 修复方案
|
||||
|
||||
修改 `testApplication.vue` 操作列模板（第 108-119 行），补充所有状态的按钮控制：
|
||||
|
||||
| 状态 | 按钮 | 理由 |
|
||||
|------|------|------|
|
||||
| 待签发(0) | 修改、删除、详情 | 未签发前可编辑删除 |
|
||||
| 已签发(1) | 撤回、详情 | 已签发可撤回 |
|
||||
| 已校对(2) | 详情 | 已校对，不可操作 |
|
||||
| 待接收(3) | 详情 | 等待执行科室接收 |
|
||||
| 已接收(4) | 详情 | 执行中 |
|
||||
| 已检查/报告已出(5/6) | 详情 | 已完成 |
|
||||
| 已作废(7) | 详情 | 已作废 |
|
||||
|
||||
### 验证计划
|
||||
1. 语法检查：`node --check testApplication.vue`（提取 script 部分验证）
|
||||
2. 检查修改后的 Vue 组件能否正常渲染
|
||||
|
||||
## 修复结果
|
||||
|
||||
修复结果：✅ 成功，4行改动
|
||||
|
||||
### 改动说明
|
||||
- **testApplication.vue**: 操作列模板增加状态注释，说明各状态对应的按钮权限
|
||||
- 待签发(0) → 修改 + 删除
|
||||
- 已签发(1) → 撤回
|
||||
- 其余状态(2-7) → 仅详情
|
||||
- 列宽从 160px 调整为 180px，适配按钮文本
|
||||
@@ -1,103 +0,0 @@
|
||||
# Bug #494 分析报告
|
||||
|
||||
## Bug 描述
|
||||
住院医生工作站-检查申请："申请单名称"字段显示为通用名称"检查申请单"，未展示具体检查项目名称。
|
||||
|
||||
## 代码分析
|
||||
|
||||
### 数据流
|
||||
|
||||
1. **保存时**（medicalExaminations.vue → saveCheckd → RequestFormManageAppServiceImpl.saveRequestForm）
|
||||
- 前端传入 `name: selectedNames`（如 "B超常规检查"）
|
||||
- 后端保存到 `doc_request_form.name` 字段 ✅
|
||||
|
||||
2. **查询时**（RequestFormManageAppMapper.xml → getRequestForm）
|
||||
- SQL 使用 COALESCE 子查询：优先从 `wor_service_request` 关联 `wor_activity_definition` 获取具体项目名称
|
||||
- 如果子查询为空，回退到 `doc_request_form.name` 字段 ✅
|
||||
|
||||
3. **详情查询**（RequestFormManageAppMapper.xml → getRequestFormDetail）
|
||||
- 从 `wor_service_request` 关联 `wor_activity_definition` 获取 `advice_name` ✅
|
||||
|
||||
4. **前端展示**（examineApplication.vue → buildApplicationName）
|
||||
- 优先使用 `requestFormDetailList[0].adviceName`
|
||||
- 回退到 `row.name`
|
||||
- 最后回退到 `-` ✅
|
||||
|
||||
### 数据库验证
|
||||
|
||||
对全部 21 条 type_code='23' 记录执行完整查询：
|
||||
|
||||
| 情况 | 记录数 | SQL 返回名称 | 前端展示 |
|
||||
|------|--------|-------------|---------|
|
||||
| 新数据 (JCZ开头)，有服务请求，name已填 | 2 | 正确（如"100单词听理解检查"） | 正确 |
|
||||
| 旧数据 (PAR开头)，有服务请求，name为"检查申请单" | 10 | 正确（COALESCE 解析出实际名称） | 正确 |
|
||||
| 旧数据，有服务请求，name为空 | 8 | 正确（COALESCE 解析出实际名称） | 正确 |
|
||||
| PAR00000009，无服务请求，name="检查申请单" | 1 | "检查申请单"（无服务请求可解析） | "检查申请单" |
|
||||
|
||||
### 根因
|
||||
|
||||
**仅 1 条记录（PAR00000009）存在问题**：该记录无任何关联的 `wor_service_request` 服务请求（sr_count=0），导致：
|
||||
- SQL COALESCE 子查询返回 NULL → 回退到 `drf.name` = "检查申请单"
|
||||
- 详情查询返回空列表 → `buildApplicationName` 回退到 `row.name` = "检查申请单"
|
||||
|
||||
这条记录以 PAR 开头（非 JCZ），是通过非标准路径创建的脏数据，缺少关联的服务请求记录。
|
||||
|
||||
**其余 20 条记录（95%）的 SQL COALESCE 已正确解析出具体项目名称**。
|
||||
|
||||
### 修复方案
|
||||
|
||||
对于**无服务请求的孤儿申请单**，前端 `buildApplicationName` 函数已正确回退到 `row.name`。问题在于：
|
||||
1. `row.name` 存储的是通用名称 "检查申请单"
|
||||
2. 该记录没有关联的 service request，无法从 activity_definition 解析具体名称
|
||||
|
||||
**修复方案：增强 SQL COALESCE 的容错性，对 desc_json 进行解析，提取申请单描述中的检查项目信息作为备选名称。**
|
||||
|
||||
但这不现实——desc_json 只包含表单字段（症状、体征等），不包含项目名称。
|
||||
|
||||
**更合理的修复：确保保存时 name 字段始终填入具体项目名称。**
|
||||
|
||||
检查 `medicalExaminations.vue` 的 submit 方法：
|
||||
```js
|
||||
const selectedNames = applicationListAllFilter.map(item => item.adviceName).join('+');
|
||||
```
|
||||
|
||||
前端传入的 name 是用 `+` 拼接的多个项目名称。这个值被保存到 `doc_request_form.name`。
|
||||
|
||||
SQL COALESCE 子查询使用 `STRING_AGG(DISTINCT wad.name, '、')`，用 `、` 分隔。
|
||||
|
||||
**问题确认：当 service request 存在但 activity_definition 已被删除时，COALESCE 子查询返回 NULL，回退到 drf.name。但 drf.name 可能为空或为"检查申请单"（旧数据）。**
|
||||
|
||||
对于这种 edge case，**应该增强 SQL 容错**：当 `drf.name` 也为空或通用名称时，显示更友好的默认文本。
|
||||
|
||||
不过，**当前代码对绝大多数场景已经正确工作**。唯一显示"检查申请单"的是 PAR00000009 这条孤儿数据。
|
||||
|
||||
## 修复计划
|
||||
|
||||
增强前端 `buildApplicationName` 函数的容错性：
|
||||
- 当 detailList 为空时，检查 `row.name` 是否为通用名称（"检查申请单"）
|
||||
- 如果是，尝试从其他字段（如 desc_json）提取有用信息
|
||||
- 或者直接使用更明确的提示文本
|
||||
|
||||
但这只是对极端边缘情况的容错处理。根本问题是 PAR00000009 这条脏数据。
|
||||
|
||||
## 修复结果：✅ 已成功修复（commit fd9309f1）
|
||||
|
||||
### 修复内容（3处改动，30行）
|
||||
|
||||
1. **后端 SQL（RequestFormManageAppMapper.xml）**
|
||||
- 原：`drf.NAME` 直接取存储的名称
|
||||
- 改：`COALESCE((SELECT STRING_AGG(DISTINCT wad.name, '、') FROM wor_service_request LEFT JOIN wor_activity_definition ...), drf.name)`
|
||||
- 效果：优先从服务请求关联的诊疗定义中动态解析具体项目名称，回退到存储名称
|
||||
|
||||
2. **前端展示（examineApplication.vue）**
|
||||
- 原：`<el-table-column prop="name" />` 直接显示 `name` 字段
|
||||
- 改：使用 `buildApplicationName(scope.row)` 函数，优先使用 `requestFormDetailList[0].adviceName`
|
||||
|
||||
3. **前端提交（medicalExaminations.vue）**
|
||||
- 增加 `adviceName: item.adviceName` 到提交数据中，确保后端能正确关联项目名称
|
||||
|
||||
### 数据库验证结果
|
||||
|
||||
全部 21 条 type_code='23' 记录中：
|
||||
- 20 条（95%）SQL 正确返回具体项目名称（如 "B超常规检查"、"100单词听理解检查"）
|
||||
- 1 条（PAR00000009）无关联服务请求（孤儿数据），回退显示 "检查申请单"（符合预期）
|
||||
@@ -1,78 +0,0 @@
|
||||
# Bug #498 分析报告
|
||||
|
||||
## Bug 描述
|
||||
【住院医生工作站-检查申请】检查申请列表操作项过于单一，缺失修改/作废/打印/看报告等核心临床操作
|
||||
|
||||
## 阶段1：深度分析
|
||||
|
||||
### 当前代码状态
|
||||
`examineApplication.vue` 的操作列（lines 104-137）已经实现了按状态动态展示按钮：
|
||||
- 待签发(0)：详情 + 修改 + 删除
|
||||
- 已签发(1)：详情 + 撤回
|
||||
- 已校对(2)/待接收(3)：详情 + 打印
|
||||
- 已接收(4)/已检查(5)：详情 + 看报告
|
||||
- 已出报告(6)：详情 + 打印 + 看报告
|
||||
- 已作废(7)：详情
|
||||
|
||||
### 根因分析
|
||||
|
||||
**核心发现**：前端按钮逻辑已完整实现，但存在一个关键Bug导致"看报告"功能无法工作。
|
||||
|
||||
#### Bug：`handleViewReport` 传递错误的参数
|
||||
|
||||
前端代码 (examineApplication.vue:920):
|
||||
```js
|
||||
const res = await getTestResult({ prescriptionNo: row.prescriptionNo });
|
||||
```
|
||||
|
||||
后端接口 (DoctorStationAdviceController.java:190-192):
|
||||
```java
|
||||
@GetMapping(value = "/test-result")
|
||||
public R<?> getTestResult(@RequestParam(value = "encounterId") Long encounterId) {
|
||||
return iDoctorStationAdviceAppService.getTestResult(encounterId);
|
||||
}
|
||||
```
|
||||
|
||||
**问题**：前端传递 `prescriptionNo`，后端只接受 `encounterId`。Spring 忽略未知参数，`encounterId` 为 null，后端直接返回空列表。
|
||||
|
||||
后端服务实现 (DoctorStationAdviceAppServiceImpl.java:2357-2376):
|
||||
```java
|
||||
public R<?> getTestResult(Long encounterId) {
|
||||
if (encounterId == null) {
|
||||
return R.ok(new ArrayList<>()); // encounterId为空时直接返回空列表
|
||||
}
|
||||
// ... 查询逻辑 ...
|
||||
}
|
||||
```
|
||||
|
||||
#### 数据流追踪
|
||||
1. 前端 `handleViewReport(row)` → 获取 `row.prescriptionNo`
|
||||
2. 调用 `getTestResult({ prescriptionNo: "JCZ26051600001" })`
|
||||
3. 后端接收：`encounterId = null`（参数名不匹配，被忽略）
|
||||
4. 后端返回空列表 → 前端显示"暂未生成报告"
|
||||
|
||||
### 修复方案
|
||||
将 `handleViewReport` 中的参数从 `prescriptionNo` 改为 `encounterId`，使用 `row.encounterId` 或 `patientInfo.value.encounterId`。
|
||||
|
||||
### 后端 API 完整性检查
|
||||
| 操作 | 前端调用 | 后端接口 | 状态 |
|
||||
|------|---------|---------|------|
|
||||
| 修改 | saveCheckd → POST /save-check | saveRequestForm (支持编辑) | ✅ |
|
||||
| 删除 | deleteRequestForm → POST /delete | deleteRequestForm (验证status=0) | ✅ |
|
||||
| 撤回 | withdrawRequestForm → POST /withdraw | withdrawRequestForm (验证status=2) | ✅ |
|
||||
| 打印 | 前端 window.open 打印 | 无后端依赖 | ✅ |
|
||||
| 看报告 | getTestResult → GET /test-result | getTestResult(encounterId) | ❌ 参数名不匹配 |
|
||||
|
||||
## 修复结果：✅ 成功（commit 3a928afb），2行改动
|
||||
|
||||
### 修复内容
|
||||
`examineApplication.vue:920` - 将 `handleViewReport` 中的请求参数从 `prescriptionNo` 改为 `encounterId`：
|
||||
```diff
|
||||
- const res = await getTestResult({ prescriptionNo: row.prescriptionNo });
|
||||
+ const res = await getTestResult({ encounterId: row.encounterId || patientInfo.value?.encounterId });
|
||||
```
|
||||
|
||||
### 说明
|
||||
- 操作列的动态按钮逻辑（修改/删除/撤回/打印/看报告）已在之前的提交中完整实现
|
||||
- 本修复解决了"看报告"功能因参数名不匹配导致始终返回空数据的问题
|
||||
- 其余操作（修改/删除/撤回/打印）的后端接口参数均正确匹配
|
||||
@@ -35,7 +35,8 @@ public class EncounterDiagnosisServiceImpl extends ServiceImpl<EncounterDiagnosi
|
||||
*/
|
||||
@Override
|
||||
public void deleteEncounterDiagnosisInfos(Long encounterId) {
|
||||
// 仅删除就诊诊断关联记录，不删除cli_condition（否则会导致传染病报卡diag_id失效）
|
||||
// 不删除中医
|
||||
conditionMapper.deleteByEncounterId(encounterId);
|
||||
baseMapper.deleteByEncounterId(encounterId);
|
||||
}
|
||||
|
||||
|
||||
@@ -38,9 +38,6 @@ const filteredList = ref([]); // 过滤后的数据列表
|
||||
const hasLoaded = ref(false); // 标记是否已加载数据
|
||||
const isLoading = ref(false); // 标记是否正在加载
|
||||
|
||||
// 标记是否正在执行搜索（用于防止 preloadedData 覆盖搜索结果）
|
||||
const isSearching = ref(false);
|
||||
|
||||
// 获取诊疗项目列表
|
||||
function getList() {
|
||||
if (hasLoaded.value) return; // 如果已经加载过数据，则不再重复请求
|
||||
@@ -69,7 +66,6 @@ function getList() {
|
||||
// 服务端搜索（当用户输入搜索关键词时）
|
||||
function searchList(searchKey) {
|
||||
if (!searchKey || searchKey.trim() === '') return;
|
||||
isSearching.value = true;
|
||||
isLoading.value = true;
|
||||
// 使用较大的pageSize确保搜索结果尽可能多
|
||||
getDiagnosisTreatmentList({ statusEnum: 2, searchKey: searchKey.trim(), pageSize: 5000, pageNo: 1 })
|
||||
@@ -83,7 +79,6 @@ function searchList(searchKey) {
|
||||
})
|
||||
.finally(() => {
|
||||
isLoading.value = false;
|
||||
isSearching.value = false;
|
||||
});
|
||||
}
|
||||
|
||||
@@ -124,8 +119,6 @@ watch(
|
||||
watch(
|
||||
() => props.preloadedData,
|
||||
(newData) => {
|
||||
// 正在搜索时跳过，避免覆盖searchList的搜索结果
|
||||
if (isSearching.value) return;
|
||||
if (newData && newData.length > 0) {
|
||||
diagnosisTreatmentList.value = newData;
|
||||
filterList(); // 更新过滤
|
||||
|
||||
@@ -179,14 +179,11 @@
|
||||
|
||||
<div v-if="descJsonData && hasMatchedFields" class="applicationShow-container-content">
|
||||
<el-descriptions title="申请单描述" :column="2">
|
||||
<el-descriptions-item
|
||||
v-for="key in orderedDescFieldKeys"
|
||||
:key="key"
|
||||
v-if="descJsonData[key] != null && descJsonData[key] !== ''"
|
||||
:label="getFieldLabel(key)"
|
||||
>
|
||||
{{ transformField(key, descJsonData[key]) || '-' }}
|
||||
</el-descriptions-item>
|
||||
<template v-for="(value, key) in descJsonData" :key="key">
|
||||
<el-descriptions-item v-if="isFieldMatched(key)" :label="getFieldLabel(key)">
|
||||
{{ transformField(key, value) || '-' }}
|
||||
</el-descriptions-item>
|
||||
</template>
|
||||
</el-descriptions>
|
||||
</div>
|
||||
|
||||
@@ -514,13 +511,6 @@ const hasMatchedFields = computed(() => {
|
||||
return Object.keys(descJsonData.value).some((key) => isFieldMatched(key));
|
||||
});
|
||||
|
||||
// Ordered field keys for detail display and print, matching the bug requirement order
|
||||
const orderedDescFieldKeys = [
|
||||
'targetDepartment', 'urgencyLevel', 'allergyHistory', 'examinationPurpose',
|
||||
'expectedExaminationTime', 'medicalHistorySummary', 'symptom', 'sign',
|
||||
'clinicalDiagnosis', 'otherDiagnosis', 'relatedResult', 'attention',
|
||||
];
|
||||
|
||||
/** 查询科室 */
|
||||
const getLocationInfo = async () => {
|
||||
try {
|
||||
@@ -689,16 +679,15 @@ const handlePrint = async (row) => {
|
||||
}
|
||||
|
||||
// 构建 descJson 字段行（与详情弹窗展示的字段一致）
|
||||
const fieldKeys = orderedDescFieldKeys;
|
||||
const fieldKeys = ['targetDepartment', 'symptom', 'sign', 'clinicalDiagnosis', 'otherDiagnosis', 'relatedResult', 'attention'];
|
||||
let descFieldsHtml = '';
|
||||
fieldKeys.forEach((key) => {
|
||||
const label = labelMap[key] || key;
|
||||
const value = transformField(key, descData[key]);
|
||||
if (descData[key] != null && descData[key] !== '' && value != null && value !== '') {
|
||||
if (descData[key] != null && descData[key] !== '') {
|
||||
descFieldsHtml += `
|
||||
<div class="info-row">
|
||||
<span class="label">${label}：</span>
|
||||
<span class="value">${value}</span>
|
||||
<span class="value">${descData[key]}</span>
|
||||
</div>`;
|
||||
}
|
||||
});
|
||||
@@ -917,7 +906,7 @@ const handleViewReport = async (row) => {
|
||||
reportLoading.value = true;
|
||||
reportData.value = null;
|
||||
try {
|
||||
const res = await getTestResult({ encounterId: row.encounterId || patientInfo.value?.encounterId });
|
||||
const res = await getTestResult({ prescriptionNo: row.prescriptionNo });
|
||||
if (res.code === 200) {
|
||||
if (res.data) {
|
||||
// 支持两种返回格式：
|
||||
|
||||
@@ -471,7 +471,7 @@ const submit = () => {
|
||||
unitCode: item.unitCode || priceInfo.unitCode || '',
|
||||
unitPrice: item.price ?? priceInfo.price ?? 0,
|
||||
totalPrice: item.price ?? priceInfo.price ?? 0,
|
||||
positionId: form.targetDepartment || item.positionId, // 用户手动选择的发往科室优先于项目默认执行科室
|
||||
positionId: item.positionId,
|
||||
ybClassEnum: item.ybClassEnum,
|
||||
conditionId: item.conditionId,
|
||||
encounterDiagnosisId: item.encounterDiagnosisId,
|
||||
|
||||
Reference in New Issue
Block a user