revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-12-20 20:40:56 +01:00
Code Issues Releases Wiki Activity

Fix incorrect check for CAN_EDIT in macro enable/disable controller

Browse source 
Summary: This CAN_EDIT capability doesn't exist. `PhabricatorMacroCapabilityManage::CAPABILITY` (checked on line 15) is used instead.

Test Plan: Disabled, then re-enabled a macro.

Reviewers: hach-que, btrahan

Reviewed By: hach-que

CC: aran

Differential Revision: https://secure.phabricator.com/D7550
This commit is contained in:
epriestley 2013-11-09 16:34:26 -08:00
parent 448d8684e8
commit 5bb646a838
1 changed files with 0 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
src/applications/macro/controller/PhabricatorMacroDisableController.php
View file

@ -19,11 +19,6 @@ final class PhabricatorMacroDisableController
$macro = id(new PhabricatorMacroQuery()) $macro = id(new PhabricatorMacroQuery())
->setViewer($user) ->setViewer($user)
->requireCapabilities(
array(
PhabricatorPolicyCapability::CAN_VIEW,
PhabricatorPolicyCapability::CAN_EDIT,
))
->withIDs(array($this->id)) ->withIDs(array($this->id))
->executeOne(); ->executeOne();
if (!$macro) { if (!$macro) {