[wp-trac] [WordPress Trac] #11419: r12389 changes a piece of logic, no?

WordPress Trac wp-trac at lists.automattic.com
Sun Dec 13 17:30:37 UTC 2009


#11419: r12389 changes a piece of logic, no?
-------------------------------+--------------------------------------------
 Reporter:  Denis-de-Bernardy  |       Owner:  westi
     Type:  defect (bug)       |      Status:  new  
 Priority:  normal             |   Milestone:  2.9  
Component:  General            |     Version:  2.9  
 Severity:  normal             |    Keywords:       
-------------------------------+--------------------------------------------

Comment(by westi):

 $user_id if set should always be a positive integer in order for it to
 make any sense.

 It is true that now if $user_id was set to 0 - which has no sensible
 meaning we will now run the db query and create a WP_User for 0 which we
 wouldn't have done before.

 I will add the second check in to remove this possibility

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


More information about the wp-trac mailing list