WordPress.org
Get WordPress

Make WordPress Core

Opened 15 years ago

Closed 13 years ago

#17545 closed defect (bug) (fixed)

Pagination Broken on Network Admin > Edit Site > Themes

Reported by: pmaiorana Owned by: nacin
Priority: normal Milestone: 3.9
Component: Themes Version: 3.2
Severity: normal Keywords: has-patch
Cc: Focuses: administration, multisite

Description

Pagination is broken in network admin on pages such as ​https://example.com/wp-admin/network/site-themes.php?id=XXXXX

You can navigate through pages using the arrow buttons, but entering a page number just reloads the page you’re currently on.

Steps to reproduce:

  • From network admin, "search sites"
  • Edit site
  • Click "themes" tab
  • Enter any valid page number in the text field
  • Hit enter

Attachments (4)

17545.diff​ (1.6 KB ) - added by kawauso 15 years ago.
17545.2.diff​ (1.1 KB ) - added by kawauso 14 years ago.
Refreshed against r21186
17545.3.diff​ (1.1 KB ) - added by jeremyfelt 13 years ago.
17545.4.diff​ (1.2 KB ) - added by jeremyfelt 13 years ago.
Add braces for new coding standards

Download all attachments as: .zip

Change History (15)

#1 @westi
15 years ago

  • Keywords needs-patch added
  • Version → 3.2

#2 @kawauso
15 years ago

Also affects site-users.php. Following patch adds request URI clean-up from site-themes.php to site-users.php and paged query variable support for both.

#3 @kawauso
15 years ago

  • Keywords has-patch added; needs-patch removed

@kawauso
15 years ago

#4 @SergeyBiryukov
15 years ago

  • Milestone Awaiting Review → 3.3

#5 @jane
15 years ago

Pinging Pete Mall.

#6 @PeteMall
15 years ago

On it for tonight.

#7 @ryan
15 years ago

  • Milestone 3.3 → Future Release

@kawauso
14 years ago

Refreshed against r21186

#8 @nacin
13 years ago

  • Component Network Admin → Themes
  • Focuses administration added

This ticket was mentioned in IRC in #wordpress-dev by nacin. ​View the logs.
13 years ago

@jeremyfelt
13 years ago

@jeremyfelt
13 years ago

Add braces for new coding standards

#10 @jeremyfelt
13 years ago

  • Milestone Future Release → 3.9

17545.4.diff​ is a refresh of @kawauso's patch against trunk.

This fixes the issue where entering a page number directly in the top right of the list tables for site-users.php and site-themes.php causes the page to reload. Things work in my testing.

#11 @nacin
13 years ago

  • Owner set to nacin
  • Resolution → fixed
  • Status new → closed

In 27715:

Network Admin: Fix pagination on site-themes and site-users.

props kawauso, jeremyfelt.
fixes #17545.

Note: See TracTickets for help on using tickets.