Expanded menu's appear in breadcrumbs

Starnox - November 8, 2008 - 18:52
Project:Menu Trails
Version:6.x-1.0
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

Shouldn't the breadcrumbs be generated by the active trail only? If you have an expended menu then they all seem to appear in the breadcrumb trail.

#1

Starnox - November 8, 2008 - 19:14
Status:active» needs review

I had a dig around and came up with this.

AttachmentSize
patch.patch 708 bytes

#2

apostate - January 9, 2009 - 19:07

Ran into the same issue using a set of all-expanded primary links. This patch works great.

#3

noahb - January 20, 2009 - 00:31
Status:needs review» reviewed & tested by the community

I had the same problem, and the patch fixes this...

#4

acstewart - January 25, 2009 - 20:16

LOL.. had the same issue. basically not honoring the current active trail.

An alternative patch (bit more readable).

AttachmentSize
menutrails-breadcrumbs.patch 550 bytes

#5

noahb - February 5, 2009 - 08:26
Status:reviewed & tested by the community» needs review

A reroll of the patch from #4, which didn't apply after the lastest dev release.

EDIT TO ADD:
Patch is against the 6--1 branch.

AttachmentSize
mt.patch 750 bytes

#6

sun - February 5, 2009 - 11:20
Status:needs review» reviewed & tested by the community

Last patch looks good and reasonable. Not only more readable, but also better in terms of performance.

Leaving final decision to Josh. Note, however, I did not test this patch.

#7

Jeff Burnz - February 12, 2009 - 02:47

Working well for me, nothing ill toward to report.

#8

eojthebrave - April 9, 2009 - 16:21

Patch works as promised for me as well. Thanks!

joshk: do you need anything else in order to get this patch applied? It's a real simple fix, and solves a pretty nasty bug.

#9

kmonty - April 23, 2009 - 23:24

Patch #5 works for me with no errors.

#10

sun - July 24, 2009 - 14:52
Status:reviewed & tested by the community» fixed

Thanks for reporting, reviewing, and testing! Committed to 6.x.

A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.

#11

System Message - August 7, 2009 - 15:00
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.