Problem/Motivation

In 10.1.0-alpha1, when enabling AJAX on a view, it adds ajax_page_state GET parameters to pager links.
The parameters appear after the first AJAX request.

Steps to reproduce

  1. Go to https://master-rv1x5j2rqmeay3sxncndouqkr9zns6f6.tugboatqa.com/admin/content
  2. Click on a pager link to trigger an AJAX request.
  3. Now, pager links have additional AJAX GET parameters.

Proposed resolution

Before #956186: Allow AJAX to use GET requests, we used to have unset($request_all['ajax_page_state']); in ViewAjaxController::ajaxView() but this was removed for some reason.
I think ajax_page_state should now be removed from both GET params before building the URL.

Comments

prudloff created an issue. See original summary.

prudloff’s picture

prudloff commented
Issue summary: View changes

I tried adding back unset($request_all['ajax_page_state']); and adding unset($query_all['ajax_page_state']); but it breaks the view style, so it is probably not the correct way to fix this.

Version: 10.1.x-dev » 11.x-dev

Drupal core is moving towards using a “main” branch. As an interim step, a new 11.x branch has been opened, as Drupal.org infrastructure cannot currently fully support a branch named main. New developments and disruptive changes should now be targeted for the 11.x branch, which currently accepts only minor-version allowed changes. For more information, see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

neclimdul’s picture

neclimdul
He/Him
commented
Status: Active » Closed (duplicate)
Related issues: +#3364088: Ajax state leaking to Views destination paths

Looks like the bug, at least as described here, has been fixed in #3364088: Ajax state leaking to Views destination paths.