From ac625fa6aee52f467243c78764625cf053dcfe81 Mon Sep 17 00:00:00 2001
From: kaiyu <404897439@qq.com>
Date: Wed, 25 Aug 2021 11:44:20 +0800
Subject: [PATCH] screen-manage         重构组织模块日志插入代码

---
 screen-manage/src/main/java/com/moral/api/service/impl/ManageAccountServiceImpl.java |  183 ++++++++++++++++++++-------------------------
 1 files changed, 81 insertions(+), 102 deletions(-)

diff --git a/screen-manage/src/main/java/com/moral/api/service/impl/ManageAccountServiceImpl.java b/screen-manage/src/main/java/com/moral/api/service/impl/ManageAccountServiceImpl.java
index f6e15a0..d43de51 100644
--- a/screen-manage/src/main/java/com/moral/api/service/impl/ManageAccountServiceImpl.java
+++ b/screen-manage/src/main/java/com/moral/api/service/impl/ManageAccountServiceImpl.java
@@ -11,10 +11,9 @@
 import com.moral.api.entity.ManageRole;
 import com.moral.api.mapper.ManageAccountMapper;
 import com.moral.api.mapper.ManageAccountRoleMapper;
-import com.moral.api.mapper.ManageMenuMapper;
 import com.moral.api.mapper.ManageRoleMapper;
 import com.moral.api.pojo.dto.account.*;
-import com.moral.api.pojo.dto.login.AccountInfoDTO;
+import com.moral.api.pojo.redisBean.AccountInfoDTO;
 import com.moral.api.pojo.dto.login.LoginDTO;
 import com.moral.api.pojo.form.account.AccountDeleteForm;
 import com.moral.api.pojo.form.account.AccountInsertForm;
@@ -26,9 +25,11 @@
 import com.baomidou.mybatisplus.extension.service.impl.ServiceImpl;
 import com.moral.api.config.mybatis.wrapper.NullFilterWrapper;
 import com.moral.api.service.ManageMenuService;
+import com.moral.api.util.CompareFieldUtils;
 import com.moral.api.util.LogUtils;
 import com.moral.constant.Constants;
 import com.moral.constant.ResponseCodeEnum;
+import com.moral.pojo.CompareFieldResult;
 import com.moral.util.*;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.boot.context.properties.ConfigurationProperties;
@@ -40,6 +41,7 @@
 
 import javax.servlet.http.HttpServletRequest;
 import java.util.*;
