From 5776cbc5f2f31f3e939cbbf792d257620f03eb89 Mon Sep 17 00:00:00 2001 From: kaiyu <404897439@qq.com> Date: Thu, 15 Jul 2021 11:54:23 +0800 Subject: [PATCH] screen-api 修改删除转换公式BUG --- screen-manage/src/main/java/com/moral/api/service/impl/ManageAccountServiceImpl.java | 90 +++----------------------------------------- 1 files changed, 7 insertions(+), 83 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 35f5d05..d17e139 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 @@ -89,21 +89,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 +120,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); //������������������ @@ -269,7 +256,7 @@ Page resultPage = manageAccountMapper.selectPage(queryPage, wrapper); List<ManageAccount> accounts = resultPage.getRecords(); //������������������������������admin������admin��������������������� - AccountInfoDTO accountInfoDTO = (AccountInfoDTO) TokenUtils.getUserInfoByToken(); + AccountInfoDTO accountInfoDTO = (AccountInfoDTO) TokenUtils.getUserInfo(); List<ManageRole> ownRoles = accountInfoDTO.getRoles(); boolean ownAdminRole = false; for (ManageRole ownRole : ownRoles) { @@ -372,7 +359,6 @@ AccountDTO accountDTO = new AccountDTO(); //������ ManageAccount manageAccount = accountUpdateForm.formConvertEntity(); - List<Integer> roleIds = accountUpdateForm.getRoleIds(); //������������������������������������������ QueryWrapper<ManageAccount> oldAccountWrapper = new QueryWrapper<>(); @@ -388,35 +374,10 @@ } //������ManageAccount��� - Map manageAccountMap = JSONObject.parseObject(JSON.toJSONString(manageAccount), Map.class);//���������Map��������������������������� - if (manageAccountMap.size() > 1) {//������������������������id��������������������������������� - manageAccountMapper.updateById(manageAccount); - } + 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); - } - } //������������������ - insertUpdateLog(accountUpdateForm, oldManageAccount, oldRoles); + insertUpdateLog(accountUpdateForm, oldManageAccount); //������token TokenUtils.destoryToken(manageAccount.getId()); //������������������ @@ -432,7 +393,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(); @@ -448,43 +409,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) { -- Gitblit v1.8.0