From 47cd9ecc0eff38ffe6b3b794b2bf197e958f4403 Mon Sep 17 00:00:00 2001
From: xiangpei <xiangpei@timesnew.cn>
Date: 星期三, 14 五月 2025 15:50:57 +0800
Subject: [PATCH] bug:学员有状态不能修改问题

---
 src/main/java/com/mindskip/xzs/controller/admin/UserController.java |   65 ++++++++++++++++++--------------
 1 files changed, 36 insertions(+), 29 deletions(-)

diff --git a/src/main/java/com/mindskip/xzs/controller/admin/UserController.java b/src/main/java/com/mindskip/xzs/controller/admin/UserController.java
index 623b48d..a645248 100644
--- a/src/main/java/com/mindskip/xzs/controller/admin/UserController.java
+++ b/src/main/java/com/mindskip/xzs/controller/admin/UserController.java
@@ -1,20 +1,19 @@
 package com.mindskip.xzs.controller.admin;
 
+import com.github.pagehelper.PageInfo;
 import com.mindskip.xzs.base.BaseApiController;
 import com.mindskip.xzs.base.RestResponse;
 import com.mindskip.xzs.domain.*;
-import com.mindskip.xzs.domain.other.KeyValue;
 import com.mindskip.xzs.domain.enums.UserStatusEnum;
+import com.mindskip.xzs.domain.other.KeyValue;
 import com.mindskip.xzs.domain.vo.UserVO;
 import com.mindskip.xzs.repository.UserDepartmentMapper;
 import com.mindskip.xzs.service.*;
 import com.mindskip.xzs.utility.DateTimeUtil;
+import com.mindskip.xzs.utility.PageInfoHelper;
 import com.mindskip.xzs.utility.convert.UserClassConvert;
 import com.mindskip.xzs.utility.excel.ExcelUtils;
 import com.mindskip.xzs.viewmodel.admin.user.*;
-import com.mindskip.xzs.utility.PageInfoHelper;
-import com.github.pagehelper.PageInfo;
-
 import org.apache.commons.lang3.ObjectUtils;
 import org.apache.commons.lang3.StringUtils;
 import org.springframework.beans.factory.annotation.Autowired;
@@ -22,10 +21,7 @@
 import org.springframework.web.multipart.MultipartFile;
 
 import javax.validation.Valid;
-import java.util.ArrayList;
-import java.util.Date;
-import java.util.List;
-import java.util.UUID;
+import java.util.*;
 import java.util.stream.Collectors;
 
 
@@ -53,20 +49,19 @@
 
     @RequestMapping(value = "/page/list", method = RequestMethod.POST)
     public RestResponse<PageInfo<UserResponseVM>> pageList(@RequestBody UserPageRequestVM model) {
-        PageInfo<User> pageInfo = userService.userPage(model);
-        PageInfo<UserResponseVM> page = PageInfoHelper.copyMap(pageInfo, d ->
-                UserResponseVM.from(d));
+        model.setDepartmentId(ObjectUtils.isNotEmpty(model.getDepartmentId()) ? model.getDepartmentId() : getAdminDeptIds());
+        PageInfo<UserResponseVM> page = userService.userPage(model);
         page.setList(page.getList().stream().map(e->{
             List<UserDepartment> userDepartments = userDepartmentMapper.selectByUserId(e.getId());
-            String deptIds = "";
+            List<Department> list = new ArrayList<>();
             for (UserDepartment userDepartment : userDepartments) {
                 Department byId = departmentService.getById(userDepartment.getDepartmentId());
-                if(byId != null){
-                    deptIds = deptIds + byId.getName();
-                    break;
+                if (ObjectUtils.isNotEmpty(byId)) {
+                    list.add(byId);
                 }
             }
-            e.setDeptNames(deptIds);
+            e.setDeptNames(StringUtils.join(list.stream().map(Department::getName).collect(Collectors.toList()), ","));
+            e.setDeptIdList(list.stream().map(Department::getId).collect(Collectors.toList()));
             e.setTagNames(tagService.selectTagNamesByUserId(e.getId()));
             return e;
         }).collect(Collectors.toList()));
@@ -87,7 +82,7 @@
 
     @RequestMapping(value = "/select/{id}", method = RequestMethod.POST)
     public RestResponse<UserResponseVM> select(@PathVariable Integer id) {
-        User user = userService.getUserById(id);
+        User user = userService.getUserInfoById(id);
         UserResponseVM userVm = UserResponseVM.from(user);
         List<UserDepartment> userDepartments = userDepartmentMapper.selectByUserId(user.getId());
         String deptIds = "";
@@ -96,6 +91,7 @@
         }
         userVm.setDeptIds(deptIds.equals("") ? "" : deptIds.substring(0,deptIds.length()-1));
         userVm.setTagIds(tagService.selectTagIdsByUserId(user.getId()));
+        userVm.setPassword(authenticationService.pwdDecode(user.getPassword()));
         return RestResponse.ok(userVm);
     }
 
@@ -140,15 +136,23 @@
             user.setModifyTime(new Date());
             userService.updateByIdFilter(user);
         }
