From dd93532526f2de7f6c7ae1a2a7b76b0451be1844 Mon Sep 17 00:00:00 2001 From: Morris Jobke Date: Wed, 4 Nov 2020 10:08:36 +0100 Subject: [PATCH] Cleanup ImageManager::getImage() Signed-off-by: Morris Jobke --- apps/theming/lib/ImageManager.php | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/apps/theming/lib/ImageManager.php b/apps/theming/lib/ImageManager.php index 7f9b410ff9..469480d1e1 100644 --- a/apps/theming/lib/ImageManager.php +++ b/apps/theming/lib/ImageManager.php @@ -101,7 +101,6 @@ class ImageManager { * @throws NotPermittedException */ public function getImage(string $key, bool $useSvg = true): ISimpleFile { - $pngFile = null; $logo = $this->config->getAppValue('theming', $key . 'Mime', false); $folder = $this->appData->getFolder('images'); if ($logo === false || !$folder->fileExists($key)) { @@ -116,17 +115,14 @@ class ImageManager { $finalIconFile->setImageFormat('png32'); $pngFile = $folder->newFile($key . '.png'); $pngFile->putContent($finalIconFile->getImageBlob()); + return $pngFile; } catch (\ImagickException $e) { $this->logger->info('The image was requested to be no SVG file, but converting it to PNG failed: ' . $e->getMessage()); - $pngFile = null; } } else { - $pngFile = $folder->getFile($key . '.png'); + return $folder->getFile($key . '.png'); } } - if ($pngFile !== null) { - return $pngFile; - } return $folder->getFile($key); }