From 77d0b2f08d33730aa0701e0b528edcade5c4879b Mon Sep 17 00:00:00 2001
From: jinpengyong <jpy123456>
Date: Wed, 23 Jun 2021 14:38:11 +0800
Subject: [PATCH] screen-api更新

---
 screen-api/src/main/java/com/moral/api/controller/MenuController.java         |    4 
 screen-api/src/main/java/com/moral/api/service/impl/GroupServiceImpl.java     |   27 ++---
 screen-api/src/main/java/com/moral/api/service/GroupMenuService.java          |    2 
 screen-api/src/main/java/com/moral/api/service/impl/GroupMenuServiceImpl.java |    6 
 screen-api/src/main/java/com/moral/api/controller/LoginController.java        |   32 ++++--
 screen-api/src/main/java/com/moral/api/service/GroupService.java              |    6 +
 screen-api/src/main/java/com/moral/api/service/impl/UserServiceImpl.java      |   83 ++++++++++------
 screen-api/src/main/java/com/moral/api/controller/UserController.java         |   31 +++---
 pom.xml                                                                       |    9 +
 screen-api/src/main/java/com/moral/api/controller/GroupController.java        |   32 +++---
 screen-api/src/main/java/com/moral/api/service/UserService.java               |   12 ++
 screen-api/src/main/java/com/moral/api/entity/Group.java                      |    2 
 screen-api/src/main/java/com/moral/api/entity/User.java                       |    7 
 screen-api/src/main/java/com/moral/api/service/impl/UserGroupServiceImpl.java |    5 
 14 files changed, 153 insertions(+), 105 deletions(-)

diff --git a/pom.xml b/pom.xml
index 247deb0..10090e3 100644
--- a/pom.xml
+++ b/pom.xml
@@ -43,6 +43,7 @@
         <swagger-bootstrap-ui.version>1.9.6</swagger-bootstrap-ui.version>
         <spring.kafka.version>2.2.2</spring.kafka.version>
         <xxl-job.version>2.3.0</xxl-job.version>
+        <aviator.version>5.2.5</aviator.version>
     </properties>
 
 
@@ -51,7 +52,6 @@
             <groupId>org.springframework.boot</groupId>
             <artifactId>spring-boot-starter-web</artifactId>
         </dependency>
-
         <dependency>
             <groupId>io.springfox</groupId>
             <artifactId>springfox-swagger2</artifactId>
@@ -126,6 +126,13 @@
             <artifactId>xxl-job-core</artifactId>
             <version>${xxl-job.version}</version>
         </dependency>
+        <!-- https://mvnrepository.com/artifact/com.googlecode.aviator/aviator -->
+        <dependency>
+            <groupId>com.googlecode.aviator</groupId>
+            <artifactId>aviator</artifactId>
+            <version>${aviator.version}</version>
+        </dependency>
+
     </dependencies>
 
 
diff --git a/screen-api/src/main/java/com/moral/api/controller/GroupController.java b/screen-api/src/main/java/com/moral/api/controller/GroupController.java
index 3223ef9..9dacceb 100644
--- a/screen-api/src/main/java/com/moral/api/controller/GroupController.java
+++ b/screen-api/src/main/java/com/moral/api/controller/GroupController.java
@@ -25,6 +25,7 @@
 import com.moral.constant.ResponseCodeEnum;
 import com.moral.constant.ResultMessage;
 import com.moral.util.PageResult;
+import com.moral.util.WebUtils;
 
 @Slf4j
 @Api(tags = {"���������"})
@@ -57,21 +58,16 @@
 
     @ApiOperation(value = "���������", notes = "���������")
     @ApiImplicitParams({
+            @ApiImplicitParam(name = "id", value = "���id", required = true, paramType = "query", dataType = "int"),
             @ApiImplicitParam(name = "token", value = "token", required = true, paramType = "header", dataType = "String")
     })
-    @RequestMapping(value = "delete", method = RequestMethod.POST)
-    public ResultMessage delete(@RequestBody Group group) {
-        if (group.getId() == null) {
+    @RequestMapping(value = "delete", method = RequestMethod.GET)
+    public ResultMessage delete(Integer id) {
+        if (id == null) {
             return ResultMessage.fail(ResponseCodeEnum.PARAMETERS_IS_MISSING.getCode(),
                     ResponseCodeEnum.PARAMETERS_IS_MISSING.getMsg());
         }
-        QueryWrapper<Group> queryWrapper = new QueryWrapper<>();
-        queryWrapper.eq("id", group.getId());
-        group = groupService.getOne(queryWrapper);
-        if (group == null) {
-            return ResultMessage.fail(ResponseCodeEnum.ACCOUNT_NOT_EXIST.getCode(), ResponseCodeEnum.ACCOUNT_NOT_EXIST.getMsg());
-        }
-        groupService.deleteGroup(group);
+        groupService.deleteGroup(id);
         return ResultMessage.ok();
     }
 
@@ -80,8 +76,7 @@
             @ApiImplicitParam(name = "token", value = "token", required = true, paramType = "header", dataType = "String")
     })
     @RequestMapping(value = "update", method = RequestMethod.POST)
