Bulk Actions default posts to Draft

[actionnerd]

Very trivial but wanted to mention it as it did provide an unexpected result during a client meeting.

Steps to reproduce bug:
Select the checkbox of a few posts in the post/page list in the Wordpress Admin area.
Change the "Bulk Actions" drop down to "Edit" (but do not hit "Apply").
Change the "Show all dates" drop down to any month and select Filter.

Instead of filtering by date, it'll update the selected posts to Draft mode.

Able to reproduce across several installs, with and without plugins and the default theme activated.

[ocean90]

Reproduced with trunk. With WP_DEBUG true I get this messages:

Notice: Undefined index: _status in /wp-admin/includes/post.php on line 284

Notice: Undefined index: _status in /wp-admin/includes/post.php on line 288

[kawauso]

Related/duplicate: #17412

[kawauso]

This occurs even if nothing is set as the filter, it's just having the Bulk Edit open when filtering that causes it.

[nacin]

Yuck.

Any attempt on a patch?

Sounds like maybe we're just not checking which submit button was pressed when trying to save bulk edit?

[actionnerd]

Sorry about that! I did a search but nothing came up — probably cause #17412 is a bit sparse on details :)

[kawauso]

Yeah, just threw it up here so it can be closed, probably in favour of this to be honest.

The bug is actually present for all Bulk Edit actions across all list tables that have Bulk Edit and Filter as far as I can tell.

[azaozz]

Looks like another rare regression. If there's a bulk edit open and the user clicks on a filter, the bulk edit should close before submitting the form. That's how it used to work when we introduced it (I think). Also checking which button was clicked to submit the form is a good idea.

[nacin]

I don't like the idea of content destruction. Moving to 3.2 and bumping severity.

[ocean90]

Patch from #15761 needs some refresh, but it works for me.

[azaozz]

In [18222]:

Do not run bulk edit when the user clicks on a filter, fixes #17733