Closed (outdated)
Project:
Drupal core
Version:
11.x-dev
Component:
menu system
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
25 Jul 2014 at 07:41 UTC
Updated:
30 Jun 2025 at 16:10 UTC
Jump to comment: Most recent
Comments
Comment #1
wim leersThis probably should be blocked on #1805054: Cache localized, access filtered, URL resolved, and rendered menu trees.
Once #1805054: Cache localized, access filtered, URL resolved, and rendered menu trees lands, we'll start seeing more different tree manipulator lists. Roughly: all *rendered* menus will start using the optimized access checking manipulator that that patch will introduce (menu block, toolbar). And menu trees built for creating the options for a
<select>would continue to use:checkAccess(). The primary/secondary menus for example already use a different tree manipulator list.That being said, I agree that this is useful for the typical use cases. However, let's not further contaminate
MenuLinkTreewith "typical/default stuff". It's bad enough that we havegetCurrentRouteMenuTreeParameters()in there (which also got renamed while I was away and is now actually incorrect — it reacts to the current active trail, not the current route) inMenuLinkTree(Interface).IMHO we should create a
DefaultMenuRendererclass/service (or something like that) that contains methods that return the default/typical parameters for bothMenuLinkTree::load()andMenuLinkTree::transform(). ThenMenuLinkTreebecomes a proper, single-purpose service, unburdened with dealing with "defaultness", which lies outside of its logical domain.Comment #15
smustgrave commentedThank you for creating this issue to improve Drupal.
We are working to decide if this task is still relevant to a currently supported version of Drupal. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or is no longer relevant. Your thoughts on this will allow a decision to be made.
Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.
Thanks!
Comment #16
smustgrave commentedSince it's been 3+ months without a follow up I'm going to close out. If still valid please re-open
Thanks all!