I utilize the hook_breadcrumb_alter to override and alter part of a breadcrumb on a custom page.
In one of the overrides I insert a custom url, with url query parameters. When the url is printed by the easy_breadcrumbs module the url is encoded and my query parameters are not printed correctly; ?id=3&reset=1 is turned into %3Freset%3D1%26id%3D3 for example.
So I made a patch that adds a checks if any query parameters are set and passes them in de l() function through the $options var. which than builds a correct link. This way the query parameters are always rendered correctly and not double encoded.
If I'm missing something, or did not utilize the easy_breadcrumbs module correctly, please do tell me!
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | easy_breadcrumb-url_encoding_fix-2923421-6.patch | 959 bytes | tatarbj |
Comments
Comment #2
tatarbjHey wannesderoy,
thanks for the issue and the patch!
For a quick reply i'd like to let you know i'm planning in the weekend to check it deeper but for the first look it seems a valid issue that is solved more-less by the patch :)
Let me get back to you in a few days!
@greg - Do we have something similar in 8.x? Maybe the issue will be there too, could you check it?
Bests,
Balazs.
Comment #3
jenlamptonThis fix is working for me, thanks @wannesderoy!
Comment #5
tatarbjComment #6
tatarbjreroll and tests are done, commit's arriving now, till that time here is the patch :)
Comment #8
tatarbj