Page MenuHomePhabricator

Add "parent" and "ancestor" information to the project.search API
ClosedPublic

Authored by epriestley on Jan 8 2017, 7:48 PM.
Tags
None
Referenced Files
F13092276: D17154.diff
Thu, Apr 25, 3:34 AM
Unknown Object (File)
Fri, Apr 19, 2:32 AM
Unknown Object (File)
Fri, Apr 19, 2:32 AM
Unknown Object (File)
Fri, Apr 19, 2:31 AM
Unknown Object (File)
Thu, Apr 11, 10:49 AM
Unknown Object (File)
Thu, Apr 11, 3:52 AM
Unknown Object (File)
Sun, Apr 7, 10:21 AM
Unknown Object (File)
Sat, Apr 6, 2:49 AM
Subscribers
None

Details

Summary

Ref T12074.

  • Adds a new "parent" property on main results. This shows an abbreviated version of the project's parent, or null if the project is a root project.
  • Adds a new "ancestor" attachment to pull the entire ancestor list.
  • Adds a new "depth" property on main results.
  • You can use "parent" or "depth" to tell if a project is a subproject or not.

These attempt to balance convenience, power, and performance: the full ancestor list can be big so I made it an attachment, but the other stuff isn't too big and is cheap and seems reasonable to always include.

Test Plan

In API results:

  • Saw null parent (root projects) and non-null parent (subprojects/milestones).
  • Used "ancestors" attchment, got full list of ancestors.
  • Saw appropriate "depth" values.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Jan 8 2017, 8:13 PM
This revision was automatically updated to reflect the committed changes.