X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=actions%2Fpassword.php;h=aef1accdcc62389d5f3f99af279aceaf8b6ce6ab;hb=506843e55d1381172c7a8e629a2447588dba67fe;hp=3a89c99d333a5f90ffd9223b1bf394767c45a7bf;hpb=3f5ededc01d8eedac2a9a75917849fbe78a3e701;p=quix0rs-gnu-social.git diff --git a/actions/password.php b/actions/password.php index 3a89c99d33..aef1accdcc 100644 --- a/actions/password.php +++ b/actions/password.php @@ -23,18 +23,26 @@ require_once(INSTALLDIR.'/lib/settingsaction.php'); class PasswordAction extends SettingsAction { + function get_instructions() { + return _('You can change your password here. Choose a good one!'); + } + function show_form($msg=NULL, $success=false) { - common_show_header(_t('Change password')); - $this->settings_menu(); - $this->message($msg, $success); - common_element_start('form', array('method' => 'POST', + $user = common_current_user(); + $this->form_header(_('Change password'), $msg, $success); + common_element_start('form', array('method' => 'post', 'id' => 'password', 'action' => common_local_url('password'))); - common_password('oldpassword', _t('Old password')); - common_password('newpassword', _t('New password')); - common_password('confirm', _t('Confirm')); - common_submit('submit', _t('Change')); + # Users who logged in with OpenID won't have a pwd + if ($user->password) { + common_password('oldpassword', _('Old password')); + } + common_password('newpassword', _('New password'), + _('6 or more characters')); + common_password('confirm', _('Confirm'), + _('same as password above')); + common_submit('submit', _('Change')); common_element_end('form'); common_show_footer(); } @@ -46,18 +54,21 @@ class PasswordAction extends SettingsAction { # FIXME: scrub input - $oldpassword = $this->arg('oldpassword'); $newpassword = $this->arg('newpassword'); $confirm = $this->arg('confirm'); if (0 != strcmp($newpassword, $confirm)) { - $this->show_form(_t('Passwords don\'t match')); + $this->show_form(_('Passwords don\'t match.')); return; } - if (!common_check_user($user->nickname, $oldpassword)) { - $this->show_form(_t('Incorrect old password')); - return; + if ($user->password) { + $oldpassword = $this->arg('oldpassword'); + + if (!common_check_user($user->nickname, $oldpassword)) { + $this->show_form(_('Incorrect old password')); + return; + } } $original = clone($user); @@ -66,15 +77,15 @@ class PasswordAction extends SettingsAction { $val = $user->validate(); if ($val !== TRUE) { - $this->show_form(_t('Error saving user; invalid.')); + $this->show_form(_('Error saving user; invalid.')); return; } - + if (!$user->update($original)) { - common_server_error(_t('Can\'t save new password.')); + common_server_error(_('Can\'t save new password.')); return; } - $this->show_form(_t('Password saved'), true); + $this->show_form(_('Password saved.'), true); } -} \ No newline at end of file +}