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/MenuServiceImpl.java | 287 +++++++++++++++++++++++++++++++++++++++++++++-----------
1 files changed, 228 insertions(+), 59 deletions(-)
diff --git a/screen-manage/src/main/java/com/moral/api/service/impl/MenuServiceImpl.java b/screen-manage/src/main/java/com/moral/api/service/impl/MenuServiceImpl.java
index efcaec9..8155f11 100644
--- a/screen-manage/src/main/java/com/moral/api/service/impl/MenuServiceImpl.java
+++ b/screen-manage/src/main/java/com/moral/api/service/impl/MenuServiceImpl.java
@@ -2,13 +2,19 @@
import com.baomidou.mybatisplus.core.conditions.query.QueryWrapper;
import com.baomidou.mybatisplus.core.conditions.update.UpdateWrapper;
-import com.moral.api.entity.ManageMenu;
+import com.moral.api.config.mybatis.wrapper.NullFilterWrapper;
+import com.moral.api.entity.Group;
import com.moral.api.entity.Menu;
+import com.moral.api.mapper.GroupMenuMapper;
import com.moral.api.mapper.MenuMapper;
import com.moral.api.pojo.dto.menu.MenuDTO;
import com.moral.api.pojo.dto.menu.MenuQueryDTO;
+import com.moral.api.pojo.dto.menu.MenuQueryNamesDTO;
import com.moral.api.pojo.form.menu.MenuDeleteForm;
import com.moral.api.pojo.form.menu.MenuInsertForm;
+import com.moral.api.pojo.form.menu.MenuQueryNamesForm;
+import com.moral.api.pojo.form.menu.MenuUpdateForm;
+import com.moral.api.service.GroupService;
import com.moral.api.service.MenuService;
import com.baomidou.mybatisplus.extension.service.impl.ServiceImpl;
import com.moral.constant.Constants;
@@ -37,32 +43,55 @@
@Autowired
MenuMapper menuMapper;
+ @Autowired
+ GroupService groupService;
+ @Autowired
+ GroupMenuMapper groupMenuMapper;
@Override
- public MenuQueryDTO queryAllMenu() {
+ public MenuQueryDTO queryAllMenus() {
//������������������
MenuQueryDTO dto = new MenuQueryDTO();
//������������������
- List<Menu> menus = menuMapper.selectList(null);
- //������menu������������
- Map<Integer, Menu> menuMap = new HashMap<>();
- for (Menu menu : menus) {
- menuMap.put(menu.getId(), menu);
- menu.setChildren(new ArrayList<>());//���������������
- }
- for (Menu menu : menus) {
- combinationParentChildrenMenus(menuMap, menu);
- }
- //������������������
- menus.removeIf(new Predicate<Menu>() {
- @Override
- public boolean test(Menu menu) {
- if (menu.getParentId().equals(0))
- return false;
- return true;
- }
- });
+ QueryWrapper wrapper = new QueryWrapper();
+ wrapper.eq("is_delete", Constants.NOT_DELETE);
+ List<Menu> menus = menuMapper.selectList(wrapper);
+ //���������������������
+ combinationParentChildrenMenus(menus);
//���������������DTO
+ List<MenuDTO> dtos = new ArrayList<>();
+ for (Menu menu : menus) {
+ MenuDTO menuDTO = new MenuDTO();
+ menuDTO.setMenu(menu);
+ dtos.add(menuDTO);
+ }
+ dto.setDtos(dtos);
+ dto.setCode(ResponseCodeEnum.SUCCESS.getCode());
+ dto.setMsg(ResponseCodeEnum.SUCCESS.getMsg());
+ return dto;
+ }
+
+ @Override
+ public MenuQueryDTO queryAdminGroupMenusByOrganizationId(Integer organizationId) {
+ //������������������
+ MenuQueryDTO dto = new MenuQueryDTO();
+ //������������admin������
+ Group group = groupService.queryAdminGroupByOrganizationId(organizationId);
+ if (ObjectUtils.isEmpty(group)) {//������������������������������������������������
+ dto.setCode(ResponseCodeEnum.ORGANIZATION_USER_NOT_EXIST.getCode());
+ dto.setMsg(ResponseCodeEnum.ORGANIZATION_USER_NOT_EXIST.getMsg());
+ return dto;
+ }
+ //���������������������������������������
+ List<Menu> allMenus = menuMapper.getMenusByGroupId(group.getId());
+ //������������������������������
+ combinationParentChildrenMenus(allMenus);
+ //������������������������������������������
+ List<Menu> menus = new ArrayList<>();
+ for (Menu menu : allMenus) {
+ getNotParentMenu(menu, menus);
+ }
+ //������������������
List<MenuDTO> dtos = new ArrayList<>();
for (Menu menu : menus) {
MenuDTO menuDTO = new MenuDTO();
@@ -94,6 +123,7 @@
dto.setMsg(ResponseCodeEnum.MENU_IS_EXIST.getMsg());
return dto;
}
+
//������������������
menuMapper.insert(menu);
//������������������
@@ -104,6 +134,7 @@
}
@Override
+ @Transactional
public MenuDTO deleteMenu(MenuDeleteForm form) {
//������������������
MenuDTO dto = new MenuDTO();
@@ -116,15 +147,25 @@
dto.setMsg(ResponseCodeEnum.MENU_IS_NULL.getMsg());
return dto;
}
- //������������������
- existMenu.setIsDelete(Constants.DELETE);
- menuMapper.updateById(existMenu);
//���������������������
List<Integer> childrenIds = getChildrenIdsByParentId(id);
- UpdateWrapper wrapper = new UpdateWrapper();
- wrapper.in("id",childrenIds);
- wrapper.set("is_delete",Constants.DELETE);
- menuMapper.update(null,wrapper);
+ if (!ObjectUtils.isEmpty(childrenIds)) {
+ UpdateWrapper wrapper = new UpdateWrapper();
+ wrapper.in("id", childrenIds);
+ wrapper.set("is_delete", Constants.DELETE);
+ menuMapper.update(null, wrapper);
+ }
+ //������������
+ existMenu.setIsDelete(Constants.DELETE);
+ menuMapper.updateById(existMenu);
+ //���������������������������������������������
+ UpdateWrapper deleteGroupMenuWrapper = new UpdateWrapper();
+ List<Integer> allMenuIds = new ArrayList<>();
+ allMenuIds.add(id);
+ allMenuIds.addAll(childrenIds);
+ deleteGroupMenuWrapper.in("menu_id",allMenuIds);
+ deleteGroupMenuWrapper.set("is_delete",Constants.DELETE);
+ groupMenuMapper.update(null,deleteGroupMenuWrapper);
//������������������
dto.setCode(ResponseCodeEnum.SUCCESS.getCode());
dto.setMsg(ResponseCodeEnum.SUCCESS.getMsg());
@@ -132,7 +173,113 @@
return dto;
}
- private void combinationParentChildrenMenus(Map<Integer, Menu> menuMap, Menu menu) {
+ @Override
+ public MenuDTO updateMenu(MenuUpdateForm form) {
+ //������������������
+ MenuDTO dto = new MenuDTO();
+ //������
+ Menu menu = form.formConvertEntity();
+ //������������������������������������������
+ QueryWrapper<Menu> oldMenuWrapper = new QueryWrapper<>();
+ Menu oldMenu = new Menu();
+ oldMenu.setId(menu.getId());
+ oldMenu.setIsDelete(Constants.NOT_DELETE);
+ oldMenuWrapper.setEntity(oldMenu);
+ oldMenu = menuMapper.selectOne(oldMenuWrapper);
+ if (ObjectUtils.isEmpty(oldMenu)) {
+ dto.setCode(ResponseCodeEnum.MENU_IS_NULL.getCode());
+ dto.setMsg(ResponseCodeEnum.MENU_IS_NULL.getMsg());
+ return dto;
+ }
+ //������
+ menuMapper.updateById(menu);
+ //������������������������
+ menu = menuMapper.selectById(menu.getId());
+ //������������������
+ dto.setMenu(menu);
+ dto.setCode(ResponseCodeEnum.SUCCESS.getCode());
+ dto.setMsg(ResponseCodeEnum.SUCCESS.getMsg());
+ return dto;
+ }
+
+ @Override
+ public MenuQueryNamesDTO queryNames(MenuQueryNamesForm form) {
+ //������������������
+ MenuQueryNamesDTO dto = new MenuQueryNamesDTO();
+ //������
+ Integer id = form.getId();
+ String name = form.getName();
+ //������������
+ NullFilterWrapper<Menu> wrapper = new NullFilterWrapper<>();
+ wrapper.eq("is_delete", Constants.NOT_DELETE);
+ wrapper.like("name", name);
+ List<Menu> menus = menuMapper.selectList(wrapper);
+ //���������id������������������������
+ List<Integer> childrenIds = getChildrenIdsByParentId(id);//���������������������id������
+ if (ObjectUtils.isEmpty(childrenIds)) {
+ menus.removeIf(new Predicate<Menu>() {
+ @Override
+ public boolean test(Menu m) {
+ if (m.getId().equals(id))
+ return true;
+ return false;
+ }
+ });
+ } else {
+ menus.removeIf(new Predicate<Menu>() {
+ @Override
+ public boolean test(Menu m) {
+ if (childrenIds.contains(m.getId()) || m.getId().equals(id))
+ return true;
+ return false;
+ }
+ });
+ }
+
+ //������������������
+ dto.setMenus(menus);
+ dto.setCode(ResponseCodeEnum.SUCCESS.getCode());
+ dto.setMsg(ResponseCodeEnum.SUCCESS.getMsg());
+ return dto;
+ }
+
+
+ /**
+ * @Description: ������������������������������������������������������
+ * @Param: [menus]
+ * @return: java.util.List<com.moral.api.entity.Menu>
+ * @Author: ���������
+ * @Date: 2021/5/13
+ */
+ public void combinationParentChildrenMenus(List<Menu> menus){
+ //������menu������������
+ Map<Integer, Menu> menuMap = new HashMap<>();
+ for (Menu menu : menus) {
+ menuMap.put(menu.getId(), menu);
+ menu.setChildren(new ArrayList<>());//���������������
+ }
+ for (Menu menu : menus) {
+ putMenuToParentMenu(menuMap, menu);
+ }
+ //������������������
+ menus.removeIf(new Predicate<Menu>() {
+ @Override
+ public boolean test(Menu menu) {
+ if (menu.getParentId().equals(0))
+ return false;
+ return true;
+ }
+ });
+ }
+
+ /**
+ * @Description: ������������menu������menuMap���������������������
+ * @Param: [menuMap, menu]
+ * @return: void
+ * @Author: ���������
+ * @Date: 2021/5/6
+ */
+ private void putMenuToParentMenu(Map<Integer, Menu> menuMap, Menu menu) {
Integer parentId = menu.getParentId();
Menu parentMenu = menuMap.get(parentId);
if (!ObjectUtils.isEmpty(parentMenu)) {
@@ -141,15 +288,17 @@
}
/**
- * @Description: ���������������id���������������children���menu������
- * @Param: [id]
- * @return: com.moral.api.entity.Menu
- * @Author: ���������
- * @Date: 2021/4/26
- */
- private Menu getMenuAndChildrenById(Integer id){
+ * @Description: ���������������id���������������children���menu������
+ * @Param: [id]
+ * @return: com.moral.api.entity.Menu
+ * @Author: ���������
+ * @Date: 2021/4/26
+ */
+ private Menu getMenuAndChildrenById(Integer id) {
//������������������
- List<Menu> menus = menuMapper.selectList(null);
+ QueryWrapper wrapper = new QueryWrapper();
+ wrapper.eq("is_delete", Constants.NOT_DELETE);
+ List<Menu> menus = menuMapper.selectList(wrapper);
//������menu������������
Map<Integer, Menu> menuMap = new HashMap<>();
for (Menu menu : menus) {
@@ -157,46 +306,66 @@
menu.setChildren(new ArrayList<>());//���������������
}
for (Menu menu : menus) {
- combinationParentChildrenMenus(menuMap, menu);
+ putMenuToParentMenu(menuMap, menu);
}
Menu menu = menuMap.get(id);
return menu;
}
/**
- * @Description: ������������������������������������id������
- * @Param: [id]
- * @return: java.util.List<java.lang.Integer>
- * @Author: ���������
- * @Date: 2021/4/26
- */
- private List<Integer> getChildrenIdsByParentId(Integer id){
+ * @Description: ������������������������������������id������
+ * @Param: [id]
+ * @return: java.util.List<java.lang.Integer>
+ * @Author: ���������
+ * @Date: 2021/4/26
+ */
+ private List<Integer> getChildrenIdsByParentId(Integer id) {
Menu parentMenu = getMenuAndChildrenById(id);
- List<Menu> children = parentMenu.getChildren();
- List<Integer> childrenIds = new ArrayList<>();
- for (Menu child : children) {
- recursiveAccess(child,childrenIds);
+ List<Integer> childrenIds = null;
+ if (!ObjectUtils.isEmpty(parentMenu.getChildren())) {
+ List<Menu> children = parentMenu.getChildren();
+ childrenIds = new ArrayList<>();
+ for (Menu child : children) {
+ recursiveAccess(child, childrenIds);
+ }
}
return childrenIds;
}
/**
- * @Description: ���������������������id���������������
- * @Param: [menu, ids]
- * @return: void
- * @Author: ���������
- * @Date: 2021/4/26
- */
- private void recursiveAccess(Menu menu,List<Integer> ids){
+ * @Description: ������������������������children���id���������������
+ * @Param: [menu, ids]
+ * @return: void
+ * @Author: ���������
+ * @Date: 2021/4/26
+ */
+ private void recursiveAccess(Menu menu, List<Integer> ids) {
ids.add(menu.getId());
List<Menu> children = menu.getChildren();
- if(!ObjectUtils.isEmpty(children)){
+ if (!ObjectUtils.isEmpty(children)) {
for (Menu child : children) {
- recursiveAccess(child,ids);
+ recursiveAccess(child, ids);
+ }
+ }
+ }
+
+ /**
+ * @Description: ������menu������������������������������������menus���
+ * @Param: [menu, menus]
+ * @return: void
+ * @Author: ���������
+ * @Date: 2021/5/13
+ */
+ private void getNotParentMenu(Menu menu, List<Menu> menus) {
+ if (ObjectUtils.isEmpty(menu.getChildren())) {
+ menus.add(menu);
+ } else {
+ List<Menu> children = menu.getChildren();
+ for (Menu child : children) {
+ getNotParentMenu(child, menus);
}
}
}
}
-
--
Gitblit v1.8.0