Page MenuHomePhabricator

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

Authored by btrahan on Feb 3 2015, 7:37 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Jan 1, 2:08 AM
Unknown Object (File)
Dec 20 2024, 5:22 PM
Unknown Object (File)
Dec 20 2024, 5:18 PM
Unknown Object (File)
Dec 20 2024, 3:04 PM
Unknown Object (File)
Nov 28 2024, 8:43 AM
Unknown Object (File)
Nov 14 2024, 2:16 AM
Unknown Object (File)
Nov 11 2024, 6:41 PM
Unknown Object (File)
Oct 31 2024, 9:39 AM
Subscribers

Details

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.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

btrahan retitled this revision from to Policy - move some owners code into an editor class and check policy better.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Feb 3 2015, 7:39 PM
This revision was automatically updated to reflect the committed changes.