[wp-trac] Re: [WordPress Trac] #5338: Preformance Tweaks for WordPress Trunk Plugin API, plugin.php

[WordPress Trac]

#5338: Preformance Tweaks for WordPress Trunk Plugin API, plugin.php
--------------------------------------------------------+-------------------
 Reporter:  darkdragon                                  |        Owner:  anonymous
     Type:  enhancement                                 |       Status:  new      
 Priority:  normal                                      |    Milestone:  2.4      
Component:  Optimization                                |      Version:  2.4      
 Severity:  normal                                      |   Resolution:           
 Keywords:  plugins.php phpdoc has-patch needs-testing  |  
--------------------------------------------------------+-------------------
Changes (by darkdragon):

 * cc: plugins.php, phpdoc, has-patch, needs-testing (removed)
  * keywords:  => plugins.php phpdoc has-patch needs-testing

Comment:

 Look at the taxonomy. Apply filters are used on objects. Therefore the
 documentation is correct.

 Replying to [comment:5 Quandary]:
 > You also need to pre-define it if you take out the array sort (I don't
 see any sorting being done on $wp_filters anymore). Otherwise, the
 priorities will get all screwed up. Pri-10 will likely wind up running
 first, because it'll usually be the first level created.
 >
 > {{{
 > <?php
 >
 > $example = array();
 > $example[10] = "This is supposed to be last";
 > $example[1] = "This is supposed to be first";
 >
 > foreach($example as $text)
 >     echo "$text\n";
 >
 > ?>
 > }}}
 >
 > === Output ===
 >
 > {{{
 > This is supposed to be last
 > This is supposed to be first
 > }}}

 This the reason I had keywords 'needs-testing', I had thought this is the
 case. Without testing, it is unknown whether this would be the case. I
 will submit a new patch that fixes this.

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