【修复】获取菜单精简信息列表接口没有排除父 ID 非 0 的节点

yudao-module-system/yudao-module-system-biz/src/main/java/cn/iocoder/yudao/module/system/controller/admin/auth/AuthController.java

@@ -109,7 +109,8 @@ public CommonResult<AuthPermissionInfoRespVO> getPermissionInfo() {
         // 1.3 获得菜单列表
         Set<Long> menuIds = permissionService.getRoleMenuListByRoleId(convertSet(roles, RoleDO::getId));
         List<MenuDO> menuList = menuService.getMenuList(menuIds);
-        menuList.removeIf(menu -> !CommonStatusEnum.ENABLE.getStatus().equals(menu.getStatus())); // 移除禁用的菜单
+        // 过滤掉关闭的菜单
+        menuList = menuService.filterClosedMenus(menuList);
 
         // 2. 拼接结果返回
         return success(AuthConvert.INSTANCE.convert(user, roles, menuList));

yudao-module-system/yudao-module-system-biz/src/main/java/cn/iocoder/yudao/module/system/controller/admin/permission/MenuController.java

@@ -72,6 +72,8 @@ public CommonResult<List<MenuRespVO>> getMenuList(MenuListReqVO reqVO) {
     public CommonResult<List<MenuSimpleRespVO>> getSimpleMenuList() {
         List<MenuDO> list = menuService.getMenuListByTenant(
                 new MenuListReqVO().setStatus(CommonStatusEnum.ENABLE.getStatus()));
+        // 过滤掉关闭的菜单及其子菜单
+        list = menuService.filterClosedMenus(list);
         list.sort(Comparator.comparing(MenuDO::getSort));
         return success(BeanUtils.toBean(list, MenuSimpleRespVO.class));
     }

yudao-module-system/yudao-module-system-biz/src/main/java/cn/iocoder/yudao/module/system/service/permission/MenuService.java

@@ -52,6 +52,14 @@ public interface MenuService {
      */
     List<MenuDO> getMenuListByTenant(MenuListReqVO reqVO);
 
+    /**
+     * 过滤掉关闭的菜单及其子菜单
+     *
+     * @param menuList
+     * @return
+     */
+    List<MenuDO> filterClosedMenus(List<MenuDO> menuList);
+
     /**
      * 筛选菜单列表
      *

yudao-module-system/yudao-module-system-biz/src/main/java/cn/iocoder/yudao/module/system/service/permission/MenuServiceImpl.java

@@ -1,6 +1,7 @@
 package cn.iocoder.yudao.module.system.service.permission;
 
 import cn.hutool.core.collection.CollUtil;
+import cn.iocoder.yudao.framework.common.enums.CommonStatusEnum;
 import cn.iocoder.yudao.framework.common.util.object.BeanUtils;
 import cn.iocoder.yudao.module.system.controller.admin.permission.vo.menu.MenuListReqVO;
 import cn.iocoder.yudao.module.system.controller.admin.permission.vo.menu.MenuSaveVO;
@@ -13,14 +14,15 @@
 import com.google.common.collect.Lists;
 import jakarta.annotation.Resource;
 import lombok.extern.slf4j.Slf4j;
+import org.apache.commons.collections4.CollectionUtils;
 import org.springframework.cache.annotation.CacheEvict;
 import org.springframework.cache.annotation.Cacheable;
 import org.springframework.context.annotation.Lazy;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
 
-import java.util.Collection;
-import java.util.List;
+import java.util.*;
+import java.util.stream.Collectors;
 
 import static cn.iocoder.yudao.framework.common.exception.util.ServiceExceptionUtil.exception;
 import static cn.iocoder.yudao.framework.common.util.collection.CollectionUtils.convertList;
@@ -106,12 +108,66 @@ public List<MenuDO> getMenuList() {
 
     @Override
     public List<MenuDO> getMenuListByTenant(MenuListReqVO reqVO) {
+        // 查询所有菜单，并过滤掉关闭的节点
         List<MenuDO> menus = getMenuList(reqVO);
         // 开启多租户的情况下，需要过滤掉未开通的菜单
         tenantService.handleTenantMenu(menuIds -> menus.removeIf(menu -> !CollUtil.contains(menuIds, menu.getId())));
         return menus;
     }
 
+    /**
+     * 过滤关闭的菜单节点及其子节点
+     *
+     * @param menuList 所有菜单列表
+     * @return
+     */
+    @Override
+    public List<MenuDO> filterClosedMenus(List<MenuDO> menuList) {
+        if(CollectionUtils.isEmpty(menuList)){
+            return Collections.emptyList();
+        }
+        List<MenuDO> allMenuList = getMenuList();
+
+        // 根据parentId快速查找子节点
+        Map<Long, List<MenuDO>> childrenMap = allMenuList.stream()
+                .collect(Collectors.groupingBy(MenuDO::getParentId));
+
+        // 所有关闭的节点ID
+        Set<Long> closedNodeIds = new HashSet<>();
+
+        // 标记所有关闭的节点
+        for (MenuDO menu : allMenuList) {
+            if (!Objects.equals(menu.getStatus(), CommonStatusEnum.ENABLE.getStatus())) {
+                markClosedNodes(menu.getId(), childrenMap, closedNodeIds);
+            }
+        }
+        // 移除掉关闭的节点及其子节点
+        return menuList.stream()
+                .filter(menu -> !closedNodeIds.contains(menu.getId()))
+                .collect(Collectors.toList());
+    }
+
+    /**
+     * 递归标记关闭的节点及其子节点
+     *
+     * @param nodeId 节点ID
+     * @param childrenMap 子节点Map
+     * @param closedNodeIds 关闭节点ID集合
+     */
+    private void markClosedNodes(Long nodeId, Map<Long,
+                                List<MenuDO>> childrenMap,
+                                 Set<Long> closedNodeIds) {
+        // 如果已经标记过，则直接返回
+        if (!closedNodeIds.add(nodeId)) {
+            return;
+        }
+        List<MenuDO> children = childrenMap.getOrDefault(nodeId,Collections.emptyList());
+        for (MenuDO child : children) {
+            markClosedNodes(child.getId(), childrenMap, closedNodeIds);
+        }
+    }
+
+
     @Override
     public List<MenuDO> getMenuList(MenuListReqVO reqVO) {
         return menuMapper.selectList(reqVO);