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-crm/src/main/java/cn/iocoder/yudao/module/crm/service/refund/CrmRefundServiceImpl.java |   48 ++++++++++++++++++++++++++++++++++--------------
 1 files changed, 34 insertions(+), 14 deletions(-)

diff --git a/yudao-module-crm/src/main/java/cn/iocoder/yudao/module/crm/service/refund/CrmRefundServiceImpl.java b/yudao-module-crm/src/main/java/cn/iocoder/yudao/module/crm/service/refund/CrmRefundServiceImpl.java
index 9d3adad..2d23a94 100644
--- a/yudao-module-crm/src/main/java/cn/iocoder/yudao/module/crm/service/refund/CrmRefundServiceImpl.java
+++ b/yudao-module-crm/src/main/java/cn/iocoder/yudao/module/crm/service/refund/CrmRefundServiceImpl.java
@@ -17,6 +17,7 @@
 import cn.iocoder.yudao.module.system.api.approval.ApprovalConfigApi;
 import cn.iocoder.yudao.module.system.api.user.AdminUserApi;
 import cn.iocoder.yudao.module.system.api.user.dto.AdminUserRespDTO;
+import com.baomidou.mybatisplus.core.conditions.update.LambdaUpdateWrapper;
 import jakarta.annotation.Resource;
 import lombok.extern.slf4j.Slf4j;
 import org.springframework.stereotype.Service;
@@ -78,23 +79,36 @@
 
     @Override
     public void updateRefund(CrmRefundSaveReqVO updateReqVO) {
-        // 1. 鏍¢獙瀛樺湪 + 瀹℃牳閫氳繃鐨勯��璐瑰崟涓嶅厑璁镐慨鏀�
+        // 1. 鏍¢獙瀛樺湪 + 鍙湁鑽夌銆佸鏍镐笉閫氳繃鍙互淇敼
         CrmRefundDO refund = validateRefundExists(updateReqVO.getId());
-        if (ObjUtil.equal(refund.getAuditStatus(), CrmAuditStatusEnum.APPROVE.getStatus())) {
-            throw exception(ErrorCodeConstants.CRM_REFUND_UPDATE_FAIL_APPROVE);
+        if (ObjUtil.notEqual(refund.getAuditStatus(), CrmAuditStatusEnum.DRAFT.getStatus())
+                && ObjUtil.notEqual(refund.getAuditStatus(), CrmAuditStatusEnum.REJECT.getStatus())) {
+            throw exception(ErrorCodeConstants.CRM_REFUND_UPDATE_FAIL_NOT_EDITABLE);
         }
-        // 2. 鏇存柊閫�璐瑰崟
+        // 2. 鏇存柊閫�璐瑰崟锛涘鎵圭姸鎬佷笌瀹℃牳缁撴灉鍙兘鐢辨彁浜�/瀹℃牳鍙樻洿锛屾澶勬樉寮忔竻绌�
         CrmRefundDO entity = BeanUtils.toBean(updateReqVO, CrmRefundDO.class);
-        entity.setAuditStatus(null); // 瀹℃壒鐘舵�佸彧鑳界敱鎻愪氦/瀹℃牳鍙樻洿
-        refundMapper.updateById(entity);
+        // 瀹℃牳鐩稿叧瀛楁鍙兘鐢辨彁浜�/瀹℃牳鍙樻洿
+        entity.setAuditStatus(null);
+        entity.setReviewerId(null);
+        entity.setReviewerName(null);
+        entity.setReviewTime(null);
+        entity.setReviewRemark(null);
+        refundMapper.update(entity, new LambdaUpdateWrapper<CrmRefundDO>()
+                .eq(CrmRefundDO::getId, entity.getId())
+                .set(CrmRefundDO::getAuditStatus, null)
+                .set(CrmRefundDO::getReviewerId, null)
+                .set(CrmRefundDO::getReviewerName, null)
+                .set(CrmRefundDO::getReviewTime, null)
+                .set(CrmRefundDO::getReviewRemark, null));
     }
 
     @Override
     public void deleteRefund(Long id) {
-        // 鏍¢獙瀛樺湪 + 瀹℃牳閫氳繃鐨勯��璐瑰崟涓嶅厑璁稿垹闄�
+        // 鏍¢獙瀛樺湪 + 鍙湁鑽夌銆佸鏍镐笉閫氳繃鍙互鍒犻櫎
         CrmRefundDO refund = validateRefundExists(id);
-        if (ObjUtil.equal(refund.getAuditStatus(), CrmAuditStatusEnum.APPROVE.getStatus())) {
-            throw exception(ErrorCodeConstants.CRM_REFUND_DELETE_FAIL_APPROVE);
+        if (ObjUtil.notEqual(refund.getAuditStatus(), CrmAuditStatusEnum.DRAFT.getStatus())
+                && ObjUtil.notEqual(refund.getAuditStatus(), CrmAuditStatusEnum.REJECT.getStatus())) {
+            throw exception(ErrorCodeConstants.CRM_REFUND_DELETE_FAIL_NOT_EDITABLE);
         }
         refundMapper.deleteById(id);
     }
@@ -129,7 +143,9 @@
         approvalConfigApi.validateApprovalEnabledAndGetApprovers(REFUND_APPROVE_BIZ_TYPE);
 
         // 3. 鐘舵�佺疆涓哄鎵逛腑锛屽苟娓呯┖涓婁竴杞鏍哥粨鏋滐紝閬垮厤娈嬬暀銆屽鏍镐笉閫氳繃銆嶇殑鍘熷洜
-        refundMapper.submitAndResetAuditInfo(id, CrmAuditStatusEnum.PROCESS.getStatus());
+        if (refundMapper.submitAndResetAuditInfo(id, CrmAuditStatusEnum.PROCESS.getStatus()) != 1) {
+            throw exception(ErrorCodeConstants.CRM_REFUND_SUBMIT_FAIL_NOT_EDITABLE);
+        }
     }
 
     @Override
@@ -154,13 +170,17 @@
             if (StrUtil.isBlank(reviewRemark)) {
                 throw exception(ErrorCodeConstants.CRM_REFUND_AUDIT_REJECT_REASON_REQUIRED);
             }
-            refundMapper.auditRefund(id, CrmAuditStatusEnum.REJECT.getStatus(),
-                    userId, getUserNickname(userId), reviewRemark);
+            if (refundMapper.auditRefund(id, CrmAuditStatusEnum.REJECT.getStatus(),
+                    userId, getUserNickname(userId), reviewRemark) != 1) {
+                throw exception(ErrorCodeConstants.CRM_REFUND_AUDIT_FAIL_NOT_PROCESS);
+            }
             return;
         }
         // 5. 瀹℃牳閫氳繃锛氳褰曞鏍镐汉涓庡鏍告剰瑙侊紝鐘舵�佺疆涓哄鏍搁�氳繃
-        refundMapper.auditRefund(id, CrmAuditStatusEnum.APPROVE.getStatus(),
-                userId, getUserNickname(userId), reviewRemark);
+        if (refundMapper.auditRefund(id, CrmAuditStatusEnum.APPROVE.getStatus(),
+                userId, getUserNickname(userId), reviewRemark) != 1) {
+            throw exception(ErrorCodeConstants.CRM_REFUND_AUDIT_FAIL_NOT_PROCESS);
+        }
     }
 
     private CrmRefundDO validateRefundExists(Long id) {

--
Gitblit v1.9.3