From a0c0918ce24f7bb7ea3fc58b0ca0f677ab0c3384 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Julius=20H=C3=A4rtl?= <jus@bitgrid.net>
Date: Fri, 20 Nov 2020 19:28:18 +0100
Subject: [PATCH] Optimize chek if background is themed
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: Julius Härtl <jus@bitgrid.net>
---
 apps/theming/lib/Util.php       |  9 +--------
 apps/theming/tests/UtilTest.php | 23 +++++------------------
 2 files changed, 6 insertions(+), 26 deletions(-)

diff --git a/apps/theming/lib/Util.php b/apps/theming/lib/Util.php
index e757d100a1..57a90b026e 100644
--- a/apps/theming/lib/Util.php
+++ b/apps/theming/lib/Util.php
@@ -250,13 +250,6 @@ class Util {
 
 	public function isBackgroundThemed() {
 		$backgroundLogo = $this->config->getAppValue('theming', 'backgroundMime', '');
-
-		$backgroundExists = true;
-		try {
-			$this->appData->getFolder('images')->getFile('background');
-		} catch (\Exception $e) {
-			$backgroundExists = false;
-		}
-		return $backgroundLogo && $backgroundLogo !== 'backgroundColor' && $backgroundExists;
+		return $backgroundLogo !== '' && $backgroundLogo !== 'backgroundColor';
 	}
 }
diff --git a/apps/theming/tests/UtilTest.php b/apps/theming/tests/UtilTest.php
index 10013e7d61..d6fe318cbc 100644
--- a/apps/theming/tests/UtilTest.php
+++ b/apps/theming/tests/UtilTest.php
@@ -216,32 +216,19 @@ class UtilTest extends TestCase {
 
 	public function dataIsBackgroundThemed() {
 		return [
-			[false, false, false],
-			['png', true, true],
-			['backgroundColor', false, false],
+			['', false],
+			['png', true],
+			['backgroundColor', false],
 		];
 	}
 	/**
 	 * @dataProvider dataIsBackgroundThemed
 	 */
-	public function testIsBackgroundThemed($backgroundMime, $fileFound, $expected) {
+	public function testIsBackgroundThemed($backgroundMime, $expected) {
 		$this->config->expects($this->once())
 			->method('getAppValue')
-			->with('theming', 'backgroundMime', false)
+			->with('theming', 'backgroundMime', '')
 			->willReturn($backgroundMime);
-		$folder = $this->createMock(ISimpleFolder::class);
-		if ($fileFound) {
-			$folder->expects($this->once())
-				->method('getFile')
-				->willReturn($this->createMock(ISimpleFile::class));
-		} else {
-			$folder->expects($this->once())
-				->method('getFile')
-				->willThrowException(new NotFoundException());
-		}
-		$this->appData->expects($this->once())
-			->method('getFolder')
-			->willReturn($folder);
 		$this->assertEquals($expected, $this->util->isBackgroundThemed());
 	}
 }