Browse Source

292: Notification email reports SUCCESS even if deployment failed

Fixing the bug reported in #292
1.0
NateC 9 years ago
parent
commit
66220d0e58
  1. 2
      Mage/Command/BuiltIn/DeployCommand.php

2
Mage/Command/BuiltIn/DeployCommand.php

@ -215,7 +215,7 @@ class DeployCommand extends AbstractCommand implements RequiresEnvironment
Console::output('Total time: <bold>' . $timeText . '</bold>.', 1, 2); Console::output('Total time: <bold>' . $timeText . '</bold>.', 1, 2);
// Send Notifications // Send Notifications
$this->sendNotification(self::$failedTasks > 0 ? false : true); $this->sendNotification(self::$deployStatus === self::SUCCEDED);
// Unlock // Unlock
if (file_exists(getcwd() . '/.mage/~working.lock')) { if (file_exists(getcwd() . '/.mage/~working.lock')) {

Loading…
Cancel
Save