HomePhabricator

Rename `PhabricatorPolicyRule` subclasses for consistency

Authored by joshuaspence.

Description

Rename PhabricatorPolicyRule subclasses for consistency

Summary: Ref T5655.

Test Plan: arc lint and arc unit

Reviewers: Blessed Reviewers, epriestley

Reviewed By: Blessed Reviewers, epriestley

Subscribers: Korvin, epriestley

Maniphest Tasks: T5655

Differential Revision: https://secure.phabricator.com/D11131

Details

Event Timeline

fabe raised a concern with this commit.Jan 2 2015, 1:28 PM
fabe added a subscriber: fabe.

Ref T6849

I think these classes need to be renamed in existing phabricator_policy.policy rules as well (it states the full class name in the DB).

I'm just going to revert this for now since T6849 is severe. We can apply it again with a migration that adjusts the data.

fabe accepted this commit.Jan 8 2015, 11:46 AM