From 100690a225167806a08f64eafff3326564a7154a Mon Sep 17 00:00:00 2001
From: cjl <276999030@qq.com>
Date: Fri, 13 Oct 2023 17:06:22 +0800
Subject: [PATCH] fix:补偿提交

---
 screen-manage/src/main/java/com/moral/api/service/impl/ManageAccountServiceImpl.java |  121 +++++++++++++++++++---------------------
 1 files changed, 57 insertions(+), 64 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 d17e139..f62a6f5 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
@@ -25,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;
@@ -50,7 +52,6 @@
  * @since 2021-03-09
  */
 @Service
-@ConfigurationProperties(prefix = "log-aspect")
 public class ManageAccountServiceImpl extends ServiceImpl<ManageAccountMapper, ManageAccount> implements ManageAccountService {
 
 
@@ -62,14 +63,6 @@
     ManageMenuService manageMenuService;
     @Autowired
     ManageAccountRoleMapper manageAccountRoleMapper;
-    @Autowired
-    LogUtils logUtils;
-
-    Map<String, String> manageAccountFormMap;
-
-    public void setManageAccountFormMap(Map<String, String> manageAccountFormMap) {
-        this.manageAccountFormMap = manageAccountFormMap;
-    }
 
     /**
      * @Description: ������������
@@ -89,7 +82,7 @@
         //������������������
         QueryWrapper<ManageAccount> wrapper = new QueryWrapper<>();
         wrapper.eq("account", account);
-        wrapper.eq("is_delete",Constants.NOT_DELETE);
+        wrapper.eq("is_delete", Constants.NOT_DELETE);
         ManageAccount manageAccount = manageAccountMapper.selectOne(wrapper);
         if (ObjectUtils.isEmpty(manageAccount)) {
             loginDTO.setCode(ResponseCodeEnum.ACCOUNT_NOT_EXIST.getCode());
@@ -129,7 +122,7 @@
         HttpServletRequest request = ((ServletRequestAttributes) RequestContextHolder.getRequestAttributes()).getRequest();
         StringBuilder content = new StringBuilder();
         content.append(manageAccount.getUserName()).append("���������������������;");
-        logUtils.saveLoginForManage(request, content.toString(), manageAccount, manageAccount.getUserName());
+        LogUtils.saveLoginForManage(request, content.toString(), manageAccount, manageAccount.getUserName());
 
         return loginDTO;
     }
@@ -192,17 +185,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;
     }
 
@@ -322,6 +309,13 @@
             accountDTO.setMsg(ResponseCodeEnum.ACCOUNT_NOT_EXIST.getMsg());
             return accountDTO;
         }
+        //���������������������������
+        AccountInfoDTO accountInfo = (AccountInfoDTO) TokenUtils.getUserInfo();
+        if(accountInfo.getAccount().getId().equals(accountId)){
+            accountDTO.setCode(ResponseCodeEnum.CANNOT_DELETE_ONESELF.getCode());
+            accountDTO.setMsg(ResponseCodeEnum.CANNOT_DELETE_ONESELF.getMsg());
+            return accountDTO;
+        }
         //������������������
         UpdateWrapper<ManageAccount> deleteAccountWrapper = new UpdateWrapper<>();
         deleteAccountWrapper.eq("id", accountId);
@@ -337,12 +331,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;
     }
 
@@ -375,9 +364,8 @@
 
         //������ManageAccount���
         manageAccountMapper.updateById(manageAccount);
-
         //������������������
-        insertUpdateLog(accountUpdateForm, oldManageAccount);
+        updateLog(oldManageAccount,manageAccount);
         //������token
         TokenUtils.destoryToken(manageAccount.getId());
         //������������������
@@ -387,45 +375,50 @@
     }
 
     /**
-     * @Description: ���������������������������
-     * @Param: [form, newAccount, oldAccount]
-     * @return: void
-     * @Author: ���������
-     * @Date: 2021/4/8
-     */
-    private void insertUpdateLog(AccountUpdateForm updateForm, ManageAccount oldAccount) {
-        //������������������
-        HttpServletRequest request = ((ServletRequestAttributes) RequestContextHolder.getRequestAttributes()).getRequest();
+    * @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(oldAccount.getUserName() + ";")
-                .append("������:" + oldAccount.getAccount() + ";");
-        //������������Map,���������������������������������
-        Map<String, Object> newParameters = JSONObject.parseObject(JSON.toJSONString(updateForm), Map.class);
-        Map<String, Object> oldParameters = JSONObject.parseObject(JSON.toJSONString(oldAccount), Map.class);
-        //������������������������Map���������������������������
-        Set<String> keys = manageAccountFormMap.keySet();
-        for (String key : keys) {
-            String value = manageAccountFormMap.get(key);//���������������������
-            if ("password".equals(key)) {//������������������������������������������
-                if (!ObjectUtils.isEmpty(updateForm.getPassword())) {//���������������������������������
-                    content.append("���������������;");
-                }
-            } else {//������������������
-                if (newParameters.get(key) != null) {
-                    String newValue = "���";
-                    String oldValue = "���";
-                    if (newParameters.get(key) != null && !newParameters.get(key).equals(" ")) {
-                        newValue = String.valueOf(newParameters.get(key));
-                    }
-                    if (oldParameters.get(key) != null && !oldParameters.get(key).equals(" ")) {
-                        oldValue = String.valueOf(oldParameters.get(key));
-                    }
-                    content.append(value + ":" + oldValue + "->" + newValue + ";");
-                }
-            }
-        }
-        logUtils.saveOperationForManage(request, content.toString(), Constants.UPDATE_OPERATE_TYPE);
+        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