[wp-trac] Re: [WordPress Trac] #5231: add_action('all', ...) causes unexpected behaviour

WordPress Trac wp-trac at lists.automattic.com
Fri Oct 19 09:09:28 GMT 2007


#5231: add_action('all', ...) causes unexpected behaviour
------------------------+---------------------------------------------------
 Reporter:  tellyworth  |        Owner:  anonymous
     Type:  defect      |       Status:  new      
 Priority:  normal      |    Milestone:  2.5      
Component:  General     |      Version:           
 Severity:  normal      |   Resolution:           
 Keywords:              |  
------------------------+---------------------------------------------------
Comment (by tellyworth):

 Fine by me.  I also found three other "all" issues in unit testing:

 1. An "all" filter will mess up the priorities of existing filters when
 merged (already reported in #4715, I can confirm this too)

 2. An "all" action (not filter) causes duplicates: after the "all" action
 has been triggered for action "foo", it exists in two places in
 wp_filters.  Subsequent calls to do_action cause additional duplicates.

 3. Removing an "all" action doesn't work.

 The unit tests will help refactoring, since they cover most of the basic
 behaviour.  They're here: http://svn.automattic.com/wordpress-tests/wp-
 testcase/test_filters.php and
 http://svn.automattic.com/wordpress-tests/wp-testcase/test_actions.php

-- 
Ticket URL: <http://trac.wordpress.org/ticket/5231#comment:5>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list