My colleagues find the strength checker confusing, but I wanted to keep the check the two passwords match. The attached patch contains the relevant code copied from modules/user/user.js in Drupal 6.12.

Comments

mcrittenden’s picture

Status: Needs review » Needs work

I think this would need a settings page, as some would probably prefer to not display the password matching info either. Marking CNW in case some kind soul wants to come along and do this.

gpk’s picture

Status: Needs work » Needs review

IMO having the password match indicator is not at all confusing and I was not expecting this module to disable it!

rimu’s picture

this patch generates a file called disablepwstrength.js, which has a javascript error on line 56.

The error can be fixed by adding this on line 20

var passwordStrength = $("span.password-strength", outerItem) so the surrounding lines look like

    var confirmInput = $("input.password-confirm", outerItem);
    var confirmResult = $("span.password-confirm", outerItem);
    var passwordStrength = $("span.password-strength", outerItem)
    var confirmChild = $("span", confirmResult);
mcrittenden’s picture

Status: Needs review » Needs work

Per #3

MBroberg’s picture

I agree, I like the password checker but not the strength warnings.

mrfelton’s picture

Status: Needs work » Needs review
StatusFileSize
new3.38 KB

Here is a revised patch with the fix from #3

mrfelton’s picture

And here is one that actually applies with the normal strip option

mrfelton’s picture

Version: 6.x-1.5 » 6.x-1.6
mrfelton’s picture

Damn it. 3rd time lucky.

mrfelton’s picture

Strangly, that last patch seems to leave you with a file thats missing a closing brace. Revised patch attached.

mrfelton’s picture

ffs. Here is one that should apply with drush make.

Any chance we can get this committed?

rafamd’s picture

+1

sachbearbeiter’s picture

+1

chinita7’s picture

#11 works for me. Thanks a lot !

jherencia’s picture

jherencia’s picture

Status: Needs review » Fixed
rafamd’s picture

Can this be done to the 7.x branch also please ?

jherencia’s picture

rafamd’s picture

Cool, thanks !

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.