Homec4science

Remove most callsites to Controller->renderHandlesForPHIDs()

Authored by epriestley <git@epriestley.com> on Mar 30 2015, 15:35.

Description

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

Details

Committed
epriestley <git@epriestley.com>Mar 31 2015, 14:48
Pushed
aubortJan 31 2017, 17:16
Parents
rPHdec03cf07683: Prepare a replacement for Controller->renderHandlesForPHIDs()
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHa8271ecd4010: Remove most callsites to Controller->renderHandlesForPHIDs() (authored by epriestley <git@epriestley.com>).Mar 31 2015, 14:48