From 9037cef5e88d18c2407f04a70a8341541b01181c Mon Sep 17 00:00:00 2001
From: kaiyu <404897439@qq.com>
Date: Mon, 07 Jun 2021 11:27:36 +0800
Subject: [PATCH] screen-manage 修复前台菜单管理删除BUG
---
screen-manage/src/main/java/com/moral/api/service/impl/MenuServiceImpl.java | 42 ++++++++++++++++++++++--------------------
1 files changed, 22 insertions(+), 20 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 b220c29..7cb06d8 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
@@ -88,7 +88,7 @@
List<Menu> ownMenus = menuMapper.getMenusByGroupId(group.getId());
/*������������������������������������������������������������
- * ���������������������������������������������������������������������������������������������������������*/
+ * ���������������������������������������������������������������������������������������������������������*/
removeMenuWithChildren(ownMenus);
//������������������
@@ -162,10 +162,12 @@
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);
+ if (!ObjectUtils.isEmpty(childrenIds)) {
+ 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());
@@ -245,13 +247,13 @@
/**
- * @Description: ������������������������������������������������������
- * @Param: [menus]
- * @return: java.util.List<com.moral.api.entity.Menu>
- * @Author: ���������
- * @Date: 2021/5/13
- */
- public void combinationParentChildrenMenus(List<Menu> menus){
+ * @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) {
@@ -351,13 +353,13 @@
}
/**
- * @Description: ������������������������������
- * @Param: [menus]
- * @return: java.util.List<com.moral.api.entity.Menu>
- * @Author: ���������
- * @Date: 2021/5/28
- */
- private void removeMenuWithChildren(List<Menu> menus){
+ * @Description: ������������������������������
+ * @Param: [menus]
+ * @return: java.util.List<com.moral.api.entity.Menu>
+ * @Author: ���������
+ * @Date: 2021/5/28
+ */
+ private void removeMenuWithChildren(List<Menu> menus) {
//������������������
QueryWrapper wrapper = new QueryWrapper();
wrapper.eq("is_delete", Constants.NOT_DELETE);
@@ -375,7 +377,7 @@
menus.removeIf(new Predicate<Menu>() {
@Override
public boolean test(Menu menu) {
- if(ObjectUtils.isEmpty(menuMap.get(menu.getId()).getChildren()))
+ if (ObjectUtils.isEmpty(menuMap.get(menu.getId()).getChildren()))
return false;
return true;
}
--
Gitblit v1.8.0