Hello,

I was having a very minor navigation issue with this module. Attached to this issue is a proposed patch that will slightly modify the functionality of building the active trail. I appreciate your consideration in adding this to your module. Here's the issue:

When a node has multiple taxonomy terms in the same field, the menu trail was always defaulting to the first taxonomy term ID in the field.

This was a little confusing, especially when the user was coming to the node from a different taxonomy term (by clicking a link in the taxonomy term's page node list).

The breadcrumb would be built for a different taxonomy term than they clicked off of, so navigating back to the original taxonomy term isn't possible via the breadcrumb.

The attached patch just sets a $_SESSION variable when the user visits a taxonomy term page. Then when a user visits a node with a taxonomy term field set to have taxonomy menu trails, the code checks if the field has the taxonomy term ID in $_SESSION, and if so sets that term as the menu trail.

Thanks!
-Joe

CommentFileSizeAuthor
taxonomy-menu-trail-context.patch1.45 KBjoecorall
Support from Acquia helps fund testing for Drupal Acquia logo