Clicking "Apply" in trash view shifts view back to All Items

[mrmist]

Scenerio -

Trash a post as an admin.
Visit the trash posts page as an author that doesn't have permission to delete said post.
Press "Apply".

The view shifts to the "All" view.
This focus change also seems to happen on "Sticky" view but does not happen for admin user, or when apply is pressed on other views (with nothing selected.)

Not major, but inconsistent UI.
(Tested in Opera)

[SergeyBiryukov]

I'm only able to reproduce the issue with “Sticky” view. Here's a patch for that. Is it necessary to select a bulk action before clicking the button?

[mrmist]

Is it necessary to select a bulk action before clicking the button?

Not for the cases I've tested.

[nacin]

Surprised I missed implementing a hidden field for that. Looks good, testing the other situations now.

[nacin]

I can only reproduce on the Trash view when I actually select a bulk action. It has nothing to do with permissions, either.

I'm noticing a few issues:

• It worked in 3.0. The only reason it did is because -- you'll like this -- the wp_redirect() on line 70 didn't have an exit, and thus the switch was evaluated (with $doaction never set), and then wp_redirect() becomes sendback. That's now fixed, but we should instead when redirect back to $sendback.

• $post_new_file is set too late in 3.1. Gah.

Patch fixes both.

[nacin]

Includes simplified version of Sergey's patch. show_sticky is boolean.

[garyc40]

nacin's patch works fine.

[ryan]

(In [17362]) Fix redirects for trashed post bulk ops. Props nacin, SergeyBiryukov. fixes #16355