![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
Opened 16 years ago
Closed 12 years ago
#16162 closed defect (bug) (fixed)
Pagination and sorting reset after bulk media deletion
| Reported by: | hakre | Owned by: | wonderboymusic |
|---|---|---|---|
| Priority: | normal | Milestone: | 4.1 |
| Component: | Administration | Version: | 3.1 |
| Severity: | normal | Keywords: | has-patch |
| Cc: | Focuses: | administration |
Description (last modified by )
When doing Bulk Deletes on paginated Media Library, after performing the delete, I'm back on page 1 regardless on which page I've been before.
Attachments (3)
Change History (25)
#2
@
16 years ago
- Keywords needs-patch added
- Milestone Awaiting Review → 3.1
- Owner set to
- Status new → reviewing
Same for categories and tags page. If we have it for posts and comments (#15416), we should add it with the other pages too.
#4
@
16 years ago
It does not reflect the sort column as well.
paged is part of the request, orderby and order are not.
#5
@
16 years ago
The patch preserves paged for bulk media deletes. sorting is not preserved as it does not gets passed in the request.
The idea I had so far is to add hidden form fields or to change the forms action attribute when things are changed by javascript.
The only hotfix I know of so far is to disable javascript which makes one of the biggest improvements of 3.1 looking pretty reduced.
#6
@
16 years ago
- Description modified (diff)
- Summary Bulk Actions and Pagination: Page Lost → Pagination and sorting reset after bulk media deletion
#9
@
16 years ago
- Keywords needs-patch added; has-patch removed
Noting that ocean90 was also able to reproduce this against edit-tags.php. A few of us were looking at this today (MarkJaquith in particular) and pagination was inconsistent across a number of screens.
#12
@
16 years ago
My patch over here also fixes pagination and sort reset for edit-tags, edit-comments, edit .
garyc40.16166.2.diff
Perhaps we should merge this ticket and #16166?
#14
@
16 years ago
- Milestone 3.1
- Resolution → duplicate
- Status reviewing → closed
Combining this, as the patch over on 16162 should fix this once it's applied to the rest of the list tables.
Duplicate of #16162
#15
@
16 years ago
#16
@
15 years ago
This is still an issue and imo shouldn't be merged with the ajax ticket. Any resolution to this
#17
@
15 years ago
- Milestone → Future Release
- Resolution duplicate
- Status closed → reopened
Yeah, need to handle this either way.
#19
@
12 years ago
- Keywords has-patch added; needs-patch removed
This has been fixed for the Edit Media list, but is still a problem on the tags and categories page.
This is because the <form>'s method attribute is set to post, rather than get, which ends up throwing off wp_get_referer(), losing the paged parameter in the link. attachment:16162.diff fixes the form method.
Related: #15416