[wp-trac] [WordPress Trac] #17817: do_action/apply_filters/etc. recursion on same filter kills underlying call

[WordPress Trac]

#17817: do_action/apply_filters/etc. recursion on same filter kills underlying call
--------------------------------------+-----------------------------
 Reporter:  kernfel                   |       Owner:  wonderboymusic
     Type:  defect (bug)              |      Status:  reviewing
 Priority:  normal                    |   Milestone:  4.4
Component:  Plugins                   |     Version:  2.2
 Severity:  normal                    |  Resolution:
 Keywords:  has-unit-tests has-patch  |     Focuses:
--------------------------------------+-----------------------------
Changes (by DrewAPicture):

 * keywords:  has-unit-tests has-patch needs-docs => has-unit-tests has-
               patch
 * owner:  DrewAPicture => wonderboymusic


Comment:

 Did a docs pass in [attachment:17817.5.diff] off of
 [attachment:17817.4.diff].

 Added:
 * A file header
 * Complete property docs
 * Complete method docs
 * Some inline-spacing and a few comments

 Changed:
 * I changed the argument order for `->add_filter()` to have the two
 optional parameters be last (and to better follow the standalone pattern
 of 'hook', 'callback', 'priority', 'args'. Besides, it's always a little
 weird when optional parameters are in between required ones.

 @wonderboymusic: Tagging you back into the ring.

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