diff --git a/Mage/Task/BuiltIn/Releases/RollbackTask.php b/Mage/Task/BuiltIn/Releases/RollbackTask.php index febafa4..7f54bc1 100644 --- a/Mage/Task/BuiltIn/Releases/RollbackTask.php +++ b/Mage/Task/BuiltIn/Releases/RollbackTask.php @@ -13,7 +13,7 @@ namespace Mage\Task\BuiltIn\Releases; use Mage\Console; use Mage\Task\Factory; use Mage\Task\AbstractTask; -use Mage\Task\Releases\BuiltIn as ReleaseTask; +use Mage\Task\Releases\IsReleaseAware; use Mage\Task\Releases\RollbackAware; use Exception; @@ -23,7 +23,7 @@ use Exception; * * @author Andrés Montañez */ -class RollbackTask extends AbstractTask implements ReleaseTask +class RollbackTask extends AbstractTask implements IsReleaseAware { /** * The Relase ID to Rollback To @@ -110,30 +110,24 @@ class RollbackTask extends AbstractTask implements ReleaseTask $completedTasks = 0; $tasksToRun = $this->getConfig()->getTasks(); $this->getConfig()->setReleaseId($releaseId); - - if (count($tasksToRun) == 0) { - Console::output('Warning! No Deployment tasks defined.', 2); - Console::output('Deployment to ' . $this->getConfig()->getHost() . ' skipped!', 1, 3); - } else { - foreach ($tasksToRun as $taskData) { - $task = Factory::get($taskData, $this->getConfig(), true, 'deploy'); - $task->init(); - Console::output('Running ' . $task->getName() . ' ... ', 2, false); + foreach ($tasksToRun as $taskData) { + $task = Factory::get($taskData, $this->getConfig(), true, 'deploy'); + $task->init(); + Console::output('Running ' . $task->getName() . ' ... ', 2, false); - if ($task instanceOf RollbackAware) { - $tasks++; - $result = $task->run(); + if ($task instanceOf RollbackAware) { + $tasks++; + $result = $task->run(); - if ($result == true) { - Console::output('OK', 0); - $completedTasks++; - } else { - Console::output('FAIL', 0); - } - } else { - Console::output('SKIPPED', 0); - } + if ($result == true) { + Console::output('OK', 0); + $completedTasks++; + } else { + Console::output('FAIL', 0); + } + } else { + Console::output('SKIPPED', 0); } }