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/UserServiceImpl.java | 99 +++++++++++++++++++++++--------------------------
1 files changed, 47 insertions(+), 52 deletions(-)
diff --git a/screen-manage/src/main/java/com/moral/api/service/impl/UserServiceImpl.java b/screen-manage/src/main/java/com/moral/api/service/impl/UserServiceImpl.java
index 3d9a977..fe313f3 100644
--- a/screen-manage/src/main/java/com/moral/api/service/impl/UserServiceImpl.java
+++ b/screen-manage/src/main/java/com/moral/api/service/impl/UserServiceImpl.java
@@ -9,10 +9,7 @@
import com.moral.api.entity.Organization;
import com.moral.api.entity.User;
import com.moral.api.entity.UserGroup;
-import com.moral.api.mapper.GroupMapper;
-import com.moral.api.mapper.OrganizationMapper;
-import com.moral.api.mapper.UserGroupMapper;
-import com.moral.api.mapper.UserMapper;
+import com.moral.api.mapper.*;
import com.moral.api.pojo.dto.user.UserDTO;
import com.moral.api.pojo.dto.user.UserQueryDTO;
import com.moral.api.pojo.form.user.UserDeleteForm;
@@ -52,6 +49,8 @@
GroupMapper groupMapper;
@Autowired
UserGroupMapper userGroupMapper;
+ @Autowired
+ GroupMenuMapper groupMenuMapper;
@Override
public UserQueryDTO queryUsers(UserQueryForm form) {
@@ -218,65 +217,61 @@
@Override
@Transactional
public UserDTO deleteUser(UserDeleteForm form) {
+ //������������������
+ UserDTO dto = new UserDTO();
//������
Integer id = form.getId();
//������������������
User user = new User();
- user.setIsDelete(Constants.NOT_DELETE);
+ user.setIsDelete(Constants.DELETE);
user.setId(id);
- //������������������
- UserDTO dto = deleteUserModel(user);
+ //������������
+ userMapper.updateById(user);
+ //������user_group
+ UpdateWrapper deleteUserGroupWrapper = new UpdateWrapper();
+ deleteUserGroupWrapper.eq("user_id", id);
+ deleteUserGroupWrapper.set("is_delete", Constants.DELETE);
+ userGroupMapper.update(null, deleteUserGroupWrapper);
//���������������admin������������
UpdateWrapper updateOrgWrapper = new UpdateWrapper();
- updateOrgWrapper.eq("admin_user_id",user.getId());
- updateOrgWrapper.set("admin_user_id",0);
- organizationMapper.update(null,updateOrgWrapper);
- return dto;
- }
-
- @Override
- @Transactional
- public UserDTO deleteUserByOrganizationId(Integer organizationId) {
- //������������������
- User user = new User();
- user.setOrganizationId(organizationId);
- //������������������
- UserDTO dto = deleteUserModel(user);
- return dto;
- }
-
-
- private UserDTO deleteUserModel(User user) {
- //������������������
- UserDTO dto = new UserDTO();
- //������������
- Integer organizationId = null;
- //������������������������������������������������
- User oldUser = null;
- //������������������������������id
- if (ObjectUtils.isEmpty(user.getOrganizationId())) {
- QueryWrapper<User> deleteUserWrapper = new QueryWrapper<>();
- deleteUserWrapper.setEntity(user);
- oldUser = userMapper.selectOne(deleteUserWrapper);
- if (ObjectUtils.isEmpty(user)) {
- dto.setCode(ResponseCodeEnum.USER_NOT_EXIST.getCode());
- dto.setMsg(ResponseCodeEnum.USER_NOT_EXIST.getMsg());
- return dto;
- }
- organizationId = oldUser.getOrganizationId();
- }else{
- organizationId = user.getOrganizationId();
- }
- //������������������������������������������������������������
- UpdateWrapper<User> deleteUserChildrenWrapper = new UpdateWrapper<>();
- deleteUserChildrenWrapper.eq("organization_id", organizationId);
- deleteUserChildrenWrapper.set("is_delete", Constants.DELETE);
- userMapper.update(null, deleteUserChildrenWrapper);
- //������������
+ updateOrgWrapper.eq("admin_user_id", user.getId());
+ updateOrgWrapper.set("admin_user_id", 0);
+ organizationMapper.update(null, updateOrgWrapper);
+ //������������������
dto.setCode(ResponseCodeEnum.SUCCESS.getCode());
dto.setMsg(ResponseCodeEnum.SUCCESS.getMsg());
return dto;
}
+ @Override
+ @Transactional
+ public UserDTO deleteUsersByOrganizationId(Integer organizationId) {
+ //������������������
+ UserDTO dto = new UserDTO();
+ //������������������������������������������������������������
+ UpdateWrapper<User> deleteUserChildrenWrapper = new UpdateWrapper<>();
+ deleteUserChildrenWrapper.eq("organization_id", organizationId);
+ deleteUserChildrenWrapper.set("is_delete", Constants.DELETE);
+ userMapper.update(null, deleteUserChildrenWrapper);
+ //���������������groupMenu
+ UpdateWrapper deleteGroupMenuWrapper = new UpdateWrapper();
+ deleteGroupMenuWrapper.eq("organization_id", organizationId);
+ deleteGroupMenuWrapper.set("is_delete", Constants.DELETE);
+ groupMenuMapper.update(null, deleteGroupMenuWrapper);
+ //���������������userGroup
+ UpdateWrapper deleteUserGroupWrapper = new UpdateWrapper();
+ deleteUserGroupWrapper.eq("organization_id", organizationId);
+ deleteUserGroupWrapper.set("is_delete", Constants.DELETE);
+ userGroupMapper.update(null, deleteUserGroupWrapper);
+ //���������������group
+ UpdateWrapper deleteGroupWrapper = new UpdateWrapper();
+ deleteGroupWrapper.eq("organization_id", organizationId);
+ deleteGroupWrapper.set("is_delete", Constants.DELETE);
+ groupMapper.update(null, deleteGroupWrapper);
+ //������������
+ dto.setCode(ResponseCodeEnum.SUCCESS.getCode());
+ dto.setMsg(ResponseCodeEnum.SUCCESS.getMsg());
+ return dto;
+ }
}
--
Gitblit v1.8.0