diff --git a/src/applications/auth/controller/PhabricatorAuthOneTimeLoginController.php b/src/applications/auth/controller/PhabricatorAuthOneTimeLoginController.php index a40ab02d89..312367d03a 100644 --- a/src/applications/auth/controller/PhabricatorAuthOneTimeLoginController.php +++ b/src/applications/auth/controller/PhabricatorAuthOneTimeLoginController.php @@ -120,7 +120,7 @@ final class PhabricatorAuthOneTimeLoginController $next = '/'; if (!PhabricatorPasswordAuthProvider::getPasswordProvider()) { $next = '/settings/panel/external/'; - } else if (PhabricatorEnv::getEnvConfig('account.editable')) { + } else { // We're going to let the user reset their password without knowing // the old one. Generate a one-time token for that. diff --git a/src/applications/settings/panel/PhabricatorSettingsPanelPassword.php b/src/applications/settings/panel/PhabricatorSettingsPanelPassword.php index eeb9b3410e..8524e5baa0 100644 --- a/src/applications/settings/panel/PhabricatorSettingsPanelPassword.php +++ b/src/applications/settings/panel/PhabricatorSettingsPanelPassword.php @@ -16,15 +16,8 @@ final class PhabricatorSettingsPanelPassword } public function isEnabled() { - // There's no sense in showing a change password panel if the user - // can't change their password... - - if (!PhabricatorEnv::getEnvConfig('account.editable')) { - return false; - } - - // ...or this install doesn't support password authentication at all. - + // There's no sense in showing a change password panel if this install + // doesn't support password authentication. if (!PhabricatorPasswordAuthProvider::getPasswordProvider()) { return false; }