[wp-trac] Re: [WordPress Trac] #9569: Installing over existing user table gives misleading instruction

WordPress Trac wp-trac at lists.automattic.com
Wed Apr 29 09:06:05 GMT 2009


#9569: Installing over existing user table gives misleading instruction
-----------------------------+----------------------------------------------
 Reporter:  coffee2code      |       Owner:                  
     Type:  enhancement      |      Status:  new             
 Priority:  lowest           |   Milestone:  2.8             
Component:  Upgrade/Install  |     Version:  2.8             
 Severity:  trivial          |    Keywords:  has-patch tested
-----------------------------+----------------------------------------------

Comment(by Denis-de-Bernardy):

 Replying to [comment:4 coffee2code]:
 > * Have `wp_install()` return `''` instead of the "...Password
 inherited..." message as the password.  The presumption being that no
 password == inherited password.  Therefore, the "...Password inherited..."
 message could just be specified in one place (`wp-admin/install.php`) and
 not also in `wp_install()`.
 >
 > * Or if we anticipate later having other non-password responses such
 that `''` can't be assumed to mean a password was inherited, maybe return
 a WP_Error object in place of the password and check for that?

 WP_Error object seems like the one that makes the most sense. That it's
 called WP_Error might confuse whoever looks into it after, though. A
 WP_Warning comes to mind. :-D

 D.

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


More information about the wp-trac mailing list