From 5bacbb17debde58d6ff2581e4a03a866ca009c02 Mon Sep 17 00:00:00 2001
From: lizijie <lzjiiie@163.com>
Date: Mon, 31 May 2021 15:33:05 +0800
Subject: [PATCH] 菜单接口合并,查询父菜单列表

---
 screen-manage/src/main/java/com/moral/api/service/impl/ManageAccountServiceImpl.java |  112 +++++++++++++++++++++++--------------------------------
 1 files changed, 47 insertions(+), 65 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 5b50ee5..2076867 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
@@ -142,7 +142,7 @@
         HttpServletRequest request = ((ServletRequestAttributes) RequestContextHolder.getRequestAttributes()).getRequest();
         StringBuilder content = new StringBuilder();
         content.append(manageAccount.getUserName()).append("���������������������;");
-        logUtils.saveLoginForManage(request, content.toString(), manageAccount.getId(), manageAccount.getUserName());
+        logUtils.saveLoginForManage(request, content.toString(), manageAccount, manageAccount.getUserName());
 
         return loginDTO;
     }
@@ -175,7 +175,7 @@
         AccountDTO accountDTO = new AccountDTO();
         //������
         ManageAccount manageAccount = accountInsertForm.formConvertEntity();
-        List<String> roleIdsStr = accountInsertForm.getRoleIds();
+        List<Integer> roleIds = accountInsertForm.getRoleIds();
 
         /*������������������������*/
         ManageAccount existAccount = new ManageAccount();
@@ -193,37 +193,28 @@
         //������
         manageAccountMapper.insert(manageAccount);
 
-        //������account_role
-        /*String to Integer*/
-        List<ManageRole> roles = null;
         //������������������������������������������������������
