menu_tree_build() gets the "current path" by checking $_GET['q'] directly. That is wrong for multiple reasons, not the least of which is that it means other modules, such as menu_position, that mess with the menu trail without altering $_GET['q'] then are unsupported.

Patch coming in a moment, but I need an issue ID for it. :-)

CommentFileSizeAuthor
#1 1524674-menu_position.patch1.01 KBCrell

Comments

Crell’s picture

Crell commented
StatusFileSize
new 1524674-menu_position.patch1.01 KB

And patch.

This may not be the correct solution, since the menu system has a dozen different definitions of "current path", none of which match, but it "works for me"(tm).

Crell’s picture

Crell commented
Status: Active » Needs review

Duh.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented

I wonder if this is related to #853462: Use menu_build_tree()

johnalbin’s picture

johnalbin
they/他
Primary language English
Location Taipei, Taiwan
commented
Status: Needs review » Fixed

Fixed! Thanks, Larry.

johnalbin’s picture

johnalbin
they/他
Primary language English
Location Taipei, Taiwan
commented

Here's the commit diff. http://drupalcode.org/project/menu_block.git/commitdiff/fa4afaa

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.