Page MenuHomePhabricator

Move ManiphestTaskQuery to EdgeLogic

Authored by epriestley on Apr 23 2015, 11:56 AM.
Referenced Files
F11626927: D12526.diff
Wed, Mar 22, 1:05 PM
Unknown Object (File)
Thu, Mar 9, 8:24 PM
Unknown Object (File)
Feb 5 2023, 4:38 AM
Unknown Object (File)
Feb 3 2023, 10:29 AM
Unknown Object (File)
Feb 2 2023, 3:51 AM
Unknown Object (File)
Jan 16 2023, 10:15 PM
Unknown Object (File)
Dec 29 2022, 4:33 AM
Unknown Object (File)
Dec 28 2022, 3:35 AM



Ref T4100. Share all edge logic code across applications.

  • Internalizes the "check that the viewer can see projects" check into edge logic.
  • Adds some convenience functions. Some of these aren't really all that convenient, but it's rare that we actually apply project constraints to queries in the applications -- and most of these callsites will go away in the long term -- so I didn't go too crazy with providing a simpler withProjectPHIDs() universal API or anything.
Test Plan
  • Grepped for all affected symbols.
  • Tried to violate policies.
  • Used workboards.
  • Used normal Maniphest queries.
  • Used maniphest.query.
  • Verified the special grouping behavior works as expected.

Diff Detail

rP Phabricator
Lint Not Applicable
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Move ManiphestTaskQuery to EdgeLogic.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.

In particular, this is a little bit cumbersome compared to the old API, but we only have about 6 callsites and half of them will vanish eventually, so it didn't seem worthwhile to define some convenience wrapper method to specify the project edge constant and/or operator for you.

btrahan edited edge metadata.
This revision is now accepted and ready to land.Apr 23 2015, 5:23 PM
This revision was automatically updated to reflect the committed changes.