D7618.diff
No OneTemporary
Actions

Size

1 KB

Referenced Files

None

Subscribers

None

D7618.diff
View Options

	Index: src/applications/auth/controller/PhabricatorAuthNeedsApprovalController.php
	===================================================================
	--- src/applications/auth/controller/PhabricatorAuthNeedsApprovalController.php
	+++ src/applications/auth/controller/PhabricatorAuthNeedsApprovalController.php
	@@ -11,6 +11,10 @@
	return false;
	}

	+ public function shouldRequireEnabledUser() {
	+ return false;
	+ }
	+
	public function processRequest() {
	$request = $this->getRequest();
	$user = $request->getUser();
	Index: src/applications/auth/controller/PhabricatorEmailVerificationController.php
	===================================================================
	--- src/applications/auth/controller/PhabricatorEmailVerificationController.php
	+++ src/applications/auth/controller/PhabricatorEmailVerificationController.php
	@@ -15,10 +15,22 @@
	return false;
	}

	+ public function shouldRequireEnabledUser() {
	+ // Unapproved users are allowed to verify their email addresses. We'll kick
	+ // disabled users out later.
	+ return false;
	+ }
	+
	public function processRequest() {
	$request = $this->getRequest();
	$user = $request->getUser();

	+ if ($user->getIsDisabled()) {
	+ // We allowed unapproved and disabled users to hit this controller, but
	+ // want to kick out disabled users now.
	+ return new Aphront400Response();
	+ }
	+
	$email = id(new PhabricatorUserEmail())->loadOneWhere(
	'userPHID = %s AND verificationCode = %s',
	$user->getPHID(),