- Issue created by @linhnm
- π³π±Netherlands mike.vindicate
Can confirm the patch from @linhnm fixes the problem with duplicate crumbs and the patch applies cleanly. Thanks @linhnm for providing it.
- Open on Drupal.org βCore: 9.5.x + Environment: PHP 8.0 & MySQL 5.7last update
about 1 year ago Waiting for branch to pass Another patch replacing the === operator with == as the latter does not care about the order of the array keys.
- Open on Drupal.org βCore: 9.5.x + Environment: PHP 8.0 & MySQL 5.7last update
about 1 year ago Waiting for branch to pass - π©πͺGermany webflo
I had the same problem and had already written a patch for it before I looked in the issue queue.
- π©πͺGermany webflo
Reviewed the patch from @linhnm
+++ b/src/MenuBasedBreadcrumbBuilder.php @@ -405,7 +405,7 @@ class MenuBasedBreadcrumbBuilder implements BreadcrumbBuilderInterface { + !array_diff($last_url->getRouteParameters(), $route_match->getRawParameters()->all())
array_diff is a good idea, but it should prporbaly use array_diff_assoc because the array index is relevant for the route parameters.
- Open on Drupal.org βCore: 9.5.x + Environment: PHP 8.0 & MySQL 5.7last update
11 months ago Waiting for branch to pass I'm having the same issue on 2.0.0-alpha0 and also agree that array_diff_assoc would be a better approach. Please review the patch.
Thanks!
- π³πΏNew Zealand xurizaemon Εtepoti, Aotearoa π
Consolidating to π Wrong breadcrumb due to route alterations RTBC as there were a few issues about this! Please let's continue there.
- Status changed to Closed: duplicate
9 months ago 2:03am 29 March 2024 - π³πΏNew Zealand xurizaemon Εtepoti, Aotearoa π
Credits transferred, closing duplicate. Head on over to π Wrong breadcrumb due to route alterations RTBC :)