From 3d35d6d3f901dec6a34d1cc35179ee3d34225e6d Mon Sep 17 00:00:00 2001 From: epriestley Date: Wed, 22 Mar 2017 05:01:58 -0700 Subject: [PATCH] Remove duplicate "Change Default Values" action in form editing workflow Summary: Fixes T12434. I accidentally copy/pasted this too much in D17442. Test Plan: Viewed a form edit page, no longer saw two copies of this action. Reviewers: chad, cspeckmim Reviewed By: chad, cspeckmim Maniphest Tasks: T12434 Differential Revision: https://secure.phabricator.com/D17530 --- .../PhabricatorEditEngineConfigurationViewController.php | 9 --------- 1 file changed, 9 deletions(-) diff --git a/src/applications/transactions/controller/PhabricatorEditEngineConfigurationViewController.php b/src/applications/transactions/controller/PhabricatorEditEngineConfigurationViewController.php index c07b2d5f3d..80939dd1c8 100644 --- a/src/applications/transactions/controller/PhabricatorEditEngineConfigurationViewController.php +++ b/src/applications/transactions/controller/PhabricatorEditEngineConfigurationViewController.php @@ -163,15 +163,6 @@ final class PhabricatorEditEngineConfigurationViewController ->setDisabled(!$can_edit)); } - $curtain->addAction( - id(new PhabricatorActionView()) - ->setName(pht('Change Default Values')) - ->setIcon('fa-paint-brush') - ->setHref($defaults_uri) - ->setWorkflow(!$can_edit) - ->setDisabled(!$can_edit)); - - $disable_uri = "{$base_uri}/disable/{$form_key}/"; if ($config->getIsDisabled()) {