Page MenuHomePhabricator

Fix `packages(project)` to work properly and add it to "MailableFunctionDatasource"
ClosedPublic

Authored by epriestley on Oct 19 2018, 11:46 AM.
Tags
None
Referenced Files
F14388467: D19747.id47178.diff
Sat, Dec 21, 4:42 PM
Unknown Object (File)
Sat, Dec 14, 5:23 PM
Unknown Object (File)
Thu, Dec 12, 10:43 PM
Unknown Object (File)
Thu, Dec 12, 10:43 PM
Unknown Object (File)
Thu, Dec 12, 10:43 PM
Unknown Object (File)
Thu, Dec 12, 10:34 PM
Unknown Object (File)
Mon, Dec 9, 10:41 PM
Unknown Object (File)
Mon, Dec 9, 2:40 PM
Subscribers
None

Details

Summary

Ref T13210. See PHI937. This function datasource isn't quite implemented correctly: it doesn't resolve package(project) properly, since the logic only handles users.

This blames back to D14013, where it looks like packages(..) was added mostly as a general nice-to-have as part of a larger modernization change.

Test Plan

Ran a packages(project) query in Differential, got accurate results (previously: no results).

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable