From b9e71660ca35e947ccac8079d3033b88ece5190e Mon Sep 17 00:00:00 2001
From: liyong <18434998025@163.com>
Date: 星期五, 22 五月 2026 16:01:21 +0800
Subject: [PATCH] fix(approve): 修复审批流程中空节点ID列表导致的异常

---
 src/main/java/com/ruoyi/approve/service/impl/ApproveNodeServiceImpl.java |  133 +++++++++++++++++++++++---------------------
 1 files changed, 70 insertions(+), 63 deletions(-)

diff --git a/src/main/java/com/ruoyi/approve/service/impl/ApproveNodeServiceImpl.java b/src/main/java/com/ruoyi/approve/service/impl/ApproveNodeServiceImpl.java
index 00a53f5..17666b7 100644
--- a/src/main/java/com/ruoyi/approve/service/impl/ApproveNodeServiceImpl.java
+++ b/src/main/java/com/ruoyi/approve/service/impl/ApproveNodeServiceImpl.java
@@ -24,14 +24,24 @@
 import com.ruoyi.project.system.service.ISysNoticeService;
 import com.ruoyi.purchase.mapper.PurchaseLedgerMapper;
 import com.ruoyi.purchase.pojo.PurchaseLedger;
-import com.ruoyi.purchase.service.impl.PurchaseLedgerServiceImpl;
-import com.ruoyi.sales.mapper.*;
+import com.ruoyi.quality.mapper.QualityInspectMapper;
+import com.ruoyi.quality.mapper.QualityInspectParamMapper;
+import com.ruoyi.quality.mapper.QualityTestStandardMapper;
+import com.ruoyi.quality.mapper.QualityTestStandardParamMapper;
+import com.ruoyi.quality.pojo.QualityInspect;
+import com.ruoyi.quality.pojo.QualityInspectParam;
+import com.ruoyi.quality.pojo.QualityTestStandard;
+import com.ruoyi.quality.pojo.QualityTestStandardParam;
+import com.ruoyi.sales.mapper.CommonFileMapper;
+import com.ruoyi.sales.mapper.SalesLedgerProductMapper;
+import com.ruoyi.sales.mapper.SalesQuotationMapper;
+import com.ruoyi.sales.mapper.ShippingInfoMapper;
 import com.ruoyi.sales.pojo.CommonFile;
 import com.ruoyi.sales.pojo.SalesLedgerProduct;
 import com.ruoyi.sales.pojo.SalesQuotation;
 import com.ruoyi.sales.pojo.ShippingInfo;
 import com.ruoyi.sales.service.impl.CommonFileServiceImpl;
-import org.springframework.beans.factory.annotation.Autowired;
+import lombok.RequiredArgsConstructor;
 import org.springframework.stereotype.Service;
 import org.springframework.util.CollectionUtils;
 
@@ -43,44 +53,26 @@
 import java.util.List;
 
 @Service
