- Issue created by @mali2022
- Merge request !16Make compatible with better exposed filters v.7 → (Merged) created by darren.fisher
- 🇬🇧United Kingdom darren.fisher
I think the MR contains the change needed to restore compatibility with the latest version of better_exposed_filters.
- Status changed to Needs review
5 months ago 2:12pm 5 June 2024 - 🇬🇧United Kingdom darren.fisher
Is there anything else I can do to progress this? This issue is preventing us upgrading better exposed filters on several sites.
- 🇺🇦Ukraine stomusic Ukraine
Tnx, I've merged this commit, but it still need review because I haven't enough time to test it
- 🇬🇧United Kingdom darren.fisher
Thanks! This progressed us a bit but it turns out that the DefaultWidget.php needed a few updates as well. Merge request above and patch is here:
https://git.drupalcode.org/project/selective_better_exposed_filters/-/me... - 🇬🇧United Kingdom darren.fisher
@mali2022 are you able to test the dev branch of selective_better_exposed_filters and my patch above and see if you can install better_exposed_filters 7.0.0-beta1 in conjunction and that everything is working as expected?
i think you have missed the point to make it fully compatible you have to edit the whole 3 pages inside the src ( DefaultWidget , Links , RadioButtons ) and change their function as shown inside the patch , i have tested it on my website and it worked all fine
- 🇮🇹Italy finex
Hi, I've tested the proposed patches and it works fine. Thank you very much.
- 🇬🇧United Kingdom darren.fisher
Thanks Osama Nuserat. I'd missed the other files as I was following error messages on my install. I'll be sure to check more thoroughly in the future. I've just tested your patch and it all looks good to me. I've updated the MR so it's easier for stomusic to merge to the dev branch.
- Status changed to Fixed
5 months ago 2:38pm 21 June 2024 Automatically closed - issue fixed for 2 weeks with no activity.