From d54fbc9d4f8981fa0f7f613b33a28a8cd4f764ea Mon Sep 17 00:00:00 2001
From: 云 <2163098428@qq.com>
Date: 星期日, 25 十月 2026 16:29:53 +0800
Subject: [PATCH] feat(system): 优化用户搜索功能并完善ERP和MES模块验证逻辑

---
 yudao-module-mes/src/main/java/cn/iocoder/yudao/module/mes/service/wm/returnvendor/MesWmReturnVendorServiceImpl.java |  238 +++++++++++++++++++++++++++++++++++++++--------------------
 1 files changed, 156 insertions(+), 82 deletions(-)

diff --git a/yudao-module-mes/src/main/java/cn/iocoder/yudao/module/mes/service/wm/returnvendor/MesWmReturnVendorServiceImpl.java b/yudao-module-mes/src/main/java/cn/iocoder/yudao/module/mes/service/wm/returnvendor/MesWmReturnVendorServiceImpl.java
index f5a6f07..e56923e 100644
--- a/yudao-module-mes/src/main/java/cn/iocoder/yudao/module/mes/service/wm/returnvendor/MesWmReturnVendorServiceImpl.java
+++ b/yudao-module-mes/src/main/java/cn/iocoder/yudao/module/mes/service/wm/returnvendor/MesWmReturnVendorServiceImpl.java
@@ -2,10 +2,16 @@
 
 import cn.hutool.core.collection.CollUtil;
 import cn.hutool.core.util.ObjUtil;
+import cn.hutool.core.util.StrUtil;
 import cn.iocoder.yudao.framework.common.pojo.PageResult;
 import cn.iocoder.yudao.framework.common.util.collection.CollectionUtils;
 import cn.iocoder.yudao.framework.common.util.object.BeanUtils;
 import cn.iocoder.yudao.framework.common.util.object.ObjectUtils;
+import cn.iocoder.yudao.module.bpm.api.task.BpmProcessInstanceApi;
+import cn.iocoder.yudao.module.bpm.api.task.dto.BpmProcessInstanceCreateReqDTO;
+import cn.iocoder.yudao.module.bpm.enums.task.BpmTaskStatusEnum;
+import cn.iocoder.yudao.module.mes.controller.admin.wm.returnvendor.vo.MesWmReturnVendorImportExcelVO;
+import cn.iocoder.yudao.module.mes.controller.admin.wm.returnvendor.vo.MesWmReturnVendorImportRespVO;
 import cn.iocoder.yudao.module.mes.controller.admin.wm.returnvendor.vo.MesWmReturnVendorPageReqVO;
 import cn.iocoder.yudao.module.mes.controller.admin.wm.returnvendor.vo.MesWmReturnVendorSaveReqVO;
 import cn.iocoder.yudao.module.mes.dal.dataobject.wm.returnvendor.MesWmReturnVendorDO;
@@ -18,25 +24,25 @@
 import cn.iocoder.yudao.module.srm.api.supplier.SrmSupplierApi;
 import cn.iocoder.yudao.module.mes.service.wm.transaction.MesWmTransactionService;
 import cn.iocoder.yudao.module.mes.service.wm.transaction.dto.MesWmTransactionSaveReqDTO;
-import cn.iocoder.yudao.module.bpm.api.task.BpmProcessInstanceApi;
-import cn.iocoder.yudao.module.bpm.api.task.dto.BpmProcessInstanceCreateReqDTO;
-import cn.iocoder.yudao.module.bpm.dal.dataobject.definition.BpmProcessDefinitionInfoDO;
-import cn.iocoder.yudao.module.bpm.enums.task.BpmTaskStatusEnum;
-import cn.iocoder.yudao.module.bpm.service.definition.BpmProcessDefinitionService;
+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 jakarta.annotation.Resource;
 import lombok.extern.slf4j.Slf4j;
-import org.flowable.engine.repository.ProcessDefinition;
-import org.springframework.security.core.Authentication;
-import org.springframework.security.core.context.SecurityContextHolder;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
 import org.springframework.validation.annotation.Validated;
 
 import java.math.BigDecimal;
