archive/phabricator
1
0
Fork 0
Code Issues 1 Pull Requests Packages Projects Releases Wiki Activity

Use grey dots for disabled users, even if a user is also unverified

Browse Source 
Summary: Fixes T12559.

Test Plan:
{F4929988}

{F4929989}

{F4929990}

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T12559

Differential Revision: https://secure.phabricator.com/D17806
This commit is contained in:
epriestley
2017-04-30 12:57:24 -07:00
parent 608dd06151
commit 89d0c8e388
2 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/applications/people/markup/PhabricatorMentionRemarkupRule.php
View File

@@ -150,7 +150,9 @@ final class PhabricatorMentionRemarkupRule extends PhutilRemarkupRule {
$tag->addClass('phabricator-remarkup-mention-nopermission'); $tag->addClass('phabricator-remarkup-mention-nopermission');
} }
if (!$user->isResponsive()) { if ($user->getIsDisabled()) {
$tag->setDotColor(PHUITagView::COLOR_GREY);
} else if (!$user->isResponsive()) {
$tag->setDotColor(PHUITagView::COLOR_VIOLET); $tag->setDotColor(PHUITagView::COLOR_VIOLET);
} else { } else {
if ($user->getAwayUntil()) { if ($user->getAwayUntil()) {

4
src/applications/people/phid/PhabricatorPeopleUserPHIDType.php
View File

@@ -61,7 +61,9 @@ final class PhabricatorPeopleUserPHIDType extends PhabricatorPHIDType {
} }
$availability = null; $availability = null;
if (!$user->isResponsive()) { if ($user->getIsDisabled()) {
$availability = PhabricatorObjectHandle::AVAILABILITY_DISABLED;
} else if (!$user->isResponsive()) {
$availability = PhabricatorObjectHandle::AVAILABILITY_NOEMAIL; $availability = PhabricatorObjectHandle::AVAILABILITY_NOEMAIL;
} else { } else {
$until = $user->getAwayUntil(); $until = $user->getAwayUntil();