Comments

aohrvetpv’s picture

Status: Active » Needs review
StatusFileSize
new7.89 KB

The purpose of password_policy.js in 6.x-1.x is to override Drupal.behaviors.password and insert translations from the Password Policy module. (There is only a single line difference between Drupal.behaviors.password in password_policy.js and module/user/user.js).

This seems to be done only to change the password meter to speak in terms of "Password quality" instead of "Password strength". This is sensible because a password that meets constraints is not necessarily "strong".

7.x-1.x did not seem to port this feature. password_policy.js is there but is unused, and is basically the D6 code.

This patch accomplishes the same as password_policy.js does in 6.x-1.x, but by overriding Drupal.settings.password instead of overriding Drupal.behaviors.password. This seems like a simpler implementation, but I am a JavaScript amateur so this is probably not following best practices.

aohrvetpv’s picture

StatusFileSize
new0 bytes

Change "PasswordPolicy" to lower camel "passwordPolicy" per coding standards.

aohrvetpv’s picture

Accidentally submitted empty patch in #2.

aohrvetpv’s picture

  • AohRveTPV committed 232976b on 7.x-1.x
    Issue #2464835 by AohRveTPV: password_policy.js unused
    
aohrvetpv’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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