Allow customizing default Phurl view and edit policies
AcceptedPublic
Actions

Authored by taavi on Sun, Dec 8, 10:37.

Details

Reviewers

aklapper
valerio.bozzolan

Group Reviewers

O1: Blessed Committers

Maniphest Tasks

T15970: phurl: Allow setting default edit policy for URLs

Summary

Before	After

Closes T15970

Test Plan

Go to http://phorge.localhost/applications/view/PhabricatorPhurlApplication/ and see two additional settings "Default View Policy" and "Default Edit Policy"
Go to http://phorge.localhost/applications/view/PhabricatorPhurlApplication/ and select "Edit Policies" to see and set two additional settings
Go to http://phorge.localhost/phurl/url/edit/form/default/ and check the "Visible To" and "Editable By" fields now showing the default page
Check existing Phurls which did not have their policies changed as that's stored in the database
run arc liberate again and see that it's already updated

Repository

rP Phorge

Branch

phurl-policy

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 1657
Build 1657: arc lint + arc unit

taavi created this revision.Sun, Dec 8, 10:37

taavi requested review of this revision.Sun, Dec 8, 10:37

Fix my local unit test config

taavi added inline comments.Sun, Dec 8, 10:42

src/applications/phurl/application/PhabricatorPhurlApplication.php
83	Note that this is different than the previous default (which allowed the URL creator only). I didn't find a way to make the policy default include the URL author, and the author will continue to have access since the URL class uses `hasAutomaticCapability` to always grant the author view/edit access.

Uuuh! Thanks <3

What happens to already-existing URLs? Maybe nice to mention in the test plan

In D25850#22726, @valerio.bozzolan wrote:

What happens to already-existing URLs? Maybe nice to mention in the test plan

The policies for those are saved in the database, so they stay as is. Done.

src/applications/phurl/application/PhabricatorPhurlApplication.php
83	I tested and this constant seems to do not exist https://we.phorge.it/source/phorge/browse/master/src/applications/policy/capability/PhabricatorPolicyCapability.php Maybe CAN_ADMIN?