Closed (fixed)
Project:
Menu Breadcrumb
Version:
2.0.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
10 Feb 2025 at 15:02 UTC
Updated:
16 Sep 2026 at 09:55 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
van.dordafog commentedComment #3
mradcliffeI am also getting this. The patch resolved the issue for taxonomy term pages via views, but there is also a duplicate crumb when visiting views pages in the menu.
Maybe this module needs something like Easy Breadcrumb's removeRepeatedSegments?
Comment #8
van.dordafog commentedComment #9
csakiistvanComment #10
csakiistvanEnvironment
Prerequisites
taxonomy_termview must be enabled (default), so that the term page route carries theview_idanddisplay_idparameters./admin/config/user-interface/menu-breadcrumband tick Append current page to breadcrumb. Without the append option the module drops the last crumb, which hides the duplicate.ddev drush uliSteps
/taxonomy/term/1and look at the breadcrumb.MenuBasedBreadcrumbBuilder::addMissingCurrentPage()the route parameter comparison changes from===to==.ddev drush cr/taxonomy/term/1again and compare the breadcrumb./taxonomy/term/2) and a node page to confirm nothing else changed.Expected results
Home › Breadcrumb Test Term— the current page appears exactly once.Actual results
Before the fix the breadcrumb on
/taxonomy/term/1readHome › Breadcrumb Test Term › Breadcrumb Test Term. The menu link stores its route parameters astaxonomy_term, display_id, view_id, while the route match returns them astaxonomy_term, view_id, display_id; both arrays hold the same key/value pairs but in a different order, so the===comparison failed andaddMissingCurrentPage()appended the current page on top of the menu link. After applying MR !36 the breadcrumb readHome › Breadcrumb Test Term. A term page without a menu link and a node page were checked afterwards and produced the same breadcrumbs as before the change, so the loosened comparison does not suppress a legitimately missing current page.Testing produced with the assistance of an LLM.
Comment #13
xurizaemonComment #16
csakiistvan