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.
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
Comment #1
mcrittenden commentedI 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.
Comment #2
gpk commentedIMO having the password match indicator is not at all confusing and I was not expecting this module to disable it!
Comment #3
rimu commentedthis 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 likeComment #4
mcrittenden commentedPer #3
Comment #5
MBroberg commentedI agree, I like the password checker but not the strength warnings.
Comment #6
mrfelton commentedHere is a revised patch with the fix from #3
Comment #7
mrfelton commentedAnd here is one that actually applies with the normal strip option
Comment #8
mrfelton commentedComment #9
mrfelton commentedDamn it. 3rd time lucky.
Comment #10
mrfelton commentedStrangly, that last patch seems to leave you with a file thats missing a closing brace. Revised patch attached.
Comment #11
mrfelton commentedffs. Here is one that should apply with drush make.
Any chance we can get this committed?
Comment #12
rafamd commented+1
Comment #13
sachbearbeiter commented+1
Comment #14
chinita7 commented#11 works for me. Thanks a lot !
Comment #15
jherencia commentedI've commited the patch:
http://drupalcode.org/project/disablepwstrength.git/commit/0a72473
Thank you all.
Comment #16
jherencia commentedComment #17
rafamd commentedCan this be done to the 7.x branch also please ?
Comment #18
jherencia commented@rafamd:
#1622322: Plans for 7.x-2.x
Comment #19
rafamd commentedCool, thanks !