revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-12-19 03:50:54 +01:00
Code Issues Releases Wiki Activity

Don't require users be activated in order to establish Web sessions

Browse source 
Summary: Ref T8496. In D13123, the condition for establishing a web session was made too strict: we need to let non-activated users establish web sessions in order to see "you are a bad disabled person" or "your account needs approval" messages. The previous behavior let them in, the new behavior incorrectly locks them out.

Test Plan: Enabled login approvals and registered a new account with username/password auth.

Reviewers: btrahan

Reviewed By: btrahan

Subscribers: joshuaspence, epriestley

Maniphest Tasks: T8496

Differential Revision: https://secure.phabricator.com/D13239
This commit is contained in:
epriestley 2015-06-10 15:34:02 -07:00
parent 5a00b5f6f6
commit 8d6209b701
1 changed files with 0 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/applications/people/storage/PhabricatorUser.php
View file

@ -118,10 +118,6 @@ final class PhabricatorUser
} }
public function canEstablishWebSessions() { public function canEstablishWebSessions() {
if (!$this->isUserActivated()) {
return false;
}
if ($this->getIsMailingList()) { if ($this->getIsMailingList()) {
return false; return false;
} }