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

WordPress Trac noreply at wordpress.org
Wed Oct 21 07:13:43 UTC 2015


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

Comment (by SergeyBiryukov):

 Replying to [comment:145 kitchin]:
 > Also I wonder if the function/method signatures in 17817.5 are so
 similar that it's confusing:
 >
 > `function add_filter( $tag, $function_to_add, $priority = 10,
 $accepted_args = 1 )`
 > `public function add_filter( $function_to_add, $tag, $priority,
 $accepted_args )`

 Good point, [attachment:17817.9.diff] makes the order of arguments for
 `WP_Hook::add_filter()`, `WP_Hook::remove_filter()`, and
 `WP_Hook::has_filter()` consistent with the respective functions.

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


More information about the wp-trac mailing list