-import java.util.*;
+import java.util.ArrayList;
+import java.util.LinkedHashMap;
+import java.util.List;
+import java.util.Set;
+import java.util.concurrent.atomic.AtomicInteger;
 
 import static cn.iocoder.yudao.framework.common.exception.util.ServiceExceptionUtil.exception;
 import static cn.iocoder.yudao.framework.common.util.collection.CollectionUtils.convertList;
+import static cn.iocoder.yudao.framework.security.core.util.SecurityFrameworkUtils.getLoginUserId;
 import static cn.iocoder.yudao.module.mes.enums.ErrorCodeConstants.*;
 
 /**
@@ -47,8 +53,23 @@
 @Slf4j
 public class MesWmReturnVendorServiceImpl implements MesWmReturnVendorService {
 
+    /**
+     * 渚涘簲鍟嗛��璐у崟瀹℃壒鐨勪笟鍔$被鍨嬬紪鐮�
+     *
+     * 涓� system_approval_config.biz_type 瀵瑰簲锛屽鏍镐汉鍦ㄣ�岀郴缁熺鐞� - 瀹℃壒閰嶇疆銆嶄腑缁存姢銆�
+     */
+    private static final String RETURN_VENDOR_APPROVE_BIZ_TYPE = "return_vendor_approve";
+
+    /**
+     * 渚涘簲鍟嗛��璐у崟瀹℃壒鐨� BPM 娴佺▼瀹氫箟 Key
+     */
+    private static final String RETURN_VENDOR_APPROVE_PROCESS_KEY = "return_vendor_approve";
+
     @Resource
     private MesWmReturnVendorMapper returnVendorMapper;
+
+    @Resource
+    private BpmProcessInstanceApi bpmProcessInstanceApi;
 
     @Resource
     private MesWmReturnVendorLineService returnVendorLineService;
@@ -59,9 +80,9 @@
     @Resource
     private MesWmTransactionService wmTransactionService;
     @Resource
-    private BpmProcessInstanceApi processInstanceApi;
+    private AdminUserApi adminUserApi;
     @Resource
-    private BpmProcessDefinitionService bpmProcessDefinitionService;
+    private ApprovalConfigApi approvalConfigApi;
 
     @Override
     @Transactional(rollbackFor = Exception.class)
@@ -113,108 +134,101 @@
 
     @Override
     @Transactional(rollbackFor = Exception.class)
