In the installer the password field is set to required when in all honesty it is possible for db to not have password (very unwise but possible).

I am not sure if this is the right place for this or put it as another issue but what i also noticed is that required check and connection check are one after the other: even after the password required msg is shown...connection is tested without a password.
If connection was successful the user is left with the password required msg...if connection was not successful it displays both password required and connection error msg.

So having the password true makes it kinda ugly, because if password was to be left as required then it would be an idea to make it so that:

if (password) {
    do connection check
}
  else {
    print password required msg.
} 

http://www.flickr.com/photos/94059573@N00/235116537/ >> shows password required+connection error
http://www.flickr.com/photos/94059573@N00/235123054/ >> shows only password required msg when connection was successful....thus we are stuck and cant move on due to form restriction.

CommentFileSizeAuthor
#4 password_install.patch500 bytesflk
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

flk’s picture

Version: 4.7.3 » x.y.z

version is suppose to be cvs not 4.7.3

webchick’s picture

Priority: Critical » Normal

critical? :)

budda’s picture

It could prevent people from installing Drupal if they don't have dbase user with password setup.

flk’s picture

Assigned: Unassigned » flk
Status: Active » Needs review
FileSize
500 bytes
Dries’s picture

Status: Needs review » Fixed

Committed to CVS HEAD.

Anonymous’s picture

Status: Fixed » Closed (fixed)