From 6ebf2b2a84b903f3600238dd084b3ae9ee4d6d3b Mon Sep 17 00:00:00 2001 From: kaiyu <404897439@qq.com> Date: Fri, 16 Apr 2021 17:33:07 +0800 Subject: [PATCH] Merge branch 'dev' of http://blit.7drlb.com:8888/r/moral into dev --- screen-api/src/main/java/com/moral/api/service/impl/UserServiceImpl.java | 103 ++++++++++++++++++++++++++++++++++++++++++--------- 1 files changed, 84 insertions(+), 19 deletions(-) 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 70b8a57..ca8fb19 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,5 +1,12 @@ package com.moral.api.service.impl; +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.web.context.request.RequestContextHolder; +import org.springframework.web.context.request.ServletRequestAttributes; + import java.util.ArrayList; import java.util.Comparator; import java.util.Date; @@ -7,13 +14,13 @@ import java.util.LinkedHashMap; import java.util.List; import java.util.Map; +import java.util.Objects; import java.util.stream.Collectors; -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 javax.servlet.http.HttpServletRequest; +import com.alibaba.fastjson.JSON; +import com.alibaba.fastjson.JSONObject; import com.baomidou.mybatisplus.core.conditions.query.QueryWrapper; import com.baomidou.mybatisplus.core.conditions.update.UpdateWrapper; import com.baomidou.mybatisplus.extension.plugins.pagination.Page; @@ -23,12 +30,14 @@ import com.moral.api.entity.Organization; import com.moral.api.entity.User; import com.moral.api.entity.UserGroup; - +import com.moral.api.entity.UserLog; import com.moral.api.mapper.MenuMapper; import com.moral.api.mapper.UserGroupMapper; +import com.moral.api.mapper.UserLogMapper; import com.moral.api.mapper.UserMapper; import com.moral.api.pojo.bo.UserBO; import com.moral.api.service.UserService; +import com.moral.api.utils.OperationLogUtils; import com.moral.constant.Constants; import com.moral.constant.ResponseCodeEnum; import com.moral.util.AESUtils; @@ -37,6 +46,7 @@ import com.moral.util.MD5Utils; import com.moral.util.RegexUtils; import com.moral.util.TokenUtils; +import com.moral.util.WebUtils; /** * <p> @@ -58,8 +68,12 @@ @Autowired private UserGroupMapper userGroupMapper; + @Autowired + private UserLogMapper userLogMapper; + @Value("${AES.KEY}") private String AESKey; + @Override public UserBO selectUserInfo(Map<String, Object> parameters) { @@ -145,6 +159,15 @@ result.put("code", ResponseCodeEnum.TOKEN_CREATE_ERROR.getCode()); result.put("msg", ResponseCodeEnum.TOKEN_CREATE_ERROR.getMsg()); } + //������ + HttpServletRequest request = ((ServletRequestAttributes) Objects.requireNonNull(RequestContextHolder.getRequestAttributes())).getRequest(); + UserLog userLog = new UserLog(); + userLog.setType(Constants.LOGIN_OPERTATE_TYPE); + userLog.setIp(WebUtils.getIpAddr(request)); + userLog.setOperateId(userBo.getId()); + userLog.setOrganizationId(userBo.getOrganizationId()); + userLog.setContent("������������������" + userBo.getAccount()); + userLogMapper.insert(userLog); return result; } @@ -206,7 +229,10 @@ @Override @Transactional - public Map<String, Object> addUser(User user, String token) { + public Map<String, Object> addUser(User user) { + HttpServletRequest request = ((ServletRequestAttributes) Objects.requireNonNull(RequestContextHolder.getRequestAttributes())).getRequest(); + String token = request.getHeader("token"); + Map<String, Object> result = new HashMap<>(); Map<String, Object> currentUserInfo = (Map<String, Object>) TokenUtils.getUserInfoByToken(token); @@ -261,8 +287,15 @@ } else { user.setExpireTime(userExpireTime); } + } else { + if (orgExpireTime != null) { + user.setExpireTime(orgExpireTime); + } } userMapper.insert(user); + //������ + String content = "������������������" + user.getAccount(); + OperationLogUtils.insertLog(request, content, Constants.INSERT_OPERATE_TYPE); return result; } @@ -273,23 +306,34 @@ //������������user user.setIsDelete(Constants.DELETE); userMapper.updateById(user); - //������������user_group + //������user_group UpdateWrapper<UserGroup> deleteWrapper = new UpdateWrapper<>(); - deleteWrapper.set("is_delete", Constants.DELETE).eq("user_id", userId); + deleteWrapper.eq("user_id", userId); + userGroupMapper.delete(deleteWrapper); //������redis if (TokenUtils.hHasKey(userId)) { String token = TokenUtils.hget(userId).toString(); TokenUtils.destoryToken(userId, token); } - userGroupMapper.update(null, deleteWrapper); + //������ + HttpServletRequest request = ((ServletRequestAttributes) Objects.requireNonNull(RequestContextHolder.getRequestAttributes())).getRequest(); + String content = "������������������" + user.getAccount(); + OperationLogUtils.insertLog(request, content, Constants.DELETE_OPERATE_TYPE); } @Override @Transactional - public Map<String, Object> updateUser(User user, String token) { + public Map<String, Object> updateUser(User user) { + HttpServletRequest request = ((ServletRequestAttributes) Objects.requireNonNull(RequestContextHolder.getRequestAttributes())).getRequest(); + String token = request.getHeader("token"); + + //��������������� + 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); - String account = userMapper.selectById(user.getId()).getAccount(); + User beforeUser = userMapper.selectById(user.getId()); + Map<String, Object> before = JSONObject.parseObject(JSON.toJSONString(beforeUser), Map.class); + String account = beforeUser.getAccount(); //account������������ user.setAccount(account); @@ -301,12 +345,14 @@ } //������������ String password = user.getPassword(); - //������������ - password = AESUtils.decrypt(password, AESKey); - if (!RegexUtils.checkPassword(password)) { - result.put("code", ResponseCodeEnum.PASSWORD_INVALID.getCode()); - result.put("msg", ResponseCodeEnum.PASSWORD_INVALID.getMsg()); - return result; + if (password != null) { + //������������ + password = AESUtils.decrypt(password, AESKey); + if (!RegexUtils.checkPassword(password)) { + result.put("code", ResponseCodeEnum.PASSWORD_INVALID.getCode()); + result.put("msg", ResponseCodeEnum.PASSWORD_INVALID.getMsg()); + return result; + } } //��������������������� @@ -331,6 +377,10 @@ } else { user.setExpireTime(userExpireTime); } + } else { + if (orgExpireTime != null) { + user.setExpireTime(orgExpireTime); + } } //������redis String userId = user.getId().toString(); @@ -339,12 +389,28 @@ TokenUtils.destoryToken(userId, deleteToken); } userMapper.updateById(user); + + //������ + StringBuilder content = new StringBuilder("������������������" + user.getAccount() + "->"); + for (String key : update.keySet()) { + Object afterValue = update.get(key); + if (!key.equals("id") && afterValue != null) { + //������������������ + Object beforeValue = before.get(key); + content.append(key).append(":").append(beforeValue).append("=>").append(afterValue).append(";"); + } + } + OperationLogUtils.insertLog(request, content.toString(), Constants.UPDATE_OPERATE_TYPE); + return result; } @Override public Page<User> selectUsers(Map<String, Object> parameters) { - Map<String, Object> currentUserInfo = (Map<String, Object>) TokenUtils.getUserInfoByToken(parameters.get("token").toString()); + HttpServletRequest request = ((ServletRequestAttributes) Objects.requireNonNull(RequestContextHolder.getRequestAttributes())).getRequest(); + String token = request.getHeader("token"); + + Map<String, Object> currentUserInfo = (Map<String, Object>) TokenUtils.getUserInfoByToken(token); Map<String, Object> orgInfo = (Map<String, Object>) currentUserInfo.get("organization"); QueryWrapper<User> queryWrapper = new QueryWrapper<>(); queryWrapper.eq("organization_id", orgInfo.get("id")) @@ -387,7 +453,6 @@ queryWrapper.orderByDesc("create_time"); } userMapper.selectPage(pageData, queryWrapper); - return pageData; } -- Gitblit v1.8.0