From 62e3463ab7914c003413d2f7b018672a9d31105b Mon Sep 17 00:00:00 2001 From: epriestley Date: Tue, 25 Mar 2014 13:59:01 -0700 Subject: [PATCH] Show primary email in approval queue interface Summary: Request from @csilvers. When approving users, the primary email address is useful for administrators. (This queue is only accessible by administrators, so this doesn't expose email information in general.) Test Plan: {F132912} Reviewers: btrahan Reviewed By: btrahan Subscribers: shadowhand, csilvers, epriestley Differential Revision: https://secure.phabricator.com/D8589 --- .../people/controller/PhabricatorPeopleListController.php | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/applications/people/controller/PhabricatorPeopleListController.php b/src/applications/people/controller/PhabricatorPeopleListController.php index ca4cd83b0a..2a84867ec1 100644 --- a/src/applications/people/controller/PhabricatorPeopleListController.php +++ b/src/applications/people/controller/PhabricatorPeopleListController.php @@ -62,6 +62,10 @@ final class PhabricatorPeopleListController extends PhabricatorPeopleController ->addAttribute($email) ->setImageURI($user->getProfileImageURI()); + if ($is_approval && $primary_email) { + $item->addAttribute($primary_email->getAddress()); + } + if ($user->getIsDisabled()) { $item->addIcon('disable', pht('Disabled')); }