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.
Comment | File | Size | Author |
---|---|---|---|
#4 | password_install.patch | 500 bytes | flk |
Comments
Comment #1
flk CreditAttribution: flk commentedversion is suppose to be cvs not 4.7.3
Comment #2
webchickcritical? :)
Comment #3
buddaIt could prevent people from installing Drupal if they don't have dbase user with password setup.
Comment #4
flk CreditAttribution: flk commentedComment #5
Dries CreditAttribution: Dries commentedCommitted to CVS HEAD.
Comment #6
(not verified) CreditAttribution: commented