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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Crell’s picture

Crell CreditAttribution: Crell commented
FileSize
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 CreditAttribution: Crell commented
Status: Active » Needs review

Duh.

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented

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

JohnAlbin’s picture

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

Fixed! Thanks, Larry.

JohnAlbin’s picture

JohnAlbin
he/they/他
Primary language English
Location Taipei, Taiwan
CreditAttribution: JohnAlbin 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.