Ajax state leaking to Views destination paths

Confirmed the issue described in the issue summary.

Patch #12 solved the issue.

@lauriii++

Patch looks good and fixes the issue I reported in #2253257-164: Use a modal for entity delete operation links. RTBC++

Reviewed this as a subsystem maintainer of both AJAX and Views.

1. +++ b/core/lib/Drupal/Core/Pager/PagerParameters.php
   @@ -36,7 +36,7 @@ public function getQueryParameters() {
   -        $request->query->all(), ['page']
   +        $request->query->all(), ['page', 'ajax_page_state']
   
   +++ b/core/modules/views/src/Controller/ViewAjaxController.php
   @@ -162,17 +162,21 @@ public function ajaxView(Request $request) {
   +        unset($param_union['ajax_page_state']);
   
   +++ b/core/modules/views/src/ViewExecutable.php
   @@ -705,7 +705,7 @@ public function getExposedInput() {
   -      foreach (['page', 'q'] as $key) {
   +      foreach (['page', 'q', 'ajax_page_state'] as $key) {
   

   These are 3 sensible places to do this. Especially glad to see the change to PagerParameters

2. +++ b/core/modules/views/src/Controller/ViewAjaxController.php
   @@ -162,17 +162,21 @@ public function ajaxView(Request $request) {
   -        $origin_destination = $path;
   +        $origin_destination = $request_clone->getBasePath() . '/' . ltrim($path ?? '/', '/');
   

   Nice :)

3. +++ b/core/modules/views/tests/src/FunctionalJavascript/PaginationAJAXTest.php
   @@ -100,6 +100,9 @@ public function testBasicPagination() {
   +    // Test that no unwanted parameters are added to the URL.
   +    $this->assertEquals('?status=All&type=All&langcode=All&items_per_page=5&order=changed&sort=asc&page=2', $link->getAttribute('href'));
   

   This assertion was incorrectly removed in #956186-152: Allow AJAX to use GET requests, glad to see it restored here

4. +++ b/core/modules/views/tests/src/Unit/Controller/ViewAjaxControllerTest.php
   @@ -224,7 +224,7 @@ public function testAjaxViewViewPathNoSlash() {
   -      ->with('test-page?ajax_page_state=drupal.settings%5B%5D&type=article');
   +      ->with('/test-page?type=article');
   

   Confirmed that adding this slash here does NOT change the intention or coverage of the test. This is a side-effect of normalizing the path with that ltrim above.

RTBC++

The other bug I discovered is that Views generates incorrect destination paths when Drupal is in a subdirectory. The fix is on the same method and is covered by the test coverage being added here so probably better to fix it here.

Nice! This has been reported also in #3130504: In subsite setup view_path is not correct for links on AJAX page refresh.

Thanks for finding and fixing the bug I introduced...

I think it would be nice if we could eventually centralise taking out ajax_page_state from incoming request URLs (rather than on generation), but that might be something that could be looked at in #3348789: Compress ajax_page_state, definitely not here.

Committed/pushed to 11.x and 10.1.x, thanks!

Hey all, thanks for your work on this! I just wanted to flag that I create a new issue that I think may be related but doesn't appear to be resolved when testing on 10.1.x-dev: https://www.drupal.org/project/drupal/issues/3366287.

Aaand there's another one: #3372678: Ajax state leaking to Views bulk operations

Thanks for working on this, here is a D9.5.x backport of patch #11

Another issue with destination path for subdirectory installation. If you also have language prefix in path it is lost: /subdirectory/de/admin/content/media becomes /subdirectory/admin/content/media