First, the second box in the password_confirm field does not get highlighted if an error exists on the field. This is fixed by the last change in the patch:

+ form_error($form['pass2']);

Multiple error messages can be displayed for the field if a module performs some validation on the field (such as password length). For example, say the module has a password_confirm field called 'password' and validates on password length. The field is required and no password has been supplied in the first text box. If the module developer chooses to use form_set_error('password', 'Blah') both the 'Blah' message and the required message from password_confirm_validate are displayed.

CommentFileSizeAuthor
form.inc.patch_1.txt645 bytessammys

Comments

sammys’s picture

Status: Active » Needs review

Bumping this to patch (code needs review).

Sammy Spets
Synerger Pty Ltd
http://www.synerger.com

sammys’s picture

See http://drupal.org/node/59528 for a better fix for this bug.

Sammy Spets
Synerger Pty Ltd
http://www.synerger.com

chx’s picture

Status: Needs review » Closed (duplicate)

then this is a duplicate of that issue.