Undefined user in passwordsettings by XRevan86

This commit is contained in:
Diogo Cordeiro 2019-04-27 15:15:11 +01:00
parent 6cf674f8f8
commit f6dbf66983

View File

@ -28,7 +28,9 @@
* @link http://status.net/ * @link http://status.net/
*/ */
if (!defined('STATUSNET')) { exit(1); } if (!defined('STATUSNET')) {
exit(1);
}
/** /**
* Change password * Change password
@ -40,7 +42,6 @@ if (!defined('STATUSNET')) { exit(1); }
* @license http://www.fsf.org/licensing/licenses/agpl-3.0.html GNU Affero General Public License version 3.0 * @license http://www.fsf.org/licensing/licenses/agpl-3.0.html GNU Affero General Public License version 3.0
* @link http://status.net/ * @link http://status.net/
*/ */
class PasswordsettingsAction extends SettingsAction class PasswordsettingsAction extends SettingsAction
{ {
/** /**
@ -49,37 +50,36 @@ class PasswordsettingsAction extends SettingsAction
* @return string Title of the page * @return string Title of the page
*/ */
function title() public function title()
{ {
// TRANS: Title for page where to change password. // TRANS: Title for page where to change password.
return _m('TITLE','Change password'); return _m('TITLE', 'Change password');
} }
/** /**
* Instructions for use * Instructions for use
* *
* @return instructions for use * @return string instructions for use
*/ */
function getInstructions() public function getInstructions()
{ {
// TRANS: Instructions for page where to change password. // TRANS: Instructions for page where to change password.
return _('Change your password.'); return _('Change your password.');
} }
function showScripts() public function showScripts()
{ {
parent::showScripts(); parent::showScripts();
$this->autofocus('oldpassword'); $this->autofocus('oldpassword');
} }
function showContent() public function showContent()
{ {
$this->elementStart('form', array('method' => 'POST', $this->elementStart('form', ['method' => 'POST',
'id' => 'form_password', 'id' => 'form_password',
'class' => 'form_settings', 'class' => 'form_settings',
'action' => 'action' => common_local_url('passwordsettings')]);
common_local_url('passwordsettings')));
$this->elementStart('fieldset'); $this->elementStart('fieldset');
// TRANS: Fieldset legend on page where to change password. // TRANS: Fieldset legend on page where to change password.
$this->element('legend', null, _('Password change')); $this->element('legend', null, _('Password change'));
@ -102,14 +102,14 @@ class PasswordsettingsAction extends SettingsAction
$this->elementEnd('li'); $this->elementEnd('li');
$this->elementStart('li'); $this->elementStart('li');
// TRANS: Field label on page where to change password. In this field the new password should be typed a second time. // TRANS: Field label on page where to change password. In this field the new password should be typed a second time.
$this->password('confirm', _m('LABEL','Confirm'), $this->password('confirm', _m('LABEL', 'Confirm'),
// TRANS: Field title on page where to change password. // TRANS: Field title on page where to change password.
_('Same as password above.')); _('Same as password above.'));
$this->elementEnd('li'); $this->elementEnd('li');
$this->elementEnd('ul'); $this->elementEnd('ul');
// TRANS: Button text on page where to change password. // TRANS: Button text on page where to change password.
$this->submit('changepass', _m('BUTTON','Change')); $this->submit('changepass', _m('BUTTON', 'Change'));
$this->elementEnd('fieldset'); $this->elementEnd('fieldset');
$this->elementEnd('form'); $this->elementEnd('form');
@ -120,14 +120,14 @@ class PasswordsettingsAction extends SettingsAction
// FIXME: scrub input // FIXME: scrub input
$newpassword = $this->arg('newpassword'); $newpassword = $this->arg('newpassword');
$confirm = $this->arg('confirm'); $confirm = $this->arg('confirm');
// Some validation // Some validation
if (strlen($newpassword) < 6) { if (strlen($newpassword) < 6) {
// TRANS: Form validation error on page where to change password. // TRANS: Form validation error on page where to change password.
throw new ClientException(_('Password must be 6 or more characters.')); throw new ClientException(_('Password must be 6 or more characters.'));
} else if (0 != strcmp($newpassword, $confirm)) { } elseif (0 != strcmp($newpassword, $confirm)) {
// TRANS: Form validation error on password change when password confirmation does not match. // TRANS: Form validation error on password change when password confirmation does not match.
throw new ClientException(_('Passwords do not match.')); throw new ClientException(_('Passwords do not match.'));
} }
@ -142,11 +142,12 @@ class PasswordsettingsAction extends SettingsAction
} }
} }
if (Event::handle('StartChangePassword', array($this->scoped, $oldpassword, $newpassword))) { if (Event::handle('StartChangePassword', [$this->scoped, $oldpassword, $newpassword])) {
//no handler changed the password, so change the password internally // no handler changed the password, so change the password internally
$user = $this->scoped->getUser();
$user->setPassword($newpassword); $user->setPassword($newpassword);
Event::handle('EndChangePassword', array($this->scoped)); Event::handle('EndChangePassword', [$this->scoped]);
} }
// TRANS: Form validation notice on page where to change password. // TRANS: Form validation notice on page where to change password.