From ffb2063003d3e735f3f8848207078b75c3f05bfa Mon Sep 17 00:00:00 2001
From: kaiyu <404897439@qq.com>
Date: Fri, 21 May 2021 16:24:40 +0800
Subject: [PATCH] screen-manage 修改删除用户BUG
---
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