Compare commits
51 Commits
d06f6ceeb0
...
zhugeliang
| Author | SHA1 | Date | |
|---|---|---|---|
| 68fe3d8e7a | |||
| 3d438838ae | |||
| 26ec2eefda | |||
| d5efbd3c6a | |||
| b3eff726db | |||
| 0fc7a8623e | |||
| ada292405a | |||
| a4370b00db | |||
| f741a1f70d | |||
| e67d3b78d7 | |||
|
|
40ca304342 | ||
|
|
3a40740538 | ||
| 7c0d103409 | |||
| ad85e4d284 | |||
| 8b6af8dd61 | |||
| e330372355 | |||
| f72bee6c95 | |||
| 6b347e9136 | |||
|
|
58e391bd2c | ||
| 9f615df3f9 | |||
| d47353a711 | |||
| dbe9fdadc1 | |||
| 7ae7cfa35c | |||
| 01da7b942a | |||
| b8666e535b | |||
| 1f7d637265 | |||
| 910f59ce9d | |||
| 0328f9642f | |||
| e6a61ea5aa | |||
| 4809b3571d | |||
| bfe544cfb3 | |||
| 37c2377b66 | |||
| 89ca306348 | |||
| 31f7950779 | |||
| ce1161caea | |||
| 046a3e4703 | |||
| e245f4ec02 | |||
| 37963dde1d | |||
| 4b2690d1ad | |||
| 8bfe4f2c23 | |||
| 08ccf9aba8 | |||
| 2d43b1cddc | |||
| 327b750c6e | |||
| 3c1087a2d1 | |||
| 4e2ee57274 | |||
| eee65a4517 | |||
| d30673ad51 | |||
| f369ea419e | |||
| 5db20ddcc2 | |||
| 1488b707e8 | |||
| 07a8e55895 |
@@ -1,27 +0,0 @@
|
||||
# Bug #545 分析报告:长效诊断标识设置保存就清空
|
||||
|
||||
## 根因定位
|
||||
|
||||
保存诊断后,前端调用 `getList()` 刷新数据,`getEncounterDiagnosis` SQL 查询未包含 `long_term_flag` 字段,且 `DiagnosisQueryDto` 缺少对应属性,导致返回数据中不含 `longTermFlag`,前端覆盖 `form.value.diagnosisList` 后下拉框清空。
|
||||
|
||||
## 数据流追踪
|
||||
|
||||
1. 前端用户在 `diagnosis.vue` 第218-231行的 el-select 下拉框选择"长期有效/临时有效",值绑定到 `scope.row.longTermFlag`
|
||||
2. 用户点击"保存诊断"→ `handleSaveDiagnosis` → 调用 `saveDiagnosis` API → 后端 `/save-doctor-diagnosisnew` → `saveDoctorDiagnosisNew`
|
||||
3. 后端 `saveDoctorDiagnosisNew` 第376行和第404行已正确保存 `encounterDiagnosis.setLongTermFlag(saveDiagnosisChildParam.getLongTermFlag())`
|
||||
4. 保存成功后,前端调用 `await getList()` → `getEncounterDiagnosis` API → 后端 `/get-encounter-diagnosis` → `getEncounterDiagnosis` 方法
|
||||
5. **断点在此**: SQL (`DoctorStationDiagnosisAppMapper.xml:122-150`) SELECT 列表缺少 `T1.long_term_flag`,DTO (`DiagnosisQueryDto.java`) 缺少 `longTermFlag` 属性
|
||||
6. 前端第351行 `form.value.diagnosisList = res.data.filter(...)` 用不含 `longTermFlag` 的数据替换了原有数据
|
||||
7. 结果:`longTermFlag` 变为 `undefined`,下拉框清空
|
||||
|
||||
## 修复方案
|
||||
|
||||
1. **SQL**: `DoctorStationDiagnosisAppMapper.xml` getEncounterDiagnosis 查询新增 `T1.long_term_flag AS longTermFlag`
|
||||
2. **DTO**: `DiagnosisQueryDto.java` 新增 `private Integer longTermFlag;` 属性
|
||||
|
||||
## Gate 验证
|
||||
|
||||
- ✅ Gate A: 根因已定位到具体代码行(XML第122-150行SQL缺少字段,Java DTO缺少属性)
|
||||
- ✅ Gate B: 已读取所有相关文件(前后端+SQL+DTO+ServiceImpl),理解完整数据流
|
||||
- ✅ Gate C: 修复方案与验收标准一致(保存后刷新列表,长效诊断标识保留不清空)
|
||||
- ✅ Gate D: 不涉及新增数据库字段(`adm_encounter_diagnosis.long_term_flag` 已存在,Entity 第89行已有定义)
|
||||
@@ -1,36 +0,0 @@
|
||||
# Bug #401 分析报告
|
||||
|
||||
## 问题描述
|
||||
门诊完诊审计日志错误:div_log 表中 pool_id 与 slot_id 存值与设计规范不符。
|
||||
|
||||
## 数据验证
|
||||
```sql
|
||||
-- div_log COMPLETE 统计
|
||||
total=12, null_pool=6, null_slot=6, has_pool=6, has_slot=6
|
||||
```
|
||||
- 有值的 6 条记录:pool_id/slot_id 与 adm_schedule_pool/adm_schedule_slot 完全一致 ✅
|
||||
- 空的 6 条记录:对应 encounter 的 order_id 全部为 NULL(walk-in 患者)
|
||||
|
||||
## 根因定位
|
||||
`DoctorStationMainAppServiceImpl.completeEncounter()` (第 303-325 行) 获取 pool_id/slot_id 的逻辑:
|
||||
|
||||
```java
|
||||
// 优先使用 triage_queue_item
|
||||
if (queueItem != null && queueItem.getPoolId() != null && queueItem.getSlotId() != null) {
|
||||
divPoolId = queueItem.getPoolId();
|
||||
divSlotId = queueItem.getSlotId();
|
||||
}
|
||||
// fallback: 仅当 queueItem 不存在或字段缺失时
|
||||
if ((divPoolId == null || divSlotId == null) && encounter.getOrderId() != null) {
|
||||
...
|
||||
}
|
||||
```
|
||||
|
||||
**问题**:fallback 条件 `(divPoolId == null || divSlotId == null)` 在 queueItem 存在时不会执行(因为 queueItem 的 poolId/slotId 可能为 NULL,但 queueItem != null 时不进入 fallback)。实际上,对于有 encounter.orderId 的患者(挂号患者),应该始终通过 order → schedule_slot 获取权威的 pool_id/slot_id。
|
||||
|
||||
## 修复方案
|
||||
调整 fallback 逻辑:只要有 encounter.orderId,就通过 order → schedule_slot 获取 pool_id/slot_id,不再依赖 queueItem 的字段值。queueItem 仅用于确定是否需要写审计日志的时机判断。
|
||||
|
||||
## 影响范围
|
||||
- 修改文件:`DoctorStationMainAppServiceImpl.java`(约 10 行调整)
|
||||
- 不涉及数据库 DDL 变更
|
||||
@@ -1,40 +0,0 @@
|
||||
# Bug #512 分析报告
|
||||
|
||||
## Bug 描述
|
||||
[住院护士站-汇总发药申请] "全选"开关功能失效,点击后下方医嘱明细未能联动勾选
|
||||
|
||||
## 根因分析
|
||||
|
||||
### 问题定位
|
||||
`index.vue` 的 `handelSwicthChange` 函数(第176-186行)和 `handleExecute` 函数(第200-202行)。
|
||||
|
||||
### 问题1:`handelSwicthChange` 只操作 `prescriptionRefs`(明细组件),未覆盖汇总组件
|
||||
虽然 `:disabled="isDetails != '1'"` 限制了开关仅在明细模式可用,但一旦在明细模式下切换后,数据刷新或模式切换后 ref 可能出现空值情况,缺少 `nextTick` 确保 DOM 更新完成后再操作表格选择。
|
||||
|
||||
### 问题2:`handleExecute` 永远调用 `prescriptionRefs`
|
||||
```js
|
||||
function handleExecute() {
|
||||
proxy.$refs['prescriptionRefs'].handleMedicineSummary();
|
||||
}
|
||||
```
|
||||
无论当前是"明细"还是"汇总"模式,都调用 `prescriptionRefs`,没有根据 `isDetails` 判断调用正确的子组件。
|
||||
|
||||
### 问题3:`summaryMedicineList.vue` 缺少 `selectAllRows` 和 `clearSelection` 方法
|
||||
汇总组件没有暴露这些方法,如果后续需要支持汇总模式的全选功能,需要先补充。
|
||||
|
||||
## 修复方案
|
||||
1. 在 `handelSwicthChange` 中添加 `nextTick` 确保 DOM 更新后再操作表格
|
||||
2. 修复 `handleExecute` 根据 `isDetails` 判断调用正确的子组件
|
||||
3. 为 `summaryMedicineList.vue` 添加 `selectAllRows` 和 `clearSelection` 方法
|
||||
|
||||
## 修复结果:✅ 成功,33行改动
|
||||
|
||||
### 修改内容
|
||||
1. `index.vue` - `handelSwicthChange` 改为 async 函数,添加 `nextTick` 确保 DOM 更新后再调用表格选择方法
|
||||
2. `index.vue` - `handelSwicthChange` 增加 `isDetails` 判断分支,覆盖明细和汇总两种模式
|
||||
3. `index.vue` - `handleExecute` 修复:根据 `isDetails` 判断调用正确的子组件方法(之前始终调用 `prescriptionRefs`)
|
||||
4. `index.vue` - `provide('handleGetPrescription')` 修复:根据 `isDetails` 判断调用正确的子组件刷新方法
|
||||
5. `index.vue` - 导入 `nextTick` from vue
|
||||
|
||||
### 构建验证
|
||||
`vite build --mode dev` 通过,无编译错误。
|
||||
@@ -1,78 +0,0 @@
|
||||
# Bug #526 分析报告
|
||||
|
||||
## Bug 描述
|
||||
[手术管理-门诊手术安排-计费] 勾选待签发状态的项目后上方签发按钮仍置灰不可用
|
||||
|
||||
## 根因定位
|
||||
|
||||
**文件**: `openhis-ui-vue3/src/views/clinicmanagement/bargain/component/prescriptionlist.vue`
|
||||
|
||||
### 签发按钮控制逻辑
|
||||
```vue
|
||||
<el-button type="primary" @click="handleSave()" :disabled="handleSaveDisabled"> 签发 </el-button>
|
||||
```
|
||||
`handleSaveDisabled` 是一个 `ref(false)`(第408行)。
|
||||
|
||||
### 两个更新机制都存在缺陷
|
||||
|
||||
**机制1:watcher(第458-475行)**
|
||||
```js
|
||||
watch(() => prescriptionList.value, (newValue) => {
|
||||
let saveList = prescriptionList.value.filter(item =>
|
||||
item.check && item.statusEnum == 1 && (Number(item.bizRequestFlag)==1 || !item.bizRequestFlag)
|
||||
)
|
||||
handleSaveDisabled.value = saveList.length == 0
|
||||
}, { immediate: true, deep: false }); // ← deep: false!
|
||||
```
|
||||
`deep: false` 意味着 watcher 只能检测到数组引用变化(push/splice),**无法检测到数组元素的 `check` 属性变化**。用户勾选/取消勾选checkbox时,watcher 不会触发。
|
||||
|
||||
**机制2:changeCheck 函数(第986-1019行)**
|
||||
```js
|
||||
groupList.value.map(k => {
|
||||
if(k.check) {
|
||||
if(k.statusEnum == 1) {
|
||||
// 待签发:设置 handleSaveDisabled = false
|
||||
}
|
||||
if(k.statusEnum == 2) {
|
||||
// 已签发:设置 handleSaveDisabled = true
|
||||
}
|
||||
}
|
||||
})
|
||||
```
|
||||
问题:
|
||||
1. **用 `.map()` 遍历逐个设置按钮状态**,后面的项会覆盖前面项的设置
|
||||
2. 如果列表中先有"待签发"项(设置 `handleSaveDisabled = false`),后面遍历时碰到一个不满足 `bizRequestFlag` 条件的项,会被覆盖为 `true`
|
||||
3. 当取消勾选最后一个项目时,`groupList` 为空,`.map()` 不执行任何操作,按钮状态保持原样
|
||||
|
||||
### 数据流
|
||||
1. `getListInfo()` 从后端加载数据 → `prescriptionList.value` 被赋值
|
||||
2. 用户勾选checkbox → `scope.row.check` 变为 true(v-model 自动)
|
||||
3. `@change` 触发 `changeCheck()` → 更新 `groupIndexList` 和 `groupList`
|
||||
4. **但 `changeCheck` 中逐个遍历设置按钮状态的逻辑不可靠** → 签发按钮可能仍为 disabled
|
||||
|
||||
## 修复方案
|
||||
|
||||
将 `handleSaveDisabled` 和 `handleSingOutDisabled` 从 `ref` + 手动管理改为 **`computed` 属性**,直接从 `prescriptionList.value` 实时计算按钮状态:
|
||||
|
||||
```js
|
||||
const handleSaveDisabled = computed(() => {
|
||||
return !prescriptionList.value.some(item =>
|
||||
item.check && item.statusEnum == 1 && (Number(item.bizRequestFlag) === 1 || !item.bizRequestFlag)
|
||||
)
|
||||
})
|
||||
|
||||
const handleSingOutDisabled = computed(() => {
|
||||
return !prescriptionList.value.some(item =>
|
||||
item.check && item.statusEnum == 2 && item.chargeStatus != 5 &&
|
||||
(Number(item.bizRequestFlag) === 1 || !item.bizRequestFlag)
|
||||
)
|
||||
})
|
||||
```
|
||||
|
||||
**优势**:
|
||||
- 响应式自动计算,任何 `check`、`statusEnum` 变化都会触发重新计算
|
||||
- 不需要 watcher(删除 deep: false 的那个)
|
||||
- 不需要在 changeCheck 中手动管理按钮状态
|
||||
- 逻辑与 `handleSave` 和 `handleSingOut` 中的筛选条件一致
|
||||
|
||||
同时从 `changeCheck` 函数中移除按钮状态管理代码(第986-1019行的 `.map()` 部分)。
|
||||
@@ -1,55 +0,0 @@
|
||||
# Bug #443 分析报告
|
||||
|
||||
## Bug 描述
|
||||
手术计费:点击"签发"耗材时异常报错
|
||||
|
||||
## 复现步骤
|
||||
1. 以"手术室护士"角色登录
|
||||
2. 进入【门诊手术安排】→ 点击【计费】
|
||||
3. 勾选一条状态为"待签发"的耗材项目
|
||||
4. 点击【签发】按钮
|
||||
|
||||
## 预期 vs 实际
|
||||
- 预期:提示"签发成功",状态变为"已签发"
|
||||
- 实际:弹出"后端程序异常"报错
|
||||
|
||||
## 代码分析
|
||||
|
||||
### 前端流程
|
||||
1. `surgicalschedule/index.vue` → `handleChargeCharge()` 打开计费弹窗
|
||||
2. 弹窗中使用 `prescriptionlist.vue` 组件,传入 `generateSourceEnum=6`, `sourceBillNo=operCode`
|
||||
3. 用户勾选"待签发"项目 → 点击"签发" → 触发 `handleSave()`
|
||||
4. `handleSave()` 过滤 `item.check && item.statusEnum == 1 && (Number(item.bizRequestFlag)==1||!item.bizRequestFlag)`
|
||||
5. 构造请求体:解析 `contentJson` + 补充顶层字段(encounterId, patientId, adviceType 等)
|
||||
6. 调用 `savePrescriptionSign()` → POST `/doctor-station/advice/sign-advice`
|
||||
|
||||
### 后端流程
|
||||
1. `DoctorStationAdviceController.signAdvice()` → `saveAdvice(param, SIGN_ADVICE)`
|
||||
2. `saveAdvice()` 校验 encounterId/patientId 非空
|
||||
3. 按 adviceType 分类:药品(1)、耗材(2)、诊疗(3)
|
||||
4. 耗材走 `handDevice(deviceList, curDate, adviceOpType)` 处理
|
||||
5. 签发后更新 DeviceRequest 状态为 ACTIVE(2)
|
||||
6. 更新 ChargeItem 状态:DRAFT(0)→PLANNED(1) 或 BILLABLE(2)→PLANNED(1)
|
||||
|
||||
### 可能根因
|
||||
|
||||
**根因1:dbOpType 语义错误**
|
||||
- 前端 `handleSave()` 对已存在的耗材发送 `dbOpType: '1'` (INSERT)
|
||||
- 后端 `handDevice` 中 `insertOrUpdateList` 通过 `requestId != null` 过滤包含这些项
|
||||
- 但对于 INSERT 操作,如果 DeviceRequest 已存在,`saveOrUpdate` 走 UPDATE 路径
|
||||
- 问题在于:INSERT 语义下某些字段(如 `bus_no`)仅在 `is_save=true` 时设置
|
||||
|
||||
**根因2:contentJson 数据一致性**
|
||||
- 前端将 `contentJson` 解析后 spread 回对象,再序列化为新的 JSON 发送
|
||||
- 后端 `handDevice` 直接将该 JSON 存入 `content_json` 字段
|
||||
- 如果原始 `content_json` 中的字段名与 `AdviceSaveDto` 不匹配(如 snake_case vs camelCase),可能导致数据丢失
|
||||
|
||||
**根因3:缺少空列表校验**
|
||||
- `handleSave()` 未校验 `saveList.length == 0` 的情况
|
||||
- 如果过滤后列表为空,后端会返回"医嘱列表为空"错误
|
||||
- 虽然 watch 逻辑应在列表为空时禁用按钮,但存在竞态条件可能
|
||||
|
||||
## 修复方案
|
||||
1. 前端 `handleSave()` 添加 `saveList.length == 0` 校验(防御性编程)
|
||||
2. 前端 `handleSave()` 对已存在记录(requestId 不为空)使用 `dbOpType: '2'` (UPDATE) 而非 '1' (INSERT)
|
||||
3. 前端 `handleSave()` 确保关键字段(quantity, unitCode)从顶层补充
|
||||
@@ -1,85 +0,0 @@
|
||||
# Bug #468 分析报告
|
||||
|
||||
## Bug 描述
|
||||
[住院医生工作站-检验申请] 列表页缺失【单据状态】列,无法闭环管理检验医嘱执行进度
|
||||
|
||||
## 阶段1:深度分析
|
||||
|
||||
### 数据流追踪
|
||||
|
||||
1. **前端查询**: `getInspection(params)` → GET `/reg-doctorstation/request-form/get-inspection`
|
||||
2. **后端控制器**: `RequestFormManageController.getInspectionRequestForm()` → 调用 `iRequestFormManageAppService.getRequestForm()`
|
||||
3. **后端服务**: `RequestFormManageAppServiceImpl.getRequestForm()` → 调用 `requestFormManageAppMapper.getRequestForm()`
|
||||
4. **SQL查询**: `RequestFormManageAppMapper.xml` 中的 `getRequestForm` 语句
|
||||
5. **状态计算**: SQL 使用 CASE WHEN 根据 `wor_service_request.status_enum` 计算 `computed_status`
|
||||
6. **前端渲染**: `parseBillStatus(scope.row.billStatus ?? scope.row.status)` 显示状态文本
|
||||
|
||||
### 状态映射关系
|
||||
|
||||
**后端 ServiceRequest.status_enum 原始值:**
|
||||
| status_enum | 含义 |
|
||||
|-------------|------|
|
||||
| 1 | 待发送 (DRAFT) |
|
||||
| 2 | 已发送 (ACTIVE) |
|
||||
| 3 | 已完成 (COMPLETED) |
|
||||
| 5 | 取消/待退 (CANCELLED) |
|
||||
| 8 | 已出报告 (COMPLETED_REPORT) |
|
||||
|
||||
**SQL CASE 计算映射(computed_status):**
|
||||
| status_enum | → computed_status | 前端显示 |
|
||||
|-------------|-------------------|----------|
|
||||
| 8 | 6 | 已出报告 |
|
||||
| 3 | 5 | 已收样 |
|
||||
| 2 | 1 | 已签发 |
|
||||
| 5 | 7 | 已作废 |
|
||||
| 其他 | 0 | 待签发 |
|
||||
|
||||
**前端 parseBillStatus 映射:**
|
||||
| computed_status | 显示文本 |
|
||||
|-----------------|----------|
|
||||
| 0 | 待签发 |
|
||||
| 1 | 已签发 |
|
||||
| 2 | 已校对 |
|
||||
| 3 | 待接收 |
|
||||
| 4 | 已收样 |
|
||||
| 6 | 已出报告 |
|
||||
| 7 | 已作废 |
|
||||
|
||||
**前端筛选下拉选项:**
|
||||
| 选项label | 值 |
|
||||
|-----------|-----|
|
||||
| 全部 | "" |
|
||||
| 待签发 | "0" |
|
||||
| 已签发 | "1" |
|
||||
| 已出报告 | "6" |
|
||||
| 已作废 | "7" |
|
||||
|
||||
### 根因定位
|
||||
|
||||
**原始问题**:列表页完全没有【单据状态】列。
|
||||
|
||||
**已有修复**(已在 develop 分支合并):
|
||||
1. 新增 `el-table-column` 单据状态列(位于申请单号之后)
|
||||
2. 新增 `parseBillStatus()` 函数用于状态码→文本转换
|
||||
3. 新增筛选表单中的单据状态下拉选择
|
||||
4. 后端 SQL 新增 `computed_status` 动态计算逻辑
|
||||
5. 前端使用 `scope.row.billStatus ?? scope.row.status` 兼容字段名
|
||||
|
||||
## 修复结果
|
||||
|
||||
✅ 成功,Bug #468 已在 develop 分支修复并合并。
|
||||
|
||||
当前 guanyu 分支与 develop 分支代码完全一致(diff 为空),无需额外代码改动。
|
||||
|
||||
已有提交记录:
|
||||
- a95c9c9f - 列表页新增单据状态列
|
||||
- ae50a704 - 列表页新增【单据状态】列
|
||||
- 02b9dc87 / e694b758 / a99ecaee - 修复前后端状态码映射不一致
|
||||
|
||||
验证通过:
|
||||
- ✅ 表格列存在(line 92-96)
|
||||
- ✅ 列位置正确(申请单号之后)
|
||||
- ✅ parseBillStatus 覆盖所有后端状态
|
||||
- ✅ 筛选表单支持状态过滤
|
||||
- ✅ 操作列按状态动态显示按钮
|
||||
- ✅ 后端 SQL computed_status 计算正确
|
||||
@@ -1,56 +0,0 @@
|
||||
# Bug #491 分析报告
|
||||
|
||||
## Bug 信息
|
||||
- **标题**: 【执行科室配置】保存配置时系统报错
|
||||
- **报错信息**: `Cannot invoke "com.openhis.administration.domain.Organization.getName()" because the return value of "com.openhis.administration.service..." is null`
|
||||
- **严重程度**: 3 | **优先级**: 3 | **类型**: codeerror
|
||||
|
||||
## 复现步骤
|
||||
1. 登录 HIS 系统 → 【系统管理】→【业务规则配置】→【执行科室配置】
|
||||
2. 左侧选择科室(如"超声诊断科")
|
||||
3. 新增或修改某行的时间区间
|
||||
4. 点击【保存】按钮
|
||||
5. 顶部弹出红色错误提示(NPE)
|
||||
|
||||
## 根因分析
|
||||
|
||||
### 文件定位
|
||||
- `openhis-server-new/.../appservice/impl/OrganizationLocationAppServiceImpl.java`(第161-175行)
|
||||
|
||||
### 根本原因
|
||||
在 `addOrEditOrgLoc` 方法中,保存时会检查时间冲突。当发现冲突时,代码需要获取冲突记录的科室名称用于错误提示:
|
||||
|
||||
```java
|
||||
// 第171-172行
|
||||
Organization org = organizationService.getById(organizationLocation.getOrganizationId());
|
||||
String organizationName = org.getName(); // NPE 这里!
|
||||
```
|
||||
|
||||
**问题**:`organizationService.getById()` 可能返回 `null`(当冲突记录的 `organizationId` 指向已被删除的机构时),直接调用 `.getName()` 导致 NPE。
|
||||
|
||||
### 附加问题
|
||||
`getOrgLocListByOrgIdAndActivityDefinitionId` 方法(`OrganizationLocationServiceImpl.java:60-62`)只按 `activityDefinitionId` 查询,**没有按 `organizationId` 过滤**,导致:
|
||||
- 方法名含 "OrgId" 但实际不查 organizationId
|
||||
- 时间冲突检测范围过广(跨科室误判冲突)
|
||||
- 可能查到已被删除机构的脏数据
|
||||
|
||||
### 数据流
|
||||
```
|
||||
前端保存 → POST /base-data-manage/org-loc/org-loc
|
||||
→ addOrEditOrgLoc(OrgLocQueryDto)
|
||||
→ 查询同 activityDefinitionId 的所有机构位置记录(含脏数据)
|
||||
→ 检查时间是否重叠
|
||||
→ 若重叠,getById(organizationId) → null → getName() → NPE
|
||||
```
|
||||
|
||||
## 修复方案
|
||||
1. `OrganizationLocationAppServiceImpl.java` 第172行:增加 `org != null` 判空,回退为 `"未知科室"`
|
||||
2. `IOrganizationLocationService.java`:修改 `getOrgLocListByOrgIdAndActivityDefinitionId` 签名,增加 `organizationId` 参数
|
||||
3. `OrganizationLocationServiceImpl.java`:查询条件增加 `.eq(OrganizationLocation::getOrganizationId, organizationId)`
|
||||
4. `OrganizationLocationAppServiceImpl.java` 第162行:调用时传入 `orgLoc.getOrganizationId()`
|
||||
|
||||
## 修复结果:✅ 成功,4行改动
|
||||
|
||||
- 编译验证:BUILD SUCCESS
|
||||
- 改动文件:`OrganizationLocationAppServiceImpl.java`、`IOrganizationLocationService.java`、`OrganizationLocationServiceImpl.java`
|
||||
- 已提交并推送到远程分支 guanyu
|
||||
@@ -1,53 +0,0 @@
|
||||
## Bug #530 分析报告
|
||||
|
||||
**标题**: [住院护士站-医嘱校对] 患者查询触发 SQL 类型匹配错误,导致勾选患者列表后后端报错
|
||||
|
||||
### 数据流追踪
|
||||
|
||||
1. `patientList.vue` → 树节点勾选触发 `handleCheckChange`
|
||||
2. `handleCheckChange` → `updatePatientInfoList(checkedPatients)` 存储选中的患者节点
|
||||
3. `handleGetPrescription()` 被触发 → `prescriptionList.vue` 的 `handleGetPrescription`
|
||||
4. 前端构造 encounterIds: `patientInfoList.value.map((i) => i.encounterId).join(',')`
|
||||
5. 后端解析: `Arrays.stream(encounterIds.split(",")).map(Long::parseLong).toList()`
|
||||
6. SQL 执行: `SELECT ... FROM ... WHERE ii.encounter_id IN (?, ?, ...)`
|
||||
|
||||
### 根因定位
|
||||
|
||||
**`patientList.vue` 第122行**:患者数据格式化时
|
||||
```javascript
|
||||
const patients = records.map((item) => ({
|
||||
id: item.id || item.encounterId, // 问题行
|
||||
name: item.patientName || '',
|
||||
leaf: true,
|
||||
...item,
|
||||
}));
|
||||
```
|
||||
|
||||
而后端 `AdmissionPatientPageDto` 中**没有 `id` 字段**(只有 `encounterId`、`patientId` 等),所以 `item.id` 为 `undefined`,此时 `item.id || item.encounterId` 回退到 `item.encounterId`。
|
||||
|
||||
但在 `prescriptionList.vue` 第186行提取 encounterId 时:
|
||||
```javascript
|
||||
let encounterIds = patientInfoList.value.map((i) => i.encounterId).join(',');
|
||||
```
|
||||
|
||||
如果 `patientInfoList` 中的某个对象其 `encounterId` 因任何原因为 `undefined`、`null` 或空字符串,`join(',')` 会产生类似 `"123,,456"` 的字符串。
|
||||
|
||||
后端解析时:
|
||||
```java
|
||||
List<Long> encounterIdList = Arrays.stream(encounterIds.split(",")).map(Long::parseLong).toList();
|
||||
```
|
||||
|
||||
`Long.parseLong("")` 会抛出 `NumberFormatException`,导致后端报错。
|
||||
|
||||
### 修复方案
|
||||
|
||||
在 `prescriptionList.vue` 的 `handleGetPrescription` 函数中,过滤掉无效的 encounterId 值:
|
||||
- 过滤 `undefined`、`null`、空字符串
|
||||
- 如果过滤后无有效 encounterId,提示用户并阻止请求
|
||||
|
||||
### 验证门禁
|
||||
|
||||
- [x] Gate A:根因已定位到 `prescriptionList.vue` 第186行,未过滤无效 encounterId
|
||||
- [x] Gate B:已读取前后端所有相关文件,理解完整数据流
|
||||
- [x] Gate C:修复方案与验收标准一致(前端防御性处理 + 正常查询)
|
||||
- [x] Gate D:不涉及数据库字段变更
|
||||
@@ -159,7 +159,7 @@ public class OrganizationLocationAppServiceImpl implements IOrganizationLocation
|
||||
String activityName = activityDef != null ? activityDef.getName() : "";
|
||||
|
||||
List<OrganizationLocation> organizationLocationList =
|
||||
organizationLocationService.getOrgLocListByOrgIdAndActivityDefinitionId(orgLoc.getOrganizationId(), orgLoc.getActivityDefinitionId());
|
||||
organizationLocationService.getOrgLocListByOrgIdAndActivityDefinitionId(orgLoc.getActivityDefinitionId());
|
||||
organizationLocationList = (orgLoc.getId() != null)
|
||||
? organizationLocationList.stream().filter(item -> !orgLoc.getId().equals(item.getId())).toList()
|
||||
: organizationLocationList;
|
||||
@@ -169,7 +169,7 @@ public class OrganizationLocationAppServiceImpl implements IOrganizationLocation
|
||||
if (DateTimeUtils.isOverlap(organizationLocation.getStartTime(), organizationLocation.getEndTime(),
|
||||
orgLoc.getStartTime(), orgLoc.getEndTime())) {
|
||||
Organization org = organizationService.getById(organizationLocation.getOrganizationId());
|
||||
String organizationName = org != null && org.getName() != null ? org.getName() : "未知科室";
|
||||
String organizationName = org != null ? org.getName() : "未知科室";
|
||||
return R.fail("当前诊疗:" + activityName + CommonConstants.Common.DASH + orgLoc.getStartTime()
|
||||
+ CommonConstants.Common.DASH + orgLoc.getEndTime() + "与" + organizationName + "时间冲突");
|
||||
}
|
||||
|
||||
@@ -96,11 +96,6 @@ public class DiagnosisQueryDto {
|
||||
*/
|
||||
private String diagnosisDoctor;
|
||||
|
||||
/**
|
||||
* 长效诊断标识
|
||||
*/
|
||||
private Integer longTermFlag;
|
||||
|
||||
/**
|
||||
* 是否已有传染病报卡(0-无,1-有)
|
||||
*/
|
||||
|
||||
@@ -13,11 +13,6 @@ import java.math.BigDecimal;
|
||||
@Accessors(chain = true)
|
||||
public class RequestFormDetailQueryDto {
|
||||
|
||||
/**
|
||||
* 诊疗活动定义ID(wor_service_request.activity_id,与开立检验时项目字典的 id / adviceDefinitionId 一致,用于编辑回显)
|
||||
*/
|
||||
private Long activityId;
|
||||
|
||||
/** 医嘱名称 */
|
||||
private String adviceName;
|
||||
|
||||
|
||||
@@ -135,7 +135,6 @@
|
||||
T1.onset_date AS onsetDate,
|
||||
T1.diagnosis_time AS diagnosisTime,
|
||||
T1.doctor AS diagnosisDoctor,
|
||||
T1.long_term_flag AS longTermFlag,
|
||||
CASE WHEN EXISTS (
|
||||
SELECT 1 FROM infectious_card T4
|
||||
WHERE T4.diag_id = T2.id AND T4.delete_flag = '0' AND T4.status >= 1
|
||||
|
||||
@@ -36,6 +36,6 @@ public interface IOrganizationLocationService extends IService<OrganizationLocat
|
||||
* @param activityDefinitionId 诊疗定义id
|
||||
* @return 诊疗的执行科室列表
|
||||
*/
|
||||
List<OrganizationLocation> getOrgLocListByOrgIdAndActivityDefinitionId(Long organizationId, Long activityDefinitionId);
|
||||
List<OrganizationLocation> getOrgLocListByOrgIdAndActivityDefinitionId(Long activityDefinitionId);
|
||||
|
||||
}
|
||||
@@ -53,14 +53,12 @@ public class OrganizationLocationServiceImpl extends ServiceImpl<OrganizationLoc
|
||||
/**
|
||||
* 查询诊疗的执行科室列表
|
||||
*
|
||||
* @param organizationId 机构id
|
||||
* @param activityDefinitionId 诊疗定义id
|
||||
* @return 诊疗的执行科室列表
|
||||
*/
|
||||
@Override
|
||||
public List<OrganizationLocation> getOrgLocListByOrgIdAndActivityDefinitionId(Long organizationId, Long activityDefinitionId) {
|
||||
public List<OrganizationLocation> getOrgLocListByOrgIdAndActivityDefinitionId(Long activityDefinitionId) {
|
||||
return baseMapper.selectList(new LambdaQueryWrapper<OrganizationLocation>()
|
||||
.eq(OrganizationLocation::getOrganizationId, organizationId)
|
||||
.eq(OrganizationLocation::getActivityDefinitionId, activityDefinitionId));
|
||||
}
|
||||
|
||||
|
||||
@@ -461,10 +461,6 @@ watch(
|
||||
console.log(prescriptionList.value,"prescriptionList.value")
|
||||
if(newValue&&newValue.length>0){
|
||||
let saveList = prescriptionList.value.filter((item) => {
|
||||
// 手术计费场景(generateSourceEnum=6)不限制 bizRequestFlag
|
||||
if (isSurgeryChargeBillingContext()) {
|
||||
return item.check && item.statusEnum == 1
|
||||
}
|
||||
return item.check && item.statusEnum == 1&&(Number(item.bizRequestFlag)==1||!item.bizRequestFlag)
|
||||
})
|
||||
console.log(saveList,"prescriptionList.value")
|
||||
@@ -1029,9 +1025,7 @@ function changeCheck(value,index,row){
|
||||
groupList.value.map(k=>{
|
||||
if(k.check){
|
||||
if(k.statusEnum == 1){//待签发
|
||||
// 手术计费场景(generateSourceEnum=6)不限制 bizRequestFlag
|
||||
const bizAllowed = isSurgeryChargeBillingContext() || Number(k.bizRequestFlag)==1||!k.bizRequestFlag
|
||||
if(bizAllowed){
|
||||
if(Number(k.bizRequestFlag)==1||!k.bizRequestFlag){
|
||||
if(handleSaveDisabled.value&&!handleSingOutDisabled.value&&groupList.value.length>1){
|
||||
proxy.$modal.msgWarning('请选择相同的状态的项目进行操作')
|
||||
return
|
||||
@@ -1046,9 +1040,7 @@ function changeCheck(value,index,row){
|
||||
}
|
||||
}
|
||||
if(k.statusEnum == 2){ //已签发
|
||||
// 手术计费场景(generateSourceEnum=6)不限制 bizRequestFlag
|
||||
const bizAllowed = isSurgeryChargeBillingContext() || Number(k.bizRequestFlag)==1||!k.bizRequestFlag
|
||||
if(bizAllowed){
|
||||
if(Number(k.bizRequestFlag)==1||!k.bizRequestFlag){
|
||||
if(!handleSaveDisabled.value&&handleSingOutDisabled.value&&groupList.value.length>1){
|
||||
proxy.$modal.msgWarning('请选择相同的状态的项目进行操作')
|
||||
return
|
||||
@@ -1075,28 +1067,28 @@ function handleSave() {
|
||||
return;
|
||||
}
|
||||
let saveList = prescriptionList.value.filter((item) => {
|
||||
// 手术计费场景(generateSourceEnum=6)不限制 bizRequestFlag,允许任何授权用户签发
|
||||
// 门诊划价场景保留 bizRequestFlag 限制,只能操作本人开立的医嘱
|
||||
if (isSurgeryChargeBillingContext()) {
|
||||
return item.check && item.statusEnum == 1
|
||||
}
|
||||
return item.check && item.statusEnum == 1&&(Number(item.bizRequestFlag)==1||!item.bizRequestFlag)
|
||||
});
|
||||
// 无可签发项目时提前返回,避免后端报"医嘱列表为空"
|
||||
if (saveList.length == 0) {
|
||||
proxy.$modal.msgWarning('当前无可签发处方');
|
||||
return;
|
||||
}
|
||||
// let saveList = prescriptionList.value
|
||||
// .filter((item) => {
|
||||
// return item.check;
|
||||
// }).filter((item) => {
|
||||
// return item.statusEnum == 1&&item.bizRequestFlag==1
|
||||
// })
|
||||
|
||||
// if (saveList.length == 0) {
|
||||
// proxy.$modal.msgWarning('当前无可签发处方');
|
||||
// return;
|
||||
// }
|
||||
// 此处签发处方和单行保存处方传参相同,后台已经将传参存为JSON字符串,此处直接转换为JSON即可
|
||||
let list = saveList.map((item) => {
|
||||
const parsedContent = item.contentJson ? JSON.parse(item.contentJson) : {};
|
||||
return {
|
||||
...parsedContent,
|
||||
requestId: item.requestId,
|
||||
// 已有 requestId 的记录走 UPDATE 路径,新记录走 INSERT 路径
|
||||
dbOpType: item.requestId ? '2' : '1',
|
||||
dbOpType: '1',
|
||||
groupId: item.groupId,
|
||||
// 补充顶层关键字段(这些可能不在 contentJson 中,需从 API 响应顶层提取)
|
||||
// 🔧 Bug #443: 补充顶层关键字段(这些不在 contentJson 中,需从 API 响应顶层提取)
|
||||
encounterId: item.encounterId,
|
||||
patientId: item.patientId,
|
||||
locationId: item.positionId,
|
||||
@@ -1104,14 +1096,9 @@ function handleSave() {
|
||||
adviceTableName: item.adviceTableName,
|
||||
adviceDefinitionId: item.adviceDefinitionId,
|
||||
chargeItemId: item.chargeItemId,
|
||||
// 补充数量、单位、批号等字段(后端 handDevice 需要这些字段)
|
||||
quantity: item.quantity,
|
||||
unitCode: item.unitCode,
|
||||
lotNumber: item.lotNumber,
|
||||
categoryEnum: item.categoryEnum,
|
||||
// 签发时显式设置手术计费关键字段,后端 generateSourceEnum 回退为默认值导致查询无法匹配
|
||||
generateSourceEnum: props.generateSourceEnum ?? parsedContent.generateSourceEnum ?? item.generateSourceEnum,
|
||||
sourceBillNo: props.sourceBillNo ?? parsedContent.sourceBillNo ?? item.sourceBillNo,
|
||||
// 🔧 Bug Fix: 签发时显式设置手术计费关键字段,避免后端 prescription_no / generateSourceEnum 回退为默认值导致查询无法匹配
|
||||
generateSourceEnum: props.generateSourceEnum ?? parsedContent.generateSourceEnum,
|
||||
sourceBillNo: props.sourceBillNo ?? parsedContent.sourceBillNo,
|
||||
};
|
||||
});
|
||||
// 确保 organizationId 不为 undefined(手术计费场景下可能缺失 orgId)
|
||||
@@ -1198,10 +1185,6 @@ function handleSingOut() {
|
||||
return item.check;
|
||||
})
|
||||
.filter((item) => {
|
||||
// 手术计费场景(generateSourceEnum=6)不限制 bizRequestFlag
|
||||
if (isSurgeryChargeBillingContext()) {
|
||||
return item.statusEnum == 2 && item.chargeStatus != 5
|
||||
}
|
||||
return item.statusEnum == 2 && item.chargeStatus != 5 && (Number(item.bizRequestFlag)==1||!item.bizRequestFlag)
|
||||
})
|
||||
.map((item) => {
|
||||
|
||||
@@ -354,11 +354,6 @@ async function getList() {
|
||||
if (!item.classification) {
|
||||
item.classification = '西医';
|
||||
}
|
||||
// 转换 longTermFlag 为字符串,以匹配 useDict 返回的字典值类型(字符串)
|
||||
// 避免 el-select 因类型不匹配(整数 1 vs 字符串 "1")导致下拉框清空
|
||||
if (item.longTermFlag != null) {
|
||||
item.longTermFlag = String(item.longTermFlag);
|
||||
}
|
||||
// 如果ybNo(诊断编码)符合传染病编码格式,添加到selectedDiseases
|
||||
if (item.ybNo && /^(01|02|03)/.test(item.ybNo)) {
|
||||
item.selectedDiseases = [item.ybNo];
|
||||
|
||||
@@ -207,8 +207,6 @@ const loadAllData = async () => {
|
||||
}
|
||||
applicationListAll.value = res.data?.records || [];
|
||||
totalCount.value = res.data?.total || 0;
|
||||
// 检验项目列表为异步加载,编辑回显必须在数据就绪后执行,否则已选区一直为空
|
||||
applyEditTransferSelection()
|
||||
} catch (e) {
|
||||
proxy.$message.error('获取检验项目列表失败');
|
||||
applicationListAll.value = [];
|
||||
@@ -345,70 +343,43 @@ watch(
|
||||
}
|
||||
);
|
||||
|
||||
/** 编辑弹窗:根据申请单明细把右侧「已选择」与 transferValue 对齐(依赖 applicationListAll 已加载) */
|
||||
const applyEditTransferSelection = () => {
|
||||
const newData = props.editData
|
||||
if (!newData?.requestFormId || !newData.requestFormDetailList?.length) {
|
||||
return
|
||||
}
|
||||
if (!applicationListAll.value.length) {
|
||||
return
|
||||
}
|
||||
const selectedIds = []
|
||||
for (const detail of newData.requestFormDetailList) {
|
||||
const idFromDetail = detail.activityId ?? detail.adviceDefinitionId
|
||||
let matched = null
|
||||
if (idFromDetail != null && idFromDetail !== '') {
|
||||
matched = applicationListAll.value.find(
|
||||
(item) => String(item.adviceDefinitionId) === String(idFromDetail)
|
||||
)
|
||||
}
|
||||
if (!matched && detail.adviceName) {
|
||||
matched = applicationListAll.value.find((item) => item.adviceName === detail.adviceName)
|
||||
}
|
||||
if (!matched && detail.adviceName) {
|
||||
const norm = (s) => String(s || '').trim()
|
||||
matched = applicationListAll.value.find(
|
||||
(item) => norm(item.adviceName) === norm(detail.adviceName)
|
||||
)
|
||||
}
|
||||
if (matched) {
|
||||
selectedIds.push(matched.adviceDefinitionId)
|
||||
}
|
||||
}
|
||||
const uniq = [...new Set(selectedIds)]
|
||||
transferValue.value = uniq
|
||||
if (newData.requestFormDetailList.length && uniq.length === 0) {
|
||||
console.warn(
|
||||
'[LaboratoryTests] 申请单明细未能在项目字典中匹配到项,请核对 activityId / 项目名称',
|
||||
newData.requestFormDetailList
|
||||
)
|
||||
}
|
||||
}
|
||||
|
||||
// 编辑模式下,回显已有数据(表单来自 descJson;项目选择在字典加载后由 applyEditTransferSelection 完成)
|
||||
// 编辑模式下,回显已有数据
|
||||
watch(
|
||||
() => props.editData,
|
||||
(newData) => {
|
||||
if (!newData || !newData.requestFormId) return
|
||||
if (!newData || !newData.requestFormId) return;
|
||||
|
||||
// 解析 descJson 回填表单
|
||||
if (newData.descJson) {
|
||||
try {
|
||||
const obj = JSON.parse(newData.descJson)
|
||||
const obj = JSON.parse(newData.descJson);
|
||||
Object.keys(form).forEach((key) => {
|
||||
if (obj[key] !== undefined) {
|
||||
form[key] = obj[key]
|
||||
form[key] = obj[key];
|
||||
}
|
||||
})
|
||||
});
|
||||
} catch (e) {
|
||||
console.error('解析 descJson 失败:', e)
|
||||
console.error('解析 descJson 失败:', e);
|
||||
}
|
||||
}
|
||||
|
||||
applyEditTransferSelection()
|
||||
// 回填已选项目
|
||||
if (newData.requestFormDetailList && newData.requestFormDetailList.length > 0) {
|
||||
// 从全部数据中匹配已选项目
|
||||
const selectedIds = [];
|
||||
newData.requestFormDetailList.forEach((detail) => {
|
||||
const matched = applicationListAll.value.find(
|
||||
(item) => item.adviceName === detail.adviceName
|
||||
);
|
||||
if (matched) {
|
||||
selectedIds.push(matched.adviceDefinitionId);
|
||||
}
|
||||
});
|
||||
transferValue.value = selectedIds;
|
||||
}
|
||||
},
|
||||
{ immediate: true, deep: true }
|
||||
)
|
||||
{ immediate: true }
|
||||
);
|
||||
|
||||
// 编辑模式下,applicationListAll 加载完成后重新回显已选项目
|
||||
watch(
|
||||
|
||||
Reference in New Issue
Block a user