[wp-trac] [WordPress Trac] #24078: Remove 'admin' as default username in install

[WordPress Trac]

#24078: Remove 'admin' as default username in install
------------------------------+-----------------------
 Reporter:  chrisrudzki       |       Owner:  ryan
     Type:  enhancement       |      Status:  reopened
 Priority:  normal            |   Milestone:  3.7
Component:  Upgrade/Install   |     Version:  3.5
 Severity:  normal            |  Resolution:
 Keywords:  has-patch commit  |
------------------------------+-----------------------
Changes (by DrewAPicture):

 * keywords:  has-patch commit 3.7-early => has-patch commit
 * status:  closed => reopened
 * resolution:  fixed =>


Comment:

 Shouldn't we also remove 'admin' in the second ternary, allowing the error
 to get tossed when `$user_name` is empty? [attachment:24078.diff]

 Replying to [comment:36 nacin]:
 > Beyond [24998], it would be nice if install.php reacted a bit better
 when there was an error. "ERROR: you must provide a valid username." is
 okay, but certainly not ideal. Needs better poka-yoke! New ticket, anyone?

 I cringed at install.php for a while last night. I'll make a ticket.

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