diff options
author | Morris Jobke <hey@morrisjobke.de> | 2015-06-23 18:40:55 +0300 |
---|---|---|
committer | Morris Jobke <hey@morrisjobke.de> | 2015-06-23 18:40:55 +0300 |
commit | 02b2b605810adcf8862a6b7548e1ed30bc56a2f6 (patch) | |
tree | bc485f1d99864fc078f140da19613276c8a7266d /lib/private/updater.php | |
parent | e01db549a04136f0ddee11245aa3a1b4e6381cca (diff) | |
parent | c86e129ce90c65de92172676b3798a8b48589b58 (diff) |
Merge pull request #17095 from owncloud/proper-error-handling
Proper error handling
Diffstat (limited to 'lib/private/updater.php')
-rw-r--r-- | lib/private/updater.php | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/lib/private/updater.php b/lib/private/updater.php index bd9e8a65363..00c6569a52f 100644 --- a/lib/private/updater.php +++ b/lib/private/updater.php @@ -189,20 +189,25 @@ class Updater extends BasicEmitter { $this->log->debug('starting upgrade from ' . $installedVersion . ' to ' . $currentVersion, array('app' => 'core')); } + $success = true; try { $this->doUpgrade($currentVersion, $installedVersion); } catch (\Exception $exception) { - $this->emit('\OC\Updater', 'failure', array($exception->getMessage())); + \OCP\Util::logException('update', $exception); + $this->emit('\OC\Updater', 'failure', array(get_class($exception) . ': ' .$exception->getMessage())); + $success = false; } - $this->emit('\OC\Updater', 'updateEnd'); + $this->emit('\OC\Updater', 'updateEnd', array($success)); - if(!$wasMaintenanceModeEnabled) { + if(!$wasMaintenanceModeEnabled && $success) { $this->config->setSystemValue('maintenance', false); $this->emit('\OC\Updater', 'maintenanceDisabled'); } else { $this->emit('\OC\Updater', 'maintenanceActive'); } + + return $success; } /** |