- Issue created by @nicrodgers
- π¬π§United Kingdom nicrodgers Monmouthshire, UK
I've pushed a fix to the MR.
- Merge request !13Updated addMissingCurrentPage to work with changes from... β (Open) created by nicrodgers
- Status changed to Closed: duplicate
9 months ago 1:15am 28 March 2024 - π³πΏNew Zealand xurizaemon Εtepoti, Aotearoa π
This looks good. Are you sure if the sorting is required? I've incorporated that part into the MR at https://git.drupalcode.org/project/menu_breadcrumb/-/merge_requests/15 over on π Wrong breadcrumb due to route alterations RTBC . Let's continue there?
- π¬π§United Kingdom nicrodgers Monmouthshire, UK
It's been a while since I worked on it, but from memory I think the sort is required because there's no guarantee that the order of params matches, so unless you sort them you could have a different order and thus it wouldn't match (even though the order is irrelevant).
- π³πΏNew Zealand xurizaemon Εtepoti, Aotearoa π
Thanks! Some further discussion & notes on π Wrong breadcrumb due to route alterations RTBC already (and a test now).