Home GnuPG

Fix expansion of projects into lists of user PHIDs
d1eed54d85bcUnpublished

Unpublished Commit ยท Learn More

Repository Importing: This repository is still importing.

Description

Fix expansion of projects into lists of user PHIDs

Summary:
Ref T11016. I think I inverted the meaning of this function by accident in D14893.

The intent is to return a list of users: direct users, and all members of all projects.

Prior to this patch actually returns direct users, and all projects they are members of.

Test Plan:

  • Created "Project with Dog".
  • Added user "dog" to project.
  • Created package "X", owning file "/x", with audit enabled.
  • Made "X" owned by "Project with Dog".
  • Modified "/x" and had user "dog" accept it.
  • Landed change.
  • Prior to change: package "X" incorrectly added as auditor.
  • After change: package "X" correctly omitted as auditor, because a member reviewed the change.

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T11016

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

Details

Provenance
epriestley <git@epriestley.com>Authored on May 24 2016, 3:05 PM
Parents
rPHAB74e117ae41dd: Don't send mail to "uninteresting" auditors
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHABd1eed54d85bc: Fix expansion of projects into lists of user PHIDs (authored by epriestley <git@epriestley.com>).May 24 2016, 3:38 PM