Merge pull request #8580 from IonBazan/bugfix/install-notifications
respect `notify-on-install` optionpull/8590/head
commit
32e2e09c89
|
@ -233,13 +233,13 @@ class Installer
|
||||||
return $res;
|
return $res;
|
||||||
}
|
}
|
||||||
} catch (\Exception $e) {
|
} catch (\Exception $e) {
|
||||||
if ($this->executeOperations) {
|
if ($this->executeOperations && $this->config->get('notify-on-install')) {
|
||||||
$this->installationManager->notifyInstalls($this->io);
|
$this->installationManager->notifyInstalls($this->io);
|
||||||
}
|
}
|
||||||
|
|
||||||
throw $e;
|
throw $e;
|
||||||
}
|
}
|
||||||
if ($this->executeOperations) {
|
if ($this->executeOperations && $this->config->get('notify-on-install')) {
|
||||||
$this->installationManager->notifyInstalls($this->io);
|
$this->installationManager->notifyInstalls($this->io);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue