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.
Applying the same url query param reserve field to BEF. All other logic is done in the parent issue.
Comment | File | Size | Author |
---|---|---|---|
#10 | preserve-url-query-params-3119947-10.patch | 806 bytes | mrinalini9 |
#2 | preserve-url-query-params-3119947-1.patch | 958 bytes | GeorgeBC |
Comments
Comment #2
GeorgeBC CreditAttribution: GeorgeBC commentedComment #3
GeorgeBC CreditAttribution: GeorgeBC commentedComment #6
apadernoComment #7
bgustafson CreditAttribution: bgustafson commentedworks for me, thanks
Comment #8
LendudeSince this is changing BetterExposedFilters.php, I'm pretty sure this should be in de BEF queue. Or am I missing something?
Comment #9
apaderno@Lendude Yes, you are right: The patched file is better_exposed_filters/src/Plugin/views/exposed_form/BetterExposedFilters.php, which clearly is a file from this project, not from Drupal core.
Also, the patch needs to be created from the directory containing the module.
Comment #10
mrinalini9 CreditAttribution: mrinalini9 at Srijan | A Material+ Company for Drupal India Association commentedRerolled patch #2 for 8.x-5.x, please review it.
Thanks!
Comment #11
mrinalini9 CreditAttribution: mrinalini9 at Srijan | A Material+ Company for Drupal India Association commentedComment #12
mglamanIs
preserve_url_query_parameters
even valid? I cannot find it in any code base.EDIT: is it in conjunction to this patch? #2681953: Allow exposed form to preserve URL query parameters
Comment #13
chucksimply CreditAttribution: chucksimply commentedI'm on Drupal 9.2.8 with BEF 5.0
First applied patch from parent issue (#25) then applied #10 patch from this issue.
All working great!
Comment #14
BramDriesenAdded related issue.
Comment #15
apadernoComment #16
NicklasMF CreditAttribution: NicklasMF as a volunteer commented#10 works for me alongside the parent issue #2681953#36.
Comment #17
aaron.ferris CreditAttribution: aaron.ferris at Zoocha commentedPatch from #10 is working well for me (along with the parent patch from #2681953), we have a search implementation that has custom query parameters that were being lost on exposed form search.
Comment #18
maxilein CreditAttribution: maxilein commentedApplies to D10.2.5 and BEF 6.0.3.
And works fine with Better Exposed Filters ( Exposed form in block:No)
It does not work with: Better Exposed Filters (with layout) from the https://www.drupal.org/project/vefl module. Issues created for this https://www.drupal.org/project/vefl/issues/3438896
Comment #19
smustgrave CreditAttribution: smustgrave at Mobomo commentedSeems this is dependent on the parent issue landing