-        if (!ObjectUtils.isEmpty(roleIdsStr)) {
-            List<Integer> roleIds = new ArrayList<>();
-            roleIdsStr.forEach(str -> roleIds.add(Integer.parseInt(str)));
-
+        if (!ObjectUtils.isEmpty(roleIds)) {
             Integer accountId = manageAccount.getId();
-            roleIdsStr.forEach(
+            roleIds.forEach(
                     value -> {
                         ManageAccountRole manageAccountRole = new ManageAccountRole();
                         manageAccountRole.setAccountId(accountId);
-                        manageAccountRole.setRoleId(Integer.parseInt(value));
+                        manageAccountRole.setRoleId(value);
                         manageAccountRoleMapper.insert(manageAccountRole);
                     }
             );
-            roles = manageRoleMapper.selectBatchIds(roleIds);
         }
         //������������������
-        accountDTO.setAccount(manageAccount);
-        accountDTO.setRoles(roles);
         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("id���" + manageAccount.getId() + ";");
-        logUtils.saveOperationForManage(request, content.toString());
+        content.append("������������������").append(manageAccount.getUserName() + ";")
+                .append("account���" + manageAccount.getAccount() + ";");
+        logUtils.saveOperationForManage(request, content.toString(), Constants.INSERT_OPERATE_TYPE);
 
         return accountDTO;
     }
@@ -240,7 +231,7 @@
         AccountQueryDTO accountQueryDTO = new AccountQueryDTO();
 
         //������
-        Integer pageCount = accountQueryForm.getPage();
+        Integer page = accountQueryForm.getPage();
         Integer size = accountQueryForm.getSize();
         String userName = accountQueryForm.getUserName();
         String email = accountQueryForm.getEmail();
@@ -250,10 +241,10 @@
         String order = accountQueryForm.getOrder();
         String orderType = accountQueryForm.getOrderType();
         Date createStartTime = accountQueryForm.getCreateStartTime();
-        Date createEndTime = DateUtils.getDateOfDay(accountQueryForm.getCreateEndTime(), 1);
+        Date createEndTime = accountQueryForm.getCreateEndTime();
 
         //������������������
-        Page<ManageAccount> page = new Page<>(pageCount, size);
+        Page<ManageAccount> queryPage = new Page<>(page, size);
         NullFilterWrapper<ManageAccount> wrapper = new NullFilterWrapper<>();
 
         wrapper.like("user_name", userName);
@@ -275,20 +266,19 @@
             wrapper.eq("is_delete", Constants.NOT_DELETE);
 
         //������������
-        Page resultPage = manageAccountMapper.selectPage(page, wrapper);
+        Page resultPage = manageAccountMapper.selectPage(queryPage, wrapper);
 
         //���������������������������
         List<ManageAccount> accounts = resultPage.getRecords();
         List<AccountDTO> accountDTOS = new ArrayList<>();
-        if (!ObjectUtils.isEmpty(accounts)) {
-            for (ManageAccount manageAccount : accounts) {
-                AccountDTO accountDTO = new AccountDTO();
-                List<ManageRole> roles = manageRoleMapper.getManageRoleByAccountId(manageAccount.getId());
-                accountDTO.setRoles(roles);
-                accountDTO.setAccount(manageAccount);
-                accountDTOS.add(accountDTO);
-            }
+        for (ManageAccount manageAccount : accounts) {
+            AccountDTO accountDTO = new AccountDTO();
+            List<ManageRole> roles = manageRoleMapper.getManageRoleByAccountId(manageAccount.getId());
+            accountDTO.setRoles(roles);
+            accountDTO.setAccount(manageAccount);
+            accountDTOS.add(accountDTO);
         }
+
 
         //������������������
         accountQueryDTO.setAccountDTOS(accountDTOS);
@@ -315,7 +305,7 @@
         AccountDTO accountDTO = new AccountDTO();
         //������
         Integer accountId = accountDeleteForm.getAccountId();
-        //������������������
+        //���������������������������������������������
         ManageAccount manageAccount = new ManageAccount();
         manageAccount.setIsDelete(Constants.NOT_DELETE);
         manageAccount.setId(accountId);
@@ -336,17 +326,17 @@
         UpdateWrapper<ManageAccountRole> deleteManageAccountRoleWrapper = new UpdateWrapper<>();
         deleteManageAccountRoleWrapper.set("is_delete", Constants.DELETE).eq("account_id", manageAccount.getId());
         manageAccountRoleMapper.update(null, deleteManageAccountRoleWrapper);
+        //������token
+        TokenUtils.destoryToken(accountId);
         //������������������
         accountDTO.setCode(ResponseCodeEnum.SUCCESS.getCode());
         accountDTO.setMsg(ResponseCodeEnum.SUCCESS.getMsg());
-        accountDTO.setAccount(existManageAccount);
-
         //������������������
         HttpServletRequest request = ((ServletRequestAttributes) RequestContextHolder.getRequestAttributes()).getRequest();
         StringBuilder content = new StringBuilder();
-        content.append("������������������").append(existManageAccount.getUserName() + ";")
-                .append("id���" + existManageAccount.getId() + ";");
-        logUtils.saveOperationForManage(request, content.toString());
+        content.append("������������������").append(existManageAccount.getUserName() + ";")
+                .append("������:" + existManageAccount.getAccount() + ";");
+        logUtils.saveOperationForManage(request, content.toString(), Constants.DELETE_OPERATE_TYPE);
 
         return accountDTO;
     }
@@ -366,26 +356,25 @@
         ManageAccount manageAccount = accountUpdateForm.formConvertEntity();
         List<Integer> roleIds = accountUpdateForm.getRoleIds();
 
-        //������������������������������������
-        QueryWrapper<ManageAccount> exitWrapper = new QueryWrapper<>();
-        ManageAccount exitManageAccount = new ManageAccount();
-        exitManageAccount.setId(manageAccount.getId());
-        exitManageAccount.setIsDelete(Constants.NOT_DELETE);
-        exitWrapper.setEntity(exitManageAccount);
-        exitManageAccount = manageAccountMapper.selectOne(exitWrapper);
-        if (ObjectUtils.isEmpty(exitManageAccount)) {
+        //������������������������������������������
+        QueryWrapper<ManageAccount> oldAccountWrapper = new QueryWrapper<>();
+        ManageAccount oldManageAccount = new ManageAccount();
+        oldManageAccount.setId(manageAccount.getId());
+        oldManageAccount.setIsDelete(Constants.NOT_DELETE);
+        oldAccountWrapper.setEntity(oldManageAccount);
+        oldManageAccount = manageAccountMapper.selectOne(oldAccountWrapper);
+        if (ObjectUtils.isEmpty(oldManageAccount)) {
             accountDTO.setCode(ResponseCodeEnum.ACCOUNT_NOT_EXIST.getCode());
             accountDTO.setMsg(ResponseCodeEnum.ACCOUNT_NOT_EXIST.getMsg());
             return accountDTO;
         }
 
         //������ManageAccount���
-        Map manageAccountMap = JSONObject.parseObject(JSON.toJSONString(manageAccount),Map.class);//���������Map���������������������������
-        if (manageAccountMap.size()>2) {//������������account������������������������������������������
+        Map manageAccountMap = JSONObject.parseObject(JSON.toJSONString(manageAccount), Map.class);//���������Map���������������������������
+        if (manageAccountMap.size() > 1) {//������������������������id���������������������������������
             manageAccountMapper.updateById(manageAccount);
         }
-        //������������������������
-        manageAccount = manageAccountMapper.selectById(manageAccount.getId());
+
         //������������������������������
         List<ManageRole> oldRoles = manageRoleMapper.getManageRoleByAccountId(manageAccount.getId());
 
@@ -395,8 +384,8 @@
          * ������roleIds���������������������������������������������������
          * ������roleIds���������������������������������������������
          * */
-        List<ManageRole> roles = null;
         if (roleIds != null) {
+            //������������������
             UpdateWrapper<ManageAccountRole> deleteWrapper = new UpdateWrapper<>();
             deleteWrapper.eq("account_id", manageAccount.getId()).set("is_delete", Constants.DELETE);
             manageAccountRoleMapper.update(null, deleteWrapper);
@@ -407,21 +396,14 @@
                 manageAccountRole.setRoleId(roleId);
                 manageAccountRoleMapper.insert(manageAccountRole);
             }
-            //������������������������
-            if (roleIds.size() != 0)
-                roles = manageRoleMapper.selectBatchIds(roleIds);
-        } else {
-            roles = manageRoleMapper.getManageRoleByAccountId(manageAccount.getId());
         }
-        accountDTO.setRoles(roles);
+        //������token
+        TokenUtils.destoryToken(manageAccount.getId());
         //������������������
         accountDTO.setCode(ResponseCodeEnum.SUCCESS.getCode());
         accountDTO.setMsg(ResponseCodeEnum.SUCCESS.getMsg());
-        accountDTO.setAccount(manageAccount);
-
         //������������������
-        insertUpdateLog(accountUpdateForm, exitManageAccount, oldRoles);
-
+        insertUpdateLog(accountUpdateForm, oldManageAccount, oldRoles);
         return accountDTO;
     }
 
@@ -436,8 +418,8 @@
         //������������������
         HttpServletRequest request = ((ServletRequestAttributes) RequestContextHolder.getRequestAttributes()).getRequest();
         StringBuilder content = new StringBuilder();
-        content.append("������������������").append(oldAccount.getUserName() + ";")
-                .append("id���" + oldAccount.getId() + ";");
+        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);
@@ -453,7 +435,7 @@
                 List<Integer> newRoleIds = updateForm.getRoleIds();
 
                 if (newRoleIds != null && 0 == newRoleIds.size()) {//���������������������������������������������������������������
-                    content.append("���������������������;");
+                    content.append("������������������������������;");
                 }
                 if ((newRoleIds != null) && (newRoleIds.size() != 0)) {//���������������������������������������������null���������������������������
                     StringBuilder oldRolesName = new StringBuilder("���");
@@ -465,7 +447,7 @@
                     }
 
                     if (!ObjectUtils.isEmpty(oldRoles)) {
-                        oldRolesName.deleteCharAt(oldRolesName.length()-1);//������ "���"
+                        oldRolesName.deleteCharAt(oldRolesName.length() - 1);//������ "���"
                         oldRolesName.append("[");
                         for (ManageRole role : oldRoles) {
                             oldRolesName.append(role.getName() + ",");
@@ -475,7 +457,7 @@
                     }
 
                     if (!ObjectUtils.isEmpty(newRoles)) {
-                        newRolesName.deleteCharAt(newRolesName.length()-1);//������ "���"
+                        newRolesName.deleteCharAt(newRolesName.length() - 1);//������ "���"
                         newRolesName.append("[");
                         for (ManageRole role : newRoles) {
                             newRolesName.append(role.getName() + ",");
@@ -484,7 +466,7 @@
                         newRolesName.append("]");
                     }
                     //������������content
-                    content.append(value+":"+oldRolesName + "->" + newRolesName + ";");
+                    content.append(value + ":" + oldRolesName + "->" + newRolesName + ";");
                 }
             } else {//������������������
                 if (newParameters.get(key) != null) {
@@ -500,7 +482,7 @@
                 }
             }
         }
-        logUtils.saveOperationForManage(request, content.toString());
+        logUtils.saveOperationForManage(request, content.toString(), Constants.UPDATE_OPERATE_TYPE);
     }
 
 

--
Gitblit v1.8.0