From 657182c7989dc77b69bb3f9adaf6c75b6466ff95 Mon Sep 17 00:00:00 2001 From: kaiyu <404897439@qq.com> Date: Wed, 09 Jun 2021 10:31:35 +0800 Subject: [PATCH] screen-manage 编辑账号接口删除分配角色功能,分配角色功能变为单独接口 --- screen-manage/src/main/java/com/moral/api/service/impl/ManageAccountServiceImpl.java | 195 ++++++++++++++++-------------------------------- 1 files changed, 66 insertions(+), 129 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..947fae4 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; @@ -40,6 +39,7 @@ import javax.servlet.http.HttpServletRequest; import java.util.*; +import java.util.function.Predicate; /** * <p> @@ -118,7 +118,7 @@ List<ManageRole> roles = manageRoleMapper.getManageRoleByAccountId(manageAccount.getId()); //������������ - List<ManageMenu> menus = null; + List<ManageMenu> menus = new ArrayList<>(); if (!ObjectUtils.isEmpty(roles)) { menus = manageMenuService.getParentChildrenMenusByRoles(roles); } @@ -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,37 @@ 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); - } + //������������������������������admin������admin��������������������� + AccountInfoDTO accountInfoDTO = (AccountInfoDTO) TokenUtils.getUserInfoByToken(); + 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(); + List<ManageRole> roles = manageRoleMapper.getManageRoleByAccountId(manageAccount.getId()); + accountDTO.setRoles(roles); + accountDTO.setAccount(manageAccount); + accountDTOS.add(accountDTO); + } + //������������������ accountQueryDTO.setAccountDTOS(accountDTOS); @@ -315,7 +323,7 @@ AccountDTO accountDTO = new AccountDTO(); //������ Integer accountId = accountDeleteForm.getAccountId(); - //������������������ + //��������������������������������������������� ManageAccount manageAccount = new ManageAccount(); manageAccount.setIsDelete(Constants.NOT_DELETE); manageAccount.setId(accountId); @@ -336,17 +344,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; } @@ -364,64 +372,30 @@ AccountDTO accountDTO = new AccountDTO(); //������ 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������������������������������������������ - manageAccountMapper.updateById(manageAccount); - } - //������������������������ - manageAccount = manageAccountMapper.selectById(manageAccount.getId()); - //������������������������������ - List<ManageRole> oldRoles = manageRoleMapper.getManageRoleByAccountId(manageAccount.getId()); + manageAccountMapper.updateById(manageAccount); - //���������������������������������ManageAccountRole��� - /* - * ������roleIds���null������������������������������������������ - * ������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); - /*������������������*/ - for (Integer roleId : roleIds) { - ManageAccountRole manageAccountRole = new ManageAccountRole(); - manageAccountRole.setAccountId(manageAccount.getId()); - manageAccountRole.setRoleId(roleId); - manageAccountRoleMapper.insert(manageAccountRole); - } - //������������������������ - if (roleIds.size() != 0) - roles = manageRoleMapper.selectBatchIds(roleIds); - } else { - roles = manageRoleMapper.getManageRoleByAccountId(manageAccount.getId()); - } - accountDTO.setRoles(roles); + //������������������ + insertUpdateLog(accountUpdateForm, oldManageAccount); + //������token + TokenUtils.destoryToken(manageAccount.getId()); //������������������ accountDTO.setCode(ResponseCodeEnum.SUCCESS.getCode()); accountDTO.setMsg(ResponseCodeEnum.SUCCESS.getMsg()); - accountDTO.setAccount(manageAccount); - - //������������������ - insertUpdateLog(accountUpdateForm, exitManageAccount, oldRoles); - return accountDTO; } @@ -432,12 +406,12 @@ * @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(); - 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); @@ -448,43 +422,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) { @@ -500,7 +437,7 @@ } } } - logUtils.saveOperationForManage(request, content.toString()); + logUtils.saveOperationForManage(request, content.toString(), Constants.UPDATE_OPERATE_TYPE); } -- Gitblit v1.8.0