+import java.util.function.Predicate;
 
 /**
  * <p>
@@ -89,21 +91,10 @@
         //������������������
         QueryWrapper<ManageAccount> wrapper = new QueryWrapper<>();
         wrapper.eq("account", account);
-        List<ManageAccount> manageAccounts = manageAccountMapper.selectList(wrapper);
-        if (ObjectUtils.isEmpty(manageAccounts)) {
-            loginDTO.setCode(ResponseCodeEnum.ACCOUNT_NOT_EXIST.getCode());
-            loginDTO.setMsg(ResponseCodeEnum.ACCOUNT_NOT_EXIST.getMsg());
-            return loginDTO;
-        }
-        //������������������������
-        ManageAccount manageAccount = null;
-        for (ManageAccount value : manageAccounts) {
-            if (Constants.NOT_DELETE.equals(value.getIsDelete()))
-                manageAccount = value;
-        }
-
+        wrapper.eq("is_delete", Constants.NOT_DELETE);
+        ManageAccount manageAccount = manageAccountMapper.selectOne(wrapper);
         if (ObjectUtils.isEmpty(manageAccount)) {
-            loginDTO.setCode(ResponseCodeEnum.ACCOUNT_IS_DELETE.getCode());
+            loginDTO.setCode(ResponseCodeEnum.ACCOUNT_NOT_EXIST.getCode());
             loginDTO.setMsg(ResponseCodeEnum.ACCOUNT_NOT_EXIST.getMsg());
             return loginDTO;
         }
@@ -131,11 +122,9 @@
 
         //������token ������������������
         String token = TokenUtils.getToken(String.valueOf(manageAccount.getId()), accountInfoDTO);
-
         //������������������
         loginDTO.setCode(ResponseCodeEnum.SUCCESS.getCode());
         loginDTO.setMsg(ResponseCodeEnum.SUCCESS.getMsg());
-        loginDTO.setAccountInfoDTO(accountInfoDTO);
         loginDTO.setToken(token);
 
         //������������������
@@ -205,17 +194,11 @@
                     }
             );
         }
+        //������������������
+        insertLog(manageAccount);
         //������������������
         accountDTO.setCode(ResponseCodeEnum.SUCCESS.getCode());
         accountDTO.setMsg(ResponseCodeEnum.SUCCESS.getMsg());
-
-        //������������������
-        HttpServletRequest request = ((ServletRequestAttributes) RequestContextHolder.getRequestAttributes()).getRequest();
-        StringBuilder content = new StringBuilder();
-        content.append("������������������").append(manageAccount.getUserName() + ";")
-                .append("account���" + manageAccount.getAccount() + ";");
-        logUtils.saveOperationForManage(request, content.toString(), Constants.INSERT_OPERATE_TYPE);
-
         return accountDTO;
     }
 
@@ -267,9 +250,27 @@
 
         //������������
         Page resultPage = manageAccountMapper.selectPage(queryPage, wrapper);
-
-        //���������������������������
         List<ManageAccount> accounts = resultPage.getRecords();
+        //������������������������������admin������admin���������������������
+        AccountInfoDTO accountInfoDTO = (AccountInfoDTO) TokenUtils.getUserInfo();
+        List<ManageRole> ownRoles = accountInfoDTO.getRoles();
+        boolean ownAdminRole = false;
+        for (ManageRole ownRole : ownRoles) {
+            String roleName = ownRole.getName();
+            if ("admin".equals(roleName))
+                ownAdminRole = true;
+        }
+        if (!ownAdminRole) {
+            accounts.removeIf(new Predicate<ManageAccount>() {
+                @Override
+                public boolean test(ManageAccount manageAccount) {
+                    if (manageAccount.getAccount().equals("admin"))
+                        return true;
+                    return false;
+                }
+            });
+        }
+        //���������������������������
         List<AccountDTO> accountDTOS = new ArrayList<>();
         for (ManageAccount manageAccount : accounts) {
             AccountDTO accountDTO = new AccountDTO();
@@ -332,12 +333,7 @@
         accountDTO.setCode(ResponseCodeEnum.SUCCESS.getCode());
         accountDTO.setMsg(ResponseCodeEnum.SUCCESS.getMsg());
         //������������������
-        HttpServletRequest request = ((ServletRequestAttributes) RequestContextHolder.getRequestAttributes()).getRequest();
-        StringBuilder content = new StringBuilder();
-        content.append("������������������").append(existManageAccount.getUserName() + ";")
-                .append("������:" + existManageAccount.getAccount() + ";");
-        logUtils.saveOperationForManage(request, content.toString(), Constants.DELETE_OPERATE_TYPE);
-
+        deleteLog(existManageAccount);
         return accountDTO;
     }
 
@@ -354,7 +350,6 @@
         AccountDTO accountDTO = new AccountDTO();
         //������
         ManageAccount manageAccount = accountUpdateForm.formConvertEntity();
-        List<Integer> roleIds = accountUpdateForm.getRoleIds();
 
         //������������������������������������������
         QueryWrapper<ManageAccount> oldAccountWrapper = new QueryWrapper<>();
@@ -370,40 +365,14 @@
         }
 
         //������ManageAccount���
-        Map manageAccountMap = JSONObject.parseObject(JSON.toJSONString(manageAccount), Map.class);//���������Map���������������������������
-        if (manageAccountMap.size() > 1) {//������������������������id���������������������������������
-            manageAccountMapper.updateById(manageAccount);
-        }
-
-        //������������������������������
-        List<ManageRole> oldRoles = manageRoleMapper.getManageRoleByAccountId(manageAccount.getId());
-
-        //���������������������������������ManageAccountRole���
-        /*
-         * ������roleIds���null������������������������������������������
-         * ������roleIds���������������������������������������������������
-         * ������roleIds���������������������������������������������
-         * */
-        if (roleIds != null) {
-            //������������������
-            UpdateWrapper<ManageAccountRole> deleteWrapper = new UpdateWrapper<>();
-            deleteWrapper.eq("account_id", manageAccount.getId()).set("is_delete", Constants.DELETE);
-            manageAccountRoleMapper.update(null, deleteWrapper);
-            /*������������������*/
-            for (Integer roleId : roleIds) {
-                ManageAccountRole manageAccountRole = new ManageAccountRole();
-                manageAccountRole.setAccountId(manageAccount.getId());
-                manageAccountRole.setRoleId(roleId);
-                manageAccountRoleMapper.insert(manageAccountRole);
-            }
-        }
+        manageAccountMapper.updateById(manageAccount);
+        //������������������
+        updateLog(oldManageAccount,manageAccount);
         //������token
         TokenUtils.destoryToken(manageAccount.getId());
         //������������������
         accountDTO.setCode(ResponseCodeEnum.SUCCESS.getCode());
         accountDTO.setMsg(ResponseCodeEnum.SUCCESS.getMsg());
-        //������������������
-        insertUpdateLog(accountUpdateForm, oldManageAccount, oldRoles);
         return accountDTO;
     }
 
