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

WordPress Trac wp-trac at lists.automattic.com
Tue Apr 28 19:31:33 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 coffee2code):

 Ah, oops.  Any other idea for something simple?  Otherwise, perhaps just
 the original patch for the time being?  Inline comments could be added to
 both places the message is defined to tell anyone updating the text that
 it needs to be updated in the other place as well.

 Other possible considerations:

 * 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?

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


More information about the wp-trac mailing list