From 5bb646a838bfdb3db0d4386a4559b79ee2ef6df2 Mon Sep 17 00:00:00 2001 From: epriestley Date: Sat, 9 Nov 2013 16:34:26 -0800 Subject: [PATCH] Fix incorrect check for CAN_EDIT in macro enable/disable controller 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 --- .../macro/controller/PhabricatorMacroDisableController.php | 5 ----- 1 file changed, 5 deletions(-) diff --git a/src/applications/macro/controller/PhabricatorMacroDisableController.php b/src/applications/macro/controller/PhabricatorMacroDisableController.php index 5b4a4f8d5e..293911f50f 100644 --- a/src/applications/macro/controller/PhabricatorMacroDisableController.php +++ b/src/applications/macro/controller/PhabricatorMacroDisableController.php @@ -19,11 +19,6 @@ final class PhabricatorMacroDisableController $macro = id(new PhabricatorMacroQuery()) ->setViewer($user) - ->requireCapabilities( - array( - PhabricatorPolicyCapability::CAN_VIEW, - PhabricatorPolicyCapability::CAN_EDIT, - )) ->withIDs(array($this->id)) ->executeOne(); if (!$macro) {