@recaptime-dev's working patches + fork for Phorge, a community fork of Phabricator. (Upstream dev and stable branches are at upstream/main and upstream/stable respectively.)
hq.recaptime.dev/wiki/Phorge
phorge
phabricator
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
+1
+1
src/applications/people/phid/PhabricatorPeoplePHIDTypeUser.php
+1
src/applications/people/phid/PhabricatorPeoplePHIDTypeUser.php
···
40
40
$handle->setFullName(
41
41
$user->getUsername().' ('.$user->getRealName().')');
42
42
$handle->setImageURI($user->loadProfileImageURI());
43
+
$handle->setDisabled($user->getIsDisabled());
43
44
if ($user->hasStatus()) {
44
45
$status = $user->getStatus();
45
46
$handle->setStatus($status->getTextStatus());