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

WordPress Trac noreply at wordpress.org
Mon Nov 25 22:02:03 UTC 2013


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

Comment (by jbrinley):

 @sc0ttkclark - I'm not quite comfortable saying that 17817.3.patch is
 ready for core, yet. As @wonderboymusic just said, it needs to be more
 performant. If core devs, or anyone else, has suggestions along those
 lines, they would be welcome. It's already improved from 17817.2.patch,
 based on feedback @nacin and @markjaquith gave at WordCamp Orlando last
 week.

 We definitely need to get this bug fixed, and it needs to be done in a
 backwards-compatible way. A minor performance impact is probably worth it.
 I'm just not sure what the threshold is for "minor". 5% is probably too
 big. 1-2%, maybe even 3%, would probably be acceptable, in my opinion.

--
Ticket URL: <http://core.trac.wordpress.org/ticket/17817#comment:44>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list