Make WordPress Core

Opened 11 years ago

Last modified 5 years ago

#22236 new defect (bug)

edit.php: Apply when Bulk Actions selected forgets tag= filter

Reported by: kw217's profile kw217 Owned by:
Milestone: Priority: normal
Severity: normal Version: 3.4.2
Component: Quick/Bulk Edit Keywords: has-patch needs-testing
Focuses: javascript Cc:

Description

Using WordPress 3.4.2, go to Posts, and click on one of the tags on one of your posts. The URL now contains tag=<tag>, and the screen shows the first 20 posts with that tag.

Now click the Apply button just next to the "Bulk Actions" drop-down at the top left.

I expect this to reload the page but otherwise have no effect.

Instead, it reloads the page without the tag=<tag> URL parameter, and shows the first 20 posts (irrespective of tag).

This is surprising and confusing - it's easy to accidentally hit "Apply" without having chosen an action, but at first glance it looks like nothing has happened. The user can then go on to do a bulk action (e.g., add category) without realising the tag they previously had selected is now no longer selected.

Note that filters (e.g., by category) that are set up using the Filter button *are* preserved when Bulk Actions / Apply is chosen - it's just tags that are lost.

Attachments (2)

22236.patch (787 bytes) - added by pankajxdx 11 years ago.
It fixes this issue
22236-2.patch (438 bytes) - added by matthusby 10 years ago.

Download all attachments as: .zip

Change History (10)

#1 @kw217
11 years ago

  • Cc keithw@… added

#2 @SergeyBiryukov
11 years ago

  • Component changed from General to Administration

#3 @SergeyBiryukov
11 years ago

  • Keywords needs-patch added

Confirmed.

@pankajxdx
11 years ago

It fixes this issue

#4 @aaroncampbell
11 years ago

pankajxdx: Unfortunately, that patch will only work in very specific conditions, the first of which is you have to have WordPress installed in the /wp/trunk directory of your site. The patch should really look at the root cause of this by seeing if the important parts of the URL can be passed with the form in some way so the URL can be rebuilt properly.

@matthusby
10 years ago

#5 @matthusby
10 years ago

Seems like this should be taken care of in JS so we don't waste time reloading the page.

#6 @matthusby
10 years ago

  • Keywords has-patch added; needs-patch removed

#7 @ocean90
10 years ago

  • Component changed from Administration to Quick/Bulk Edit
  • Focuses javascript added

#8 @chriscct7
8 years ago

  • Keywords needs-testing added
Note: See TracTickets for help on using tickets.