Problem/Motivation
When going through the installation process and fixing a validation error on the password field, the validation message is wrongly displayed again after the installation process finishes.
Steps to reproduce:
1. Disable javascript (*)
2. Start the installation process, enter mismatching passwords (which will trigger the validation message)
3. Fix the validation errors and continue installation.
4. On the welcome page, the message "1 error has been found: Password" is displayed.
(*) Instead of disabling javascript, you can also reproduce by entering '0' as password/confirm password in step 2. (until #2563751: Password field errors on user create/edit/login when password is (literally) 0 is fixed)
Remaining tasks
Found cause, write patch.
API changes
None
Data model changes
None
Comment | File | Size | Author |
---|---|---|---|
#3 | install_error_shown_on_welcome_page.jpg | 64.66 KB | LoMo |
Comments
Comment #2
andypostThere was issue when user is not logged in after submiting this form, so looks that session saved with messages from previous request
Manual testing shows that session contains "messages" from previous request
Comment #3
LoMo CreditAttribution: LoMo as a volunteer commentedHi, andypost.
I've also encountered this oddity and in neither case was JS disabled nor did I use '0' for a password, although it being a local testing installation, the password was more than trivial.
Steps to reproduce:
1) While creating new D8 testing environment on initial installation, make a typo in one of the password fields and ignore any possible message about non-matching passwords.
2) Submit form and see validation error (that the passwords do not match)
3) Correct entry to ensure both password fields have the same (non-zero) value
4) The error will still be shown after re-submission of the final installation step (on the "Welcome page"), where the link in the error message, of course, is also broken.
Comment #4
xjmComment #8
SKAUGHTthis repeating message error occurs without IFE enabled. it has been happening in all version of 8.x.x.
Comment #17
smustgrave CreditAttribution: smustgrave at Mobomo commentedVerified in 9.5
Downgrading to minor though because doesn't seem to stop any installation.