Add missing needReviewerAuthority() for differential.createcomment
ClosedPublic
Actions

Authored by epriestley on Mar 12 2014, 7:40 PM.

Details

Reviewers

btrahan
dctrwatson

Commits

Restricted Diffusion Commit
rP4173367f8765: Add missing needReviewerAuthority() for differential.createcomment

Summary

For some actions (like accept) we need to load reviewer authority so we can figure out if the actor can act on behalf of project reviewers, etc.

Test Plan

Will make @dctrwatson do it.

Diff Detail

Repository

rP Phabricator

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

epriestley updated this revision to Diff 20180.Mar 12 2014, 7:40 PM

epriestley retitled this revision from to Add missing needReviewerAuthority() for differential.createcomment.

epriestley updated this object.

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

epriestley added reviewers: btrahan, dctrwatson.

epriestley added a subscriber: dctrwatson.

Herald added subscribers: epriestley, aran.Mar 12 2014, 7:40 PM

Also handle the oldschool 'none' action.
Also remove some dead code.

happycat

awesometown

btrahan accepted this revision.Mar 12 2014, 9:48 PM

btrahan edited edge metadata.

This revision is now accepted and ready to land.Mar 12 2014, 9:48 PM

Closed by commit rP4173367f8765 (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

applications/

differential/

conduit/

ConduitAPI_differential_createcomment_Method.php

3 lines

constants/

DifferentialAction.php

30 lines

Diff 20188

View Options

src/applications/differential/conduit/ConduitAPI_differential_createcomment_Method.php