[wp-trac] Re: [WordPress Trac] #5418: remove unused variables from wp-admin/*.php

WordPress Trac wp-trac at lists.automattic.com
Sun Dec 9 00:07:50 GMT 2007


#5418: remove unused variables from wp-admin/*.php
-----------------------+----------------------------------------------------
 Reporter:  DD32       |        Owner:  anonymous
     Type:  defect     |       Status:  reopened 
 Priority:  normal     |    Milestone:  2.4      
Component:  General    |      Version:  2.4      
 Severity:  normal     |   Resolution:           
 Keywords:  has-patch  |  
-----------------------+----------------------------------------------------
Comment (by JeremyVisser):

 Replying to [comment:13 DD32]:
 > However, that filter is used elsewhere, and in those cases, the output
 ''may'' be modified.

 But in this instance, the value is being ignored, so ''semantically'', I
 believe it's better to do {{{do_action()}}}. (Although, I think
 {{{do_action()}}} uses {{{apply_filters()}}} internally, so it's
 definitely not for optimisation reasons.)

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


More information about the wp-trac mailing list