Trivial fixes from D14467
ClosedPublic
Actions

Authored by epriestley on Dec 24 2015, 12:31 AM.

Details

Reviewers

joshuaspence
chad

Commits

Restricted Diffusion Commit
rPdc5397b2db5d: Trivial fixes from D14467

Summary

See D14467. Just teasing this apart so I can be a little more confident in my commandeering. These are the unambiguous cleanup changes from D14467.

Test Plan

inspection / clicked stuff / no impact

Diff Detail

Repository

rP Phabricator

Branch

projpolicy1

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 9762
Build 11728: Run Core Tests
Build 11727: arc lint + arc unit

Event Timeline

epriestley updated this revision to Diff 35943.Dec 24 2015, 12:31 AM

epriestley retitled this revision from to Trivial fixes from D14467.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added reviewers: joshuaspence, chad.

joshuaspence accepted this revision.Dec 24 2015, 12:32 AM

joshuaspence edited edge metadata.

This revision is now accepted and ready to land.Dec 24 2015, 12:32 AM

Closed by commit rPdc5397b2db5d: Trivial fixes from D14467 (authored by epriestley, committed by epriestley). · Explain WhyDec 24 2015, 1:19 AM

This revision was automatically updated to reflect the committed changes.

epriestley mentioned this in D14467: Add a "members of project" policy rule.Dec 24 2015, 1:21 AM

Revision Contents
Changeset List

Path

Size

src/

applications/

policy/

rule/

PhabricatorPolicyRule.php

19 lines

project/

controller/

PhabricatorProjectProfileController.php

3 lines

Diff 35943

View Options

src/applications/policy/rule/PhabricatorPolicyRule.php