From f05a7d377838ba6b809329829aff058a8c455736 Mon Sep 17 00:00:00 2001
From: jinpengyong <jpy123456>
Date: Wed, 14 Apr 2021 17:26:36 +0800
Subject: [PATCH] 日志接口重构

---
 screen-api/src/main/java/com/moral/api/service/impl/UserServiceImpl.java |   46 ++++++++++++++++++++++++++--------------------
 1 files changed, 26 insertions(+), 20 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 6ccef8c..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;
@@ -9,13 +16,6 @@
 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 org.springframework.web.context.request.RequestContextHolder;
-import org.springframework.web.context.request.ServletRequestAttributes;
 
 import javax.servlet.http.HttpServletRequest;
 
@@ -30,7 +30,6 @@
 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;
@@ -163,6 +162,7 @@
         //������
         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());
@@ -229,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);
 
@@ -291,9 +294,8 @@
         }
         userMapper.insert(user);
         //������
-        HttpServletRequest request = ((ServletRequestAttributes) Objects.requireNonNull(RequestContextHolder.getRequestAttributes())).getRequest();
         String content = "������������������" + user.getAccount();
-        OperationLogUtils.insertLog(request, content);
+        OperationLogUtils.insertLog(request, content, Constants.INSERT_OPERATE_TYPE);
         return result;
     }
 
@@ -304,24 +306,27 @@
         //������������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);
+        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<>();
@@ -386,7 +391,6 @@
         userMapper.updateById(user);
 
         //������
-        HttpServletRequest request = ((ServletRequestAttributes) Objects.requireNonNull(RequestContextHolder.getRequestAttributes())).getRequest();
         StringBuilder content = new StringBuilder("������������������" + user.getAccount() + "->");
         for (String key : update.keySet()) {
             Object afterValue = update.get(key);
@@ -396,14 +400,17 @@
                 content.append(key).append(":").append(beforeValue).append("=>").append(afterValue).append(";");
             }
         }
-        OperationLogUtils.insertLog(request, content.toString());
+        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"))
@@ -446,7 +453,6 @@
             queryWrapper.orderByDesc("create_time");
         }
         userMapper.selectPage(pageData, queryWrapper);
-
         return pageData;
     }
 

--
Gitblit v1.8.0