HomePhabricator

Policy - move some owners code into an editor class and check policy better

Description

Policy - move some owners code into an editor class and check policy better

Summary: Ref T7094. We basically need to make sure folks can see repositories before making owners packages about code within. This cleans up things a little bit by moving a bunch of logic out of the storage class and into an editor class.

Test Plan: made a package and it worked! deleted a package and it worked! discovered buggy behavior in more complicated edits and filed T7127; note this bug exists before and after this diff.

Reviewers: epriestley

Reviewed By: epriestley

Subscribers: Korvin, epriestley

Maniphest Tasks: T7094

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

Details

Provenance
btrahanAuthored on
btrahanPushed on Feb 3 2015, 7:41 PM
Reviewer
epriestley
Differential Revision
D11652: Policy - move some owners code into an editor class and check policy better
Parents
rPc65b58b21c32: Clean up a ConduitException around Diffusion merges
Branches
Unknown
Tags
Unknown
Tasks
T7094: Clean up T603