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/OrganizationServiceImpl.java | 149 ++++++++++++++++++++++++++-----------------------
1 files changed, 78 insertions(+), 71 deletions(-)
diff --git a/screen-manage/src/main/java/com/moral/api/service/impl/OrganizationServiceImpl.java b/screen-manage/src/main/java/com/moral/api/service/impl/OrganizationServiceImpl.java
index d7ac38c..54c25ba 100644
--- a/screen-manage/src/main/java/com/moral/api/service/impl/OrganizationServiceImpl.java
+++ b/screen-manage/src/main/java/com/moral/api/service/impl/OrganizationServiceImpl.java
@@ -7,19 +7,21 @@
import com.baomidou.mybatisplus.extension.plugins.pagination.Page;
import com.moral.api.config.mybatis.wrapper.NullFilterWrapper;
import com.moral.api.entity.Organization;
+import com.moral.api.entity.User;
import com.moral.api.mapper.OrganizationMapper;
+import com.moral.api.mapper.UserMapper;
import com.moral.api.pojo.dto.organization.OrganizationDTO;
import com.moral.api.pojo.dto.organization.OrganizationQueryDTO;
-import com.moral.api.pojo.dto.organization.OrganizationQueryNameDTO;
+import com.moral.api.pojo.dto.organization.OrganizationQueryNamesDTO;
import com.moral.api.pojo.form.organization.*;
import com.moral.api.service.OrganizationService;
import com.baomidou.mybatisplus.extension.service.impl.ServiceImpl;
+import com.moral.api.service.UserService;
import com.moral.api.util.LogUtils;
import com.moral.constant.Constants;
import com.moral.constant.ResponseCodeEnum;
import com.moral.util.ConvertUtils;
import com.moral.util.DateUtils;
-import lombok.Data;
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.boot.context.properties.ConfigurationProperties;
import org.springframework.stereotype.Service;
@@ -49,6 +51,12 @@
@Autowired
LogUtils logUtils;
+
+ @Autowired
+ UserService userService;
+
+ @Autowired
+ UserMapper userMapper;
Map<String, String> organizationFormMap;
@@ -83,26 +91,10 @@
organizationDTO.setMsg(ResponseCodeEnum.ORGANIZATION_EXIST.getMsg());
return organizationDTO;
}
- //���������������������������
- Integer parentId = organization.getParentId();
- Organization parentOrganization = new Organization();
- if (!ObjectUtils.isEmpty(parentId)) {
- parentOrganization.setId(parentId);
- parentOrganization.setIsDelete(Constants.NOT_DELETE);
- queryWrapper.setEntity(parentOrganization);
- parentOrganization = organizationMapper.selectOne(queryWrapper);
- if (ObjectUtils.isEmpty(parentOrganization)) {
- organizationDTO.setCode(ResponseCodeEnum.ORGANIZATION_PARENT_NOT_EXIST.getCode());
- organizationDTO.setMsg(ResponseCodeEnum.ORGANIZATION_PARENT_NOT_EXIST.getMsg());
- return organizationDTO;
- }
- }
//������������
organizationMapper.insert(organization);
//������DTO������
- organizationDTO.setParentOrganization(parentOrganization);
- organizationDTO.setOrganization(organization);
organizationDTO.setCode(ResponseCodeEnum.SUCCESS.getCode());
organizationDTO.setMsg(ResponseCodeEnum.SUCCESS.getMsg());
@@ -128,34 +120,19 @@
//form���entity
Organization organization = organizationUpdateForm.formConvertEntity();
- //������������������������
- QueryWrapper<Organization> existWrapper = new QueryWrapper<>();
- Organization existOrganization = new Organization();
- existOrganization.setId(organization.getId());
- existOrganization.setIsDelete(Constants.NOT_DELETE);
- existWrapper.setEntity(existOrganization);
- existOrganization = organizationMapper.selectOne(existWrapper);
- if (ObjectUtils.isEmpty(existOrganization)) {
+ //������������������������������������������
+ QueryWrapper<Organization> oldWrapper = new QueryWrapper<>();
+ Organization oldOrganization = new Organization();
+ oldOrganization.setId(organization.getId());
+ oldOrganization.setIsDelete(Constants.NOT_DELETE);
+ oldWrapper.setEntity(oldOrganization);
+ oldOrganization = organizationMapper.selectOne(oldWrapper);
+ if (ObjectUtils.isEmpty(oldOrganization)) {
organizationDTO.setCode(ResponseCodeEnum.ORGANIZATION_NOT_EXIST.getCode());
organizationDTO.setMsg(ResponseCodeEnum.ORGANIZATION_NOT_EXIST.getMsg());
return organizationDTO;
}
- //������������������������������������������������������
- Integer parentId = organization.getParentId();
- Organization parentOrganization = new Organization();
- if (!ObjectUtils.isEmpty(parentId) && parentId != 0) {
- QueryWrapper<Organization> existParentWrapper = new QueryWrapper<>();
- parentOrganization.setId(parentId);
- parentOrganization.setIsDelete(Constants.NOT_DELETE);
- existParentWrapper.setEntity(parentOrganization);
- parentOrganization = organizationMapper.selectOne(existParentWrapper);
- if (ObjectUtils.isEmpty(parentOrganization)) {
- organizationDTO.setCode(ResponseCodeEnum.ORGANIZATION_PARENT_NOT_EXIST.getCode());
- organizationDTO.setMsg(ResponseCodeEnum.ORGANIZATION_PARENT_NOT_EXIST.getMsg());
- return organizationDTO;
- }
- }
//������������
organizationMapper.updateById(organization);
@@ -164,16 +141,16 @@
organization = organizationMapper.selectById(organization.getId());
//������DTO������
- organizationDTO.setParentOrganization(parentOrganization);
- organizationDTO.setOrganization(organization);
organizationDTO.setCode(ResponseCodeEnum.SUCCESS.getCode());
organizationDTO.setMsg(ResponseCodeEnum.SUCCESS.getMsg());
//������������������
- insertUpdateLog(organizationUpdateForm, organization, existOrganization);
+ insertUpdateLog(organizationUpdateForm, organization, oldOrganization);
return organizationDTO;
}
+
+
/**
* @Description: ������������������
@@ -189,7 +166,7 @@
//������
Integer id = form.getOrganizationId();
- //������������������������
+ //������������������������������������������
Organization existOrganization = new Organization();
existOrganization.setIsDelete(Constants.NOT_DELETE);
existOrganization.setId(id);
@@ -208,10 +185,13 @@
deleteWrapper.set("is_delete", Constants.DELETE);
organizationMapper.update(null, deleteWrapper);
+ //������������������
+ userService.deleteUsersByOrganizationId(id);
+
//���������������������������
- List<Organization> children = getAllChildrenOrganization(existOrganization);
+ List<Organization> children = getAllChildrenOrganization(existOrganization.getId());
if (!ObjectUtils.isEmpty(children)) {//���������������������
- //������������������������������������������������������������������������������������������������������������������
+ //���������������������������������������������������������������������������������������������������������������������������
UpdateWrapper updateWrapper = new UpdateWrapper();
if (form.getDeleteChildren().equals(Constants.DELETE_CHILDREN_ORG)) {//���������������������
//������������������������id���������������
@@ -223,6 +203,9 @@
updateWrapper.in("id", childrenId);
updateWrapper.set("is_delete", Constants.DELETE);
organizationMapper.update(null, updateWrapper);
+ //���������������������������
+ childrenId.forEach(value->userService.deleteUsersByOrganizationId(value));
+
} else {//���������
//���������������������������id
List<Integer> childrenId = new ArrayList<>();
@@ -238,7 +221,6 @@
}
//������������������
- dto.setOrganization(existOrganization);
dto.setCode(ResponseCodeEnum.SUCCESS.getCode());
dto.setMsg(ResponseCodeEnum.SUCCESS.getMsg());
//������������������
@@ -318,12 +300,22 @@
Page<Organization> resultPage = organizationMapper.selectPage(page, queryWrapper);
List<Organization> organizations = resultPage.getRecords();
List<OrganizationDTO> organizationDTOS = new ArrayList<>();
- //������������������������������������������organization���DTO���
- for (Organization child : organizations) {
+ //���������������������������������admin������������������organization���DTO���
+ for (Organization organization : organizations) {
OrganizationDTO resultDto = new OrganizationDTO();
- Organization parent = organizationMapper.selectById(child.getParentId());//���������������
- resultDto.setOrganization(child);
+ //���������������
+ Organization parent = organizationMapper.selectById(organization.getParentId());
+ resultDto.setOrganization(organization);
resultDto.setParentOrganization(parent);
+ //������admin������
+ QueryWrapper userWrapper = new QueryWrapper();
+ User adminUser = new User();
+ adminUser.setIsAdmin(true);
+ adminUser.setOrganizationId(organization.getId());
+ adminUser.setIsDelete(Constants.NOT_DELETE);
+ userWrapper.setEntity(adminUser);
+ adminUser = userMapper.selectOne(userWrapper);
+ resultDto.setAdminUser(adminUser);
organizationDTOS.add(resultDto);
}
@@ -331,31 +323,47 @@
dto.setMsg(ResponseCodeEnum.SUCCESS.getMsg());
dto.setOrganizationDTOS(organizationDTOS);
dto.setCurrent(page.getCurrent());
- dto.setPage(page.getPages());
+ dto.setPages(page.getPages());
dto.setSize(page.getSize());
dto.setTotal(page.getTotal());
return dto;
}
/**
- * @Description: ������������������������������Id���������������
- * @Param: [organizationQueryNameForm]
- * @return: com.moral.api.pojo.dto.organization.OrganizationQueryNameDTO
- * @Author: ���������
- * @Date: 2021/4/13
- */
+ * @Description: ������form���id���������������������������������
+ * ������id������������������������id������id������������������������������������
+ * @Param: [from]
+ * @return: com.moral.api.pojo.dto.organization.OrganizationQueryNamesDTO
+ * @Author: ���������
+ * @Date: 2021/5/7
+ */
@Override
- public OrganizationQueryNameDTO queryOrganizationNames(OrganizationQueryNameForm form) {
- OrganizationQueryNameDTO dto = new OrganizationQueryNameDTO();
+ public OrganizationQueryNamesDTO queryNames(OrganizationQueryNamesForm form) {
+ OrganizationQueryNamesDTO dto = new OrganizationQueryNamesDTO();
//������
- String name = form.getName();
+ Integer id = form.getId();
//������������������
- NullFilterWrapper<Organization> queryWrapper = new NullFilterWrapper();
- queryWrapper.like("name", name);
+ QueryWrapper<Organization> queryWrapper = new QueryWrapper();
queryWrapper.eq("is_delete", Constants.NOT_DELETE);
- //������
+ //������������������
List<Organization> organizations = organizationMapper.selectList(queryWrapper);
-
+ //������form������������id,���������������������������������������������
+ if(!ObjectUtils.isEmpty(id)){
+ List<Organization> children = getAllChildrenOrganization(id);
+ List<Integer> thisAndchildrenIds = new ArrayList<>();//���id���������������������������id������
+ thisAndchildrenIds.add(id);
+ for (Organization child : children) {
+ thisAndchildrenIds.add(child.getId());
+ }
+ organizations.removeIf(new Predicate<Organization>() {//������
+ @Override
+ public boolean test(Organization organization) {
+ if(thisAndchildrenIds.contains(organization.getId()))
+ return true;
+ return false;
+ }
+ });
+ }
//������������������
dto.setCode(ResponseCodeEnum.SUCCESS.getCode());
dto.setMsg(ResponseCodeEnum.SUCCESS.getMsg());
@@ -427,9 +435,9 @@
* @Author: ���������
* @Date: 2021/4/14
*/
- private List<Organization> getAllChildrenOrganization(Organization parent) {
+ private List<Organization> getAllChildrenOrganization(Integer parentId) {
List<Organization> children = new ArrayList<>();
- recursionQueryChildren(parent, children);
+ recursionQueryChildren(parentId, children);
return children;
}
@@ -440,8 +448,7 @@
* @Author: ���������
* @Date: 2021/4/14
*/
- private void recursionQueryChildren(Organization parent, List<Organization> children) {
- Integer parentId = parent.getId();
+ private void recursionQueryChildren(Integer parentId, List<Organization> children) {
QueryWrapper<Organization> queryWrapper = new QueryWrapper();
queryWrapper.eq("is_delete", Constants.NOT_DELETE);
queryWrapper.eq("parent_id", parentId);
@@ -449,7 +456,7 @@
if (!ObjectUtils.isEmpty(organizations)) {
children.addAll(organizations);
for (Organization organization : organizations) {
- recursionQueryChildren(organization, children);
+ recursionQueryChildren(organization.getId(), children);
}
} else {
return;
--
Gitblit v1.8.0