Page MenuHomePhabricator

Simplify custom policies before saving, and reject meaningless policies
ClosedPublic

Authored by epriestley on Oct 14 2013, 11:10 PM.
Tags
None
Referenced Files
F13324294: D7314.diff
Fri, Jun 14, 11:32 PM
F13311883: D7314.diff
Tue, Jun 11, 5:44 AM
F13306083: D7314.id.diff
Sun, Jun 9, 7:12 AM
F13306015: D7314.diff
Sun, Jun 9, 6:22 AM
F13300249: D7314.diff
Fri, Jun 7, 8:59 AM
F13284562: D7314.diff
Mon, Jun 3, 6:34 AM
F13266709: D7314.diff
Tue, May 28, 6:11 PM
F13258509: D7314.id16475.diff
Sun, May 26, 9:58 AM
Subscribers

Details

Summary

Ref T603. Do a little more sanity checking on custom policies, so policies like this:

[ Allow ] [ Users ] [ <no users> ]

...that don't specify anything and thus which aren't meaningful raise errors.

Test Plan

{F69570}

Diff Detail

Branch
psimplify
Lint
Lint Passed
Unit
Tests Passed

Event Timeline

epriestley updated this revision to Unknown Object (????).Oct 14 2013, 11:12 PM
  • Save the simplified version of the data.