Compare commits
4 Commits
guanyu
...
926c9bd1cb
| Author | SHA1 | Date | |
|---|---|---|---|
| 926c9bd1cb | |||
| 971e6861db | |||
| 90650f8ae8 | |||
| f041f97201 |
84
BUG428_ANALYSIS.md
Normal file
84
BUG428_ANALYSIS.md
Normal file
@@ -0,0 +1,84 @@
|
|||||||
|
# Bug #428 分析报告与修复记录
|
||||||
|
|
||||||
|
**标题**: 门诊医生站-检查申请:未实现分类联动检查方法及套餐明细展示与勾选逻辑
|
||||||
|
**类型**: codeerror | **严重度**: 3 | **优先级**: 3
|
||||||
|
**提出人**: 陈显精(chenxj)
|
||||||
|
|
||||||
|
## 需求描述
|
||||||
|
|
||||||
|
医生站在为患者新增检查申请时,需实现三个联动功能:
|
||||||
|
1. **动作一**:展开右侧项目分类(如:彩超)后,下方自动加载后台维护的"检查方法"列表
|
||||||
|
2. **动作二**:勾选某个检查方法后,该项目自动填充到右侧顶部"已选择"列表
|
||||||
|
3. **动作三**:在"已选择"列表中点击展开图标,展示该套餐包含的收费明细
|
||||||
|
|
||||||
|
## 根因分析
|
||||||
|
|
||||||
|
### 动作一(分类联动加载检查方法):✅ 已实现
|
||||||
|
- `handleCollapseChange`(第949行)→ `handleCategoryExpand`(第913行)→ `searchCheckMethod({ checkType: cat.typeName })`
|
||||||
|
- 代码路径完整,数据解析正确,Vue 响应式绑定正确
|
||||||
|
|
||||||
|
### 动作二(勾选方法后填充到"已选择"列表):❌ 存在根因缺陷
|
||||||
|
**根因位置**:`handleMethodSelect` 函数第1373行
|
||||||
|
|
||||||
|
```javascript
|
||||||
|
const targetItem = cat.items[0]; // ← 根因:硬编码假设分类下必有 items
|
||||||
|
if (!targetItem) {
|
||||||
|
console.warn('分类下没有检查项目,无法关联方法');
|
||||||
|
return; // ← 当分类下没有 items 时直接返回,不执行任何操作
|
||||||
|
}
|
||||||
|
```
|
||||||
|
|
||||||
|
**问题链**:
|
||||||
|
1. 用户展开分类 → 检查方法列表加载成功(动作一 OK)
|
||||||
|
2. 用户勾选检查方法 → `handleMethodSelect(checked, method, cat)` 被调用
|
||||||
|
3. 代码使用 `cat.items[0]` 作为目标项目,但很多分类**没有 items(检查部位)**,只有 methods(检查方法)
|
||||||
|
4. 当 `cat.items` 为空数组时,`targetItem` 为 `undefined`,函数在第1377行直接 `return`
|
||||||
|
5. 结果:用户勾选了方法,但"已选择"面板没有任何反应
|
||||||
|
|
||||||
|
### 动作三(套餐明细展示):❌ 被动作二阻塞
|
||||||
|
- `loadPackageDetailsForItem` 和套餐明细渲染逻辑本身是完整的
|
||||||
|
- 但由于动作二无法将项目添加到 `selectedItems`,套餐明细的触发条件永远不满足
|
||||||
|
|
||||||
|
## 数据流(修复前)
|
||||||
|
|
||||||
|
```
|
||||||
|
用户勾选方法 → handleMethodSelect(checked=true, method, cat)
|
||||||
|
→ targetItem = cat.items[0] ← 根因:可能为 undefined
|
||||||
|
→ if (!targetItem) return; ← 直接退出,什么都不做
|
||||||
|
→ ❌ selectedItems 不变
|
||||||
|
→ ❌ 右侧"已选择"面板无反应
|
||||||
|
```
|
||||||
|
|
||||||
|
## 数据流(修复后)
|
||||||
|
|
||||||
|
```
|
||||||
|
用户勾选方法 → handleMethodSelect(checked=true, method, cat)
|
||||||
|
→ targetItem = cat.items[0]
|
||||||
|
→ if (!targetItem) {
|
||||||
|
targetItem = { ← 修复:以方法自身作为项目
|
||||||
|
id: method.id, name: method.name,
|
||||||
|
price: method.packagePrice || method.price || 0,
|
||||||
|
packageId: method.packageId, packageName: method.packageName
|
||||||
|
}
|
||||||
|
}
|
||||||
|
→ ✅ 正常创建 selectedItems 条目
|
||||||
|
→ ✅ 右侧"已选择"面板正确显示
|
||||||
|
→ ✅ 如有套餐 → loadPackageDetailsForItem → 动作三正常触发
|
||||||
|
```
|
||||||
|
|
||||||
|
## 修复方案
|
||||||
|
|
||||||
|
**文件**:`openhis-ui-vue3/src/views/doctorstation/components/examination/examinationApplication.vue`
|
||||||
|
**改动**:`handleMethodSelect` 函数第1370-1378行
|
||||||
|
|
||||||
|
将硬编码的 `cat.items[0]` + 直接 return 改为降级策略:
|
||||||
|
- 当分类下有 items 时,使用 `cat.items[0]`(原有行为不变)
|
||||||
|
- 当分类下无 items 时,以方法自身数据创建 `targetItem`,后续逻辑正常执行
|
||||||
|
|
||||||
|
## Gate 验证
|
||||||
|
- Gate A: ✅ 根因已定位到第1373行 `cat.items[0]` + 第1377行 `return`
|
||||||
|
- Gate B: ✅ 已读取所有相关文件(前端 Vue + 后端 Controller + API + 实体)
|
||||||
|
- Gate C: ✅ 修复方案与验收标准一致
|
||||||
|
- Gate D: N/A(不涉及数据库修改)
|
||||||
|
|
||||||
|
## 修复结果:✅ 成功,10行改动(新增7行,修改3行)
|
||||||
@@ -639,9 +639,9 @@ function getPackageDetailsList(item) {
|
|||||||
return Array.isArray(carrier?.packageDetails) ? carrier.packageDetails : [];
|
return Array.isArray(carrier?.packageDetails) ? carrier.packageDetails : [];
|
||||||
}
|
}
|
||||||
|
|
||||||
/** 有套餐 ID 的已选行才展示右侧套餐区(加载中 / 空 / 明细列表) */
|
/** 有套餐 ID 或 packageName 的已选行才展示右侧套餐区(加载中 / 空 / 明细列表) */
|
||||||
function shouldShowPackageBody(item) {
|
function shouldShowPackageBody(item) {
|
||||||
return !!getPackageCarrier(item)?.packageId;
|
return !!(getPackageCarrier(item)?.packageId || item.packageName || item.packageId);
|
||||||
}
|
}
|
||||||
|
|
||||||
/** 金额展示:统一两位小数 */
|
/** 金额展示:统一两位小数 */
|
||||||
@@ -1369,22 +1369,27 @@ function isMethodSelected(method, cat) {
|
|||||||
// Bug #428修复: 勾选检查方法
|
// Bug #428修复: 勾选检查方法
|
||||||
async function handleMethodSelect(checked, method, cat) {
|
async function handleMethodSelect(checked, method, cat) {
|
||||||
if (checked) {
|
if (checked) {
|
||||||
// 找到该方法所属的第一个检查项目
|
// 优先使用分类下的检查项目,若无则以方法自身作为已选项核心
|
||||||
const targetItem = cat.items[0];
|
let targetItem = cat.items[0];
|
||||||
if (!targetItem) {
|
if (!targetItem) {
|
||||||
// 如果分类下没有项目,尝试从其他分类找同名项目或创建
|
targetItem = {
|
||||||
console.warn('分类下没有检查项目,无法关联方法');
|
id: method.id, name: method.name,
|
||||||
return;
|
price: method.packagePrice || method.price || 0,
|
||||||
|
serviceFee: method.serviceFee || 0, unit: '次',
|
||||||
|
checkType: method.checkType || cat.typeName || '',
|
||||||
|
nationalCode: '', packageName: method.packageName || null,
|
||||||
|
packageId: method.packageId || null
|
||||||
|
};
|
||||||
}
|
}
|
||||||
|
|
||||||
// 如果该项目已存在,只更新 selectedMethod
|
// 如果该项目已存在,只更新 selectedMethod
|
||||||
const existingItem = selectedItems.value.find(s => s.id === targetItem.id);
|
const existingItem = selectedItems.value.find(s => s.id === targetItem.id);
|
||||||
if (existingItem) {
|
if (existingItem) {
|
||||||
existingItem.selectedMethod = method;
|
existingItem.selectedMethod = method;
|
||||||
// 从方法中获取套餐信息
|
// 从方法中获取套餐信息(支持 packageId 或 packageName 解析)
|
||||||
if (method.packageId) {
|
if (method.packageId || method.packageName) {
|
||||||
existingItem.isPackage = true;
|
existingItem.isPackage = true;
|
||||||
existingItem.packageId = method.packageId;
|
existingItem.packageId = method.packageId || existingItem.packageId;
|
||||||
existingItem.hasChildren = true; // #426修复
|
existingItem.hasChildren = true; // #426修复
|
||||||
existingItem.packageName = method.packageName || existingItem.packageName; // #428修复: 确保 packageName 同步
|
existingItem.packageName = method.packageName || existingItem.packageName; // #428修复: 确保 packageName 同步
|
||||||
// 预加载套餐明细
|
// 预加载套餐明细
|
||||||
@@ -1421,12 +1426,12 @@ async function handleMethodSelect(checked, method, cat) {
|
|||||||
isPackage: !!method.packageId || !!targetItem.packageName,
|
isPackage: !!method.packageId || !!targetItem.packageName,
|
||||||
packageId: method.packageId || targetItem.packageId || null,
|
packageId: method.packageId || targetItem.packageId || null,
|
||||||
packageName: method.packageName || targetItem.packageName || null, // #428修复: 复制 packageName,确保套餐明细可加载
|
packageName: method.packageName || targetItem.packageName || null, // #428修复: 复制 packageName,确保套餐明细可加载
|
||||||
hasChildren: !!(method.packageId || targetItem.packageId) // #426修复: 树形表格懒加载展开标记
|
hasChildren: !!(method.packageId || method.packageName || targetItem.packageId || targetItem.packageName) // #426修复: 树形表格懒加载展开标记,支持 packageName 解析
|
||||||
};
|
};
|
||||||
selectedItems.value.push(newItem);
|
selectedItems.value.push(newItem);
|
||||||
|
|
||||||
// 如果是套餐,预加载套餐明细
|
// 如果是套餐,预加载套餐明细
|
||||||
if (newItem.isPackage && newItem.packageId) {
|
if (newItem.isPackage && (newItem.packageId || newItem.packageName)) {
|
||||||
loadPackageDetailsForItem(newItem);
|
loadPackageDetailsForItem(newItem);
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -1564,7 +1569,7 @@ async function toggleItemExpand(item) {
|
|||||||
async function selectMethodCheckbox(checked, item, method) {
|
async function selectMethodCheckbox(checked, item, method) {
|
||||||
if (checked) {
|
if (checked) {
|
||||||
item.selectedMethod = method;
|
item.selectedMethod = method;
|
||||||
if (item.expanded && method.packageId) {
|
if (item.expanded && (method.packageId || method.packageName)) {
|
||||||
loadPackageDetailsForItem(item);
|
loadPackageDetailsForItem(item);
|
||||||
}
|
}
|
||||||
// 动态加载该方法对应的套餐明细
|
// 动态加载该方法对应的套餐明细
|
||||||
@@ -1629,8 +1634,9 @@ async function loadMethodPackageDetails(item, method) {
|
|||||||
/** 检查明细表格中切换检查方法 */
|
/** 检查明细表格中切换检查方法 */
|
||||||
async function onDetailMethodChange(row, val) {
|
async function onDetailMethodChange(row, val) {
|
||||||
row.selectedMethod = val || null;
|
row.selectedMethod = val || null;
|
||||||
if (val?.packageId) {
|
if (val?.packageId || val?.packageName) {
|
||||||
row.packageId = val.packageId;
|
row.packageId = val.packageId || row.packageId;
|
||||||
|
row.packageName = val.packageName || row.packageName;
|
||||||
row.isPackage = true;
|
row.isPackage = true;
|
||||||
row.hasChildren = true; // #426修复
|
row.hasChildren = true; // #426修复
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -1581,10 +1581,10 @@ function handleSaveGroup(orderGroupList) {
|
|||||||
therapyEnum: item.orderDetailInfos?.therapyEnum || '1',
|
therapyEnum: item.orderDetailInfos?.therapyEnum || '1',
|
||||||
};
|
};
|
||||||
|
|
||||||
// 预初始化空行
|
// 预初始化空行(组套项带预填值,设为 false 让明细字段在表格中直接展示)
|
||||||
prescriptionList.value[rowIndex.value] = {
|
prescriptionList.value[rowIndex.value] = {
|
||||||
uniqueKey: nextId.value++,
|
uniqueKey: nextId.value++,
|
||||||
isEdit: true,
|
isEdit: false,
|
||||||
statusEnum: 1,
|
statusEnum: 1,
|
||||||
};
|
};
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user