From fd2593e8abc39c13f21f8ce3814af8cd55ddb62d Mon Sep 17 00:00:00 2001 From: epriestley Date: Thu, 1 Aug 2013 14:50:45 -0700 Subject: [PATCH] Restore setting "disabled" on user handles of disabled users Summary: Fixes T3666. D6585 updated the User handles, but accidentally dropped this unusual property. We should get rid of this -- it doesn't really make any sense on Handles -- but restore the previous beahvior to fix T3666 until we can nuke it. Test Plan: Clicked some pages? (Actually testing this properly is a bit of a pain and I am super lazy.) Reviewers: btrahan Reviewed By: btrahan CC: aran Maniphest Tasks: T3666 Differential Revision: https://secure.phabricator.com/D6644 --- src/applications/people/phid/PhabricatorPeoplePHIDTypeUser.php | 1 + 1 file changed, 1 insertion(+) diff --git a/src/applications/people/phid/PhabricatorPeoplePHIDTypeUser.php b/src/applications/people/phid/PhabricatorPeoplePHIDTypeUser.php index 2f6910d7af..b9dbc51422 100644 --- a/src/applications/people/phid/PhabricatorPeoplePHIDTypeUser.php +++ b/src/applications/people/phid/PhabricatorPeoplePHIDTypeUser.php @@ -40,6 +40,7 @@ final class PhabricatorPeoplePHIDTypeUser extends PhabricatorPHIDType { $handle->setFullName( $user->getUsername().' ('.$user->getRealName().')'); $handle->setImageURI($user->loadProfileImageURI()); + $handle->setDisabled($user->getIsDisabled()); if ($user->hasStatus()) { $status = $user->getStatus(); $handle->setStatus($status->getTextStatus());