-    public ResultMessage update(@RequestBody Group group, HttpServletRequest request) {
-        String token = request.getHeader("token");
+    public ResultMessage update(@RequestBody Group group) {
         Map<String, Object> result = groupService.updateGroup(group);
         if (!result.isEmpty()) {
             return ResultMessage.fail((int) result.get("code"), result.get("msg").toString());
@@ -98,8 +93,9 @@
             @ApiImplicitParam(name = "token", value = "token", required = true, paramType = "header", dataType = "String"),
             @ApiImplicitParam(name = "groupName", value = "������������������", required = false, paramType = "query", dataType = "String")
     })
-    @RequestMapping(value = "select", method = RequestMethod.POST)
-    public ResultMessage select(@RequestBody Map<String, Object> parameters) {
+    @RequestMapping(value = "select", method = RequestMethod.GET)
+    public ResultMessage select(HttpServletRequest request) {
+        Map<String, Object> parameters = WebUtils.getParametersStartingWith(request, null);
         Page<Group> userPage = groupService.selectGroups(parameters);
         PageResult<Group> pageResult = new PageResult<>(
                 userPage.getTotal(), userPage.getPages(), userPage.getRecords()
@@ -109,10 +105,16 @@
 
     @ApiOperation(value = "���������������������ids", notes = "���������������������ids")
     @ApiImplicitParams({
+            @ApiImplicitParam(name = "userId", value = "������id", required = true, paramType = "query", dataType = "int"),
             @ApiImplicitParam(name = "token", value = "token", required = true, paramType = "header", dataType = "String")
     })
-    @RequestMapping(value = "get-group-ids", method = RequestMethod.GET)
+    @RequestMapping(value = "getGroupIds", method = RequestMethod.GET)
     public ResultMessage getGroupIds(Integer userId) {
+        System.out.println(userId);
+        if (userId == null) {
+            return ResultMessage.fail(ResponseCodeEnum.PARAMETERS_IS_MISSING.getCode(),
+                    ResponseCodeEnum.PARAMETERS_IS_MISSING.getMsg());
+        }
         List<Integer> groupIds = userGroupService.getGroupIds(userId);
         return ResultMessage.ok(groupIds);
     }
diff --git a/screen-api/src/main/java/com/moral/api/controller/LoginController.java b/screen-api/src/main/java/com/moral/api/controller/LoginController.java
index 2f34eee..ea704fe 100644
--- a/screen-api/src/main/java/com/moral/api/controller/LoginController.java
+++ b/screen-api/src/main/java/com/moral/api/controller/LoginController.java
@@ -7,6 +7,7 @@
 import lombok.extern.slf4j.Slf4j;
 
 import java.io.IOException;
+import java.util.HashMap;
 import java.util.Map;
 
 import javax.servlet.http.HttpServletRequest;
@@ -22,7 +23,6 @@
 import com.baomidou.mybatisplus.core.conditions.query.QueryWrapper;
 import com.moral.api.entity.SysConfig;
 import com.moral.api.service.SysConfigService;
-import com.moral.api.service.UserLogService;
 import com.moral.api.service.UserService;
 import com.moral.constant.Constants;
 import com.moral.constant.ResponseCodeEnum;
@@ -50,25 +50,35 @@
                     ResponseCodeEnum.PARAMETERS_IS_MISSING.getMsg());
         }
         Map<String, Object> result = userService.login(parameters);
-        if (!result.containsKey("data")) {
+        if (!result.containsKey("token")) {
             return ResultMessage.fail((int) result.get("code"), (String) result.get("msg"));
         }
-        Map<String, Object> data = (Map<String, Object>) result.get("data");
-        return ResultMessage.ok(data);
+        return ResultMessage.ok(result);
     }
 
-    @ApiOperation(value = "������", notes = "������")
-    @RequestMapping(value = "logout", method = RequestMethod.POST)
+    @ApiOperation(value = "������������������", notes = "������������������")
     @ApiImplicitParams({
             @ApiImplicitParam(name = "token", value = "token", required = true, paramType = "header", dataType = "String")
     })
-    public ResultMessage logout(@RequestBody Map<String, Object> parameters, HttpServletRequest request) {
-        if (!parameters.containsKey("uid")) {
+    @RequestMapping(value = "getUserInfo", method = RequestMethod.POST)
+    public ResultMessage getUserInfo(HttpServletRequest request) {
+        String token = request.getHeader("token");
+        Map<String, Object> result = userService.getUserInfo(token);
+        return ResultMessage.ok(result);
+    }
+
+    @ApiOperation(value = "������", notes = "������")
+    @RequestMapping(value = "logout", method = RequestMethod.GET)
+    @ApiImplicitParams({
+            @ApiImplicitParam(name = "token", value = "token", required = true, paramType = "header", dataType = "String")
+    })
+    public ResultMessage logout(Integer userId, HttpServletRequest request) {
+        if (userId == null) {
             return ResultMessage.fail(ResponseCodeEnum.PARAMETERS_IS_MISSING.getCode(),
                     ResponseCodeEnum.PARAMETERS_IS_MISSING.getMsg());
         }
         String token = request.getHeader("token");
-        TokenUtils.destoryToken(parameters.get("uid").toString(), token);
+        TokenUtils.destoryToken(userId.toString(), token);
         return ResultMessage.ok();
     }
 
@@ -79,9 +89,9 @@
         queryWrapper.eq(Constants.FRONT_VERIFICATIONCODE_COLUMN, Constants.FRONT_ERIFICATIONCODE_VALUE);
         SysConfig sysConfig = sysConfigService.getOne(queryWrapper);
         if (Constants.VERFICATIONCODE_OPEN.equals(sysConfig.getCode())) {
-            return new ResultMessage(ResponseCodeEnum.VERIFICATION_OPEN,null);
+            return new ResultMessage(ResponseCodeEnum.VERIFICATION_OPEN, null);
         }
-        return new ResultMessage(ResponseCodeEnum.VERIFICATION_CLOSE,null);
+        return new ResultMessage(ResponseCodeEnum.VERIFICATION_CLOSE, null);
     }
 
     @ApiOperation(value = "���������������", notes = "���������������")
diff --git a/screen-api/src/main/java/com/moral/api/controller/MenuController.java b/screen-api/src/main/java/com/moral/api/controller/MenuController.java
index f8b69ba..7ae682a 100644
--- a/screen-api/src/main/java/com/moral/api/controller/MenuController.java
+++ b/screen-api/src/main/java/com/moral/api/controller/MenuController.java
@@ -67,7 +67,7 @@
     @ApiImplicitParams({
             @ApiImplicitParam(name = "token", value = "token", required = true, paramType = "header", dataType = "String"),
     })
-    @RequestMapping(value = "select", method = RequestMethod.POST)
+    @RequestMapping(value = "select", method = RequestMethod.GET)
     public ResultMessage select(HttpServletRequest request) {
         String token = request.getHeader("token");
         Map<String, Object> currentUserInfo = (Map<String, Object>) TokenUtils.getUserInfoByToken(token);
@@ -80,7 +80,7 @@
     @ApiImplicitParams({
             @ApiImplicitParam(name = "token", value = "token", required = true, paramType = "header", dataType = "String")
     })
-    @RequestMapping(value = "get-menu-ids", method = RequestMethod.GET)
+    @RequestMapping(value = "getMenuIds", method = RequestMethod.GET)
     public ResultMessage getMenuIds(Integer groupId) {
         List<Integer> menusIds = groupMenuService.getMenusIds(groupId);
         return ResultMessage.ok(menusIds);
diff --git a/screen-api/src/main/java/com/moral/api/controller/UserController.java b/screen-api/src/main/java/com/moral/api/controller/UserController.java
index ba6d30a..39b7c5a 100644
--- a/screen-api/src/main/java/com/moral/api/controller/UserController.java
+++ b/screen-api/src/main/java/com/moral/api/controller/UserController.java
@@ -24,6 +24,7 @@
 import com.moral.constant.ResponseCodeEnum;
 import com.moral.constant.ResultMessage;
 import com.moral.util.PageResult;
+import com.moral.util.WebUtils;
 
 @Slf4j
 @Api(tags = {"������������"})
@@ -40,6 +41,7 @@
     })
     @RequestMapping(value = "insert", method = RequestMethod.POST)
     public ResultMessage insert(@RequestBody User user) {
+        System.out.println(user);
         if (user.getAccount().isEmpty() || user.getPassword().isEmpty()) {
             return ResultMessage.fail(ResponseCodeEnum.PARAMETERS_IS_MISSING.getCode(),
                     ResponseCodeEnum.PARAMETERS_IS_MISSING.getMsg());
@@ -53,22 +55,16 @@
 
     @ApiOperation(value = "������������", notes = "������������")
     @ApiImplicitParams({
+            @ApiImplicitParam(name = "id", value = "������id", required = true, paramType = "query", dataType = "int"),
             @ApiImplicitParam(name = "token", value = "token", required = true, paramType = "header", dataType = "String")
     })
-    @RequestMapping(value = "delete", method = RequestMethod.POST)
-    public ResultMessage delete(@RequestBody User user) {
-        if (user.getId() == null) {
+    @RequestMapping(value = "delete", method = RequestMethod.GET)
+    public ResultMessage delete(Integer id) {
+        if (id == null) {
             return ResultMessage.fail(ResponseCodeEnum.PARAMETERS_IS_MISSING.getCode(),
                     ResponseCodeEnum.PARAMETERS_IS_MISSING.getMsg());
         }
-        QueryWrapper<User> queryWrapper = new QueryWrapper<>();
-        queryWrapper.eq("id", user.getId());
-        user = userService.getOne(queryWrapper);
-        if (user == null) {
-            return ResultMessage.fail(ResponseCodeEnum.ACCOUNT_NOT_EXIST.getCode(),
-                    ResponseCodeEnum.ACCOUNT_NOT_EXIST.getMsg());
-        }
-        userService.deleteUser(user);
+        userService.deleteUser(id);
         return ResultMessage.ok();
     }
 
@@ -91,8 +87,8 @@
 
     @ApiOperation(value = "������������������������", notes = "������������������������")
     @ApiImplicitParams({
-            @ApiImplicitParam(name = "page", value = "���������", required = false, paramType = "query", dataType = "Integer"),
-            @ApiImplicitParam(name = "size", value = "������������", required = false, paramType = "query", dataType = "Integer"),
+            @ApiImplicitParam(name = "page", value = "���������", required = false, paramType = "query", dataType = "int"),
+            @ApiImplicitParam(name = "size", value = "������������", required = false, paramType = "query", dataType = "int"),
             @ApiImplicitParam(name = "order", value = "������������", required = false, paramType = "query", dataType = "String"),
             @ApiImplicitParam(name = "orderType", value = "������������������������0������������1", defaultValue = "0", required = false, paramType = "query", dataType = "String"),
             @ApiImplicitParam(name = "token", value = "token", required = true, paramType = "header", dataType = "String"),
@@ -103,8 +99,13 @@
             @ApiImplicitParam(name = "wechat", value = "������������������", required = false, paramType = "query", dataType = "String"),
 
     })
-    @RequestMapping(value = "select", method = RequestMethod.POST)
-    public ResultMessage select(@RequestBody Map<String, Object> parameters) {
+    @RequestMapping(value = "select", method = RequestMethod.GET)
+    public ResultMessage select(HttpServletRequest request) {
+        Map<String, Object> parameters = WebUtils.getParametersStartingWith(request, null);
+        if (!parameters.containsKey("page") || !parameters.containsKey("size")) {
+            return ResultMessage.fail(ResponseCodeEnum.PARAMETERS_IS_MISSING.getCode(),
+                    ResponseCodeEnum.PARAMETERS_IS_MISSING.getMsg());
+        }
         Page<User> userPage = userService.selectUsers(parameters);
         PageResult<User> pageResult = new PageResult<>(
                 userPage.getTotal(), userPage.getPages(), userPage.getRecords()
diff --git a/screen-api/src/main/java/com/moral/api/entity/Group.java b/screen-api/src/main/java/com/moral/api/entity/Group.java
index dc0d63e..0b79a9b 100644
--- a/screen-api/src/main/java/com/moral/api/entity/Group.java
+++ b/screen-api/src/main/java/com/moral/api/entity/Group.java
@@ -22,7 +22,7 @@
  */
 @Data
 @EqualsAndHashCode(callSuper = false)
-@TableName("`groupMenu`")
+@TableName("`group`")
 public class Group extends Model<Group> {
 
     private static final long serialVersionUID = 1L;
diff --git a/screen-api/src/main/java/com/moral/api/entity/User.java b/screen-api/src/main/java/com/moral/api/entity/User.java
index 678ff19..cf22cd8 100644
--- a/screen-api/src/main/java/com/moral/api/entity/User.java
+++ b/screen-api/src/main/java/com/moral/api/entity/User.java
@@ -13,9 +13,7 @@
 
 import lombok.Data;
 import lombok.EqualsAndHashCode;
-
-import javax.persistence.Transient;
-
+import org.springframework.format.annotation.DateTimeFormat;
 
 /**
  * <p>
@@ -93,7 +91,8 @@
     /**
      * ������������
      */
-    @JsonFormat(pattern = "yyyy-MM-dd HH:mm:ss", timezone = "GMT+8")
+    @JsonFormat(pattern = "yyyy-MM-dd", timezone = "GMT+8")
+    @DateTimeFormat(pattern = "yyyy-MM-dd")
     private Date expireTime;
 
     /**
diff --git a/screen-api/src/main/java/com/moral/api/service/GroupMenuService.java b/screen-api/src/main/java/com/moral/api/service/GroupMenuService.java
index 45edad9..70d23fb 100644
--- a/screen-api/src/main/java/com/moral/api/service/GroupMenuService.java
+++ b/screen-api/src/main/java/com/moral/api/service/GroupMenuService.java
@@ -16,7 +16,9 @@
  */
 public interface GroupMenuService extends IService<GroupMenu> {
 
+    //������������
     void allotMenus(Map<String, Object> parameters);
 
+    //���������������������������������������
     List<Integer> getMenusIds(Integer groupId);
 }
diff --git a/screen-api/src/main/java/com/moral/api/service/GroupService.java b/screen-api/src/main/java/com/moral/api/service/GroupService.java
index 02c1ea8..1bebb59 100644
--- a/screen-api/src/main/java/com/moral/api/service/GroupService.java
+++ b/screen-api/src/main/java/com/moral/api/service/GroupService.java
@@ -18,12 +18,16 @@
  */
 public interface GroupService extends IService<Group> {
 
+    //���������������������
     Map<String, Object> addGroup(Group group);
 
-    void deleteGroup(Group group);
+    //���������������������
+    void deleteGroup(Integer id);
 
+    //���������������������
     Map<String, Object> updateGroup(Group group);
 
+    //���������������������������
     Page<Group> selectGroups(Map<String, Object> parameters);
 
 }
diff --git a/screen-api/src/main/java/com/moral/api/service/UserService.java b/screen-api/src/main/java/com/moral/api/service/UserService.java
index 5680a37..90e5581 100644
--- a/screen-api/src/main/java/com/moral/api/service/UserService.java
+++ b/screen-api/src/main/java/com/moral/api/service/UserService.java
@@ -20,18 +20,28 @@
  */
 public interface UserService extends IService<User> {
 
+    //������
     Map<String, Object> login(Map<String, Object> parameters);
 
+    //������token������������������
+    Map<String,Object> getUserInfo(String token);
+
+    //������������
     Map<String, Object> getMenus(Map<String, Object> parameters);
 
+    //������������
     Map<String, Object> addUser(User user);
 
-    void deleteUser(User user);
+    //������������
+    void deleteUser(Integer userId);
 
+    //������������
     Map<String, Object> updateUser(User user);
 
+    //������������������
     Page<User> selectUsers(Map<String, Object> parameters);
 
+    //������������������������������������
     UserBO selectUserInfo(Map<String, Object> parameters);
 
 }
diff --git a/screen-api/src/main/java/com/moral/api/service/impl/GroupMenuServiceImpl.java b/screen-api/src/main/java/com/moral/api/service/impl/GroupMenuServiceImpl.java
index 7ddf506..ec6709b 100644
--- a/screen-api/src/main/java/com/moral/api/service/impl/GroupMenuServiceImpl.java
+++ b/screen-api/src/main/java/com/moral/api/service/impl/GroupMenuServiceImpl.java
@@ -20,6 +20,7 @@
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
+import org.springframework.util.ObjectUtils;
 import org.springframework.web.context.request.RequestContextHolder;
 import org.springframework.web.context.request.ServletRequestAttributes;
 
@@ -62,7 +63,7 @@
         deleteWrapper.eq("group_id", groupId).eq("channel_key", Constants.WEB_CHANNEL);
         groupMenuMapper.delete(deleteWrapper);
         List<String> menus = new ArrayList<>();
-        if (menuIds != null && !menuIds.isEmpty()) {
+        if (!ObjectUtils.isEmpty(menuIds)) {
             menuIds.forEach(menuId -> {
                 menus.add(menuMapper.selectById(menuId).getName());
                 GroupMenu groupMenu = new GroupMenu();
@@ -74,10 +75,9 @@
             });
         }
 
-
         //������
         String groupName = groupMapper.selectById((Integer) parameters.get("groupId")).getGroupName();
-        String content = "���������" + groupName + "������������������" + menus.toString();
+        String content = "���������" + groupName + "������������������" + menus;
         operationLogUtils.insertLog(request, content, Constants.UPDATE_OPERATE_TYPE);
     }
 
diff --git a/screen-api/src/main/java/com/moral/api/service/impl/GroupServiceImpl.java b/screen-api/src/main/java/com/moral/api/service/impl/GroupServiceImpl.java
index f03442c..4152336 100644
--- a/screen-api/src/main/java/com/moral/api/service/impl/GroupServiceImpl.java
+++ b/screen-api/src/main/java/com/moral/api/service/impl/GroupServiceImpl.java
@@ -10,6 +10,7 @@
 import com.baomidou.mybatisplus.extension.plugins.pagination.Page;
 import com.moral.api.entity.Group;
 import com.moral.api.entity.GroupMenu;
+import com.moral.api.entity.User;
 import com.moral.api.entity.UserGroup;
 import com.moral.api.mapper.GroupMapper;
 import com.moral.api.mapper.GroupMenuMapper;
@@ -80,21 +81,22 @@
     }
 
     @Override
-    public void deleteGroup(Group group) {
+    public void deleteGroup(Integer id) {
         //������������group
-        group.setIsDelete(Constants.DELETE);
-        groupMapper.updateById(group);
+        UpdateWrapper<Group> updateWrapper = new UpdateWrapper<>();
+        updateWrapper.eq("id", id).set("is_delete", Constants.DELETE);
+        groupMapper.update(null, updateWrapper);
         //������group_menu
         UpdateWrapper<GroupMenu> deleteGroupWrapper = new UpdateWrapper<>();
-        deleteGroupWrapper.eq("group_id", group.getId());
+        deleteGroupWrapper.eq("group_id", id);
         groupMenuMapper.delete(deleteGroupWrapper);
         //������user_group
         UpdateWrapper<UserGroup> deleteUserGroupWrapper = new UpdateWrapper<>();
-        deleteUserGroupWrapper.eq("group_id", group.getId());
+        deleteUserGroupWrapper.eq("group_id", id);
         userGroupMapper.delete(deleteUserGroupWrapper);
         //������
         HttpServletRequest request = ((ServletRequestAttributes) Objects.requireNonNull(RequestContextHolder.getRequestAttributes())).getRequest();
-        String content = "���������������" + group.getGroupName();
+        String content = "���������������" + groupMapper.selectById(id).getGroupName();
         operationLogUtils.insertLog(request, content, Constants.DELETE_OPERATE_TYPE);
     }
 
@@ -136,7 +138,8 @@
         Integer orgId = (Integer) orgInfo.get("id");
         QueryWrapper<Group> queryWrapper = new QueryWrapper<>();
         queryWrapper.eq("organization_id", orgId)
-                .eq("is_delete", Constants.NOT_DELETE);
+                .eq("is_delete", Constants.NOT_DELETE)
+                .ne("group_name", "admin");
         Object order = parameters.get("order");
         Object orderType = parameters.get("orderType");
         Object groupName = parameters.get("groupName");
@@ -145,18 +148,9 @@
             queryWrapper.like("group_name", groupName);
         }
 
-        //������������
-        if (!parameters.containsKey("page")) {
-            parameters.put("page", 0);
-        }
-        if (!parameters.containsKey("size")) {
-            parameters.put("size", 10);
-        }
-
         int page = Integer.parseInt(parameters.get("page").toString());
         int size = Integer.parseInt(parameters.get("size").toString());
         Page<Group> pageData = new Page<>(page, size);
-        //������������,������id������
         //������������,������create_time������
         if (order != null && orderType != null) {
             if (Constants.ORDER_ASC.equals(orderType)) {
@@ -170,5 +164,4 @@
         groupMapper.selectPage(pageData, queryWrapper);
         return pageData;
     }
-
 }
diff --git a/screen-api/src/main/java/com/moral/api/service/impl/UserGroupServiceImpl.java b/screen-api/src/main/java/com/moral/api/service/impl/UserGroupServiceImpl.java
index 4515798..75a25f0 100644
--- a/screen-api/src/main/java/com/moral/api/service/impl/UserGroupServiceImpl.java
+++ b/screen-api/src/main/java/com/moral/api/service/impl/UserGroupServiceImpl.java
@@ -20,6 +20,7 @@
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
+import org.springframework.util.ObjectUtils;
 import org.springframework.web.context.request.RequestContextHolder;
 import org.springframework.web.context.request.ServletRequestAttributes;
 
@@ -62,7 +63,7 @@
         deleteWrapper.eq("user_id", userId);
         userGroupMapper.delete(deleteWrapper);
         List<String> groups = new ArrayList<>();
-        if (groupIds != null && !groupIds.isEmpty()) {
+        if (!ObjectUtils.isEmpty(groupIds)) {
             groupIds.forEach(groupId -> {
                 groups.add(groupMapper.selectById(groupId).getGroupName());
                 UserGroup userGroup = new UserGroup();
@@ -75,7 +76,7 @@
 
         //������
         String account = userMapper.selectById((Integer) parameters.get("userId")).getAccount();
-        String content = "������������" + account + "���������������" + groups.toString();
+        String content = "������������" + account + "���������������" + groups;
         operationLogUtils.insertLog(request, content, Constants.UPDATE_OPERATE_TYPE);
     }
 
diff --git a/screen-api/src/main/java/com/moral/api/service/impl/UserServiceImpl.java b/screen-api/src/main/java/com/moral/api/service/impl/UserServiceImpl.java
index 9a82667..12dad29 100644
--- a/screen-api/src/main/java/com/moral/api/service/impl/UserServiceImpl.java
+++ b/screen-api/src/main/java/com/moral/api/service/impl/UserServiceImpl.java
@@ -1,9 +1,11 @@
 package com.moral.api.service.impl;
 
+import lombok.extern.slf4j.Slf4j;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.beans.factory.annotation.Value;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
+import org.springframework.util.ObjectUtils;
 import org.springframework.web.context.request.RequestContextHolder;
 import org.springframework.web.context.request.ServletRequestAttributes;
 
@@ -57,6 +59,7 @@
  * @since 2021-03-09
  */
 @Service
+@Slf4j
 public class UserServiceImpl extends ServiceImpl<UserMapper, User> implements UserService {
 
     @Autowired
@@ -76,7 +79,6 @@
 
     @Value("${AES.KEY}")
     private String AESKey;
-
 
     @Override
     public UserBO selectUserInfo(Map<String, Object> parameters) {
@@ -153,14 +155,12 @@
         try {
             //������token������������redis
             String token = TokenUtils.getToken(userBo.getId().toString(), userInfo);
-            Map<String, Object> data = new HashMap<>();
-            data.put("token", token);
-            data.put("user", userInfo);
-            result.put("data", data);
+            result.put("token", token);
         } catch (Exception e) {
             log.error("token���������������" + e.getMessage());
             result.put("code", ResponseCodeEnum.TOKEN_CREATE_ERROR.getCode());
             result.put("msg", ResponseCodeEnum.TOKEN_CREATE_ERROR.getMsg());
+            return result;
         }
         //������
         HttpServletRequest request = ((ServletRequestAttributes) Objects.requireNonNull(RequestContextHolder.getRequestAttributes())).getRequest();
@@ -172,6 +172,12 @@
         userLog.setContent("������������������" + userBo.getAccount());
         userLogMapper.insert(userLog);
         return result;
+    }
+
+    @Override
+    public Map<String, Object> getUserInfo(String token) {
+        Map<String, Object> userInfo = (Map<String, Object>) TokenUtils.getUserInfoByToken(token);
+        return userInfo;
     }
 
     @Override
@@ -195,10 +201,10 @@
         oneMenu.forEach(o -> {
             Map<String, Object> menuMap = new LinkedHashMap<>();
             menuMap.put("id", o.getId());
-            menuMap.put("name", o.getName());
+            menuMap.put("label", o.getName());
             menuMap.put("url", o.getUrl());
             menuMap.put("icon", o.getIcon());
-            menuMap.put("child", getMenusByRecursion(o, allMenus));
+            menuMap.put("children", getMenusByRecursion(o, allMenus));
             newList.add(menuMap);
         });
         resultMap.put("menus", newList);
@@ -242,24 +248,25 @@
         QueryWrapper<User> queryWrapper = new QueryWrapper<>();
         queryWrapper.eq("account", user.getAccount()).eq("is_delete", Constants.NOT_DELETE);
         //������������������������
-        if (user.selectOne(queryWrapper) != null) {
+        if (user.selectCount(queryWrapper) > 0) {
             result.put("code", ResponseCodeEnum.ACCOUNT_EXIST.getCode());
             result.put("msg", ResponseCodeEnum.ACCOUNT_EXIST.getMsg());
             return result;
         }
-        //������������������
+        //������������
         if (!RegexUtils.checkAccount(user.getAccount())) {
             result.put("code", ResponseCodeEnum.ACCOUNT_INVALID.getCode());
             result.put("msg", ResponseCodeEnum.ACCOUNT_INVALID.getMsg());
             return result;
         }
-        //������������������
-        if (user.getEmail() != null && !RegexUtils.checkEmail(user.getEmail())) {
+
+        //������������
+        if (!ObjectUtils.isEmpty(user.getEmail()) && !RegexUtils.checkEmail(user.getEmail())) {
             result.put("code", ResponseCodeEnum.EMAIL_INVALID.getCode());
             result.put("msg", ResponseCodeEnum.EMAIL_INVALID.getMsg());
             return result;
         }
-        //������������������
+        //������������
         String password = user.getPassword();
         //������������
         password = AESUtils.decrypt(password, AESKey);
@@ -268,8 +275,8 @@
             result.put("msg", ResponseCodeEnum.PASSWORD_INVALID.getMsg());
             return result;
         }
-        //���������������������
-        if (user.getMobile() != null && !RegexUtils.checkMobile(user.getMobile())) {
+        //���������������
+        if (!ObjectUtils.isEmpty(user.getMobile()) && !RegexUtils.checkMobile(user.getMobile())) {
             result.put("code", ResponseCodeEnum.MOBILE_INVALID.getCode());
             result.put("msg", ResponseCodeEnum.MOBILE_INVALID.getMsg());
             return result;
@@ -304,19 +311,20 @@
 
     @Override
     @Transactional
-    public void deleteUser(User user) {
-        String userId = user.getId().toString();
+    public void deleteUser(Integer userId) {
+        User user = userMapper.selectById(userId);
         //������������user
-        user.setIsDelete(Constants.DELETE);
-        userMapper.updateById(user);
+        UpdateWrapper<User> updateWrapper = new UpdateWrapper<>();
+        updateWrapper.eq("id", userId).set("is_delete", Constants.DELETE);
+        userMapper.update(null, updateWrapper);
         //������user_group
         UpdateWrapper<UserGroup> deleteWrapper = new UpdateWrapper<>();
         deleteWrapper.eq("user_id", userId);
         userGroupMapper.delete(deleteWrapper);
-        //������redis
-        if (TokenUtils.hHasKey(userId)) {
-            String token = TokenUtils.hget(userId).toString();
-            TokenUtils.destoryToken(userId, token);
+        //������redis
+        if (TokenUtils.hHasKey(userId.toString())) {
+            String token = TokenUtils.hget(userId.toString()).toString();
+            TokenUtils.destoryToken(userId.toString(), token);
         }
         //������
         HttpServletRequest request = ((ServletRequestAttributes) Objects.requireNonNull(RequestContextHolder.getRequestAttributes())).getRequest();
@@ -330,25 +338,36 @@
         HttpServletRequest request = ((ServletRequestAttributes) Objects.requireNonNull(RequestContextHolder.getRequestAttributes())).getRequest();
         String token = request.getHeader("token");
 
+        Map<String, Object> result = new HashMap<>();
         //���������������
         Map<String, Object> update = JSONObject.parseObject(JSON.toJSONString(user), Map.class);
-        Map<String, Object> result = new HashMap<>();
         Map<String, Object> currentUserInfo = (Map<String, Object>) TokenUtils.getUserInfoByToken(token);
         User beforeUser = userMapper.selectById(user.getId());
         Map<String, Object> before = JSONObject.parseObject(JSON.toJSONString(beforeUser), Map.class);
-        String account = beforeUser.getAccount();
+        /*String account = beforeUser.getAccount();
         //account������������
-        user.setAccount(account);
+        user.setAccount(account);*/
+        System.out.println(user);
+        if (!ObjectUtils.isEmpty(user.getAccount())) {
+            QueryWrapper<User> queryWrapper = new QueryWrapper<>();
+            queryWrapper.eq("is_delete", Constants.NOT_DELETE).eq("account", user.getAccount());
+            if (userMapper.selectCount(queryWrapper) > 0) {
+                result.put("code", ResponseCodeEnum.ACCOUNT_EXIST.getCode());
+                result.put("msg", ResponseCodeEnum.ACCOUNT_EXIST.getMsg());
+                return result;
+            }
+        }
 
         //������������
-        if (user.getEmail() != null && !RegexUtils.checkEmail(user.getEmail())) {
+        if (!ObjectUtils.isEmpty(user.getEmail()) && !RegexUtils.checkEmail(user.getEmail())) {
             result.put("code", ResponseCodeEnum.EMAIL_INVALID.getCode());
             result.put("msg", ResponseCodeEnum.EMAIL_INVALID.getMsg());
             return result;
         }
         //������������
         String password = user.getPassword();
-        if (password != null) {
+        System.out.println("password==" + password);
+        if (!ObjectUtils.isEmpty(password)) {
             //������������
             password = AESUtils.decrypt(password, AESKey);
             if (!RegexUtils.checkPassword(password)) {
@@ -359,7 +378,7 @@
         }
 
         //���������������������
-        if (user.getMobile() != null && !RegexUtils.checkMobile(user.getMobile())) {
+        if (!ObjectUtils.isEmpty(user.getMobile()) && !RegexUtils.checkMobile(user.getMobile())) {
             result.put("code", ResponseCodeEnum.MOBILE_INVALID.getCode());
             result.put("msg", ResponseCodeEnum.MOBILE_INVALID.getMsg());
             return result;
@@ -404,7 +423,6 @@
             }
         }
         operationLogUtils.insertLog(request, content.toString(), Constants.UPDATE_OPERATE_TYPE);
-
         return result;
     }
 
@@ -417,7 +435,8 @@
         Map<String, Object> orgInfo = (Map<String, Object>) currentUserInfo.get("organization");
         QueryWrapper<User> queryWrapper = new QueryWrapper<>();
         queryWrapper.eq("organization_id", orgInfo.get("id"))
-                .eq("is_delete", Constants.NOT_DELETE);
+                .eq("is_delete", Constants.NOT_DELETE)
+                .eq("is_admin", false);
         Object order = parameters.get("order");
         Object orderType = parameters.get("orderType");
         Object account = parameters.get("account");
@@ -442,8 +461,8 @@
             queryWrapper.like("wechat", wechat);
         }
 
-        Integer page = (Integer) parameters.get("page");
-        Integer size = (Integer) parameters.get("size");
+        int page = Integer.parseInt(parameters.get("page").toString());
+        int size = Integer.parseInt(parameters.get("size").toString());
         Page<User> pageData = new Page<>(page, size);
         //������������,������create_time������
         if (order != null && orderType != null) {

--
Gitblit v1.8.0