@@ -414,7 +383,7 @@
      * @Author: ���������
      * @Date: 2021/4/8
      */
-    private void insertUpdateLog(AccountUpdateForm updateForm, ManageAccount oldAccount, List<ManageRole> oldRoles) {
+    private void insertUpdateLog(AccountUpdateForm updateForm, ManageAccount oldAccount) {
         //������������������
         HttpServletRequest request = ((ServletRequestAttributes) RequestContextHolder.getRequestAttributes()).getRequest();
         StringBuilder content = new StringBuilder();
@@ -430,43 +399,6 @@
             if ("password".equals(key)) {//������������������������������������������
                 if (!ObjectUtils.isEmpty(updateForm.getPassword())) {//���������������������������������
                     content.append("���������������;");
-                }
-            } else if ("roleIds".equals(key)) {//������������������������Id���������������������
-                List<Integer> newRoleIds = updateForm.getRoleIds();
-
-                if (newRoleIds != null && 0 == newRoleIds.size()) {//���������������������������������������������������������������
-                    content.append("������������������������������;");
-                }
-                if ((newRoleIds != null) && (newRoleIds.size() != 0)) {//���������������������������������������������null���������������������������
-                    StringBuilder oldRolesName = new StringBuilder("���");
-                    StringBuilder newRolesName = new StringBuilder("���");
-
-                    List<ManageRole> newRoles = null;
-                    if (!ObjectUtils.isEmpty(updateForm.getRoleIds())) {
-                        newRoles = manageRoleMapper.selectBatchIds(updateForm.getRoleIds());
-                    }
-
-                    if (!ObjectUtils.isEmpty(oldRoles)) {
-                        oldRolesName.deleteCharAt(oldRolesName.length() - 1);//������ "���"
-                        oldRolesName.append("[");
-                        for (ManageRole role : oldRoles) {
-                            oldRolesName.append(role.getName() + ",");
-                        }
-                        oldRolesName.deleteCharAt(oldRolesName.length() - 1);//������������������������
-                        oldRolesName.append("]");
-                    }
-
-                    if (!ObjectUtils.isEmpty(newRoles)) {
-                        newRolesName.deleteCharAt(newRolesName.length() - 1);//������ "���"
-                        newRolesName.append("[");
-                        for (ManageRole role : newRoles) {
-                            newRolesName.append(role.getName() + ",");
-                        }
-                        newRolesName.deleteCharAt(newRolesName.length() - 1);//������������������������
-                        newRolesName.append("]");
-                    }
-                    //������������content
-                    content.append(value + ":" + oldRolesName + "->" + newRolesName + ";");
                 }
             } else {//������������������
                 if (newParameters.get(key) != null) {
@@ -486,4 +418,51 @@
     }
 
 
+    /**
+    * @Description: ������������������������
+            * @Param: [oldAccount, newAccount]
+            * @return: void
+            * @Author: ���������
+            * @Date: 2021/8/25
+            */
+    private void updateLog(ManageAccount oldAccount,ManageAccount newAccount){
+        List<CompareFieldResult> results = CompareFieldUtils.compare(ManageAccount.class, oldAccount, newAccount);
+        StringBuilder content = new StringBuilder("���������������������"+";");
+        content.append("���������"+oldAccount.getAccount()+";");
+        //������������������
+        if(!ObjectUtils.isEmpty(newAccount.getPassword()))
+            content.append("���������������");
+        String resultContent = CompareFieldUtils.resultsConvertContent(results,content.toString());
+        logUtils.saveOperationForManage(resultContent, Constants.UPDATE_OPERATE_TYPE);
+    }
+
+    /**
+    * @Description: ������������������������
+            * @Param: [account]
+            * @return: void
+            * @Author: ���������
+            * @Date: 2021/8/25
+            */
+    private void insertLog(ManageAccount manageAccount){
+        StringBuilder content = new StringBuilder();
+        content.append("������������������������").append(manageAccount.getUserName() + ";")
+                .append("account���" + manageAccount.getAccount() + ";");
+        logUtils.saveOperationForManage(content.toString(), Constants.INSERT_OPERATE_TYPE);
+    }
+
+
+    /**
+    * @Description: ������������������������
+            * @Param: [manageAccount]
+            * @return: void
+            * @Author: ���������
+            * @Date: 2021/8/25
+            */
+    private void deleteLog(ManageAccount manageAccount){
+        StringBuilder content = new StringBuilder();
+        content.append("������������������").append(manageAccount.getUserName() + ";")
+                .append("������:" + manageAccount.getAccount() + ";");
+        logUtils.saveOperationForManage( content.toString(), Constants.DELETE_OPERATE_TYPE);
+    }
 }
+

--
Gitblit v1.8.0