- ๐ณ๐ฟNew Zealand xurizaemon ลtepoti, Aotearoa ๐
Needs test coverage and/or steps to reproduce.
- Status changed to Active
8 months ago 4:30pm 8 August 2024 - ๐บ๐ธUnited States jlstrecker Athens, Ohio, USA
Steps to reproduce / testing the patch:
- Install
menu_breadcrumb 2.0.0-alpha0
. - Enable "Show current page as link". Leave rest of settings at defaults.
- Add breadcrumb block to all pages.
- Create pages:
- Second โ
/second
- Third โ
/third
- Params maybe โ
/second/params
- Params again โ
/third/params
- Second โ
- Set up main menu:
- Second โ
/second
- No params โ
/second/params
- Yes params โ
/second/params?has=1
- No params โ
- Third โ
/third
- Yes params 2 โ
/third/params?hello=1
- No params 2 โ
/third/params
- Yes params 2 โ
- Second โ
Breadcrumbs without the patch:
- No params โ Second > No params
- Yes params โ Second > No params
- Yes params 2 โ Third > Yes params 2
- No params 2 โ Third > Yes params 2
Breadcrumbs with the patch and "Exclude menu items with query in URLs" enabled:
- No params โ Second > No params
- Yes params โ Second > No params
- Yes params 2 โ Third > Params again
- No params 2 โ Third > Params again
This patch solve a problem for me where I had two menu links to the same search page โ one with no query parameters ("Search"), the other with query parameters that would narrow the search ("Specific Category") โ and I wanted the breadcrumb for both to be the first menu link / page title ("Search").
- Install
- First commit to issue fork.
- ๐ฎ๐ณIndia anwar_max
anwar_max โ changed the visibility of the branch 3274684-allow-urls-that to hidden.