Support views AJAX GET requests after https://www.drupal.org/project/drupal/issues/956186 / 10.1.x

Slight tweaks to issue summary.

Updating issue summary to reference issue and Gitlab MR that removed GET support.

Merge request is up!

Requires contrib modules to test:

• Facets
• better exposed filters
• views_ajax_get

Testing steps:

1. Enable the required contrib modules
2. Login to your site
3. Visit /admin/structure/views/settings/advanced and enable views_ajax_get extender
4. Create a view with ajax and GET requests enabled
5. Setup checkbox facets to associate with the new view
6. Edit the view to add the newly created facets as filters through the facets views plugin integration
7. Edit the view to enable better exposed filters, ideally with auto-submit enabled
8. Save the view settings
9. Test the view and request type by filtering with the facets and observing the network tab in your browser's debugger
10. Verify you're seeing GET requests instead of POST
11. Edit the view's ajax settings to disable GET requests
12. Save the view settings
13. Test the view with your network tab and verify you're now seeing POST requests (you may need to clear caches)

I think we should create a sub-module facets_ajax that has dependencies to better_exposed_filters and views_ajax_get.
In facets itself we should throw an exception if a view uses ajax and a facet gets rendered without facets_ajax beeing active.

📌 Allow AJAX to use GET requests Fixed landed in 10.1.x, is some of this needed to work with the core change now?

Changing this to a bug now.

When using facets 3.0.x all ajax paginations are now broken.

Steps to reproduce
Install facets
Edit admin/content view by enabling ajax
Add enough content or edit view to trigger pagination.
Pagination no longer works.

Can also be seen in the media library widget.

Applying the MR resolved the problem.

Updating the issue title. The RouteAlterSubscriber probably won't be necessary to add support for views_ajax_get any more (since that module should hopefully be deprecated from 10.1.x onwards), but most/all the rest of the changes look generic to support AJAX GET requests at all.

@mkalkbrenner I guess the approach should change now since the views_ajax_get is redundant?

Then again how would we handle BC for < 10.1.x sites?

https://www.drupal.org/node/3193798 →

This patch contains the current state of the merge request. See "Patches from drupal.org merge request URLs are dangerous?" for more information.

I've used a filename that was not websafe. My bad. I can't delete my previous comment. here is the patch with the commit hash as filename.
This patch contains the current state of the merge request. See "Patches from drupal.org merge request URLs are dangerous?" for more information.