1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-11-10 08:52:39 +01:00

Exclude disabled (disapproved) users from count on People application on homepage

Summary:
The People application shows users awaiting approval, but incorrectly counts disabled users (i.e., users who were not approved).

Instead, count only non-disabled, non-approved users.

Test Plan: My homepage count dropped from 4 to 1, corresponding to the actual number of accounts.

Reviewers: btrahan

Reviewed By: btrahan

CC: aran, spicyj

Differential Revision: https://secure.phabricator.com/D8486
This commit is contained in:
epriestley 2014-03-10 16:20:49 -07:00
parent c857f8cacb
commit 0a779b60a2

View file

@ -80,6 +80,7 @@ final class PhabricatorApplicationPeople extends PhabricatorApplication {
$need_approval = id(new PhabricatorPeopleQuery())
->setViewer($user)
->withIsApproved(false)
->withIsDisabled(false)
->execute();
if (!$need_approval) {