Page MenuHomePhabricator
Differential D10890

Add default View, Edit, and Join Policies to Projects
ClosedPublic
Actions

Authored by chad on Nov 21 2014, 5:07 AM.

Details

Reviewers
epriestley
btrahan
Maniphest Tasks
T6567: Add ability to set default Project Policy
Commits
Restricted Diffusion Commit
rP32b1ffdf894c: Add default View, Edit, and Join Policies to Projects
Summary

Fixes T6567, lets admins set a default policy for new Projects.

Test Plan

Changed the default Policy, created a new Project. Project had correct defaults.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad updated this revision to Diff 26155.Nov 21 2014, 5:07 AM
chad retitled this revision from to Add default View, Edit, and Join Policies to Projects.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added reviewers: epriestley, btrahan.
chad added a task: T6567: Add ability to set default Project Policy.
Herald added subscribers: epriestley, Korvin. · View Herald TranscriptNov 21 2014, 5:07 AM
Harbormaster completed remote builds in B3120: Diff 26155.Nov 21 2014, 5:08 AM
btrahan awarded a token.Nov 21 2014, 6:37 AM
Harbormaster completed remote builds in B3120: Diff 26155.Nov 21 2014, 2:20 PM
epriestley accepted this revision.Nov 21 2014, 7:07 PM
epriestley edited edge metadata.
epriestley added inline comments.
src/applications/project/capability/ProjectDefaultEditCapability.php
13–15

You should be able to just omit this; the default is false.

This revision is now accepted and ready to land.Nov 21 2014, 7:07 PM
chad updated this revision to Diff 26160.Nov 21 2014, 7:20 PM
chad edited edge metadata.
  • Remove unneeded functions
Harbormaster completed remote builds in B3122: Diff 26160.Nov 21 2014, 7:21 PM
Closed by commit rP32b1ffdf894c: Add default View, Edit, and Join Policies to Projects (authored by chad, committed by chad). · Explain WhyNov 21 2014, 7:22 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 26161

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/project/application/PhabricatorProjectApplication.php

Loading...
View Options

src/applications/project/capability/ProjectDefaultEditCapability.php

Loading...
View Options

src/applications/project/capability/ProjectDefaultJoinCapability.php

Loading...
View Options

src/applications/project/capability/ProjectDefaultViewCapability.php

Loading...
View Options

src/applications/project/storage/PhabricatorProject.php

Loading...
Phabricator · Built by Phacility