demianturner / sgl-docs-tickets-migration-test

0 stars 0 forks source link

Password edit notify #1632

Closed demianturner closed 11 years ago

demianturner commented 11 years ago

On changing a user password, the user can choose to have the new password sent to their email. The value sent by the form for the option (checkbox) is '1' or '' but in the Mgr it's looking for the value 'On' so using the checkbox to 'notify' doesn't work.

suggest changing this line

http://trac.seagullproject.org/browser/modules/0.6/user/classes/UserPasswordMgr.php#L80

to this:

{{{ $input->passwdResetNotify = ($req->get('frmPasswdResetNotify') == '1') ? 1 : 0; }}}

demianturner commented 11 years ago

[ed209] see http://trac.seagullproject.org/browser/modules/0.6/user/templates/userPasswordEdit.html#L51 for the form input

demianturner commented 11 years ago

[demian] debugging shows the value is "on", see attachment