From 66220d0e586b398515ca39e33b50430461fece31 Mon Sep 17 00:00:00 2001 From: NateC Date: Fri, 15 Apr 2016 16:49:10 -0500 Subject: [PATCH] 292: Notification email reports SUCCESS even if deployment failed Fixing the bug reported in #292 --- Mage/Command/BuiltIn/DeployCommand.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Mage/Command/BuiltIn/DeployCommand.php b/Mage/Command/BuiltIn/DeployCommand.php index 8dd6c9d..e03051b 100644 --- a/Mage/Command/BuiltIn/DeployCommand.php +++ b/Mage/Command/BuiltIn/DeployCommand.php @@ -215,7 +215,7 @@ class DeployCommand extends AbstractCommand implements RequiresEnvironment Console::output('Total time: ' . $timeText . '.', 1, 2); // Send Notifications - $this->sendNotification(self::$failedTasks > 0 ? false : true); + $this->sendNotification(self::$deployStatus === self::SUCCEDED); // Unlock if (file_exists(getcwd() . '/.mage/~working.lock')) {