From 61873e4e218c4dcde8bb13d35c0bfec8a94267ed Mon Sep 17 00:00:00 2001 From: =?utf8?q?J=C3=B6rn=20Friedrich=20Dreyer?= Date: Wed, 8 Oct 2014 18:02:42 +0200 Subject: [PATCH] throw exception in writeBack, the returned boolean is checked nowhere --- lib/private/files/objectstore/objectstorestorage.php | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/lib/private/files/objectstore/objectstorestorage.php b/lib/private/files/objectstore/objectstorestorage.php index 241864bcccd..ae8bff52896 100644 --- a/lib/private/files/objectstore/objectstorestorage.php +++ b/lib/private/files/objectstore/objectstorestorage.php @@ -349,7 +349,7 @@ class ObjectStoreStorage extends \OC\Files\Storage\Common { public function writeBack($tmpFile) { if (!isset(self::$tmpFiles[$tmpFile])) { - return false; + return; } $path = self::$tmpFiles[$tmpFile]; @@ -375,9 +375,8 @@ class ObjectStoreStorage extends \OC\Files\Storage\Common { } catch (\Exception $ex) { $this->getCache()->remove($path); \OCP\Util::writeLog('objectstore', 'Could not create object: ' . $ex->getMessage(), \OCP\Util::ERROR); - return false; + throw $ex; // make this bubble up } - return true; } /** -- 2.39.5