Page MenuHomePhabricator

Move handle fetching into tokenizer Datasources
ClosedPublic

Authored by epriestley on Mar 31 2015, 1:47 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 20, 11:10 AM
Unknown Object (File)
Fri, Dec 20, 10:32 AM
Unknown Object (File)
Fri, Dec 20, 4:19 AM
Unknown Object (File)
Wed, Dec 4, 6:34 PM
Unknown Object (File)
Sat, Nov 30, 1:34 AM
Unknown Object (File)
Oct 20 2024, 3:30 PM
Unknown Object (File)
Oct 18 2024, 11:44 AM
Unknown Object (File)
Sep 30 2024, 10:45 PM
Subscribers

Details

Summary

Ref T7689. This serves two goals:

  • I want to remove Controller->loadViewerHandles(). A nontrivial number of these callsites are loading handles to pass to tokenizers. Since tokenizers need to take strings eventually anyway, we can do less work by letting them take PHIDs now.
  • A few changes out, I want tokenizers to accept parameterized tokens (like viewer(), members(differential), etc.), so the setValues() signature needs to change eventually anyway.

I made this work and converted a handful of callsites as an example; upcoming changes will convert more.

Test Plan
  • Viewed Almanac binding editor; used "Interface".
  • Edited Almanac services; used "Projects".
  • Edited Almanac devices; used "Projects".
  • Searched for commits; used "Auditors"; "Commit Authors", "Repositories".
  • Searched for calendar events; used "Created By"; used "Invited".

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Move handle fetching into tokenizer Datasources.
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 31 2015, 8:44 PM
This revision was automatically updated to reflect the committed changes.