Page MenuHomePhabricator

Improve handle semantics with HandlePool / HandleList
ClosedPublic

Authored by epriestley on Mar 30 2015, 1:27 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Dec 15, 11:58 AM
Unknown Object (File)
Wed, Dec 4, 4:59 PM
Unknown Object (File)
Tue, Dec 3, 10:19 PM
Unknown Object (File)
Nov 21 2024, 5:42 AM
Unknown Object (File)
Nov 19 2024, 4:43 AM
Unknown Object (File)
Nov 15 2024, 3:02 AM
Unknown Object (File)
Nov 14 2024, 3:40 AM
Unknown Object (File)
Nov 12 2024, 9:34 PM
Subscribers
Tokens
"Grey Medal" token, awarded by chad.

Details

Summary

Ref T7689, which discusses some of the motivation here. Briefly, these methods are awkward:

  • Controller->loadHandles()
  • Controller->loadViewerHandles()
  • Controller->renderHandlesForPHIDs()

This moves us toward better semantics, less awkwardness, and a more reasonable attack on T7688 which won't double-fetch a bunch of data.

Test Plan
  • Added unit tests.
  • Converted one controller to the new stuff.
    • Viewed countdown lists, saw handles render.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Improve handle semantics with HandlePool / HandleList.
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, 7:54 PM
This revision was automatically updated to reflect the committed changes.