Page MenuHomePhabricator

Remove most callsites to Controller->renderHandlesForPHIDs()
ClosedPublic

Authored by epriestley on Mar 30 2015, 2:05 PM.
Tags
None
Referenced Files
F13240965: D12207.id29357.diff
Wed, May 22, 6:11 PM
F13240526: D12207.id29330.diff
Wed, May 22, 2:51 PM
F13240490: D12207.id.diff
Wed, May 22, 2:36 PM
F13217382: D12207.diff
Sat, May 18, 4:54 AM
F13204489: D12207.diff
Wed, May 15, 12:37 AM
F13187453: D12207.diff
Sat, May 11, 4:32 AM
F13174523: D12207.id29330.diff
Tue, May 7, 11:36 PM
Unknown Object (File)
Tue, May 7, 7:40 AM

Details

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".

Diff Detail

Repository
rP Phabricator
Branch
param4
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 5038
Build 5056: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley retitled this revision from to Remove most callsites to Controller->renderHandlesForPHIDs().
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Mar 30 2015, 8:03 PM
This revision was automatically updated to reflect the committed changes.