diff options
| author | Julius Haertl <jus@bitgrid.net> | 2016-12-09 12:44:01 +0100 |
|---|---|---|
| committer | Julius Haertl <jus@bitgrid.net> | 2016-12-09 12:54:03 +0100 |
| commit | 5fbadee95ff11429739dc7cf818d93d0c549a73f (patch) | |
| tree | d9d5ff9ae970d665899c32c1bea4ada7f0426faa | |
| parent | e3b2832ee192a9cc961342420efe6ec077638461 (diff) | |
| download | nextcloud-server-5fbadee95ff11429739dc7cf818d93d0c549a73f.tar.gz nextcloud-server-5fbadee95ff11429739dc7cf818d93d0c549a73f.zip | |
Check if image files for theming are empty
Signed-off-by: Julius Haertl <jus@bitgrid.net>
| -rw-r--r-- | apps/theming/lib/Controller/IconController.php | 3 | ||||
| -rw-r--r-- | apps/theming/lib/IconBuilder.php | 2 |
2 files changed, 4 insertions, 1 deletions
diff --git a/apps/theming/lib/Controller/IconController.php b/apps/theming/lib/Controller/IconController.php index 4c25d911e5e..7c4e209d0df 100644 --- a/apps/theming/lib/Controller/IconController.php +++ b/apps/theming/lib/Controller/IconController.php @@ -94,6 +94,9 @@ class IconController extends Controller { $iconFile = $this->imageManager->getCachedImage("icon-" . $app . '-' . str_replace("/","_",$image)); } catch (NotFoundException $exception) { $icon = $this->iconBuilder->colorSvg($app, $image); + if ($icon === false || $icon === "") { + return new NotFoundResponse(); + } $iconFile = $this->imageManager->setCachedImage("icon-" . $app . '-' . str_replace("/","_",$image), $icon); } if ($iconFile !== false) { diff --git a/apps/theming/lib/IconBuilder.php b/apps/theming/lib/IconBuilder.php index 9cf0d051304..205ee3a89a0 100644 --- a/apps/theming/lib/IconBuilder.php +++ b/apps/theming/lib/IconBuilder.php @@ -172,7 +172,7 @@ class IconBuilder { return false; } $svg = file_get_contents($imageFile); - if ($svg !== false) { + if ($svg !== false && $svg !== "") { $color = $this->util->elementColor($this->themingDefaults->getMailHeaderColor()); $svg = $this->util->colorizeSvg($svg, $color); return $svg; |