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

WordPress Trac wp-trac at lists.automattic.com
Fri Dec 7 09:18:47 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 Nazgul):

 That last patch introduced this:
 '''apply_filters('wp_create_file_in_uploads', $file); // replicate'''

 The output of the filter was removed, because it wasn't used in the rest
 of the code, but doesn't that imply that the entire apply_filters call can
 be removed as well? Why filter if you throw away the output.

 This happens in wp-app.php (415), wp-admin/custom-header.php (231) and wp-
 admin/link-import.php (126).

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


More information about the wp-trac mailing list