-    public void submitReturnVendorApproval(Long id, String processDefinitionKey) {
-        // 鏍¢獙瀛樺湪 + 鑽夌鐘舵��
+    public void submitReturnVendorApproval(Long id) {
+        // 1. 鏍¢獙瀛樺湪 + 鑽夌鐘舵��
         validateReturnVendorExistsAndPrepare(id);
-        // 鏍¢獙鑷冲皯鏈変竴鏉¤
+        // 2. 鏍¢獙鑷冲皯鏈変竴鏉¤
         List<MesWmReturnVendorLineDO> lines = returnVendorLineService.getReturnVendorLineListByReturnId(id);
         if (CollUtil.isEmpty(lines)) {
             throw exception(WM_RETURN_VENDOR_NO_LINE);
         }
-        // 鍒涘缓 BPM 娴佺▼瀹炰緥
-        String processInstanceId = processInstanceApi.createProcessInstance(getCurrentUserId(),
+        // 3. 鏍¢獙瀹℃壒閰嶇疆宸插惎鐢ㄤ笖閰嶇疆浜嗘湁鏁堝鏍镐汉锛堟湭閰嶇疆鏃舵姏鍑哄甫鏄庣‘鎻愮ず鐨勪笟鍔″紓甯革級
+        approvalConfigApi.validateApprovalEnabledAndGetApprovers(RETURN_VENDOR_APPROVE_BIZ_TYPE);
+        // 4. 鎻愪氦瀹℃壒锛堣崏绋� 鈫� 寰呮嫞璐э級锛屽苟娓呯┖涓婁竴杞鏍哥粨鏋�
+        returnVendorMapper.submitAndResetAuditInfo(id, MesWmReturnVendorStatusEnum.APPROVING.getStatus());
+        // 5. 鍒涘缓 BPM 娴佺▼瀹炰緥锛屼娇瀹℃壒鍑虹幇鍦ㄣ�屽崗鍚屽姙鍏� - 鎴戠殑娴佺▼銆�
+        bpmProcessInstanceApi.createProcessInstance(getLoginUserId(),
                 new BpmProcessInstanceCreateReqDTO()
-                        .setProcessDefinitionKey(processDefinitionKey)
+                        .setProcessDefinitionKey(RETURN_VENDOR_APPROVE_PROCESS_KEY)
                         .setBusinessKey(String.valueOf(id)));
-        // 鎻愪氦瀹℃壒锛堣崏绋� 鈫� 瀹℃壒涓級
-        returnVendorMapper.updateById(new MesWmReturnVendorDO()
-                .setId(id)
-                .setStatus(MesWmReturnVendorStatusEnum.APPROVING.getStatus())
-                .setProcessInstanceId(processInstanceId));
-    }
-
-    @Override
-    public List<Map<String, Object>> getReturnVendorApprovalProcessDefinitionList() {
-        return getProcessDefinitionListByCategory(APPROVE_CATEGORY_CODE);
     }
 
     @Override
     @Transactional(rollbackFor = Exception.class)
-    public void updateReturnVendorAuditStatus(Long id, Integer bpmResult) {
-        // 1. 鏍¢獙瀛樺湪
-        MesWmReturnVendorDO returnVendor = validateReturnVendorExists(id);
-        if (!MesWmReturnVendorStatusEnum.APPROVING.getStatus().equals(returnVendor.getStatus())) {
-            log.warn("[updateReturnVendorAuditStatus] 渚涘簲鍟嗛��璐у崟({}) 涓嶅浜庡鎵逛腑鐘舵��", id);
+    public void updateReturnVendorAuditStatus(Long id, Integer bpmStatus, String reason) {
+        // BPM 瀹℃壒缁撴灉鏄犲皠涓哄崟鎹姸鎬侊細閫氳繃=寰呮墽琛岄��璐� / 涓嶉�氳繃=鑽夌 / 鍙栨秷=宸插彇娑�
+        Integer status;
+        if (BpmTaskStatusEnum.APPROVE.getStatus().equals(bpmStatus)) {
+            status = MesWmReturnVendorStatusEnum.APPROVED.getStatus();
+        } else if (BpmTaskStatusEnum.REJECT.getStatus().equals(bpmStatus)) {
+            status = MesWmReturnVendorStatusEnum.PREPARE.getStatus();
+        } else if (BpmTaskStatusEnum.CANCEL.getStatus().equals(bpmStatus)) {
+            status = MesWmReturnVendorStatusEnum.CANCELED.getStatus();
+        } else {
             return;
         }
-        // 2. 鏍规嵁瀹℃壒缁撴灉鏇存柊鐘舵��
-        Integer newStatus = convertBpmResultToStatus(bpmResult);
-        if (newStatus != null) {
-            returnVendorMapper.updateById(new MesWmReturnVendorDO().setId(id).setStatus(newStatus));
+        if (returnVendorMapper.updateAuditStatusByBpm(id, status, reason) != 1) {
+            log.warn("[updateReturnVendorAuditStatus] 渚涘簲鍟嗛��璐у崟({}) 涓嶅浜庡鎵逛腑鐘舵�侊紝蹇界暐 BPM 缁撴灉({})", id, bpmStatus);
         }
     }
 
-    private Integer convertBpmResultToStatus(Integer bpmResult) {
-        if (BpmTaskStatusEnum.APPROVE.getStatus().equals(bpmResult)) {
-            return MesWmReturnVendorStatusEnum.APPROVED.getStatus();
-        }
-        if (BpmTaskStatusEnum.REJECT.getStatus().equals(bpmResult)) {
-            return MesWmReturnVendorStatusEnum.PREPARE.getStatus();
-        }
-        if (BpmTaskStatusEnum.CANCEL.getStatus().equals(bpmResult)) {
-            return MesWmReturnVendorStatusEnum.CANCELED.getStatus();
-        }
-        return null;
+    @Override
+    public Set<Long> getReturnVendorApproverUserIds() {
+        return approvalConfigApi.getApproverUserIds(RETURN_VENDOR_APPROVE_BIZ_TYPE);
     }
 
-    private List<Map<String, Object>> getProcessDefinitionListByCategory(String categoryCode) {
-        List<BpmProcessDefinitionInfoDO> definitionInfoList = bpmProcessDefinitionService
-                .getProcessDefinitionInfoListByCategory(categoryCode);
-        if (CollUtil.isEmpty(definitionInfoList)) {
-            return Collections.emptyList();
+    @Override
+    @Transactional(rollbackFor = Exception.class)
+    public void auditReturnVendor(Long id, Boolean pass, String reviewRemark) {
+        // 1. 鏍¢獙瀛樺湪
+        MesWmReturnVendorDO returnVendor = validateReturnVendorExists(id);
+        // 2. 鏍¢獙澶勪簬寰呮嫞璐х姸鎬�
+        if (!MesWmReturnVendorStatusEnum.APPROVING.getStatus().equals(returnVendor.getStatus())) {
+            throw exception(WM_RETURN_VENDOR_NOT_APPROVING);
         }
-        // 杩囨护婵�娲荤姸鎬侊紝淇濈暀鏈�鏂扮増鏈�
-        Map<String, ProcessDefinition> latestVersionMap = new HashMap<>();
-        for (BpmProcessDefinitionInfoDO info : definitionInfoList) {
-            ProcessDefinition pd = bpmProcessDefinitionService.getProcessDefinition(info.getProcessDefinitionId());
-            if (pd == null || pd.isSuspended()) {
-                continue;
+        // 3. 鏍¢獙褰撳墠鐧诲綍鐢ㄦ埛鏄涓氬姟绫诲瀷鐨勫鏍镐汉锛堟垨绛撅細浠讳竴浜哄潎鍙鏍革級
+        Long userId = getLoginUserId();
+        approvalConfigApi.validateApprover(RETURN_VENDOR_APPROVE_BIZ_TYPE, userId);
+        // 4. 瀹℃牳涓嶉�氳繃锛氬繀椤诲~鍐欏師鍥狅紝閫�鍥炶崏绋垮彲淇敼鍚庨噸鏂版彁浜�
+        if (!Boolean.TRUE.equals(pass)) {
+            if (StrUtil.isBlank(reviewRemark)) {
+                throw exception(WM_RETURN_VENDOR_REJECT_REASON_REQUIRED);
             }
-            ProcessDefinition existing = latestVersionMap.get(pd.getKey());
-            if (existing == null || pd.getVersion() > existing.getVersion()) {
-                latestVersionMap.put(pd.getKey(), pd);
-            }
+            returnVendorMapper.auditReturnVendor(id, MesWmReturnVendorStatusEnum.PREPARE.getStatus(),
+                    userId, getUserNickname(userId), reviewRemark);
+            return;
         }
-        List<Map<String, Object>> result = new ArrayList<>();
-        for (ProcessDefinition pd : latestVersionMap.values()) {
-            Map<String, Object> item = new HashMap<>();
-            item.put("id", pd.getId());
-            item.put("key", pd.getKey());
-            item.put("name", pd.getName());
-            result.add(item);
-        }
-        return result;
+        // 5. 瀹℃牳閫氳繃锛氳褰曞鏍镐汉涓庡鏍告剰瑙侊紝鐘舵�佺疆涓哄緟鎵ц閫�璐э紙鍙畬鎴愰��璐у苟鎵e噺搴撳瓨锛�
+        returnVendorMapper.auditReturnVendor(id, MesWmReturnVendorStatusEnum.APPROVED.getStatus(),
+                userId, getUserNickname(userId), reviewRemark);
     }
 
-    private Long getCurrentUserId() {
-        Authentication authentication = SecurityContextHolder.getContext().getAuthentication();
-        if (authentication != null && authentication.getPrincipal() instanceof Long) {
-            return (Long) authentication.getPrincipal();
+    /**
+     * 鑾峰彇鐢ㄦ埛鏄电О
+     *
+     * @param userId 鐢ㄦ埛缂栧彿
+     * @return 鏄电О锛岀敤鎴蜂笉瀛樺湪鏃惰繑鍥� null
+     */
+    private String getUserNickname(Long userId) {
+        if (userId == null) {
+            return null;
         }
-        return 1L; // 榛樿鐢ㄦ埛
+        AdminUserRespDTO user = adminUserApi.getUser(userId);
+        return user == null ? null : user.getNickname();
     }
 
     @Override
     @Transactional(rollbackFor = Exception.class)
     public void stockReturnVendor(Long id) {
-        // 鏍¢獙瀛樺湪
+        // 鏍¢獙瀛樺湪 + 宸插鎵癸紙鐘舵�佸彧鑳界敱 audit 鎺ㄨ繘锛欰PPROVING 鈫� APPROVED锛�
         MesWmReturnVendorDO returnVendor = validateReturnVendorExists(id);
-        if (ObjUtil.notEqual(MesWmReturnVendorStatusEnum.APPROVING.getStatus(), returnVendor.getStatus())) {
+        if (ObjUtil.notEqual(MesWmReturnVendorStatusEnum.APPROVED.getStatus(), returnVendor.getStatus())) {
             throw exception(WM_RETURN_VENDOR_STATUS_INVALID);
         }
-        // 鎵ц鎷h揣锛堝緟鎷h揣 -> 寰呮墽琛岄��璐э級
-        returnVendorMapper.updateById(new MesWmReturnVendorDO()
-                .setId(id).setStatus(MesWmReturnVendorStatusEnum.APPROVED.getStatus()));
+        // 浠呮牎楠岃鏁伴噺涓庢槑缁嗘暟閲忎竴鑷达紝涓嶄慨鏀瑰崟鎹姸鎬�
+        if (!Boolean.TRUE.equals(checkReturnVendorQuantity(id))) {
+            throw exception(WM_RETURN_VENDOR_DETAIL_QUANTITY_MISMATCH);
+        }
     }
 
     @Override
@@ -230,6 +244,10 @@
         if (CollUtil.isEmpty(details)) {
             throw exception(WM_RETURN_VENDOR_NO_DETAIL);
         }
+        // 1.3 鏍¢獙姣忚鏄庣粏鏁伴噺涔嬪拰绛変簬琛岄��璐ф暟閲忥紝閬垮厤鍑哄簱鏁伴噺涓庡崟鎹笉涓�鑷�
+        if (!Boolean.TRUE.equals(checkReturnVendorQuantity(id))) {
+            throw exception(WM_RETURN_VENDOR_DETAIL_QUANTITY_MISMATCH);
+        }
 
         // 2. 閬嶅巻鎵�鏈夋槑缁嗭紝鍒涘缓搴撳瓨浜嬪姟锛堟墸鍑忓簱瀛� + 璁板綍娴佹按锛�
         createTransactionList(returnVendor, details);
@@ -242,12 +260,22 @@
     private void createTransactionList(MesWmReturnVendorDO returnVendor, List<MesWmReturnVendorDetailDO> details) {
         wmTransactionService.createTransactionList(convertList(details, detail -> new MesWmTransactionSaveReqDTO()
                 .setType(MesWmTransactionTypeEnum.OUT.getType()).setItemId(detail.getItemId())
-                .setQuantity(detail.getQuantity().negate()) // 鍑哄簱鏁伴噺涓鸿礋鏁�
+                .setQuantity(negateQuantity(detail.getQuantity())) // 鍑哄簱鏁伴噺涓鸿礋鏁�
                 .setBatchId(detail.getBatchId()).setBatchCode(detail.getBatchCode())
                 .setWarehouseId(detail.getWarehouseId()).setLocationId(detail.getLocationId()).setAreaId(detail.getAreaId())
                 .setVendorId(returnVendor.getVendorId())
                 .setBizType(MesBizTypeConstants.WM_RETURN_VENDOR).setBizId(returnVendor.getId())
                 .setBizCode(returnVendor.getCode()).setBizLineId(detail.getLineId())));
+    }
+
+    /**
+     * 鍑哄簱鏁伴噺鍙栬礋锛氭暟閲忎负绌烘椂鎶涘嚭鏄庣‘鐨勪笟鍔″紓甯革紝閬垮厤 NPE
+     */
+    private BigDecimal negateQuantity(BigDecimal quantity) {
+        if (quantity == null) {
+            throw exception(WM_RETURN_VENDOR_DETAIL_QUANTITY_INVALID);
+        }
+        return quantity.negate();
     }
 
     @Override
@@ -331,4 +359,50 @@
         return returnVendorMapper.selectCountByVendorId(vendorId);
     }
 
+    @Override
+    @Transactional(rollbackFor = Exception.class)
+    public MesWmReturnVendorImportRespVO importReturnVendorList(List<MesWmReturnVendorImportExcelVO> importList, boolean updateSupport) {
+        if (CollUtil.isEmpty(importList)) {
+            throw exception(WM_RETURN_VENDOR_IMPORT_LIST_IS_EMPTY);
+        }
+        MesWmReturnVendorImportRespVO respVO = MesWmReturnVendorImportRespVO.builder()
+                .createCodes(new ArrayList<>()).updateCodes(new ArrayList<>())
+                .failureCodes(new LinkedHashMap<>()).build();
+        AtomicInteger index = new AtomicInteger(1);
+        importList.forEach(importItem -> {
+            int currentIndex = index.getAndIncrement();
+            String code = importItem.getCode();
+            String key = StrUtil.blankToDefault(code, "绗� " + currentIndex + " 琛�");
+            if (StrUtil.isBlank(code)) {
+                respVO.getFailureCodes().put(key, "鍗曞彿涓嶈兘涓虹┖");
+                return;
+            }
+            if (StrUtil.isBlank(importItem.getName())) {
+                respVO.getFailureCodes().put(key, "鍚嶇О涓嶈兘涓虹┖");
+                return;
+            }
+            MesWmReturnVendorDO existing = returnVendorMapper.selectByCode(code);
+            if (existing != null && updateSupport) {
+                if (!MesWmReturnVendorStatusEnum.PREPARE.getStatus().equals(existing.getStatus())) {
+                    respVO.getFailureCodes().put(key, "鍗曟嵁闈炶崏绋跨姸鎬侊紝涓嶅厑璁歌鐩栨洿鏂�");
+                    return;
+                }
+                MesWmReturnVendorDO updateObj = BeanUtils.toBean(importItem, MesWmReturnVendorDO.class);
+                updateObj.setId(existing.getId());
+                updateObj.setStatus(existing.getStatus());
+                returnVendorMapper.updateById(updateObj);
+                respVO.getUpdateCodes().add(code);
+            } else if (existing != null) {
+                respVO.getFailureCodes().put(key, "鍗曞彿宸插瓨鍦細" + code);
+            } else {
+                MesWmReturnVendorDO createObj = BeanUtils.toBean(importItem, MesWmReturnVendorDO.class);
+                createObj.setId(null);
+                createObj.setStatus(MesWmReturnVendorStatusEnum.PREPARE.getStatus());
+                returnVendorMapper.insert(createObj);
+                respVO.getCreateCodes().add(code);
+            }
+        });
+        return respVO;
+    }
+
 }

--
Gitblit v1.9.3