[wp-trac] Re: [WordPress Trac] #8220: Unused variables/variable typo's cleanup

WordPress Trac wp-trac at lists.automattic.com
Sat Nov 15 06:19:35 GMT 2008


#8220: Unused variables/variable typo's cleanup
-----------------------+----------------------------------------------------
 Reporter:  DD32       |        Owner:  anonymous
     Type:  defect     |       Status:  new      
 Priority:  normal     |    Milestone:  2.7      
Component:  General    |      Version:  2.7      
 Severity:  normal     |   Resolution:           
 Keywords:  has-patch  |  
-----------------------+----------------------------------------------------
Comment (by DD32):

 Few more notes:
  * wp-includes/pluggable.php: get_avatar();
 {{{$defaults[$avatar_default][1]}}} doesnt exist, As far as i could tell,
 'custom' shouldnt be passed to that function, instead, a URL should be, So
 i've assumed that branch was supposed to add the size arg to the url..
  * Most of those changes are untested, In the sense that the script I
 refer to for checking didnt come up with any other uses of the variables,
 And i couldnt locate anything which was refering to them.

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


More information about the wp-trac mailing list