diff --git a/bug547_analysis.md b/bug547_analysis.md index e71e5a3e..895d4aee 100644 --- a/bug547_analysis.md +++ b/bug547_analysis.md @@ -5,21 +5,23 @@ ## 根因定位 -**核心问题在 `OrganizationLocationAppServiceImpl.java:161-177`** +**核心问题在 `OrganizationLocationAppServiceImpl.java:161-162`** -时间冲突检测的查询范围过大: +时间冲突检测使用了 `getOrgLocListByActivityDefinitionId()` 跨科室查询同一诊疗定义下的**所有**科室配置记录。 + +### 缺陷:跨科室误报冲突 -### 缺陷:跨科室冲突检测导致误报 ```java -// 查询同一 activityDefinitionId 的所有记录(跨科室) +// 修复前:查询同一诊疗的所有科室配置(跨科室) organizationLocationService.getOrgLocListByActivityDefinitionId(orgLoc.getActivityDefinitionId()); ``` + 该查询返回**所有科室**中同一诊疗项目的配置记录。当其他科室(非当前操作科室)已配置了相同诊疗且时间重叠时,会被误判为冲突。 "执行科室配置"的业务语义是:为某个科室配置它可执行的诊疗项目及时段。不同科室配置同一诊疗的不同时段是完全合理的(如检验科 08:00-12:00,放射科 14:00-18:00)。跨科室时间重叠不应视为冲突。 ### 附带缺陷:"未知科室"错误提示 -当冲突记录关联的科室被软删除(`delete_flag='1'`)时,`organizationService.getById()` 受 `@TableLogic` 注解影响查不到该科室,返回 null,之前的代码 `continue` 跳过了,但跨科室检测的根本问题未解决。 +当冲突记录关联的科室被软删除(`delete_flag='1'`)时,`organizationService.getById()` 受 `@TableLogic` 注解影响查不到该科室,返回 null,错误提示变成"与未知科室时间冲突"。 ## 修复方案 @@ -29,8 +31,6 @@ organizationLocationService.getOrgLocListByActivityDefinitionId(orgLoc.getActivi **修复状态**: ✅ 成功 -**提交**: `7c6e35dc` Fix Bug #547: 冲突检测改为同科室范围 - **改动行数**: +3/-1(`OrganizationLocationAppServiceImpl.java`) **变更内容**: @@ -39,3 +39,5 @@ organizationLocationService.getOrgLocListByActivityDefinitionId(orgLoc.getActivi +organizationLocationService.getOrgLocListByOrgIdAndActivityDefinitionId(orgLoc.getOrganizationId(), + orgLoc.getActivityDefinitionId()); ``` + +编译验证通过。