[wp-trac] [WordPress Trac] #13317: Code Impriovement in get_userdata

WordPress Trac wp-trac at lists.automattic.com
Wed May 12 06:49:58 UTC 2010


#13317: Code Impriovement in get_userdata
--------------------------+-------------------------------------------------
 Reporter:  hakre         |       Owner:           
     Type:  defect (bug)  |      Status:  new      
 Priority:  lowest        |   Milestone:  3.0      
Component:  General       |     Version:           
 Severity:  normal        |    Keywords:  has-patch
--------------------------+-------------------------------------------------

Comment(by hakre):

 I finally must admit that those useless comments might have distracted me
 somehow and in kind of a bad influence.

 Naturally my original patch is totally okay and nacin's suggestion to it
 are infact silly.

 It's common practice in core code when it comes to a comparison against
 zero for a userid to do it with the numerical value and the standard php
 comparison operator {{{==}}}. Using {{{false}}} or {{{!}}} would be
 misleading.

 So my new choice is A.

 I hope some other comitter is able to concentrate on the actual work
 instead of trying to find an argumentation to no do things. It's
 especially a problem for me to see a really useless discussion here
 instead of a quick and straight ticket handling. It's not me who is
 wasting time in the end. Please deal with such minor changes more
 efficient next time to safe us all some time to concentrate on more
 important stuff. Thanks.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/13317#comment:4>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list