Page MenuHomePhabricator

Move some files around
ClosedPublic

Authored by joshuaspence on Nov 13 2015, 11:12 PM.
Tags
None
Referenced Files
F12854612: D14478.id35059.diff
Fri, Mar 29, 8:46 AM
Unknown Object (File)
Mon, Mar 25, 8:23 PM
Unknown Object (File)
Sat, Mar 23, 3:10 PM
Unknown Object (File)
Sat, Mar 23, 3:09 PM
Unknown Object (File)
Sat, Mar 23, 1:01 PM
Unknown Object (File)
Wed, Mar 6, 5:15 AM
Unknown Object (File)
Wed, Mar 6, 5:14 AM
Unknown Object (File)
Tue, Mar 5, 7:02 PM
Subscribers

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
Restricted Diffusion Commit
rPcf2eb0dd5f92: Move some files around
Summary

Move some PhabricatorPolicyRule implementations to a subdirectory of the parent application.

Test Plan

N/A

Diff Detail

Repository
rP Phabricator
Branch
master
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 8829
Build 10303: Run Core Tests
Build 10302: arc lint + arc unit

Event Timeline

joshuaspence retitled this revision from to Move some files around.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
src/applications/project/policyrule/PhabricatorProjectProjectMembersPolicyRule.php
3

I renamed this class to remove ambiguity for D14467.

epriestley edited edge metadata.

I expect that the rename breaks existing policies (which use class names in the database, and are not migrated). Moves are fine.

This revision now requires changes to proceed.Nov 15 2015, 4:40 PM
joshuaspence edited edge metadata.
joshuaspence marked an inline comment as done.

Leave class name intact (will rename in D14467 instead)

epriestley edited edge metadata.
This revision is now accepted and ready to land.Nov 16 2015, 4:42 PM
This revision was automatically updated to reflect the committed changes.