Make WordPress Core

Opened 2 years ago

Last modified 13 months ago

#22236 new defect (bug)

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

Reported by: kw217 Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version: 3.4.2
Component: Quick/Bulk Edit Keywords: has-patch
Focuses: javascript Cc:


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 22 months ago.
It fixes this issue
22236-2.patch (438 bytes) - added by matthusby 16 months ago.

Download all attachments as: .zip

Change History (9)

comment:1 @kw2172 years ago

  • Cc keithw@… added

comment:2 @SergeyBiryukov2 years ago

  • Component changed from General to Administration

comment:3 @SergeyBiryukov2 years ago

  • Keywords needs-patch added


@pankajxdx22 months ago

It fixes this issue

comment:4 @aaroncampbell22 months 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.

@matthusby16 months ago

comment:5 @matthusby16 months ago

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

comment:6 @matthusby16 months ago

  • Keywords has-patch added; needs-patch removed

comment:7 @ocean9013 months ago

  • Component changed from Administration to Quick/Bulk Edit
  • Focuses javascript added
Note: See TracTickets for help on using tickets.