+        Map<Integer, UserDepartment> oldDeptUser = userDepartmentMapper.selectByUserId(user.getId()).stream().collect(Collectors.toMap(UserDepartment::getDepartmentId, userDept -> userDept));
         userDepartmentMapper.removeByUserId(user.getId());
-        for (String s : model.getDeptIds().split(",")) {
-            UserDepartment userDepartment = new UserDepartment();
-            userDepartment.setUserId(user.getId());
-            userDepartment.setDepartmentId(Integer.parseInt(s));
-            userDepartmentMapper.insert(userDepartment);
+        if (ObjectUtils.isNotEmpty(model.getDeptIds())) {
+            for (Integer s : model.getDeptIds()) {
+                UserDepartment userDepartment = new UserDepartment();
+                userDepartment.setUserId(user.getId());
+                userDepartment.setDepartmentId(s);
+                if (Objects.nonNull(oldDeptUser.get(s))) {
+                    userDepartment.setDeptAdmin(oldDeptUser.get(s).getDeptAdmin());
+                } else {
+                    userDepartment.setDeptAdmin("0");
+                }
+                userDepartmentMapper.insert(userDepartment);
+            }
         }
+        tagService.removeUserTagByUserId(user.getId());
         if (ObjectUtils.isNotEmpty(model.getTagIds())) {
-            tagService.removeUserTagByUserId(user.getId());
             tagService.saveBatchUserTag(model.getTagIds().stream().map(
                     tagId -> new UserTag() {{
                         setUserId(user.getId());
@@ -156,7 +160,6 @@
                     }}
             ).collect(Collectors.toList()));
         }
-        user.setDeptIds(model.getDeptIds());
         return RestResponse.ok(user);
     }
 
@@ -185,7 +188,7 @@
 
     @RequestMapping(value = "/delete/{id}", method = RequestMethod.POST)
     public RestResponse delete(@PathVariable Integer id) {
-        User user = userService.getUserById(id);
+        User user = userService.getUserInfoById(id);
         user.setDeleted(true);
         userService.updateByIdFilter(user);
         return RestResponse.ok();
@@ -230,10 +233,14 @@
         return RestResponse.ok();
     }
 
-    @RequestMapping(value = "/setStatus", method = RequestMethod.POST)
-    public RestResponse<String> setStatus(@RequestBody UserVO user) {
-        userService.setStatus(user);
-        return RestResponse.ok("鎿嶄綔鎴愬姛");
+    @GetMapping("/getFailExamUser/{examPaperId}")
+    public RestResponse<List<User>> getFailExamUser(@PathVariable Integer examPaperId) {
+        return RestResponse.ok(userService.getFailExamUser(examPaperId));
+    }
+
+    @GetMapping("/getFailTemplateUser/{templateId}")
+    public RestResponse<List<User>> getFailTemplateUser(@PathVariable Integer templateId) {
+        return RestResponse.ok(userService.getFailTemplateUser(templateId));
     }
 
 }

--
Gitblit v1.8.0