[wp-trac] [WordPress Trac] #16031: New bulk actions hook missing catcher behavior

[WordPress Trac]

#16031: New bulk actions hook missing catcher behavior
----------------------------+-----------------------------
 Reporter:  Veraxus         |       Owner:  DrewAPicture
     Type:  enhancement     |      Status:  accepted
 Priority:  normal          |   Milestone:  Future Release
Component:  Administration  |     Version:  3.1
 Severity:  normal          |  Resolution:
 Keywords:  has-patch       |     Focuses:
----------------------------+-----------------------------
Changes (by DrewAPicture):

 * owner:   => DrewAPicture
 * status:  assigned => accepted


Comment:

 I'm not inclined to support opening up the bulk actions filter while still
 lacking a solid approach for a catcher behavior.

 @Veraxus: While I appreciate the effort in
 [attachment:16031-comprehensive.patch], there's an awful lot of fairly
 unrelated stuff going on in there.

 I think our best bet would be to introduce a `handle_bulk_actions()`
 method in `WP_List_Table` that's called at the top of `prepare_items()` in
 the sub-classes and contains a dynamic action hook using the screen ID.

 At its base, this approach would give a more straightforward path for
 "catching" the request data on the other end. I think wherever possible,
 we'd want to adapt core list tables leveraging bulk actions to extend that
 new `handle_bulk_actions()` method.

 This isn't "no we can't have this" or wontfix even, but it's not going to
 happen in time for 4.4. Realistically, we'd need to get a patch together
 at the beginning of the 4.5 cycle to give any outlying bugs time to
 surface.

 All that said, I look forward to working with you on this in 4.5 and
 getting it over the finish line. Let's do this thing :-)

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