Page MenuHomePhabricator

Move ManiphestTaskQuery to EdgeLogic
ClosedPublic

Authored by epriestley on Apr 23 2015, 11:56 AM.
Tags
None
Referenced Files
F13478705: D12526.diff
Fri, Jul 19, 3:49 AM
F13475713: D12526.id30072.diff
Thu, Jul 18, 6:31 AM
F13475704: D12526.id30086.diff
Thu, Jul 18, 6:24 AM
F13475703: D12526.id.diff
Thu, Jul 18, 6:24 AM
F13433920: D12526.diff
Wed, Jul 10, 6:52 PM
F13424732: D12526.diff
Tue, Jul 9, 3:40 AM
F13405676: D12526.diff
Fri, Jul 5, 3:34 PM
F13394372: D12526.diff
Wed, Jul 3, 6:55 AM
Subscribers

Details

Summary

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

Repository
rP Phabricator
Branch
search4
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 5467
Build 5485: [Placeholder Plan] Wait for 30 Seconds

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.
src/applications/home/controller/PhabricatorHomeMainController.php
182–185

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.