Page MenuHomePhabricator

Remove Controller->getHandle() and Controller->loadHandles()
ClosedPublic

Authored by epriestley on Mar 30 2015, 3:25 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Apr 21, 4:19 PM
Unknown Object (File)
Sat, Apr 20, 7:49 PM
Unknown Object (File)
Thu, Apr 11, 7:29 AM
Unknown Object (File)
Sun, Apr 7, 7:37 PM
Unknown Object (File)
Sun, Apr 7, 11:49 AM
Unknown Object (File)
Tue, Apr 2, 1:52 AM
Unknown Object (File)
Fri, Mar 29, 10:41 AM
Unknown Object (File)
Mar 24 2024, 7:12 AM
Subscribers

Details

Summary

Ref T7689. Modernize all callsites of these methods.

Test Plan
  • Poked at dashboards.
    • Pretty sure this code is technically unreachable right now.
  • Viewed commit; viewed "Audit Status".
  • Viewed a fund; viewed "Payable to"; viewed "Owner".
  • Viewed herald rules; viewed "Author"; viewed "Applies To".
  • Viewed a Legalpad document; viewed "Contributors".
  • Viewed Phame post list; viewed blog; viewed post (viewed "Blog", viewed "Blogger").
  • Viewed a macro; viewed "Audio".
  • Viewed a Phriction page; viewed "Last Author".
  • Viewed a Ponder question; viewed "Author".
  • Viewed a Ponder answer; viewed header.
    • Behavior changed very slightly here; whatevs.
  • Viewed a Countdown; viewed "Author".

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Remove Controller->getHandle() and Controller->loadHandles().
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:06 PM
This revision was automatically updated to reflect the committed changes.