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
F13085960: D11652.diff
Thu, Apr 25, 12:00 AM
Unknown Object (File)
Sun, Apr 14, 2:42 PM
Unknown Object (File)
Thu, Apr 11, 8:17 AM
Unknown Object (File)
Mon, Apr 8, 8:38 PM
Unknown Object (File)
Mar 26 2024, 4:10 AM
Unknown Object (File)
Mar 26 2024, 4:10 AM
Unknown Object (File)
Mar 5 2024, 1:16 AM
Unknown Object (File)
Feb 14 2024, 11:16 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.