Remove most callsites to Controller->renderHandlesForPHIDs()

Summary: Ref T7689. This moves most of the easy/testable callsites off `Controller->renderHandlesForPHIDs()`.

Test Plan:
- Viewed a file; viewed author; viewed "attached" tab.
- Viewed a mock; viewed attached tasks.
- Viewed a credential; viewed "Used By".
- Viewed a paste; viewed author; viewed forks; viewed forked from.
- Viewed a dashboard; viewed panel list.
- Viewed a dashboard panel; viewed "Appears On".
- Viewed a Phortune account; viewed "Members"; viewed payment methods.
- Viewed a Phortune merchant account; viewed "Members".
- Viewed Phortune account switcher; viewed "Accounts".
  - I just removed "Members:" here since it felt kind of out-of-place anyway.
- Viewed a Phragment fragment, viewed "Latest Version", viewed "Snapshots".
- Viewed a Phargment snapshot, viewed "Fragment".

Reviewers: btrahan

Reviewed By: btrahan

Subscribers: hach-que, epriestley

Maniphest Tasks: T7689

Differential Revision: https://secure.phabricator.com/D12207

Author: epriestley

src/applications/dashboard/controller/PhabricatorDashboardManageController.php

@@ -165,12 +165,9 @@ private function buildPropertyView(PhabricatorDashboard $dashboard) {
       pht('Editable By'),
       $descriptions[PhabricatorPolicyCapability::CAN_EDIT]);
 
-    $panel_phids = $dashboard->getPanelPHIDs();
-    $this->loadHandles($panel_phids);
-
     $properties->addProperty(
       pht('Panels'),
-      $this->renderHandlesForPHIDs($panel_phids));
+      $viewer->renderHandleList($dashboard->getPanelPHIDs()));
 
     return $properties;
   }

src/applications/dashboard/controller/PhabricatorDashboardPanelViewController.php

@@ -163,15 +163,14 @@ private function buildPropertyView(PhabricatorDashboardPanel $panel) {
     $dashboard_phids = PhabricatorEdgeQuery::loadDestinationPHIDs(
       $panel->getPHID(),
       PhabricatorDashboardPanelHasDashboardEdgeType::EDGECONST);
-    $this->loadHandles($dashboard_phids);
 
     $does_not_appear = pht(
       'This panel does not appear on any dashboards.');
 
     $properties->addProperty(
       pht('Appears On'),
       $dashboard_phids
-        ? $this->renderHandlesForPHIDs($dashboard_phids)
+        ? $viewer->renderHandleList($dashboard_phids)
         : phutil_tag('em', array(), $does_not_appear));
 
     return $properties;

src/applications/files/controller/PhabricatorFileInfoController.php

@@ -39,11 +39,6 @@ public function processRequest() {
 
     $phid = $file->getPHID();
 
-    $handle_phids = array_merge(
-      array($file->getAuthorPHID()),
-      $file->getObjectPHIDs());
-
-    $this->loadHandles($handle_phids);
     $header = id(new PHUIHeaderView())
       ->setUser($user)
       ->setPolicyObject($file)
@@ -185,15 +180,14 @@ private function buildPropertyViews(
     $request = $this->getRequest();
     $user = $request->getUser();
 
-
     $properties = id(new PHUIPropertyListView());
     $properties->setActionList($actions);
     $box->addPropertyList($properties, pht('Details'));
 
     if ($file->getAuthorPHID()) {
       $properties->addProperty(
         pht('Author'),
-        $this->getHandle($file->getAuthorPHID())->renderLink());
+        $user->renderHandle($file->getAuthorPHID()));
     }
 
     $properties->addProperty(
@@ -270,7 +264,7 @@ private function buildPropertyViews(
 
       $attached->addProperty(
         pht('Attached To'),
-        $this->renderHandlesForPHIDs($phids));
+        $user->renderHandleList($phids));
     }
 
 

src/applications/passphrase/controller/PassphraseCredentialViewController.php

@@ -189,10 +189,9 @@ private function buildPropertyView(
       PhabricatorCredentialsUsedByObjectEdgeType::EDGECONST);
 
     if ($used_by_phids) {
-      $this->loadHandles($used_by_phids);
       $properties->addProperty(
         pht('Used By'),
-        $this->renderHandlesForPHIDs($used_by_phids));
+        $viewer->renderHandleList($used_by_phids));
     }
 
     $description = $credential->getDescription();

src/applications/paste/controller/PhabricatorPasteViewController.php

@@ -176,35 +176,35 @@ private function buildPropertyView(
     PhabricatorPaste $paste,
     array $child_phids,
     PhabricatorActionListView $actions) {
+    $viewer = $this->getViewer();
 
-    $user = $this->getRequest()->getUser();
     $properties = id(new PHUIPropertyListView())
-      ->setUser($user)
+      ->setUser($viewer)
       ->setObject($paste)
       ->setActionList($actions);
 
     $properties->addProperty(
       pht('Author'),
-      $this->getHandle($paste->getAuthorPHID())->renderLink());
+      $viewer->renderHandle($paste->getAuthorPHID()));
 
     $properties->addProperty(
       pht('Created'),
-      phabricator_datetime($paste->getDateCreated(), $user));
+      phabricator_datetime($paste->getDateCreated(), $viewer));
 
     if ($paste->getParentPHID()) {
       $properties->addProperty(
         pht('Forked From'),
-        $this->getHandle($paste->getParentPHID())->renderLink());
+        $viewer->renderHandle($paste->getParentPHID()));
     }
 
     if ($child_phids) {
       $properties->addProperty(
         pht('Forks'),
-        $this->renderHandlesForPHIDs($child_phids));
+        $viewer->renderHandleList($child_phids));
     }
 
     $descriptions = PhabricatorPolicyQuery::renderPolicyDescriptions(
-      $user,
+      $viewer,
       $paste);
 
     return $properties;

