Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.I think the title says it all. I suspect that using parse_url or drupal_parse_url on the URL components, and passing the resulting $options to l() would make this possible
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | 1353812.1-rules-breadcrumbs-querystring.patch | 1022 bytes | mrfelton |
| |||
Comments
Comment #1
mrfelton CreditAttribution: mrfelton commentedSeems to do the job.
Comment #2
fagoMakes sense to me, though the help text still says just paths are supported. I guess we should improve it then to say query strings and fragments are supported.
Comment #3
mwallenberg CreditAttribution: mwallenberg commentedIs there any progress on this issue? I'm currently building a site which could really use this functionality, and I would really like to stick to Rules for the breadcrumbs (and not add Custom breadcrumbs just for a single content type).
Comment #4
mitchell CreditAttribution: mitchell commentedIf there's a known error in phpdocs with an open issue tracking the necessary update, there's no harm in making the commit.. I think it makes more sense to err toward executable code completeness over correctness, especially when correctness is easily replaced with partial/pseudo-correctness when not erring toward incompleteness, as shown by #1 + #3.
...We could even change the status options to include 'needs change notification' or 'needs docs updates' too!
Comment #5
fagoNop, updating the UI texts is the very least that we need to do. I won't commit anything that works contrary to what the UI says, thus let's just fix the patch to update the help text accordingly.
Comment #6
mitchell CreditAttribution: mitchell commentedUpdated component.
Comment #7
TR CreditAttribution: TR commentedPatch still applies, but #5 has not been addressed.
Also need to add a test case - we already have a test case for setting breadcrumbs, so it should be trivial to copy that and append a query string to the test action.
Comment #8
TR CreditAttribution: TR commentedComment #9
TR CreditAttribution: TR commentedThis action isn't going to be ported to D8 (See #2351979: Port "Set breadcrumb" action to D8)
If it's going to be put into D7, then someone needs to work on it - no one has for the past 8 years, and it's not on my to-do list so I'm marking this as postponed. Feel free to re-open it if you're willing to write the tests and update the above patch as requested in #5.
Comment #10
TR CreditAttribution: TR commentedNo response, I guess no one wants this anymore.