X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=actions%2Fpassword.php;h=603ae58eaf0405c9884b775c0c2b0b50cdc879c0;hb=d0559fdf4dafcaf8446b437f4af089c944a23d09;hp=31831d3a935e5a7db189b7fd455f16eac50ef24a;hpb=fac522f4d7cce9a35e605fac2bba0b2d23616ad0;p=quix0rs-gnu-social.git diff --git a/actions/password.php b/actions/password.php index 31831d3a93..603ae58eaf 100644 --- a/actions/password.php +++ b/actions/password.php @@ -1,74 +1,61 @@ . */ -if (!defined('LACONICA')) { exit(1) } +if (!defined('LACONICA')) { exit(1); } + +require_once(INSTALLDIR.'/lib/settingsaction.php'); class PasswordAction extends SettingsAction { - - function handle($args) { - parent::handle($args); - if (!common_logged_in()) { - common_user_error(_t('Not logged in.')); - return; - } - if ($this->arg('METHOD') == 'POST') { - $this->handle_post(); - } else { - $this->show_form(); - } - } function show_form($msg=NULL, $success=false) { - common_show_header(_t('Change password')); - $this->settings_menu(); + common_show_header(_t('Change password'), NULL, NULL, array($this, 'settings_menu')); if ($msg) { - common_element('div', ($success) ? 'success' : 'error', - $msg); + $this->message($msg, $success); + } else { + common_element('div', 'instructions', + _t('You can change your password here. Choose a good one!')); } - common_start_element('form', array('method' => 'POST', + common_element_start('form', array('method' => 'POST', 'id' => 'password', - 'action' => + 'action' => common_local_url('password'))); common_password('oldpassword', _t('Old password')); - common_password('newpassword', _t('New password')); - common_password('confirm', _t('Confirm')); - common_element('input', array('name' => 'submit', - 'type' => 'submit', - 'id' => 'submit'), - _t('Login')); - common_element('input', array('name' => 'cancel', - 'type' => 'button', - 'id' => 'cancel'), - _t('Cancel')); + common_password('newpassword', _t('New password'), + _t('6 or more characters')); + common_password('confirm', _t('Confirm'), + _t('same as password above')); + common_submit('submit', _t('Change')); + common_element_end('form'); + common_show_footer(); } - + function handle_post() { $user = common_current_user(); assert(!is_null($user)); # should already be checked - + # 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')); return; @@ -78,14 +65,22 @@ class PasswordAction extends SettingsAction { $this->show_form(_t('Incorrect old password')); return; } - + + $original = clone($user); + $user->password = common_munge_password($newpassword, $user->id); - - if (!$user->update()) { + + $val = $user->validate(); + if ($val !== TRUE) { + $this->show_form(_t('Error saving user; invalid.')); + return; + } + + if (!$user->update($original)) { common_server_error(_t('Can\'t save new password.')); return; } - + $this->show_form(_t('Password saved'), true); } } \ No newline at end of file