-//@RequiredArgsConstructor
+@RequiredArgsConstructor
 public class ApproveNodeServiceImpl extends ServiceImpl<ApproveNodeMapper, ApproveNode> implements IApproveNodeService {
 
-    @Autowired
-    private  ApproveNodeMapper approveNodeMapper;
-    @Autowired
-    private ApproveProcessMapper approveProcessMapper;
-    @Autowired
-    private SysUserMapper sysUserMapper;
-
-    @Autowired
-    private TempFileServiceImpl tempFileService;
-    @Autowired
-    private  ISysNoticeService sysNoticeService;
-
-    @Autowired
-    private CommonFileMapper fileMapper;
-    @Autowired
-    private DeviceRepairMapper deviceRepairMapper;
-
-    @Autowired
-    private PurchaseLedgerMapper purchaseLedgerMapper;
-
-    @Autowired
-    private SalesQuotationMapper salesQuotationMapper;
-
-    @Autowired
-    private ShippingInfoMapper shippingInfoMapper;
-
-    @Autowired
-    private CommonFileServiceImpl commonFileService;
-    @Autowired
-    private StockUtils stockUtils;
-    @Autowired
-    private SalesLedgerProductMapper salesLedgerProductMapper;
-
-    @Autowired
-    private PurchaseLedgerServiceImpl purchaseLedgerServiceImpl;
+    private final ApproveNodeMapper approveNodeMapper;
+    private final ApproveProcessMapper approveProcessMapper;
+    private final SysUserMapper sysUserMapper;
+    private final TempFileServiceImpl tempFileService;
+    private final ISysNoticeService sysNoticeService;
+    private final CommonFileMapper fileMapper;
+    private final DeviceRepairMapper deviceRepairMapper;
+    private final PurchaseLedgerMapper purchaseLedgerMapper;
+    private final SalesQuotationMapper salesQuotationMapper;
+    private final ShippingInfoMapper shippingInfoMapper;
+    private final CommonFileServiceImpl commonFileService;
+    private final StockUtils stockUtils;
+    private final SalesLedgerProductMapper salesLedgerProductMapper;
+    private final QualityInspectMapper qualityInspectMapper;
+    private final QualityTestStandardMapper qualityTestStandardMapper;
+    private final QualityTestStandardParamMapper qualityTestStandardParamMapper;
+    private final QualityInspectParamMapper qualityInspectParamMapper;
 
 
     public ApproveProcess getApproveById(String id) {
@@ -94,7 +86,7 @@
     }
 
     @Override
-    public void initApproveNodes(String approveUserIds,String approveID,Long tenantId) {
+    public void initApproveNodes(String approveUserIds, String approveID, Long tenantId) {
         Long userId = SecurityUtils.getLoginUser().getUser().getUserId();
         String[] names = approveUserIds.split(",");
         for (int i = 0; i < names.length; i++) {
@@ -102,7 +94,7 @@
             if (sysUser == null) continue;
             ApproveNode approveNode = new ApproveNode();
             approveNode.setApproveProcessId(approveID);
-            approveNode.setApproveNodeOrder(i +1);
+            approveNode.setApproveNodeOrder(i + 1);
             approveNode.setApproveNodeUser(sysUser.getNickName());
             approveNode.setApproveNodeUserId(sysUser.getUserId());
             approveNode.setApproveNodeTime(new Date());
@@ -132,17 +124,17 @@
 //                .eq(ApproveProcess::getApproveStatus, 0)
                 .last("limit 1");
         ApproveProcess approveProcess = approveProcessMapper.selectOne(approveProcessLambdaQueryWrapper);
-        if(approveProcess != null && approveProcess.getApproveStatus() == 3){
+        if (approveProcess != null && approveProcess.getApproveStatus() == 3) {
             return list;
         }
         for (ApproveNode approveNode : list) {
             List<CommonFile> commonFiles = fileMapper.selectList(new LambdaQueryWrapper<CommonFile>()
                     .eq(CommonFile::getCommonId, approveNode.getId())
                     .eq(CommonFile::getType, FileNameType.ApproveNode.getValue()));
-            if(!CollectionUtils.isEmpty(commonFiles)){
+            if (!CollectionUtils.isEmpty(commonFiles)) {
                 approveNode.setUrl(commonFiles.get(0).getUrl());
             }
-            if(approveNode.getApproveNodeStatus() == 1){
+            if (approveNode.getApproveNodeStatus() == 1) {
                 continue;
             }
             approveNode.setIsShen(true);
@@ -151,13 +143,13 @@
         return list;
     }
 
-    public void updateApproveProcessStatus(ApproveNode approveNode,Integer status) throws IOException {
+    public void updateApproveProcessStatus(ApproveNode approveNode, Integer status) throws IOException {
         LambdaQueryWrapper<ApproveProcess> approveProcessLambdaQueryWrapper = new LambdaQueryWrapper<>();
         approveProcessLambdaQueryWrapper.eq(ApproveProcess::getApproveId, approveNode.getApproveProcessId())
                 .eq(ApproveProcess::getApproveDelete, 0)
                 .last("limit 1");
         ApproveProcess approveProcess = approveProcessMapper.selectOne(approveProcessLambdaQueryWrapper);
-        if(approveProcess == null) throw new RuntimeException("瀹℃壒涓嶅瓨鍦�");
+        if (approveProcess == null) throw new RuntimeException("瀹℃壒涓嶅瓨鍦�");
         LambdaQueryWrapper<ApproveNode> approveNodeLambdaQueryWrapper = new LambdaQueryWrapper<>();
         approveNodeLambdaQueryWrapper.eq(ApproveNode::getApproveProcessId, approveNode.getApproveProcessId())
                 .eq(ApproveNode::getApproveNodeOrder, approveNode.getApproveNodeOrder() + 1)
@@ -166,26 +158,14 @@
                 .last("limit 1");
         ApproveNode approveNode1 = approveNodeMapper.selectOne(approveNodeLambdaQueryWrapper);
         approveProcess.setApproveStatus(status);
-        if(approveNode1 != null){
+        if (approveNode1 != null) {
             approveProcess.setApproveUserCurrentId(approveNode1.getApproveNodeUserId());
             approveProcess.setApproveUserCurrentName(approveNode1.getApproveNodeUser());
         }
-        if(approveProcess.getApproveStatus().equals(2) || approveProcess.getApproveStatus().equals(3) || approveProcess.getApproveStatus().equals(4)){
+        if (approveProcess.getApproveStatus().equals(2) || approveProcess.getApproveStatus().equals(3) || approveProcess.getApproveStatus().equals(4)) {
             approveProcess.setApproveOverTime(new Date());
         }
         approveProcessMapper.updateById(approveProcess);
-
-        DeviceRepair deviceRepair = deviceRepairMapper.selectById(approveProcess.getDeviceRepairId());
-        if(ObjectUtils.isNotNull(deviceRepair)) {
-            if(approveProcess.getApproveStatus().equals(2)){
-                // 鍚屾剰
-                deviceRepair.setStatus(1);
-            }else if(approveProcess.getApproveStatus().equals(3)){
-                // 鎷掔粷
-                deviceRepair.setStatus(2);
-            }
-            deviceRepairMapper.updateById(deviceRepair);
-        }
 
         //閲囪喘瀹℃牳
         if(approveProcess.getApproveType().equals(5)){
@@ -201,10 +181,9 @@
                     for (SalesLedgerProduct salesLedgerProduct : salesLedgerProducts) {
                         // 璐ㄦ
                         if (salesLedgerProduct.getIsChecked()) {
-                            purchaseLedgerServiceImpl.addQualityInspect(purchaseLedger, salesLedgerProduct);
-                        }else {
+                            addQualityInspect(purchaseLedger, salesLedgerProduct);                        }else {
                             //鐩存帴鍏ュ簱
-                            stockUtils.addStock(salesLedgerProduct.getProductModelId(), salesLedgerProduct.getQuantity(), StockInQualifiedRecordTypeEnum.PURCHASE_STOCK_IN.getCode(), purchaseLedger.getId());
+                            stockUtils.addStock(salesLedgerProduct.getProductModelId(), salesLedgerProduct.getQuantity(), StockInQualifiedRecordTypeEnum.PURCHASE_STOCK_IN.getCode(), purchaseLedger.getId(),"-","-","-");
                         }
                     }
                 } else if (status.equals(3)) {
@@ -329,4 +308,32 @@
         return null;
     }
 
+    private void addQualityInspect(PurchaseLedger purchaseLedger, SalesLedgerProduct saleProduct) {
+        QualityInspect qualityInspect = new QualityInspect();
+        qualityInspect.setInspectType(0);
+        qualityInspect.setSupplier(purchaseLedger.getSupplierName());
+        qualityInspect.setPurchaseLedgerId(purchaseLedger.getId());
+        qualityInspect.setProductId(saleProduct.getProductId());
+        qualityInspect.setProductName(saleProduct.getProductCategory());
+        qualityInspect.setModel(saleProduct.getSpecificationModel());
+        qualityInspect.setProductModelId(saleProduct.getProductModelId());
+        qualityInspect.setUnit(saleProduct.getUnit());
+        qualityInspect.setQuantity(saleProduct.getQuantity());
+        qualityInspectMapper.insert(qualityInspect);
+        List<QualityTestStandard> qualityTestStandard = qualityTestStandardMapper.getQualityTestStandardByProductId(saleProduct.getProductId(), 0,null);
+        if (qualityTestStandard.size()>0){
+            qualityInspect.setTestStandardId(qualityTestStandard.get(0).getId());
+            qualityInspectMapper.updateById(qualityInspect);
+            qualityTestStandardParamMapper.selectList(Wrappers.<QualityTestStandardParam>lambdaQuery()
+                            .eq(QualityTestStandardParam::getTestStandardId,qualityTestStandard.get(0).getId()))
+                    .forEach(qualityTestStandardParam -> {
+                        QualityInspectParam param = new QualityInspectParam();
+                        com.ruoyi.common.utils.bean.BeanUtils.copyProperties(qualityTestStandardParam, param);
+                        param.setId(null);
+                        param.setInspectId(qualityInspect.getId());
+                        qualityInspectParamMapper.insert(param);
+                    });
+        }
+    }
+
 }

--
Gitblit v1.9.3