Merge pull request #293 from natechicago/292-bugfix

292: Notification email reports SUCCESS even if deployment failed
andres-montanez · Jan 2, 2017 · 62c575f · 62c575f
2 parents 35d61cc + 66220d0
commit 62c575f
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/Mage/Command/BuiltIn/DeployCommand.php b/Mage/Command/BuiltIn/DeployCommand.php
@@ -215,7 +215,7 @@ public function run()
         Console::output('Total time: <bold>' . $timeText . '</bold>.', 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')) {