From a6a3bb04cdaa334a2868d09b20518ed98575bccb Mon Sep 17 00:00:00 2001
From: xiangpei <xiangpei@timesnew.cn>
Date: 星期二, 02 七月 2024 18:08:25 +0800
Subject: [PATCH] Merge remote-tracking branch 'origin/master'

---
 src/main/java/com/mindskip/xzs/controller/admin/UserController.java |   41 +++++++++++++++++++++--------------------
 1 files changed, 21 insertions(+), 20 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 a76d70a..8478851 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()));
@@ -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,11 +136,17 @@
             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(",")) {
+        for (Integer s : model.getDeptIds()) {
             UserDepartment userDepartment = new UserDepartment();
             userDepartment.setUserId(user.getId());
-            userDepartment.setDepartmentId(Integer.parseInt(s));
+            userDepartment.setDepartmentId(s);
+            if (Objects.nonNull(oldDeptUser.get(s))) {
+                userDepartment.setDeptAdmin(oldDeptUser.get(s).getDeptAdmin());
+            } else {
+                userDepartment.setDeptAdmin("0");
+            }
             userDepartmentMapper.insert(userDepartment);
         }
         if (ObjectUtils.isNotEmpty(model.getTagIds())) {
@@ -156,7 +158,6 @@
                     }}
             ).collect(Collectors.toList()));
         }
-        user.setDeptIds(model.getDeptIds());
         return RestResponse.ok(user);
     }
 
@@ -173,7 +174,7 @@
 
     @RequestMapping(value = "/changeStatus/{id}", method = RequestMethod.POST)
     public RestResponse<Integer> changeStatus(@PathVariable Integer id) {
-        User user = userService.getUserById(id);
+        User user = userService.selectById(id);
         UserStatusEnum userStatusEnum = UserStatusEnum.fromCode(user.getStatus());
         Integer newStatus = userStatusEnum == UserStatusEnum.Enable ? UserStatusEnum.Disable.getCode() : UserStatusEnum.Enable.getCode();
         user.setStatus(newStatus);

--
Gitblit v1.8.0