From fa75a03b725f6db69e1c41d812772ecff0777178 Mon Sep 17 00:00:00 2001 From: vrana Date: Wed, 24 Oct 2012 12:42:18 -0700 Subject: [PATCH] Fix didAbortWorkflow() without workflow Summary: This is a BC break but it was introduced recently. Test Plan: $ arc unit x No more: > Fatal error: Argument 2 passed to ArcanistConfiguration::didAbortWorkflow() must be an instance of ArcanistBaseWorkflow, null given Reviewers: epriestley Reviewed By: epriestley CC: aran, Korvin Differential Revision: https://secure.phabricator.com/D3812 --- src/configuration/ArcanistConfiguration.php | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/src/configuration/ArcanistConfiguration.php b/src/configuration/ArcanistConfiguration.php index f8af6909..ea781630 100644 --- a/src/configuration/ArcanistConfiguration.php +++ b/src/configuration/ArcanistConfiguration.php @@ -93,11 +93,7 @@ class ArcanistConfiguration { // This is a hook. } - public function didAbortWorkflow( - $command, - ArcanistBaseWorkflow $workflow, - Exception $ex) { - + public function didAbortWorkflow($command, $workflow, Exception $ex) { // This is a hook. }