[wp-trac] [WordPress Trac] #19826: Error behavior for deleting trashed posts is different for Bulk Delete versus Empty Trash

[WordPress Trac]

#19826: Error behavior for deleting trashed posts is different for Bulk Delete
versus Empty Trash
--------------------------------------+------------------------------
 Reporter:  jpbellona                 |       Owner:
     Type:  defect (bug)              |      Status:  new
 Priority:  normal                    |   Milestone:  Awaiting Review
Component:  Posts, Post Types         |     Version:  3.1.2
 Severity:  minor                     |  Resolution:
 Keywords:  good-first-bug has-patch  |     Focuses:
--------------------------------------+------------------------------
Changes (by piontkowski):

 * keywords:  needs-patch good-first-bug => good-first-bug has-patch


Comment:

 Ignore my last question. I uploaded a patch to implement the behavior
 described in [comment:1 comment 1]. The code will take a count of how many
 posts were not able to be deleted and, if applicable, return the counts
 formatted as an error.

 While fixing the behavior when deleting a post, I noticed that the
 behavior is the same for trashing and restoring posts. Same, as in, the
 code issues a wp_die() when it comes across the first post that is not
 able to be trashed or restored by the user. So I also modified the trash
 and restore behavior too.

 Tested on regular posts and custom post types.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/19826#comment:8>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform