Page MenuHomePhabricator

Add a "template" parameter to application default policies
ClosedPublic

Authored by epriestley on Jun 11 2015, 7:08 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Dec 8, 4:45 AM
Unknown Object (File)
Thu, Nov 28, 5:32 AM
Unknown Object (File)
Thu, Nov 28, 5:32 AM
Unknown Object (File)
Thu, Nov 28, 5:31 AM
Unknown Object (File)
Thu, Nov 28, 5:11 AM
Unknown Object (File)
Tue, Nov 26, 1:50 AM
Unknown Object (File)
Mon, Nov 25, 11:40 AM
Unknown Object (File)
Thu, Nov 21, 2:34 AM
Subscribers

Details

Summary

Ref T5681. Ref T6860. This doesn't do anything interesting on its own, just makes the next diff smaller.

In the next diff, policies become aware of the types of objects they're acting on. We need to specify which object type all the "Default View/Edit" settings are for so they get the right rules.

For example, a rule like "Allow task author" is OK for "View Policy" on a task, and also OK for "Default View Policy" on ManiphestApplication. But it's not OK for "Can Create Tasks" on ManiphestApplication.

So annotate all the "template"/"default" policies with their types. The next diff will use these to let you select appropriate rules for the given object type.

Test Plan
  • Used grep to find these.
  • This change has no effect.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable