Page MenuHomePhabricator

Convert all tokenizers to take token/scalar inputs
ClosedPublic

Authored by epriestley on Mar 31 2015, 8:20 PM.
Tags
None
Referenced Files
F14054414: D12224.diff
Sat, Nov 16, 1:39 AM
F14043468: D12224.diff
Tue, Nov 12, 10:26 AM
F14025426: D12224.diff
Thu, Nov 7, 5:46 PM
F14009822: D12224.diff
Wed, Oct 30, 11:52 PM
F13997293: D12224.diff
Thu, Oct 24, 3:29 AM
F13968089: D12224.diff
Oct 16 2024, 6:29 PM
Unknown Object (File)
Oct 6 2024, 9:34 AM
Unknown Object (File)
Oct 3 2024, 10:02 AM
Subscribers

Details

Summary

Ref T7689. Ref T4100. This advances the goals of removing loadViewerHandles() (only 67 callsites remain!) and letting tokenizers some day take token functions like viewer() and members(differential).

Test Plan
  • Sent a new message; used "To".
    • I simplified the cancel URI construction slightly because it's moot in all normal cases.
  • Edited a thread; used "Add Participants".
  • Searched rooms; used "Participants".
  • Searched countdowns; used "Authors".
  • Created a diff; used "Repository".
  • Edited a revision; edited "Projects"; edited "Reveiwers"; edited "Subscribers".
  • Searched for revisions; edited "responsible users"; "authors"; "reviwers"; "subscribers"; "repositories".
  • Added revision comments; edited "Add Reveiwers"; "Add Subscribers".
  • Commented on a commit; edited "Add Auditors"; "Add subscribers".
  • Edited a commit; edited "Projects".
  • Edited a repository; edited "Projects".
  • Searched feed, used "include Users"; "include Proejcts".
  • Searched files, used "authors".
  • Edited initiative; edited "Projects".
  • Searched backers; used "Backers".
  • Searched initiatives; used "Owners".
  • Edited build plans; edited "Run Command".
  • Searched Herald; used "Authors".
  • Added signature exemption in Legalpad.
  • Searhced legalpad; used "creators"; used "contributors".
  • Searched signatures; used "documents"; used "signers".
  • Created meme.
  • Searched macros; used "Authors".
  • Used "Projects" in Maniphest reports.
  • Used Maniphest comment actions.
  • Edited Maniphest tasks; edited "Assigned To"; edited "CC"; edited "projects".
  • Used "parent" in Maniphest task creation workflow.
  • Searched for projects; used "assigned to"; "in any projec"; "in all projects"; "not in projects"; "in users' projects"; "authors"; "subscribers".
  • Edited Maniphest bug filing domains, used "Default Author".
  • Searched for OAuth applications, used "Creators".
  • Edited Owners pacakge; edited "Primary Owner"; edited "Owners".
  • Searched for Owners packages; used "Owner".
    • OMG this UI is OLD
  • Edited a paste; edited "Projects".
  • Searched for paste; used "Authors".
  • Searched user activity log; used "Actors"; used "Users".
  • Edited a mock; edited "Projects"; edited "CC".
  • Searched for mocks; used "Authors".
  • Edited Phortune account; edited "Members".
  • Edited Phortune merchant account; edited "Members".
  • Searched Phrequent; used "Users".
  • Edited Ponder question; sued "projects".
  • Searched Ponder; used "Authors"; used "Answered By".
  • Added project members.
  • Searched for projects; used "Members".
  • Edited a Releeph product; edited "Pushers".
  • Searched pull requests; searched "Requestors".
  • Edited an arcanist project; used "Uses Symbols From".
  • Searhced push logs; used "Repositories"; used "Pushers".
  • Searched repositories; used "In nay project".
  • Used global search; used Authors/owners/Subscribers/In Any Project.
  • Edited a slowvote; used "Projects".
  • Searched slovotes; used "Authors".
  • Created a custom "Users" field; edited and searched for it.
  • Made a whole lot of typos in this list. ^^^^^^

Did not test:

  • Lint is nontrivial to test locally, I'll test it in production.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Convert all tokenizers to take token/scalar inputs.
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:43 PM
This revision was automatically updated to reflect the committed changes.