Commit b41bb318b431b793c824e1708b0aa7f448439f7c
1 parent
90a621a8
♻️ refactor(machinery): 移除设备存在性校验中的启用状态检查
简化设备存在性校验逻辑,移除对设备启用状态的检查,确保代码更清晰且易于维护。
Showing
7 changed files
with
5 additions
and
23 deletions
yudao-module-mes/src/main/java/cn/iocoder/yudao/module/mes/service/dv/checkplan/MesDvCheckPlanMachineryServiceImpl.java
| @@ -42,7 +42,7 @@ public class MesDvCheckPlanMachineryServiceImpl implements MesDvCheckPlanMachine | @@ -42,7 +42,7 @@ public class MesDvCheckPlanMachineryServiceImpl implements MesDvCheckPlanMachine | ||
| 42 | // 1.1 校验方案草稿状态 | 42 | // 1.1 校验方案草稿状态 |
| 43 | MesDvCheckPlanDO currentPlan = checkPlanService.validateCheckPlanPrepare(createReqVO.getPlanId()); | 43 | MesDvCheckPlanDO currentPlan = checkPlanService.validateCheckPlanPrepare(createReqVO.getPlanId()); |
| 44 | // 1.2 校验设备存在 | 44 | // 1.2 校验设备存在 |
| 45 | - machineryService.validateMachineryExistsAndEnable(createReqVO.getMachineryId()); | 45 | + machineryService.validateMachineryExists(createReqVO.getMachineryId()); |
| 46 | // 1.3 校验同一方案下设备不重复 | 46 | // 1.3 校验同一方案下设备不重复 |
| 47 | if (checkPlanMachineryMapper.selectByPlanIdAndMachineryId(createReqVO.getPlanId(), createReqVO.getMachineryId()) != null) { | 47 | if (checkPlanMachineryMapper.selectByPlanIdAndMachineryId(createReqVO.getPlanId(), createReqVO.getMachineryId()) != null) { |
| 48 | throw exception(DV_CHECK_PLAN_MACHINERY_DUPLICATE); | 48 | throw exception(DV_CHECK_PLAN_MACHINERY_DUPLICATE); |
yudao-module-mes/src/main/java/cn/iocoder/yudao/module/mes/service/dv/checkrecord/MesDvCheckRecordServiceImpl.java
| @@ -170,7 +170,7 @@ public class MesDvCheckRecordServiceImpl implements MesDvCheckRecordService { | @@ -170,7 +170,7 @@ public class MesDvCheckRecordServiceImpl implements MesDvCheckRecordService { | ||
| 170 | 170 | ||
| 171 | private void validateCheckRecordSave(MesDvCheckRecordSaveReqVO reqVO) { | 171 | private void validateCheckRecordSave(MesDvCheckRecordSaveReqVO reqVO) { |
| 172 | // 校验设备是否存在 | 172 | // 校验设备是否存在 |
| 173 | - machineryService.validateMachineryExistsAndEnable(reqVO.getMachineryId()); | 173 | + machineryService.validateMachineryExists(reqVO.getMachineryId()); |
| 174 | // 校验点检计划是否存在 | 174 | // 校验点检计划是否存在 |
| 175 | if (reqVO.getPlanId() != null) { | 175 | if (reqVO.getPlanId() != null) { |
| 176 | checkPlanService.validateCheckPlanExistsAndType(reqVO.getPlanId(), MesDvCheckPlanTypeEnum.CHECK.getType()); | 176 | checkPlanService.validateCheckPlanExistsAndType(reqVO.getPlanId(), MesDvCheckPlanTypeEnum.CHECK.getType()); |
yudao-module-mes/src/main/java/cn/iocoder/yudao/module/mes/service/dv/machinery/MesDvMachineryService.java
| @@ -76,13 +76,6 @@ public interface MesDvMachineryService { | @@ -76,13 +76,6 @@ public interface MesDvMachineryService { | ||
| 76 | void validateMachineryExists(Long id); | 76 | void validateMachineryExists(Long id); |
| 77 | 77 | ||
| 78 | /** | 78 | /** |
| 79 | - * 校验设备存在且启用 | ||
| 80 | - * | ||
| 81 | - * @param id 编号 | ||
| 82 | - */ | ||
| 83 | - void validateMachineryExistsAndEnable(Long id); | ||
| 84 | - | ||
| 85 | - /** | ||
| 86 | * 获得设备列表 | 79 | * 获得设备列表 |
| 87 | * | 80 | * |
| 88 | * @return 设备列表 | 81 | * @return 设备列表 |
yudao-module-mes/src/main/java/cn/iocoder/yudao/module/mes/service/dv/machinery/MesDvMachineryServiceImpl.java
| @@ -132,17 +132,6 @@ public class MesDvMachineryServiceImpl implements MesDvMachineryService { | @@ -132,17 +132,6 @@ public class MesDvMachineryServiceImpl implements MesDvMachineryService { | ||
| 132 | } | 132 | } |
| 133 | } | 133 | } |
| 134 | 134 | ||
| 135 | - @Override | ||
| 136 | - public void validateMachineryExistsAndEnable(Long id) { | ||
| 137 | - MesDvMachineryDO machinery = machineryMapper.selectById(id); | ||
| 138 | - if (machinery == null) { | ||
| 139 | - throw exception(DV_MACHINERY_NOT_EXISTS); | ||
| 140 | - } | ||
| 141 | - if (ObjUtil.notEqual(CommonStatusEnum.ENABLE.getStatus(), machinery.getStatus())) { | ||
| 142 | - throw exception(DV_MACHINERY_IS_DISABLE); | ||
| 143 | - } | ||
| 144 | - } | ||
| 145 | - | ||
| 146 | private void validateMachineryCodeUnique(Long id, String code) { | 135 | private void validateMachineryCodeUnique(Long id, String code) { |
| 147 | if (code == null) { | 136 | if (code == null) { |
| 148 | return; | 137 | return; |
yudao-module-mes/src/main/java/cn/iocoder/yudao/module/mes/service/dv/maintenrecord/MesDvMaintenRecordServiceImpl.java
| @@ -106,7 +106,7 @@ public class MesDvMaintenRecordServiceImpl implements MesDvMaintenRecordService | @@ -106,7 +106,7 @@ public class MesDvMaintenRecordServiceImpl implements MesDvMaintenRecordService | ||
| 106 | 106 | ||
| 107 | private void validateMaintenRecordRelation(MesDvMaintenRecordSaveReqVO reqVO) { | 107 | private void validateMaintenRecordRelation(MesDvMaintenRecordSaveReqVO reqVO) { |
| 108 | // 校验设备是否存在 | 108 | // 校验设备是否存在 |
| 109 | - machineryService.validateMachineryExistsAndEnable(reqVO.getMachineryId()); | 109 | + machineryService.validateMachineryExists(reqVO.getMachineryId()); |
| 110 | // 校验保养计划是否存在 | 110 | // 校验保养计划是否存在 |
| 111 | if (reqVO.getPlanId() != null) { | 111 | if (reqVO.getPlanId() != null) { |
| 112 | checkPlanService.validateCheckPlanExistsAndType(reqVO.getPlanId(), MesDvCheckPlanTypeEnum.MAINTENANCE.getType()); | 112 | checkPlanService.validateCheckPlanExistsAndType(reqVO.getPlanId(), MesDvCheckPlanTypeEnum.MAINTENANCE.getType()); |
yudao-module-mes/src/main/java/cn/iocoder/yudao/module/mes/service/dv/repair/MesDvRepairServiceImpl.java
| @@ -79,7 +79,7 @@ public class MesDvRepairServiceImpl implements MesDvRepairService { | @@ -79,7 +79,7 @@ public class MesDvRepairServiceImpl implements MesDvRepairService { | ||
| 79 | // 校验编码唯一 | 79 | // 校验编码唯一 |
| 80 | validateCodeUnique(reqVO.getId(), reqVO.getCode()); | 80 | validateCodeUnique(reqVO.getId(), reqVO.getCode()); |
| 81 | // 校验设备存在 | 81 | // 校验设备存在 |
| 82 | - machineryService.validateMachineryExistsAndEnable(reqVO.getMachineryId()); | 82 | + machineryService.validateMachineryExists(reqVO.getMachineryId()); |
| 83 | } | 83 | } |
| 84 | 84 | ||
| 85 | private void validateCodeUnique(Long id, String code) { | 85 | private void validateCodeUnique(Long id, String code) { |
yudao-module-mes/src/main/java/cn/iocoder/yudao/module/mes/service/md/workstation/MesMdWorkstationMachineServiceImpl.java
| @@ -47,7 +47,7 @@ public class MesMdWorkstationMachineServiceImpl implements MesMdWorkstationMachi | @@ -47,7 +47,7 @@ public class MesMdWorkstationMachineServiceImpl implements MesMdWorkstationMachi | ||
| 47 | 47 | ||
| 48 | private void validateWorkstationMachineSaveData(MesMdWorkstationMachineSaveReqVO reqVO) { | 48 | private void validateWorkstationMachineSaveData(MesMdWorkstationMachineSaveReqVO reqVO) { |
| 49 | // 校验设备是否存在 | 49 | // 校验设备是否存在 |
| 50 | - machineryService.validateMachineryExistsAndEnable(reqVO.getMachineryId()); | 50 | + machineryService.validateMachineryExists(reqVO.getMachineryId()); |
| 51 | // 校验该设备是否已分配到其他工作站(一台设备只能分配到一个工作站) | 51 | // 校验该设备是否已分配到其他工作站(一台设备只能分配到一个工作站) |
| 52 | MesMdWorkstationMachineDO existing = workstationMachineMapper.selectByMachineryId(reqVO.getMachineryId()); | 52 | MesMdWorkstationMachineDO existing = workstationMachineMapper.selectByMachineryId(reqVO.getMachineryId()); |
| 53 | if (existing != null) { | 53 | if (existing != null) { |