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