menu_tree_page_data should accept a path or menu_item as argument

Created on 21 August 2012, almost 13 years ago

Updated 17 July 2023, almost 2 years ago

We can now change the #520106: Allow setting the active menu trail for dynamically-generated menu paths. → . For me this look like a hack but I have to say the menu system is complicated and I perhaps don't get all the incidence.

Now the signature of the function is

menu_tree_page_data($menu_name, $max_depth = NULL, $only_active_trail = FALSE);

But you have now way to build a breadcrumb based of an other node than the active one. Anyways it look cleaner to me to always give the active_path instead of finding it inside the function.

So we can add active_path as an argument and use this signature

menu_tree_page_data($menu_name, $max_depth = NULL, $only_active_trail = FALSE, $active_path = NULL);

✨ Feature request

Status

Postponed: needs info

Version

9.5

Component

Menu system →

Last updated 3 days ago

Maintained by
🇩🇪Germany @dawehner
🇺🇸United States @pwolanin

Created by

🇨🇭Switzerland gagarine

Live updates comments and jobs are added and updated live.

Incomplete comments

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Comment almost 2 years ago →
🇺🇸United States smustgrave
Since this was turned to a service in https://www.drupal.org/node/2226481 → wonder if this is still a needed issue?
Status changed to Closed: works as designed 2 months ago2:28am 2 May 2025
Comment 2 months ago →
🇳🇿New Zealand quietone
There has been no discussion here to support this feature request.

The proposal doesn't met the Criteria for evaluating proposed changes → . In this case, there is not demonstrated demand and support for the change.

If there is interest in this re-open the issue and add a comment. Or open a new issue and reference this one.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024