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

WordPress Trac wp-trac at lists.automattic.com
Sat Dec 8 08:30:59 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 DD32):

 Replying to [comment:12 JeremyVisser]:
 > In that case, {{{do_action('wp_create_file_in_uploads', $file);}}} would
 suffice.

 I thought so too, However, That filter is used elsewhere, and in those
 cases, the output ''may'' be modified.

 In the lines pointed out, the return value was simply going to be ignored
 anyway, Yet i thought other scripts ''may'' want to hook those events too.

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


More information about the wp-trac mailing list