src/applications/people/storage/PhabricatorUser.php

@@ -1,7 +1,8 @@
 <?php
 
 /**
- * @task factors  Multi-Factor Authentication
+ * @task factors Multi-Factor Authentication
+ * @task handles Managing Handles
  */
 final class PhabricatorUser
   extends PhabricatorUserDAO
@@ -801,7 +802,7 @@ public static function getOmnipotentUser() {
   }
 
 
-/* -(  Handles  )------------------------------------------------------------ */
+/* -(  Managing Handles  )--------------------------------------------------- */
 
 
   /**
@@ -810,6 +811,7 @@ public static function getOmnipotentUser() {
    *
    * @param list<phid> List of PHIDs to load.
    * @return PhabricatorHandleList Handle list object.
+   * @task handle
    */
   public function loadHandles(array $phids) {
     if ($this->handlePool === null) {
@@ -821,6 +823,34 @@ public function loadHandles(array $phids) {
   }
 
 
+  /**
+   * Get a @{class:PHUIHandleView} for a single handle.
+   *
+   * This benefits from the viewer's internal handle pool.
+   *
+   * @param phid PHID to render a handle for.
+   * @return PHUIHandleView View of the handle.
+   * @task handle
+   */
+  public function renderHandle($phid) {
+    return $this->loadHandles(array($phid))->renderHandle($phid);
+  }
+
+
+  /**
+   * Get a @{class:PHUIHandleListView} for a list of handles.
+   *
+   * This benefits from the viewer's internal handle pool.
+   *
+   * @param list<phid> List of PHIDs to render.
+   * @return PHUIHandleListView View of the handles.
+   * @task handle
+   */
+  public function renderHandleList(array $phids) {
+    return $this->loadHandles($phids)->renderList();
+  }
+
+
 /* -(  PhabricatorPolicyInterface  )----------------------------------------- */
 
 

src/applications/pholio/controller/PholioMockViewController.php

@@ -42,8 +42,6 @@ public function processRequest() {
       $mock->getPHID(),
       PholioMockHasTaskEdgeType::EDGECONST);
     $this->setManiphestTaskPHIDs($phids);
-    $phids[] = $mock->getAuthorPHID();
-    $this->loadHandles($phids);
 
     $engine = id(new PhabricatorMarkupEngine())
       ->setViewer($user);
@@ -167,7 +165,7 @@ private function buildPropertyView(
 
     $properties->addProperty(
       pht('Author'),
-      $this->getHandle($mock->getAuthorPHID())->renderLink());
+      $user->renderHandle($mock->getAuthorPHID()));
 
     $properties->addProperty(
       pht('Created'),
@@ -176,7 +174,7 @@ private function buildPropertyView(
     if ($this->getManiphestTaskPHIDs()) {
       $properties->addProperty(
         pht('Maniphest Tasks'),
-        $this->renderHandlesForPHIDs($this->getManiphestTaskPHIDs()));
+        $user->renderHandleList($this->getManiphestTaskPHIDs()));
     }
 
     $properties->invokeWillRenderEvent();

src/applications/phortune/controller/PhortuneAccountListController.php

@@ -36,12 +36,10 @@ public function processRequest() {
 
     foreach ($accounts as $account) {
       $this->loadHandles($account->getMemberPHIDs());
-      $members = $this->renderHandlesForPHIDs($account->getMemberPHIDs(), ',');
       $item = id(new PHUIObjectItemView())
         ->setObjectName(pht('Account %d', $account->getID()))
         ->setHeader($account->getName())
         ->setHref($this->getApplicationURI($account->getID().'/'))
-        ->addAttribute(pht('Members: %s', $members))
         ->setObject($account);
 
       $payment_list->addItem($item);

src/applications/phortune/controller/PhortuneAccountViewController.php

@@ -57,11 +57,9 @@ public function handleRequest(AphrontRequest $request) {
       ->setObject($account)
       ->setUser($viewer);
 
-    $this->loadHandles($account->getMemberPHIDs());
-
     $properties->addProperty(
       pht('Members'),
-      $this->renderHandlesForPHIDs($account->getMemberPHIDs()));
+      $viewer->renderHandleList($account->getMemberPHIDs()));
 
     $status_items = $this->getStatusItemsForAccount($account, $invoices);
     $status_view = new PHUIStatusListView();
@@ -137,10 +135,6 @@ private function buildPaymentMethodsSection(PhortuneAccount $account) {
       ->withAccountPHIDs(array($account->getPHID()))
       ->execute();
 
-    if ($methods) {
-      $this->loadHandles(mpull($methods, 'getAuthorPHID'));
-    }
-
     foreach ($methods as $method) {
       $id = $method->getID();
 

src/applications/phortune/controller/PhortuneMerchantViewController.php

@@ -135,11 +135,9 @@ private function buildPropertyListView(
 
     $view->addProperty(pht('Status'), $status_view);
 
-    $this->loadHandles($merchant->getMemberPHIDs());
-
     $view->addProperty(
       pht('Members'),
-      $this->renderHandlesForPHIDs($merchant->getMemberPHIDs()));
+      $viewer->renderHandleList($merchant->getMemberPHIDs()));
 
     $view->invokeWillRenderEvent();