Opened 2 years ago

Closed 2 years ago

#16248 closed defect (bug) (invalid)

Back to Authors and Users link redirects to nowhere from a paginated user-edit screen

Reported by: linuxologos Owned by: PeteMall
Priority: high Milestone: 3.1
Component: Administration Version: 3.1
Severity: critical Keywords: has-patch
Cc:

Description

Testing against 3.1-RC2-17283.

Go to Users and go to any page other than the first one (the screen has to be paginated). Edit a User at that page and save. The Back to Authors and Users link redirects to a page just showing "-1". The problem is quite obvious even from the browser's address bar, which reveals the strange value for wp_http_referer (it must be related to ajaxified screens).

Attachments (3)

16248.diff (4.5 KB) - added by scribu 2 years ago.
16248.2.diff (4.5 KB) - added by PeteMall 2 years ago.
16248.3.diff (5.1 KB) - added by PeteMall 2 years ago.
Fixed ajax search query vars.

Download all attachments as: .zip

Change History (13)

  • Milestone changed from Awaiting Review to 3.1

Screens affected:

  • site admin users
  • net admin users
  • net admin site-users
  • Priority changed from normal to high
  • Severity changed from normal to critical

This is happening because the referrer is changed to admin-ajax.php with the JS pagination/search. We'll need to fix this for all list tables.

It should be a simple matter of passing _wp_http_referrer to the 'fetch-list' ajax action.

  • Owner set to scribu
  • Status changed from new to accepted
  • Keywords has-patch added

The problem was that these screens used 'wp_http_referrer' instead of '_wp_http_referrer'.

scribu2 years ago

  • Keywords commit added
  • Keywords commit removed

PeteMall said this doesn't work in all cases; he'll work through it tomorrow. Something about paging.

Fixed ajax search query vars.

  • Owner changed from scribu to PeteMall
  • Resolution set to invalid
  • Status changed from accepted to closed

Closing this as invalid because of #16262. See #16275 for a related no-js bug.

Note: See TracTickets for help on using tickets.