The module is not compatible with the user_registrationpassword module that allows users to set a password on the registration form - even if the email has to be validated.
The problem is that password policy checks if user.settings.verify_mail is checked and the user_registrationpassword module replace this check with custom options.
I dont know if this is an issue for user_registrationpassword module or for password_policy module.
Maybe the check to know if password policy should be verified should be done on the presence or not of the password field and not on something not really related as user.settings.verify_mail.
Comments
Comment #2
marabak CreditAttribution: marabak at Lumini commentedComment #3
Kristen PolThanks to everyone for the work on this issue.
I'm going through all the 8.x issues.
As the 8.x is no longer supported, I'm postponing this issue for now and need feedback as to whether or not this issue is relevant to 4.0.x.
If it is, please reopen and change the version, make sure the issue summary is clear and complete, including concrete steps to reproduce, and reroll the patch. If it's not, please close.
If there is no response to this in a month addressing the above, it can be closed.
Comment #4
mably CreditAttribution: mably commentedOur solution was to override the
password_policy.validation_manager
in a custom module.Related code below which checks for
user_registrationpassword
configuration and acts accordingly:May be password policy could provides two hooks to allow other modules to modify by reference the boolean value checked for the
return FALSE
part and to modify theuser_roles
array. Both were needed to make Password Policy work with theuser_registrationpassword
module.But may be other modules will have other needs.
Comment #5
mably CreditAttribution: mably commentedStarted implementing my compatibility fix for the user_registrationpassword module in a new contrib module: https://www.drupal.org/project/password_policy_extras
This new project could be used as a sandbox for the Password Policy module.