From 962f78c8383e7c884ddb08c951c15e5ba6eb4cb8 Mon Sep 17 00:00:00 2001
From: liu <2021943741@qq.com>
Date: 星期五, 23 十月 2026 15:43:33 +0800
Subject: [PATCH] fix: 完善审批流程与业务状态校验

---
 yudao-module-hrm/src/main/java/cn/iocoder/yudao/module/hrm/service/attendance/HrmAttendanceExceptionServiceImpl.java |   37 ++++++++++++++++++++++++++++---------
 1 files changed, 28 insertions(+), 9 deletions(-)

diff --git a/yudao-module-hrm/src/main/java/cn/iocoder/yudao/module/hrm/service/attendance/HrmAttendanceExceptionServiceImpl.java b/yudao-module-hrm/src/main/java/cn/iocoder/yudao/module/hrm/service/attendance/HrmAttendanceExceptionServiceImpl.java
index e4579e9..8f59d69 100644
--- a/yudao-module-hrm/src/main/java/cn/iocoder/yudao/module/hrm/service/attendance/HrmAttendanceExceptionServiceImpl.java
+++ b/yudao-module-hrm/src/main/java/cn/iocoder/yudao/module/hrm/service/attendance/HrmAttendanceExceptionServiceImpl.java
@@ -84,7 +84,8 @@
     @Override
     public void updateAttendanceException(HrmAttendanceExceptionSaveReqVO updateReqVO) {
         HrmAttendanceExceptionDO attendanceException = validateAttendanceExceptionExists(updateReqVO.getId());
-        if (!HrmAuditStatusEnum.DRAFT.getStatus().equals(attendanceException.getStatus())) {
+        if (!HrmAuditStatusEnum.DRAFT.getStatus().equals(attendanceException.getStatus())
+                && !HrmAuditStatusEnum.REJECT.getStatus().equals(attendanceException.getStatus())) {
             throw exception(ATTENDANCE_EXCEPTION_UPDATE_FAIL_NOT_DRAFT);
         }
         HrmAttendanceExceptionDO updateObj = BeanUtils.toBean(updateReqVO, HrmAttendanceExceptionDO.class);
@@ -95,7 +96,8 @@
     @Override
     public void deleteAttendanceException(Long id) {
         HrmAttendanceExceptionDO attendanceException = validateAttendanceExceptionExists(id);
-        if (!HrmAuditStatusEnum.DRAFT.getStatus().equals(attendanceException.getStatus())) {
+        if (!HrmAuditStatusEnum.DRAFT.getStatus().equals(attendanceException.getStatus())
+                && !HrmAuditStatusEnum.REJECT.getStatus().equals(attendanceException.getStatus())) {
             throw exception(ATTENDANCE_EXCEPTION_DELETE_FAIL_NOT_DRAFT);
         }
         storageAttachmentApi.deleteAttachmentsByRecord("hrm_attendance_exception", id);
@@ -130,11 +132,19 @@
             throw exception(ATTENDANCE_EXCEPTION_SUBMIT_FAIL_NOT_EDITABLE);
         }
 
-        // 2. 鏍¢獙瀹℃壒閰嶇疆宸插惎鐢ㄤ笖閰嶇疆浜嗘湁鏁堝鎵逛汉锛堟湭閰嶇疆鏃舵姏鍑哄甫鏄庣‘鎻愮ず鐨勪笟鍔″紓甯革級
+        // 2. 琛ュ崱绫诲瀷蹇呴』濉啓琛ュ崱鏃堕棿锛岄伩鍏嶅鏍搁�氳繃鍚庡洖濉�冨嫟璁板綍鏃跺嚭鐜扮┖鎸囬拡
+        if (HrmAttendanceExceptionTypeEnum.SUPPLEMENT.getType().equals(exception.getType())
+                && exception.getExceptionTime() == null) {
+            throw exception(ATTENDANCE_EXCEPTION_TIME_REQUIRED);
+        }
+
+        // 3. 鏍¢獙瀹℃壒閰嶇疆宸插惎鐢ㄤ笖閰嶇疆浜嗘湁鏁堝鎵逛汉锛堟湭閰嶇疆鏃舵姏鍑哄甫鏄庣‘鎻愮ず鐨勪笟鍔″紓甯革級
         approvalConfigApi.validateApprovalEnabledAndGetApprovers(ATTENDANCE_APPROVE_BIZ_TYPE);
 
-        // 3. 鐘舵�佺疆涓哄鎵逛腑锛屽苟娓呯┖涓婁竴杞鏍哥粨鏋滐紝閬垮厤娈嬬暀銆屽鏍镐笉閫氳繃銆嶇殑鍘熷洜
-        attendanceExceptionMapper.submitAndResetAuditInfo(id, HrmAuditStatusEnum.PROCESS.getStatus());
+        // 4. 鐘舵�佺疆涓哄鎵逛腑锛屽苟娓呯┖涓婁竴杞鏍哥粨鏋滐紝閬垮厤娈嬬暀銆屽鏍镐笉閫氳繃銆嶇殑鍘熷洜
+        if (attendanceExceptionMapper.submitAndResetAuditInfo(id, HrmAuditStatusEnum.PROCESS.getStatus()) != 1) {
+            throw exception(ATTENDANCE_EXCEPTION_SUBMIT_FAIL_NOT_EDITABLE);
+        }
     }
 
     @Override
@@ -151,6 +161,11 @@
         if (!HrmAuditStatusEnum.PROCESS.getStatus().equals(exception.getStatus())) {
             throw exception(ATTENDANCE_EXCEPTION_AUDIT_FAIL_NOT_PROCESS);
         }
+        // 2.1 琛ュ崱绫诲瀷蹇呴』濉啓琛ュ崱鏃堕棿锛岄伩鍏嶅鏍搁�氳繃鍚庡洖濉�冨嫟璁板綍鏃跺嚭鐜扮┖鎸囬拡
+        if (HrmAttendanceExceptionTypeEnum.SUPPLEMENT.getType().equals(exception.getType())
+                && exception.getExceptionTime() == null) {
+            throw exception(ATTENDANCE_EXCEPTION_TIME_REQUIRED);
+        }
         // 3. 鏍¢獙褰撳墠鐧诲綍鐢ㄦ埛鏄涓氬姟绫诲瀷鐨勫鎵逛汉锛堟垨绛撅細浠讳竴浜哄潎鍙鏍革級
         Long userId = getLoginUserId();
         approvalConfigApi.validateApprover(ATTENDANCE_APPROVE_BIZ_TYPE, userId);
@@ -159,13 +174,17 @@
             if (StrUtil.isBlank(reviewRemark)) {
                 throw exception(ATTENDANCE_EXCEPTION_AUDIT_REJECT_REASON_REQUIRED);
             }
-            attendanceExceptionMapper.auditAttendanceException(id, HrmAuditStatusEnum.REJECT.getStatus(),
-                    userId, getUserNickname(userId), reviewRemark);
+            if (attendanceExceptionMapper.auditAttendanceException(id, HrmAuditStatusEnum.REJECT.getStatus(),
+                    userId, getUserNickname(userId), reviewRemark) != 1) {
+                throw exception(ATTENDANCE_EXCEPTION_AUDIT_FAIL_NOT_PROCESS);
+            }
             return;
         }
         // 5. 瀹℃牳閫氳繃锛氳褰曞鏍镐汉涓庡鏍告剰瑙侊紝鐘舵�佺疆涓哄鏍搁�氳繃
-        attendanceExceptionMapper.auditAttendanceException(id, HrmAuditStatusEnum.APPROVE.getStatus(),
-                userId, getUserNickname(userId), reviewRemark);
+        if (attendanceExceptionMapper.auditAttendanceException(id, HrmAuditStatusEnum.APPROVE.getStatus(),
+                userId, getUserNickname(userId), reviewRemark) != 1) {
+            throw exception(ATTENDANCE_EXCEPTION_AUDIT_FAIL_NOT_PROCESS);
+        }
 
         // 6. 瀹℃牳閫氳繃鍚庣疆涓氬姟閫昏緫锛氳ˉ鍗″洖濉笂涓嬬彮鏃堕棿 / 鍔犵彮鍥炲啓鍔犵彮鏃堕暱
         if (HrmAttendanceExceptionTypeEnum.SUPPLEMENT.getType().equals(exception.getType())) {

--
Gitblit v1.9.3