[wp-trac] Re: [WordPress Trac] #5232: Get the current action and current filter name

WordPress Trac wp-trac at lists.automattic.com
Thu Oct 25 03:43:20 GMT 2007


#5232: Get the current action and current filter name
-------------------------+--------------------------------------------------
 Reporter:  tellyworth   |        Owner:  anonymous
     Type:  enhancement  |       Status:  new      
 Priority:  normal       |    Milestone:  2.4      
Component:  General      |      Version:           
 Severity:  normal       |   Resolution:           
 Keywords:  has-patch    |  
-------------------------+--------------------------------------------------
Comment (by tellyworth):

 I wrote a few draft patches for this yesterday, and after reviewing those
 I came to two conclusions:

 1. The filter/action code is time critical.  Anything that adds an extra
 function call, object dereference or similar can have a significant impact
 on performance.

 2. Properly encapsulating the current_filter data requires a singleton
 object; a function with a static variable is messy and involves several
 branches and function calls (which affect performance).  Short of
 refactoring the filter/action API, I can't see a good way to overcome
 that.

 wp-current-filter-revised-r6278.patch is a compromise.  It uses only one
 global, and provides a current_filter() API function for fetching the
 filter or action name.  Only 2 lines of code are needed in each of the
 time critical functions, with no branching or user function calls.

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


More information about the wp-trac mailing list