the only reason we *need* to reset the password is to support including %password in the emails. however:
a) the 1-time login URLs are better anyway (easier to use, require the user to reset their password, etc).
b) the fact this module will now always reset a users's password whenever it goes from blocked to active is a little wonky.
it'd be better to just have a setting to control if the password is reset, which then makes %password an available placeholder in the email bodies. this would default to off, so that the module never resets passwords unless site admins go out of their way to configure it to work that way.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | user_status_passwd_reset_setting.patch.txt | 1.9 KB | dww |
Comments
Comment #1
dwwneeds testing, but i think this should do it.
Comment #2
dwwJust did another round of heavy testing, and I'm happy with this. committed to HEAD, DRUPAL-5, and DRUPAL-4-7. Included in the 5.x-1.1 and 4.7.x-1.2 releases i just created.
Comment #3
(not verified) commented