HomePhabricator

When publishing project transactions, load ancestor members

Description

When publishing project transactions, load ancestor members

Summary: Ref T10010. Fixes T10107. When we publish a transaction about a project, we perform visibility checks for many different users. We need to know all of the ancestors' members to perform these checks.

Test Plan:

  • Before patch: when updating a subproject, daemons fatal trying to publish things because they can not test visibility of parent projects.
  • After patch: daemons successfully publish subproject updates.
  • Also added a unit test.

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T10010, T10107

Differential Revision: https://secure.phabricator.com/D15054