nextcloud / server

apps/theming/lib/ThemingDefaults.php

Indentation +503 added lines, -503 removed lines

@@ -22,507 +22,507 @@
 
 class ThemingDefaults extends \OC_Defaults {
 
-	private string $name;
-	private string $title;
-	private string $entity;
-	private string $productName;
-	private string $url;
-	private string $backgroundColor;
-	private string $primaryColor;
-	private string $docBaseUrl;
-
-	private string $iTunesAppId;
-	private string $iOSClientUrl;
-	private string $AndroidClientUrl;
-	private string $FDroidClientUrl;
-
-	/**
-	 * ThemingDefaults constructor.
-	 */
-	public function __construct(
-		private IConfig $config,
-		private IAppConfig $appConfig,
-		private IL10N $l,
-		private IUserSession $userSession,
-		private IURLGenerator $urlGenerator,
-		private ICacheFactory $cacheFactory,
-		private Util $util,
-		private ImageManager $imageManager,
-		private IAppManager $appManager,
-		private INavigationManager $navigationManager,
-		private BackgroundService $backgroundService,
-	) {
-		parent::__construct();
-
-		$this->name = parent::getName();
-		$this->title = parent::getTitle();
-		$this->entity = parent::getEntity();
-		$this->productName = parent::getProductName();
-		$this->url = parent::getBaseUrl();
-		$this->primaryColor = parent::getColorPrimary();
-		$this->backgroundColor = parent::getColorBackground();
-		$this->iTunesAppId = parent::getiTunesAppId();
-		$this->iOSClientUrl = parent::getiOSClientUrl();
-		$this->AndroidClientUrl = parent::getAndroidClientUrl();
-		$this->FDroidClientUrl = parent::getFDroidClientUrl();
-		$this->docBaseUrl = parent::getDocBaseUrl();
-	}
-
-	public function getName() {
-		return strip_tags($this->config->getAppValue('theming', 'name', $this->name));
-	}
-
-	public function getHTMLName() {
-		return $this->config->getAppValue('theming', 'name', $this->name);
-	}
-
-	public function getTitle() {
-		return strip_tags($this->config->getAppValue('theming', 'name', $this->title));
-	}
-
-	public function getEntity() {
-		return strip_tags($this->config->getAppValue('theming', 'name', $this->entity));
-	}
-
-	public function getProductName() {
-		return strip_tags($this->config->getAppValue('theming', 'productName', $this->productName));
-	}
-
-	public function getBaseUrl() {
-		return $this->config->getAppValue('theming', 'url', $this->url);
-	}
-
-	/**
-	 * We pass a string and sanitizeHTML will return a string too in that case
-	 * @psalm-suppress InvalidReturnStatement
-	 * @psalm-suppress InvalidReturnType
-	 */
-	public function getSlogan(?string $lang = null) {
-		return \OCP\Util::sanitizeHTML($this->config->getAppValue('theming', 'slogan', parent::getSlogan($lang)));
-	}
-
-	public function getImprintUrl() {
-		return (string)$this->config->getAppValue('theming', 'imprintUrl', '');
-	}
-
-	public function getPrivacyUrl() {
-		return (string)$this->config->getAppValue('theming', 'privacyUrl', '');
-	}
-
-	public function getDocBaseUrl() {
-		return (string)$this->config->getAppValue('theming', 'docBaseUrl', $this->docBaseUrl);
-	}
-
-	public function getShortFooter() {
-		$slogan = $this->getSlogan();
-		$baseUrl = $this->getBaseUrl();
-		$entity = $this->getEntity();
-		$footer = '';
-
-		if ($entity !== '') {
-			if ($baseUrl !== '') {
-				$footer = '<a href="' . $baseUrl . '" target="_blank"'
-					. ' rel="noreferrer noopener" class="entity-name">' . $entity . '</a>';
-			} else {
-				$footer = '<span class="entity-name">' . $entity . '</span>';
-			}
-		}
-		$footer .= ($slogan !== '' ? ' – ' . $slogan : '');
-
-		$links = [
-			[
-				'text' => $this->l->t('Legal notice'),
-				'url' => (string)$this->getImprintUrl()
-			],
-			[
-				'text' => $this->l->t('Privacy policy'),
-				'url' => (string)$this->getPrivacyUrl()
-			],
-		];
-
-		$navigation = $this->navigationManager->getAll(INavigationManager::TYPE_GUEST);
-		$guestNavigation = array_map(function ($nav) {
-			return [
-				'text' => $nav['name'],
-				'url' => $nav['href']
-			];
-		}, $navigation);
-		$links = array_merge($links, $guestNavigation);
-
-		$legalLinks = '';
-		$divider = '';
-		foreach ($links as $link) {
-			if ($link['url'] !== ''
-				&& filter_var($link['url'], FILTER_VALIDATE_URL)
-			) {
-				$legalLinks .= $divider . '<a href="' . $link['url'] . '" class="legal" target="_blank"'
-					. ' rel="noreferrer noopener">' . $link['text'] . '</a>';
-				$divider = ' · ';
-			}
-		}
-		if ($legalLinks !== '') {
-			$footer .= '<br/><span class="footer__legal-links">' . $legalLinks . '</span>';
-		}
-
-		return $footer;
-	}
-
-	/**
-	 * Color that is used for highlighting elements like important buttons
-	 * If user theming is enabled then the user defined value is returned
-	 */
-	public function getColorPrimary(): string {
-		$user = $this->userSession->getUser();
-
-		// admin-defined primary color
-		$defaultColor = $this->getDefaultColorPrimary();
-
-		if ($this->isUserThemingDisabled()) {
-			return $defaultColor;
-		}
-
-		// user-defined primary color
-		if (!empty($user)) {
-			$userPrimaryColor = $this->config->getUserValue($user->getUID(), Application::APP_ID, 'primary_color', '');
-			if (preg_match('/^\#([0-9a-f]{3}|[0-9a-f]{6})$/i', $userPrimaryColor)) {
-				return $userPrimaryColor;
-			}
-		}
-
-		// Finally, return the system global primary color
-		return $defaultColor;
-	}
-
-	/**
-	 * Color that is used for the page background (e.g. the header)
-	 * If user theming is enabled then the user defined value is returned
-	 */
-	public function getColorBackground(): string {
-		$user = $this->userSession->getUser();
-
-		// admin-defined background color
-		$defaultColor = $this->getDefaultColorBackground();
-
-		if ($this->isUserThemingDisabled()) {
-			return $defaultColor;
-		}
-
-		// user-defined background color
-		if (!empty($user)) {
-			$userBackgroundColor = $this->config->getUserValue($user->getUID(), Application::APP_ID, 'background_color', '');
-			if (preg_match('/^\#([0-9a-f]{3}|[0-9a-f]{6})$/i', $userBackgroundColor)) {
-				return $userBackgroundColor;
-			}
-		}
-
-		// Finally, return the system global background color
-		return $defaultColor;
-	}
-
-	/**
-	 * Return the default primary color - only taking admin setting into account
-	 */
-	public function getDefaultColorPrimary(): string {
-		// try admin color
-		$defaultColor = $this->appConfig->getValueString(Application::APP_ID, 'primary_color', '');
-		if (preg_match('/^\#([0-9a-f]{3}|[0-9a-f]{6})$/i', $defaultColor)) {
-			return $defaultColor;
-		}
-
-		// fall back to default primary color
-		return $this->primaryColor;
-	}
-
-	/**
-	 * Default background color only taking admin setting into account
-	 */
-	public function getDefaultColorBackground(): string {
-		$defaultColor = $this->appConfig->getValueString(Application::APP_ID, 'background_color');
-		if (preg_match('/^\#([0-9a-f]{3}|[0-9a-f]{6})$/i', $defaultColor)) {
-			return $defaultColor;
-		}
-
-		return $this->backgroundColor;
-	}
-
-	/**
-	 * Themed logo url
-	 *
-	 * @param bool $useSvg Whether to point to the SVG image or a fallback
-	 * @return string
-	 */
-	public function getLogo($useSvg = true): string {
-		$logo = $this->config->getAppValue('theming', 'logoMime', '');
-
-		// short cut to avoid setting up the filesystem just to check if the logo is there
-		//
-		// explanation: if an SVG is requested and the app config value for logoMime is set then the logo is there.
-		// otherwise we need to check it and maybe also generate a PNG from the SVG (that's done in getImage() which
-		// needs to be called then)
-		if ($useSvg === true && $logo !== false) {
-			$logoExists = true;
-		} else {
-			try {
-				$this->imageManager->getImage('logo', $useSvg);
-				$logoExists = true;
-			} catch (\Exception $e) {
-				$logoExists = false;
-			}
-		}
-
-		$cacheBusterCounter = $this->config->getAppValue('theming', 'cachebuster', '0');
-
-		if (!$logo || !$logoExists) {
-			if ($useSvg) {
-				$logo = $this->urlGenerator->imagePath('core', 'logo/logo.svg');
-			} else {
-				$logo = $this->urlGenerator->imagePath('core', 'logo/logo.png');
-			}
-			return $logo . '?v=' . $cacheBusterCounter;
-		}
-
-		return $this->urlGenerator->linkToRoute('theming.Theming.getImage', [ 'key' => 'logo', 'useSvg' => $useSvg, 'v' => $cacheBusterCounter ]);
-	}
-
-	/**
-	 * Themed background image url
-	 *
-	 * @param bool $darkVariant if the dark variant (if available) of the background should be used
-	 * @return string
-	 */
-	public function getBackground(bool $darkVariant = false): string {
-		return $this->imageManager->getImageUrl('background' . ($darkVariant ? 'Dark' : ''));
-	}
-
-	/**
-	 * @return string
-	 */
-	public function getiTunesAppId() {
-		return $this->config->getAppValue('theming', 'iTunesAppId', $this->iTunesAppId);
-	}
-
-	/**
-	 * @return string
-	 */
-	public function getiOSClientUrl() {
-		return $this->config->getAppValue('theming', 'iOSClientUrl', $this->iOSClientUrl);
-	}
-
-	/**
-	 * @return string
-	 */
-	public function getAndroidClientUrl() {
-		return $this->config->getAppValue('theming', 'AndroidClientUrl', $this->AndroidClientUrl);
-	}
-
-	/**
-	 * @return string
-	 */
-	public function getFDroidClientUrl() {
-		return $this->config->getAppValue('theming', 'FDroidClientUrl', $this->FDroidClientUrl);
-	}
-
-	/**
-	 * @return array scss variables to overwrite
-	 * @deprecated since Nextcloud 22 - https://github.com/nextcloud/server/issues/9940
-	 */
-	public function getScssVariables() {
-		$cacheBuster = $this->config->getAppValue('theming', 'cachebuster', '0');
-		$cache = $this->cacheFactory->createDistributed('theming-' . $cacheBuster . '-' . $this->urlGenerator->getBaseUrl());
-		if ($value = $cache->get('getScssVariables')) {
-			return $value;
-		}
-
-		$variables = [
-			'theming-cachebuster' => "'" . $cacheBuster . "'",
-			'theming-logo-mime' => "'" . $this->config->getAppValue('theming', 'logoMime') . "'",
-			'theming-background-mime' => "'" . $this->config->getAppValue('theming', 'backgroundMime') . "'",
-			'theming-logoheader-mime' => "'" . $this->config->getAppValue('theming', 'logoheaderMime') . "'",
-			'theming-favicon-mime' => "'" . $this->config->getAppValue('theming', 'faviconMime') . "'"
-		];
-
-		$variables['image-logo'] = "url('" . $this->imageManager->getImageUrl('logo') . "')";
-		$variables['image-logoheader'] = "url('" . $this->imageManager->getImageUrl('logoheader') . "')";
-		$variables['image-favicon'] = "url('" . $this->imageManager->getImageUrl('favicon') . "')";
-		$variables['image-login-background'] = "url('" . $this->imageManager->getImageUrl('background') . "')";
-		$variables['image-login-plain'] = 'false';
-
-		if ($this->appConfig->getValueString(Application::APP_ID, 'primary_color', '') !== '') {
-			$variables['color-primary'] = $this->getColorPrimary();
-			$variables['color-primary-text'] = $this->getTextColorPrimary();
-			$variables['color-primary-element'] = $this->util->elementColor($this->getColorPrimary());
-		}
-
-		if ($this->config->getAppValue('theming', 'backgroundMime', '') === 'backgroundColor') {
-			$variables['image-login-plain'] = 'true';
-		}
-
-		$variables['has-legal-links'] = 'false';
-		if ($this->getImprintUrl() !== '' || $this->getPrivacyUrl() !== '') {
-			$variables['has-legal-links'] = 'true';
-		}
-
-		$cache->set('getScssVariables', $variables);
-		return $variables;
-	}
-
-	/**
-	 * Check if the image should be replaced by the theming app
-	 * and return the new image location then
-	 *
-	 * @param string $app name of the app
-	 * @param string $image filename of the image
-	 * @return bool|string false if image should not replaced, otherwise the location of the image
-	 */
-	public function replaceImagePath($app, $image) {
-		if ($app === '' || $app === 'files_sharing') {
-			$app = 'core';
-		}
-		$cacheBusterValue = $this->config->getAppValue('theming', 'cachebuster', '0');
-
-		$route = false;
-		if ($image === 'favicon.ico' && ($this->imageManager->shouldReplaceIcons() || $this->getCustomFavicon() !== null)) {
-			$route = $this->urlGenerator->linkToRoute('theming.Icon.getFavicon', ['app' => $app]);
-		}
-		if (($image === 'favicon-touch.png' || $image === 'favicon-fb.png') && ($this->imageManager->shouldReplaceIcons() || $this->getCustomFavicon() !== null)) {
-			$route = $this->urlGenerator->linkToRoute('theming.Icon.getTouchIcon', ['app' => $app]);
-		}
-		if ($image === 'manifest.json') {
-			try {
-				$appPath = $this->appManager->getAppPath($app);
-				if (file_exists($appPath . '/img/manifest.json')) {
-					return false;
-				}
-			} catch (AppPathNotFoundException $e) {
-			}
-			$route = $this->urlGenerator->linkToRoute('theming.Theming.getManifest', ['app' => $app ]);
-		}
-		if (str_starts_with($image, 'filetypes/') && file_exists(\OC::$SERVERROOT . '/core/img/' . $image)) {
-			$route = $this->urlGenerator->linkToRoute('theming.Icon.getThemedIcon', ['app' => $app, 'image' => $image]);
-		}
-
-		if ($route) {
-			return $route . '?v=' . $this->util->getCacheBuster();
-		}
-
-		return false;
-	}
-
-	protected function getCustomFavicon(): ?ISimpleFile {
-		try {
-			return $this->imageManager->getImage('favicon');
-		} catch (NotFoundException $e) {
-			return null;
-		}
-	}
-
-	/**
-	 * Increases the cache buster key
-	 */
-	public function increaseCacheBuster(): void {
-		$cacheBusterKey = (int)$this->config->getAppValue('theming', 'cachebuster', '0');
-		$this->config->setAppValue('theming', 'cachebuster', (string)($cacheBusterKey + 1));
-		$this->cacheFactory->createDistributed('theming-')->clear();
-		$this->cacheFactory->createDistributed('imagePath')->clear();
-	}
-
-	/**
-	 * Update setting in the database
-	 *
-	 * @param string $setting
-	 * @param string $value
-	 */
-	public function set($setting, $value): void {
-		$this->appConfig->setValueString('theming', $setting, $value);
-		$this->increaseCacheBuster();
-	}
-
-	/**
-	 * Revert all settings to the default value
-	 */
-	public function undoAll(): void {
-		// Remember the current cachebuster value, as we do not want to reset this value
-		// Otherwise this can lead to caching issues as the value might be known to a browser already
-		$cacheBusterKey = $this->config->getAppValue('theming', 'cachebuster', '0');
-		$this->config->deleteAppValues('theming');
-		$this->config->setAppValue('theming', 'cachebuster', $cacheBusterKey);
-		$this->increaseCacheBuster();
-	}
-
-	/**
-	 * Revert admin settings to the default value
-	 *
-	 * @param string $setting setting which should be reverted
-	 * @return string default value
-	 */
-	public function undo($setting): string {
-		$this->config->deleteAppValue('theming', $setting);
-		$this->increaseCacheBuster();
-
-		$returnValue = '';
-		switch ($setting) {
-			case 'name':
-				$returnValue = $this->getEntity();
-				break;
-			case 'url':
-				$returnValue = $this->getBaseUrl();
-				break;
-			case 'slogan':
-				$returnValue = $this->getSlogan();
-				break;
-			case 'primary_color':
-				$returnValue = BackgroundService::DEFAULT_COLOR;
-				break;
-			case 'background_color':
-				// If a background image is set we revert to the mean image color
-				if ($this->imageManager->hasImage('background')) {
-					$file = $this->imageManager->getImage('background');
-					$returnValue = $this->backgroundService->setGlobalBackground($file->read()) ?? '';
-				}
-				break;
-			case 'logo':
-			case 'logoheader':
-			case 'background':
-			case 'favicon':
-				$this->imageManager->delete($setting);
-				$this->config->deleteAppValue('theming', $setting . 'Mime');
-				break;
-		}
-
-		return $returnValue;
-	}
-
-	/**
-	 * Color of text in the header menu
-	 *
-	 * @return string
-	 */
-	public function getTextColorBackground() {
-		return $this->util->invertTextColor($this->getColorBackground()) ? '#000000' : '#ffffff';
-	}
-
-	/**
-	 * Color of text on primary buttons and other elements
-	 *
-	 * @return string
-	 */
-	public function getTextColorPrimary() {
-		return $this->util->invertTextColor($this->getColorPrimary()) ? '#000000' : '#ffffff';
-	}
-
-	/**
-	 * Color of text in the header and primary buttons
-	 *
-	 * @return string
-	 */
-	public function getDefaultTextColorPrimary() {
-		return $this->util->invertTextColor($this->getDefaultColorPrimary()) ? '#000000' : '#ffffff';
-	}
-
-	/**
-	 * Has the admin disabled user customization
-	 */
-	public function isUserThemingDisabled(): bool {
-		return $this->appConfig->getValueBool(Application::APP_ID, 'disable-user-theming');
-	}
+    private string $name;
+    private string $title;
+    private string $entity;
+    private string $productName;
+    private string $url;
+    private string $backgroundColor;
+    private string $primaryColor;
+    private string $docBaseUrl;
+
+    private string $iTunesAppId;
+    private string $iOSClientUrl;
+    private string $AndroidClientUrl;
+    private string $FDroidClientUrl;
+
+    /**
+     * ThemingDefaults constructor.
+     */
+    public function __construct(
+        private IConfig $config,
+        private IAppConfig $appConfig,
+        private IL10N $l,
+        private IUserSession $userSession,
+        private IURLGenerator $urlGenerator,
+        private ICacheFactory $cacheFactory,
+        private Util $util,
+        private ImageManager $imageManager,
+        private IAppManager $appManager,
+        private INavigationManager $navigationManager,
+        private BackgroundService $backgroundService,
+    ) {
+        parent::__construct();
+
+        $this->name = parent::getName();
+        $this->title = parent::getTitle();
+        $this->entity = parent::getEntity();
+        $this->productName = parent::getProductName();
+        $this->url = parent::getBaseUrl();
+        $this->primaryColor = parent::getColorPrimary();
+        $this->backgroundColor = parent::getColorBackground();
+        $this->iTunesAppId = parent::getiTunesAppId();
+        $this->iOSClientUrl = parent::getiOSClientUrl();
+        $this->AndroidClientUrl = parent::getAndroidClientUrl();
+        $this->FDroidClientUrl = parent::getFDroidClientUrl();
+        $this->docBaseUrl = parent::getDocBaseUrl();
+    }
+
+    public function getName() {
+        return strip_tags($this->config->getAppValue('theming', 'name', $this->name));
+    }
+
+    public function getHTMLName() {
+        return $this->config->getAppValue('theming', 'name', $this->name);
+    }
+
+    public function getTitle() {
+        return strip_tags($this->config->getAppValue('theming', 'name', $this->title));
+    }
+
+    public function getEntity() {
+        return strip_tags($this->config->getAppValue('theming', 'name', $this->entity));
+    }
+
+    public function getProductName() {
+        return strip_tags($this->config->getAppValue('theming', 'productName', $this->productName));
+    }
+
+    public function getBaseUrl() {
+        return $this->config->getAppValue('theming', 'url', $this->url);
+    }
+
+    /**
+     * We pass a string and sanitizeHTML will return a string too in that case
+     * @psalm-suppress InvalidReturnStatement
+     * @psalm-suppress InvalidReturnType
+     */
+    public function getSlogan(?string $lang = null) {
+        return \OCP\Util::sanitizeHTML($this->config->getAppValue('theming', 'slogan', parent::getSlogan($lang)));
+    }
+
+    public function getImprintUrl() {
+        return (string)$this->config->getAppValue('theming', 'imprintUrl', '');
+    }
+
+    public function getPrivacyUrl() {
+        return (string)$this->config->getAppValue('theming', 'privacyUrl', '');
+    }
+
+    public function getDocBaseUrl() {
+        return (string)$this->config->getAppValue('theming', 'docBaseUrl', $this->docBaseUrl);
+    }
+
+    public function getShortFooter() {
+        $slogan = $this->getSlogan();
+        $baseUrl = $this->getBaseUrl();
+        $entity = $this->getEntity();
+        $footer = '';
+
+        if ($entity !== '') {
+            if ($baseUrl !== '') {
+                $footer = '<a href="' . $baseUrl . '" target="_blank"'
+                    . ' rel="noreferrer noopener" class="entity-name">' . $entity . '</a>';
+            } else {
+                $footer = '<span class="entity-name">' . $entity . '</span>';
+            }
+        }
+        $footer .= ($slogan !== '' ? ' – ' . $slogan : '');
+
+        $links = [
+            [
+                'text' => $this->l->t('Legal notice'),
+                'url' => (string)$this->getImprintUrl()
+            ],
+            [
+                'text' => $this->l->t('Privacy policy'),
+                'url' => (string)$this->getPrivacyUrl()
+            ],
+        ];
+
+        $navigation = $this->navigationManager->getAll(INavigationManager::TYPE_GUEST);
+        $guestNavigation = array_map(function ($nav) {
+            return [
+                'text' => $nav['name'],
+                'url' => $nav['href']
+            ];
+        }, $navigation);
+        $links = array_merge($links, $guestNavigation);
+
+        $legalLinks = '';
+        $divider = '';
+        foreach ($links as $link) {
+            if ($link['url'] !== ''
+                && filter_var($link['url'], FILTER_VALIDATE_URL)
+            ) {
+                $legalLinks .= $divider . '<a href="' . $link['url'] . '" class="legal" target="_blank"'
+                    . ' rel="noreferrer noopener">' . $link['text'] . '</a>';
+                $divider = ' · ';
+            }
+        }
+        if ($legalLinks !== '') {
+            $footer .= '<br/><span class="footer__legal-links">' . $legalLinks . '</span>';
+        }
+
+        return $footer;
+    }
+
+    /**
+     * Color that is used for highlighting elements like important buttons
+     * If user theming is enabled then the user defined value is returned
+     */
+    public function getColorPrimary(): string {
+        $user = $this->userSession->getUser();
+
+        // admin-defined primary color
+        $defaultColor = $this->getDefaultColorPrimary();
+
+        if ($this->isUserThemingDisabled()) {
+            return $defaultColor;
+        }
+
+        // user-defined primary color
+        if (!empty($user)) {
+            $userPrimaryColor = $this->config->getUserValue($user->getUID(), Application::APP_ID, 'primary_color', '');
+            if (preg_match('/^\#([0-9a-f]{3}|[0-9a-f]{6})$/i', $userPrimaryColor)) {
+                return $userPrimaryColor;
+            }
+        }
+
+        // Finally, return the system global primary color
+        return $defaultColor;
+    }
+
+    /**
+     * Color that is used for the page background (e.g. the header)
+     * If user theming is enabled then the user defined value is returned
+     */
+    public function getColorBackground(): string {
+        $user = $this->userSession->getUser();
+
+        // admin-defined background color
+        $defaultColor = $this->getDefaultColorBackground();
+
+        if ($this->isUserThemingDisabled()) {
+            return $defaultColor;
+        }
+
+        // user-defined background color
+        if (!empty($user)) {
+            $userBackgroundColor = $this->config->getUserValue($user->getUID(), Application::APP_ID, 'background_color', '');
+            if (preg_match('/^\#([0-9a-f]{3}|[0-9a-f]{6})$/i', $userBackgroundColor)) {
+                return $userBackgroundColor;
+            }
+        }
+
+        // Finally, return the system global background color
+        return $defaultColor;
+    }
+
+    /**
+     * Return the default primary color - only taking admin setting into account
+     */
+    public function getDefaultColorPrimary(): string {
+        // try admin color
+        $defaultColor = $this->appConfig->getValueString(Application::APP_ID, 'primary_color', '');
+        if (preg_match('/^\#([0-9a-f]{3}|[0-9a-f]{6})$/i', $defaultColor)) {
+            return $defaultColor;
+        }
+
+        // fall back to default primary color
+        return $this->primaryColor;
+    }
+
+    /**
+     * Default background color only taking admin setting into account
+     */
+    public function getDefaultColorBackground(): string {
+        $defaultColor = $this->appConfig->getValueString(Application::APP_ID, 'background_color');
+        if (preg_match('/^\#([0-9a-f]{3}|[0-9a-f]{6})$/i', $defaultColor)) {
+            return $defaultColor;
+        }
+
+        return $this->backgroundColor;
+    }
+
+    /**
+     * Themed logo url
+     *
+     * @param bool $useSvg Whether to point to the SVG image or a fallback
+     * @return string
+     */
+    public function getLogo($useSvg = true): string {
+        $logo = $this->config->getAppValue('theming', 'logoMime', '');
+
+        // short cut to avoid setting up the filesystem just to check if the logo is there
+        //
+        // explanation: if an SVG is requested and the app config value for logoMime is set then the logo is there.
+        // otherwise we need to check it and maybe also generate a PNG from the SVG (that's done in getImage() which
+        // needs to be called then)
+        if ($useSvg === true && $logo !== false) {
+            $logoExists = true;
+        } else {
+            try {
+                $this->imageManager->getImage('logo', $useSvg);
+                $logoExists = true;
+            } catch (\Exception $e) {
+                $logoExists = false;
+            }
+        }
+
+        $cacheBusterCounter = $this->config->getAppValue('theming', 'cachebuster', '0');
+
+        if (!$logo || !$logoExists) {
+            if ($useSvg) {
+                $logo = $this->urlGenerator->imagePath('core', 'logo/logo.svg');
+            } else {
+                $logo = $this->urlGenerator->imagePath('core', 'logo/logo.png');
+            }
+            return $logo . '?v=' . $cacheBusterCounter;
+        }
+
+        return $this->urlGenerator->linkToRoute('theming.Theming.getImage', [ 'key' => 'logo', 'useSvg' => $useSvg, 'v' => $cacheBusterCounter ]);
+    }
+
+    /**
+     * Themed background image url
+     *
+     * @param bool $darkVariant if the dark variant (if available) of the background should be used
+     * @return string
+     */
+    public function getBackground(bool $darkVariant = false): string {
+        return $this->imageManager->getImageUrl('background' . ($darkVariant ? 'Dark' : ''));
+    }
+
+    /**
+     * @return string
+     */
+    public function getiTunesAppId() {
+        return $this->config->getAppValue('theming', 'iTunesAppId', $this->iTunesAppId);
+    }
+
+    /**
+     * @return string
+     */
+    public function getiOSClientUrl() {
+        return $this->config->getAppValue('theming', 'iOSClientUrl', $this->iOSClientUrl);
+    }
+
+    /**
+     * @return string
+     */
+    public function getAndroidClientUrl() {
+        return $this->config->getAppValue('theming', 'AndroidClientUrl', $this->AndroidClientUrl);
+    }
+
+    /**
+     * @return string
+     */
+    public function getFDroidClientUrl() {
+        return $this->config->getAppValue('theming', 'FDroidClientUrl', $this->FDroidClientUrl);
+    }
+
+    /**
+     * @return array scss variables to overwrite
+     * @deprecated since Nextcloud 22 - https://github.com/nextcloud/server/issues/9940
+     */
+    public function getScssVariables() {
+        $cacheBuster = $this->config->getAppValue('theming', 'cachebuster', '0');
+        $cache = $this->cacheFactory->createDistributed('theming-' . $cacheBuster . '-' . $this->urlGenerator->getBaseUrl());
+        if ($value = $cache->get('getScssVariables')) {
+            return $value;
+        }
+
+        $variables = [
+            'theming-cachebuster' => "'" . $cacheBuster . "'",
+            'theming-logo-mime' => "'" . $this->config->getAppValue('theming', 'logoMime') . "'",
+            'theming-background-mime' => "'" . $this->config->getAppValue('theming', 'backgroundMime') . "'",
+            'theming-logoheader-mime' => "'" . $this->config->getAppValue('theming', 'logoheaderMime') . "'",
+            'theming-favicon-mime' => "'" . $this->config->getAppValue('theming', 'faviconMime') . "'"
+        ];
+
+        $variables['image-logo'] = "url('" . $this->imageManager->getImageUrl('logo') . "')";
+        $variables['image-logoheader'] = "url('" . $this->imageManager->getImageUrl('logoheader') . "')";
+        $variables['image-favicon'] = "url('" . $this->imageManager->getImageUrl('favicon') . "')";
+        $variables['image-login-background'] = "url('" . $this->imageManager->getImageUrl('background') . "')";
+        $variables['image-login-plain'] = 'false';
+
+        if ($this->appConfig->getValueString(Application::APP_ID, 'primary_color', '') !== '') {
+            $variables['color-primary'] = $this->getColorPrimary();
+            $variables['color-primary-text'] = $this->getTextColorPrimary();
+            $variables['color-primary-element'] = $this->util->elementColor($this->getColorPrimary());
+        }
+
+        if ($this->config->getAppValue('theming', 'backgroundMime', '') === 'backgroundColor') {
+            $variables['image-login-plain'] = 'true';
+        }
+
+        $variables['has-legal-links'] = 'false';
+        if ($this->getImprintUrl() !== '' || $this->getPrivacyUrl() !== '') {
+            $variables['has-legal-links'] = 'true';
+        }
+
+        $cache->set('getScssVariables', $variables);
+        return $variables;
+    }
+
+    /**
+     * Check if the image should be replaced by the theming app
+     * and return the new image location then
+     *
+     * @param string $app name of the app
+     * @param string $image filename of the image
+     * @return bool|string false if image should not replaced, otherwise the location of the image
+     */
+    public function replaceImagePath($app, $image) {
+        if ($app === '' || $app === 'files_sharing') {
+            $app = 'core';
+        }
+        $cacheBusterValue = $this->config->getAppValue('theming', 'cachebuster', '0');
+
+        $route = false;
+        if ($image === 'favicon.ico' && ($this->imageManager->shouldReplaceIcons() || $this->getCustomFavicon() !== null)) {
+            $route = $this->urlGenerator->linkToRoute('theming.Icon.getFavicon', ['app' => $app]);
+        }
+        if (($image === 'favicon-touch.png' || $image === 'favicon-fb.png') && ($this->imageManager->shouldReplaceIcons() || $this->getCustomFavicon() !== null)) {
+            $route = $this->urlGenerator->linkToRoute('theming.Icon.getTouchIcon', ['app' => $app]);
+        }
+        if ($image === 'manifest.json') {
+            try {
+                $appPath = $this->appManager->getAppPath($app);
+                if (file_exists($appPath . '/img/manifest.json')) {
+                    return false;
+                }
+            } catch (AppPathNotFoundException $e) {
+            }
+            $route = $this->urlGenerator->linkToRoute('theming.Theming.getManifest', ['app' => $app ]);
+        }
+        if (str_starts_with($image, 'filetypes/') && file_exists(\OC::$SERVERROOT . '/core/img/' . $image)) {
+            $route = $this->urlGenerator->linkToRoute('theming.Icon.getThemedIcon', ['app' => $app, 'image' => $image]);
+        }
+
+        if ($route) {
+            return $route . '?v=' . $this->util->getCacheBuster();
+        }
+
+        return false;
+    }
+
+    protected function getCustomFavicon(): ?ISimpleFile {
+        try {
+            return $this->imageManager->getImage('favicon');
+        } catch (NotFoundException $e) {
+            return null;
+        }
+    }
+
+    /**
+     * Increases the cache buster key
+     */
+    public function increaseCacheBuster(): void {
+        $cacheBusterKey = (int)$this->config->getAppValue('theming', 'cachebuster', '0');
+        $this->config->setAppValue('theming', 'cachebuster', (string)($cacheBusterKey + 1));
+        $this->cacheFactory->createDistributed('theming-')->clear();
+        $this->cacheFactory->createDistributed('imagePath')->clear();
+    }
+
+    /**
+     * Update setting in the database
+     *
+     * @param string $setting
+     * @param string $value
+     */
+    public function set($setting, $value): void {
+        $this->appConfig->setValueString('theming', $setting, $value);
+        $this->increaseCacheBuster();
+    }
+
+    /**
+     * Revert all settings to the default value
+     */
+    public function undoAll(): void {
+        // Remember the current cachebuster value, as we do not want to reset this value
+        // Otherwise this can lead to caching issues as the value might be known to a browser already
+        $cacheBusterKey = $this->config->getAppValue('theming', 'cachebuster', '0');
+        $this->config->deleteAppValues('theming');
+        $this->config->setAppValue('theming', 'cachebuster', $cacheBusterKey);
+        $this->increaseCacheBuster();
+    }
+
+    /**
+     * Revert admin settings to the default value
+     *
+     * @param string $setting setting which should be reverted
+     * @return string default value
+     */
+    public function undo($setting): string {
+        $this->config->deleteAppValue('theming', $setting);
+        $this->increaseCacheBuster();
+
+        $returnValue = '';
+        switch ($setting) {
+            case 'name':
+                $returnValue = $this->getEntity();
+                break;
+            case 'url':
+                $returnValue = $this->getBaseUrl();
+                break;
+            case 'slogan':
+                $returnValue = $this->getSlogan();
+                break;
+            case 'primary_color':
+                $returnValue = BackgroundService::DEFAULT_COLOR;
+                break;
+            case 'background_color':
+                // If a background image is set we revert to the mean image color
+                if ($this->imageManager->hasImage('background')) {
+                    $file = $this->imageManager->getImage('background');
+                    $returnValue = $this->backgroundService->setGlobalBackground($file->read()) ?? '';
+                }
+                break;
+            case 'logo':
+            case 'logoheader':
+            case 'background':
+            case 'favicon':
+                $this->imageManager->delete($setting);
+                $this->config->deleteAppValue('theming', $setting . 'Mime');
+                break;
+        }
+
+        return $returnValue;
+    }
+
+    /**
+     * Color of text in the header menu
+     *
+     * @return string
+     */
+    public function getTextColorBackground() {
+        return $this->util->invertTextColor($this->getColorBackground()) ? '#000000' : '#ffffff';
+    }
+
+    /**
+     * Color of text on primary buttons and other elements
+     *
+     * @return string
+     */
+    public function getTextColorPrimary() {
+        return $this->util->invertTextColor($this->getColorPrimary()) ? '#000000' : '#ffffff';
+    }
+
+    /**
+     * Color of text in the header and primary buttons
+     *
+     * @return string
+     */
+    public function getDefaultTextColorPrimary() {
+        return $this->util->invertTextColor($this->getDefaultColorPrimary()) ? '#000000' : '#ffffff';
+    }
+
+    /**
+     * Has the admin disabled user customization
+     */
+    public function isUserThemingDisabled(): bool {
+        return $this->appConfig->getValueBool(Application::APP_ID, 'disable-user-theming');
+    }
 }

apps/theming/tests/ThemingDefaultsTest.php

Indentation +781 added lines, -781 removed lines

@@ -26,785 +26,785 @@
 use Test\TestCase;
 
 class ThemingDefaultsTest extends TestCase {
-	private IAppConfig&MockObject $appConfig;
-	private IConfig&MockObject $config;
-	private \OC_Defaults $defaults;
-	private IL10N|MockObject $l10n;
-	private IUserSession&MockObject $userSession;
-	private IURLGenerator&MockObject $urlGenerator;
-	private ICacheFactory&MockObject $cacheFactory;
-	private Util&MockObject $util;
-	private ICache&MockObject $cache;
-	private IAppManager&MockObject $appManager;
-	private ImageManager&MockObject $imageManager;
-	private INavigationManager&MockObject $navigationManager;
-	private BackgroundService&MockObject $backgroundService;
-	private ThemingDefaults $template;
-
-	protected function setUp(): void {
-		parent::setUp();
-		$this->appConfig = $this->createMock(IAppConfig::class);
-		$this->config = $this->createMock(IConfig::class);
-		$this->l10n = $this->createMock(IL10N::class);
-		$this->userSession = $this->createMock(IUserSession::class);
-		$this->urlGenerator = $this->createMock(IURLGenerator::class);
-		$this->cacheFactory = $this->createMock(ICacheFactory::class);
-		$this->cache = $this->createMock(ICache::class);
-		$this->util = $this->createMock(Util::class);
-		$this->imageManager = $this->createMock(ImageManager::class);
-		$this->appManager = $this->createMock(IAppManager::class);
-		$this->navigationManager = $this->createMock(INavigationManager::class);
-		$this->backgroundService = $this->createMock(BackgroundService::class);
-		$this->defaults = new \OC_Defaults();
-		$this->urlGenerator
-			->expects($this->any())
-			->method('getBaseUrl')
-			->willReturn('');
-		$this->template = new ThemingDefaults(
-			$this->config,
-			$this->appConfig,
-			$this->l10n,
-			$this->userSession,
-			$this->urlGenerator,
-			$this->cacheFactory,
-			$this->util,
-			$this->imageManager,
-			$this->appManager,
-			$this->navigationManager,
-			$this->backgroundService,
-		);
-	}
-
-	public function testGetNameWithDefault(): void {
-		$this->config
-			->expects($this->once())
-			->method('getAppValue')
-			->with('theming', 'name', 'Nextcloud')
-			->willReturn('Nextcloud');
-
-		$this->assertEquals('Nextcloud', $this->template->getName());
-	}
-
-	public function testGetNameWithCustom(): void {
-		$this->config
-			->expects($this->once())
-			->method('getAppValue')
-			->with('theming', 'name', 'Nextcloud')
-			->willReturn('MyCustomCloud');
-
-		$this->assertEquals('MyCustomCloud', $this->template->getName());
-	}
-
-	public function testGetHTMLNameWithDefault(): void {
-		$this->config
-			->expects($this->once())
-			->method('getAppValue')
-			->with('theming', 'name', 'Nextcloud')
-			->willReturn('Nextcloud');
-
-		$this->assertEquals('Nextcloud', $this->template->getHTMLName());
-	}
-
-	public function testGetHTMLNameWithCustom(): void {
-		$this->config
-			->expects($this->once())
-			->method('getAppValue')
-			->with('theming', 'name', 'Nextcloud')
-			->willReturn('MyCustomCloud');
-
-		$this->assertEquals('MyCustomCloud', $this->template->getHTMLName());
-	}
-
-	public function testGetTitleWithDefault(): void {
-		$this->config
-			->expects($this->once())
-			->method('getAppValue')
-			->with('theming', 'name', 'Nextcloud')
-			->willReturn('Nextcloud');
-
-		$this->assertEquals('Nextcloud', $this->template->getTitle());
-	}
-
-	public function testGetTitleWithCustom(): void {
-		$this->config
-			->expects($this->once())
-			->method('getAppValue')
-			->with('theming', 'name', 'Nextcloud')
-			->willReturn('MyCustomCloud');
-
-		$this->assertEquals('MyCustomCloud', $this->template->getTitle());
-	}
-
-
-	public function testGetEntityWithDefault(): void {
-		$this->config
-			->expects($this->once())
-			->method('getAppValue')
-			->with('theming', 'name', 'Nextcloud')
-			->willReturn('Nextcloud');
-
-		$this->assertEquals('Nextcloud', $this->template->getEntity());
-	}
-
-	public function testGetEntityWithCustom(): void {
-		$this->config
-			->expects($this->once())
-			->method('getAppValue')
-			->with('theming', 'name', 'Nextcloud')
-			->willReturn('MyCustomCloud');
-
-		$this->assertEquals('MyCustomCloud', $this->template->getEntity());
-	}
-
-	public function testGetBaseUrlWithDefault(): void {
-		$this->config
-			->expects($this->once())
-			->method('getAppValue')
-			->with('theming', 'url', $this->defaults->getBaseUrl())
-			->willReturn($this->defaults->getBaseUrl());
-
-		$this->assertEquals($this->defaults->getBaseUrl(), $this->template->getBaseUrl());
-	}
-
-	public function testGetBaseUrlWithCustom(): void {
-		$this->config
-			->expects($this->once())
-			->method('getAppValue')
-			->with('theming', 'url', $this->defaults->getBaseUrl())
-			->willReturn('https://example.com/');
-
-		$this->assertEquals('https://example.com/', $this->template->getBaseUrl());
-	}
-
-	public static function legalUrlProvider(): array {
-		return [
-			[''],
-			['https://example.com/legal.html'],
-		];
-	}
-
-	#[\PHPUnit\Framework\Attributes\DataProvider('legalUrlProvider')]
-	public function testGetImprintURL(string $imprintUrl): void {
-		$this->config
-			->expects($this->once())
-			->method('getAppValue')
-			->with('theming', 'imprintUrl', '')
-			->willReturn($imprintUrl);
-
-		$this->assertEquals($imprintUrl, $this->template->getImprintUrl());
-	}
-
-	#[\PHPUnit\Framework\Attributes\DataProvider('legalUrlProvider')]
-	public function testGetPrivacyURL(string $privacyUrl): void {
-		$this->config
-			->expects($this->once())
-			->method('getAppValue')
-			->with('theming', 'privacyUrl', '')
-			->willReturn($privacyUrl);
-
-		$this->assertEquals($privacyUrl, $this->template->getPrivacyUrl());
-	}
-
-	public function testGetSloganWithDefault(): void {
-		$this->config
-			->expects($this->once())
-			->method('getAppValue')
-			->with('theming', 'slogan', $this->defaults->getSlogan())
-			->willReturn($this->defaults->getSlogan());
-
-		$this->assertEquals($this->defaults->getSlogan(), $this->template->getSlogan());
-	}
-
-	public function testGetSloganWithCustom(): void {
-		$this->config
-			->expects($this->once())
-			->method('getAppValue')
-			->with('theming', 'slogan', $this->defaults->getSlogan())
-			->willReturn('My custom Slogan');
-
-		$this->assertEquals('My custom Slogan', $this->template->getSlogan());
-	}
-
-	public function testGetShortFooter(): void {
-		$this->config
-			->expects($this->exactly(5))
-			->method('getAppValue')
-			->willReturnMap([
-				['theming', 'url', $this->defaults->getBaseUrl(), 'url'],
-				['theming', 'name', 'Nextcloud', 'Name'],
-				['theming', 'slogan', $this->defaults->getSlogan(), 'Slogan'],
-				['theming', 'imprintUrl', '', ''],
-				['theming', 'privacyUrl', '', ''],
-			]);
-
-		$this->assertEquals('<a href="url" target="_blank" rel="noreferrer noopener" class="entity-name">Name</a> – Slogan', $this->template->getShortFooter());
-	}
-
-	public function testGetShortFooterEmptyUrl(): void {
-		$this->navigationManager->expects($this->once())->method('getAll')->with(INavigationManager::TYPE_GUEST)->willReturn([]);
-		$this->config
-			->expects($this->exactly(5))
-			->method('getAppValue')
-			->willReturnMap([
-				['theming', 'url', $this->defaults->getBaseUrl(), ''],
-				['theming', 'name', 'Nextcloud', 'Name'],
-				['theming', 'slogan', $this->defaults->getSlogan(), 'Slogan'],
-				['theming', 'imprintUrl', '', ''],
-				['theming', 'privacyUrl', '', ''],
-			]);
-
-		$this->assertEquals('<span class="entity-name">Name</span> – Slogan', $this->template->getShortFooter());
-	}
-
-	public function testGetShortFooterEmptySlogan(): void {
-		$this->navigationManager->expects($this->once())->method('getAll')->with(INavigationManager::TYPE_GUEST)->willReturn([]);
-		$this->config
-			->expects($this->exactly(5))
-			->method('getAppValue')
-			->willReturnMap([
-				['theming', 'url', $this->defaults->getBaseUrl(), 'url'],
-				['theming', 'name', 'Nextcloud', 'Name'],
-				['theming', 'slogan', $this->defaults->getSlogan(), ''],
-				['theming', 'imprintUrl', '', ''],
-				['theming', 'privacyUrl', '', ''],
-			]);
-
-		$this->assertEquals('<a href="url" target="_blank" rel="noreferrer noopener" class="entity-name">Name</a>', $this->template->getShortFooter());
-	}
-
-	public function testGetShortFooterImprint(): void {
-		$this->navigationManager->expects($this->once())->method('getAll')->with(INavigationManager::TYPE_GUEST)->willReturn([]);
-		$this->config
-			->expects($this->exactly(5))
-			->method('getAppValue')
-			->willReturnMap([
-				['theming', 'url', $this->defaults->getBaseUrl(), 'url'],
-				['theming', 'name', 'Nextcloud', 'Name'],
-				['theming', 'slogan', $this->defaults->getSlogan(), 'Slogan'],
-				['theming', 'imprintUrl', '', 'https://example.com/imprint'],
-				['theming', 'privacyUrl', '', ''],
-			]);
-
-		$this->l10n
-			->expects($this->any())
-			->method('t')
-			->willReturnArgument(0);
-
-		$this->assertEquals('<a href="url" target="_blank" rel="noreferrer noopener" class="entity-name">Name</a> – Slogan<br/><span class="footer__legal-links"><a href="https://example.com/imprint" class="legal" target="_blank" rel="noreferrer noopener">Legal notice</a></span>', $this->template->getShortFooter());
-	}
-
-	public function testGetShortFooterPrivacy(): void {
-		$this->navigationManager->expects($this->once())->method('getAll')->with(INavigationManager::TYPE_GUEST)->willReturn([]);
-		$this->config
-			->expects($this->exactly(5))
-			->method('getAppValue')
-			->willReturnMap([
-				['theming', 'url', $this->defaults->getBaseUrl(), 'url'],
-				['theming', 'name', 'Nextcloud', 'Name'],
-				['theming', 'slogan', $this->defaults->getSlogan(), 'Slogan'],
-				['theming', 'imprintUrl', '', ''],
-				['theming', 'privacyUrl', '', 'https://example.com/privacy'],
-			]);
-
-		$this->l10n
-			->expects($this->any())
-			->method('t')
-			->willReturnArgument(0);
-
-		$this->assertEquals('<a href="url" target="_blank" rel="noreferrer noopener" class="entity-name">Name</a> – Slogan<br/><span class="footer__legal-links"><a href="https://example.com/privacy" class="legal" target="_blank" rel="noreferrer noopener">Privacy policy</a></span>', $this->template->getShortFooter());
-	}
-
-	public function testGetShortFooterAllLegalLinks(): void {
-		$this->navigationManager->expects($this->once())->method('getAll')->with(INavigationManager::TYPE_GUEST)->willReturn([]);
-		$this->config
-			->expects($this->exactly(5))
-			->method('getAppValue')
-			->willReturnMap([
-				['theming', 'url', $this->defaults->getBaseUrl(), 'url'],
-				['theming', 'name', 'Nextcloud', 'Name'],
-				['theming', 'slogan', $this->defaults->getSlogan(), 'Slogan'],
-				['theming', 'imprintUrl', '', 'https://example.com/imprint'],
-				['theming', 'privacyUrl', '', 'https://example.com/privacy'],
-			]);
-
-		$this->l10n
-			->expects($this->any())
-			->method('t')
-			->willReturnArgument(0);
-
-		$this->assertEquals('<a href="url" target="_blank" rel="noreferrer noopener" class="entity-name">Name</a> – Slogan<br/><span class="footer__legal-links"><a href="https://example.com/imprint" class="legal" target="_blank" rel="noreferrer noopener">Legal notice</a> · <a href="https://example.com/privacy" class="legal" target="_blank" rel="noreferrer noopener">Privacy policy</a></span>', $this->template->getShortFooter());
-	}
-
-	public static function invalidLegalUrlProvider(): array {
-		return [
-			['example.com/legal'],  # missing scheme
-			['https:///legal'],     # missing host
-		];
-	}
-
-	#[\PHPUnit\Framework\Attributes\DataProvider('invalidLegalUrlProvider')]
-	public function testGetShortFooterInvalidImprint(string $invalidImprintUrl): void {
-		$this->navigationManager->expects($this->once())->method('getAll')->with(INavigationManager::TYPE_GUEST)->willReturn([]);
-		$this->config
-			->expects($this->exactly(5))
-			->method('getAppValue')
-			->willReturnMap([
-				['theming', 'url', $this->defaults->getBaseUrl(), 'url'],
-				['theming', 'name', 'Nextcloud', 'Name'],
-				['theming', 'slogan', $this->defaults->getSlogan(), 'Slogan'],
-				['theming', 'imprintUrl', '', $invalidImprintUrl],
-				['theming', 'privacyUrl', '', ''],
-			]);
-
-		$this->assertEquals('<a href="url" target="_blank" rel="noreferrer noopener" class="entity-name">Name</a> – Slogan', $this->template->getShortFooter());
-	}
-
-	#[\PHPUnit\Framework\Attributes\DataProvider('invalidLegalUrlProvider')]
-	public function testGetShortFooterInvalidPrivacy(string $invalidPrivacyUrl): void {
-		$this->navigationManager->expects($this->once())->method('getAll')->with(INavigationManager::TYPE_GUEST)->willReturn([]);
-		$this->config
-			->expects($this->exactly(5))
-			->method('getAppValue')
-			->willReturnMap([
-				['theming', 'url', $this->defaults->getBaseUrl(), 'url'],
-				['theming', 'name', 'Nextcloud', 'Name'],
-				['theming', 'slogan', $this->defaults->getSlogan(), 'Slogan'],
-				['theming', 'imprintUrl', '', ''],
-				['theming', 'privacyUrl', '', $invalidPrivacyUrl],
-			]);
-
-		$this->assertEquals('<a href="url" target="_blank" rel="noreferrer noopener" class="entity-name">Name</a> – Slogan', $this->template->getShortFooter());
-	}
-
-	public function testGetColorPrimaryWithDefault(): void {
-		$this->appConfig
-			->expects(self::once())
-			->method('getValueBool')
-			->with('theming', 'disable-user-theming')
-			->willReturn(false);
-		$this->appConfig
-			->expects(self::once())
-			->method('getValueString')
-			->with('theming', 'primary_color', '')
-			->willReturn($this->defaults->getColorPrimary());
-
-		$this->assertEquals($this->defaults->getColorPrimary(), $this->template->getColorPrimary());
-	}
-
-	public function testGetColorPrimaryWithCustom(): void {
-		$this->appConfig
-			->expects(self::once())
-			->method('getValueBool')
-			->with('theming', 'disable-user-theming')
-			->willReturn(false);
-		$this->appConfig
-			->expects(self::once())
-			->method('getValueString')
-			->with('theming', 'primary_color', '')
-			->willReturn('#fff');
-
-		$this->assertEquals('#fff', $this->template->getColorPrimary());
-	}
-
-	public static function dataGetColorPrimary(): array {
-		return [
-			'with fallback default' => [
-				'disableTheming' => false,
-				'primaryColor' => '',
-				'userPrimaryColor' => '',
-				'expected' => BackgroundService::DEFAULT_COLOR,
-			],
-			'with custom admin primary' => [
-				'disableTheming' => false,
-				'primaryColor' => '#aaa',
-				'userPrimaryColor' => '',
-				'expected' => '#aaa',
-			],
-			'with custom invalid admin primary' => [
-				'disableTheming' => false,
-				'primaryColor' => 'invalid',
-				'userPrimaryColor' => '',
-				'expected' => BackgroundService::DEFAULT_COLOR,
-			],
-			'with custom invalid user primary' => [
-				'disableTheming' => false,
-				'primaryColor' => '',
-				'userPrimaryColor' => 'invalid-name',
-				'expected' => BackgroundService::DEFAULT_COLOR,
-			],
-			'with custom user primary' => [
-				'disableTheming' => false,
-				'primaryColor' => '',
-				'userPrimaryColor' => '#bbb',
-				'expected' => '#bbb',
-			],
-			'with disabled user theming primary' => [
-				'disableTheming' => true,
-				'primaryColor' => '#aaa',
-				'userPrimaryColor' => '#bbb',
-				'expected' => '#aaa',
-			],
-		];
-	}
-
-	#[\PHPUnit\Framework\Attributes\DataProvider('dataGetColorPrimary')]
-	public function testGetColorPrimary(bool $disableTheming, string $primaryColor, string $userPrimaryColor, string $expected): void {
-		$user = $this->createMock(IUser::class);
-		$this->userSession->expects($this->any())
-			->method('getUser')
-			->willReturn($user);
-		$user->expects($this->any())
-			->method('getUID')
-			->willReturn('user');
-		$this->appConfig
-			->expects(self::any())
-			->method('getValueBool')
-			->with('theming', 'disable-user-theming')
-			->willReturn($disableTheming);
-		$this->appConfig
-			->expects(self::any())
-			->method('getValueString')
-			->with('theming', 'primary_color', '')
-			->willReturn($primaryColor);
-		$this->config
-			->expects($this->any())
-			->method('getUserValue')
-			->with('user', 'theming', 'primary_color', '')
-			->willReturn($userPrimaryColor);
-
-		$this->assertEquals($expected, $this->template->getColorPrimary());
-	}
-
-	public function testSet(): void {
-		$this->config
-			->expects($this->once())
-			->method('setAppValue')
-			->with('theming', 'cachebuster', 16);
-		$this->appConfig
-			->expects($this->once())
-			->method('setValueString')
-			->with('theming', 'MySetting', 'MyValue');
-		$this->config
-			->expects($this->once())
-			->method('getAppValue')
-			->with('theming', 'cachebuster', '0')
-			->willReturn('15');
-		$this->cacheFactory
-			->expects($this->exactly(2))
-			->method('createDistributed')
-			->willReturnMap([
-				['theming-', $this->cache],
-				['imagePath', $this->cache],
-			]);
-		$this->cache
-			->expects($this->any())
-			->method('clear')
-			->with('');
-		$this->template->set('MySetting', 'MyValue');
-	}
-
-	public function testUndoName(): void {
-		$this->config
-			->expects($this->once())
-			->method('deleteAppValue')
-			->with('theming', 'name');
-		$this->config
-			->expects($this->exactly(2))
-			->method('getAppValue')
-			->willReturnMap([
-				['theming', 'cachebuster', '0', '15'],
-				['theming', 'name', 'Nextcloud', 'Nextcloud'],
-			]);
-		$this->config
-			->expects($this->once())
-			->method('setAppValue')
-			->with('theming', 'cachebuster', 16);
-
-		$this->assertSame('Nextcloud', $this->template->undo('name'));
-	}
-
-	public function testUndoBaseUrl(): void {
-		$this->config
-			->expects($this->once())
-			->method('deleteAppValue')
-			->with('theming', 'url');
-		$this->config
-			->expects($this->exactly(2))
-			->method('getAppValue')
-			->willReturnMap([
-				['theming', 'cachebuster', '0', '15'],
-				['theming', 'url', $this->defaults->getBaseUrl(), $this->defaults->getBaseUrl()],
-			]);
-		$this->config
-			->expects($this->once())
-			->method('setAppValue')
-			->with('theming', 'cachebuster', 16);
-
-		$this->assertSame($this->defaults->getBaseUrl(), $this->template->undo('url'));
-	}
-
-	public function testUndoSlogan(): void {
-		$this->config
-			->expects($this->once())
-			->method('deleteAppValue')
-			->with('theming', 'slogan');
-		$this->config
-			->expects($this->exactly(2))
-			->method('getAppValue')
-			->willReturnMap([
-				['theming', 'cachebuster', '0', '15'],
-				['theming', 'slogan', $this->defaults->getSlogan(), $this->defaults->getSlogan()],
-			]);
-		$this->config
-			->expects($this->once())
-			->method('setAppValue')
-			->with('theming', 'cachebuster', 16);
-
-		$this->assertSame($this->defaults->getSlogan(), $this->template->undo('slogan'));
-	}
-
-	public function testUndoPrimaryColor(): void {
-		$this->config
-			->expects($this->once())
-			->method('deleteAppValue')
-			->with('theming', 'primary_color');
-		$this->config
-			->expects($this->once())
-			->method('getAppValue')
-			->with('theming', 'cachebuster', '0')
-			->willReturn('15');
-		$this->config
-			->expects($this->once())
-			->method('setAppValue')
-			->with('theming', 'cachebuster', 16);
-
-		$this->assertSame($this->defaults->getColorPrimary(), $this->template->undo('primary_color'));
-	}
-
-	public function testUndoDefaultAction(): void {
-		$this->config
-			->expects($this->once())
-			->method('deleteAppValue')
-			->with('theming', 'defaultitem');
-		$this->config
-			->expects($this->once())
-			->method('getAppValue')
-			->with('theming', 'cachebuster', '0')
-			->willReturn('15');
-		$this->config
-			->expects($this->once())
-			->method('setAppValue')
-			->with('theming', 'cachebuster', 16);
-
-		$this->assertSame('', $this->template->undo('defaultitem'));
-	}
-
-	public function testGetBackground(): void {
-		$this->imageManager
-			->expects($this->once())
-			->method('getImageUrl')
-			->with('background')
-			->willReturn('custom-background?v=0');
-		$this->assertEquals('custom-background?v=0', $this->template->getBackground());
-	}
-
-	private function getLogoHelper($withName, $useSvg) {
-		$this->imageManager->expects($this->any())
-			->method('getImage')
-			->with('logo')
-			->willThrowException(new NotFoundException());
-		$this->config
-			->expects($this->exactly(2))
-			->method('getAppValue')
-			->willReturnMap([
-				['theming', 'logoMime', '', ''],
-				['theming', 'cachebuster', '0', '0'],
-			]);
-		$this->urlGenerator->expects($this->once())
-			->method('imagePath')
-			->with('core', $withName)
-			->willReturn('core-logo');
-		$this->assertEquals('core-logo?v=0', $this->template->getLogo($useSvg));
-	}
-
-	public function testGetLogoDefaultWithSvg(): void {
-		$this->getLogoHelper('logo/logo.svg', true);
-	}
-
-	public function testGetLogoDefaultWithoutSvg(): void {
-		$this->getLogoHelper('logo/logo.png', false);
-	}
-
-	public function testGetLogoCustom(): void {
-		$this->config
-			->expects($this->exactly(2))
-			->method('getAppValue')
-			->willReturnMap([
-				['theming', 'logoMime', '', 'image/svg+xml'],
-				['theming', 'cachebuster', '0', '0'],
-			]);
-		$this->urlGenerator->expects($this->once())
-			->method('linkToRoute')
-			->with('theming.Theming.getImage')
-			->willReturn('custom-logo?v=0');
-		$this->assertEquals('custom-logo' . '?v=0', $this->template->getLogo());
-	}
-
-	public function testGetScssVariablesCached(): void {
-		$this->config->expects($this->any())->method('getAppValue')->with('theming', 'cachebuster', '0')->willReturn('1');
-		$this->cacheFactory->expects($this->once())
-			->method('createDistributed')
-			->with('theming-1-')
-			->willReturn($this->cache);
-		$this->cache->expects($this->once())->method('get')->with('getScssVariables')->willReturn(['foo' => 'bar']);
-		$this->assertEquals(['foo' => 'bar'], $this->template->getScssVariables());
-	}
-
-	public function testGetScssVariables(): void {
-		$this->config
-			->expects($this->any())
-			->method('getAppValue')
-			->willReturnMap([
-				['theming', 'cachebuster', '0', '0'],
-				['theming', 'logoMime', '', 'jpeg'],
-				['theming', 'backgroundMime', '', 'jpeg'],
-				['theming', 'logoheaderMime', '', 'jpeg'],
-				['theming', 'faviconMime', '', 'jpeg'],
-			]);
-
-		$this->appConfig
-			->expects(self::atLeastOnce())
-			->method('getValueString')
-			->willReturnMap([
-				['theming', 'primary_color', '', false, $this->defaults->getColorPrimary()],
-				['theming', 'primary_color', $this->defaults->getColorPrimary(), false, $this->defaults->getColorPrimary()],
-			]);
-
-		$this->util->expects($this->any())->method('invertTextColor')->with($this->defaults->getColorPrimary())->willReturn(false);
-		$this->util->expects($this->any())->method('elementColor')->with($this->defaults->getColorPrimary())->willReturn('#aaaaaa');
-		$this->cacheFactory->expects($this->once())
-			->method('createDistributed')
-			->with('theming-0-')
-			->willReturn($this->cache);
-		$this->cache->expects($this->once())->method('get')->with('getScssVariables')->willReturn(null);
-		$this->imageManager->expects($this->exactly(4))
-			->method('getImageUrl')
-			->willReturnMap([
-				['logo', 'custom-logo?v=0'],
-				['logoheader', 'custom-logoheader?v=0'],
-				['favicon', 'custom-favicon?v=0'],
-				['background', 'custom-background?v=0'],
-			]);
-
-		$expected = [
-			'theming-cachebuster' => '\'0\'',
-			'theming-logo-mime' => '\'jpeg\'',
-			'theming-background-mime' => '\'jpeg\'',
-			'image-logo' => "url('custom-logo?v=0')",
-			'image-login-background' => "url('custom-background?v=0')",
-			'color-primary' => $this->defaults->getColorPrimary(),
-			'color-primary-text' => '#ffffff',
-			'image-login-plain' => 'false',
-			'color-primary-element' => '#aaaaaa',
-			'theming-logoheader-mime' => '\'jpeg\'',
-			'theming-favicon-mime' => '\'jpeg\'',
-			'image-logoheader' => "url('custom-logoheader?v=0')",
-			'image-favicon' => "url('custom-favicon?v=0')",
-			'has-legal-links' => 'false'
-		];
-		$this->assertEquals($expected, $this->template->getScssVariables());
-	}
-
-	public function testGetDefaultAndroidURL(): void {
-		$this->config
-			->expects($this->once())
-			->method('getAppValue')
-			->with('theming', 'AndroidClientUrl', 'https://play.google.com/store/apps/details?id=com.nextcloud.client')
-			->willReturn('https://play.google.com/store/apps/details?id=com.nextcloud.client');
-
-		$this->assertEquals('https://play.google.com/store/apps/details?id=com.nextcloud.client', $this->template->getAndroidClientUrl());
-	}
-
-	public function testGetCustomAndroidURL(): void {
-		$this->config
-			->expects($this->once())
-			->method('getAppValue')
-			->with('theming', 'AndroidClientUrl', 'https://play.google.com/store/apps/details?id=com.nextcloud.client')
-			->willReturn('https://play.google.com/store/apps/details?id=com.mycloud.client');
-
-		$this->assertEquals('https://play.google.com/store/apps/details?id=com.mycloud.client', $this->template->getAndroidClientUrl());
-	}
-
-	public function testGetDefaultiOSURL(): void {
-		$this->config
-			->expects($this->once())
-			->method('getAppValue')
-			->with('theming', 'iOSClientUrl', 'https://geo.itunes.apple.com/us/app/nextcloud/id1125420102?mt=8')
-			->willReturn('https://geo.itunes.apple.com/us/app/nextcloud/id1125420102?mt=8');
-
-		$this->assertEquals('https://geo.itunes.apple.com/us/app/nextcloud/id1125420102?mt=8', $this->template->getiOSClientUrl());
-	}
-
-	public function testGetCustomiOSURL(): void {
-		$this->config
-			->expects($this->once())
-			->method('getAppValue')
-			->with('theming', 'iOSClientUrl', 'https://geo.itunes.apple.com/us/app/nextcloud/id1125420102?mt=8')
-			->willReturn('https://geo.itunes.apple.com/us/app/nextcloud/id1234567890?mt=8');
-
-		$this->assertEquals('https://geo.itunes.apple.com/us/app/nextcloud/id1234567890?mt=8', $this->template->getiOSClientUrl());
-	}
-
-	public function testGetDefaultiTunesAppId(): void {
-		$this->config
-			->expects($this->once())
-			->method('getAppValue')
-			->with('theming', 'iTunesAppId', '1125420102')
-			->willReturn('1125420102');
-
-		$this->assertEquals('1125420102', $this->template->getiTunesAppId());
-	}
-
-	public function testGetCustomiTunesAppId(): void {
-		$this->config
-			->expects($this->once())
-			->method('getAppValue')
-			->with('theming', 'iTunesAppId', '1125420102')
-			->willReturn('1234567890');
-
-		$this->assertEquals('1234567890', $this->template->getiTunesAppId());
-	}
-
-	public static function dataReplaceImagePath(): array {
-		return [
-			['core', 'test.png', false],
-			['core', 'manifest.json'],
-			['core', 'favicon.ico'],
-			['core', 'favicon-touch.png']
-		];
-	}
-
-	#[\PHPUnit\Framework\Attributes\DataProvider('dataReplaceImagePath')]
-	public function testReplaceImagePath(string $app, string $image, string|bool $result = 'themingRoute?v=1234abcd'): void {
-		$this->cache->expects($this->any())
-			->method('get')
-			->with('shouldReplaceIcons')
-			->willReturn(true);
-		$this->config
-			->expects($this->any())
-			->method('getAppValue')
-			->with('theming', 'cachebuster', '0')
-			->willReturn('0');
-		$this->urlGenerator
-			->expects($this->any())
-			->method('linkToRoute')
-			->willReturn('themingRoute');
-		if ($result) {
-			$this->util
-				->expects($this->once())
-				->method('getCacheBuster')
-				->willReturn('1234abcd');
-		}
-		$this->assertEquals($result, $this->template->replaceImagePath($app, $image));
-	}
+    private IAppConfig&MockObject $appConfig;
+    private IConfig&MockObject $config;
+    private \OC_Defaults $defaults;
+    private IL10N|MockObject $l10n;
+    private IUserSession&MockObject $userSession;
+    private IURLGenerator&MockObject $urlGenerator;
+    private ICacheFactory&MockObject $cacheFactory;
+    private Util&MockObject $util;
+    private ICache&MockObject $cache;
+    private IAppManager&MockObject $appManager;
+    private ImageManager&MockObject $imageManager;
+    private INavigationManager&MockObject $navigationManager;
+    private BackgroundService&MockObject $backgroundService;
+    private ThemingDefaults $template;
+
+    protected function setUp(): void {
+        parent::setUp();
+        $this->appConfig = $this->createMock(IAppConfig::class);
+        $this->config = $this->createMock(IConfig::class);
+        $this->l10n = $this->createMock(IL10N::class);
+        $this->userSession = $this->createMock(IUserSession::class);
+        $this->urlGenerator = $this->createMock(IURLGenerator::class);
+        $this->cacheFactory = $this->createMock(ICacheFactory::class);
+        $this->cache = $this->createMock(ICache::class);
+        $this->util = $this->createMock(Util::class);
+        $this->imageManager = $this->createMock(ImageManager::class);
+        $this->appManager = $this->createMock(IAppManager::class);
+        $this->navigationManager = $this->createMock(INavigationManager::class);
+        $this->backgroundService = $this->createMock(BackgroundService::class);
+        $this->defaults = new \OC_Defaults();
+        $this->urlGenerator
+            ->expects($this->any())
+            ->method('getBaseUrl')
+            ->willReturn('');
+        $this->template = new ThemingDefaults(
+            $this->config,
+            $this->appConfig,
+            $this->l10n,
+            $this->userSession,
+            $this->urlGenerator,
+            $this->cacheFactory,
+            $this->util,
+            $this->imageManager,
+            $this->appManager,
+            $this->navigationManager,
+            $this->backgroundService,
+        );
+    }
+
+    public function testGetNameWithDefault(): void {
+        $this->config
+            ->expects($this->once())
+            ->method('getAppValue')
+            ->with('theming', 'name', 'Nextcloud')
+            ->willReturn('Nextcloud');
+
+        $this->assertEquals('Nextcloud', $this->template->getName());
+    }
+
+    public function testGetNameWithCustom(): void {
+        $this->config
+            ->expects($this->once())
+            ->method('getAppValue')
+            ->with('theming', 'name', 'Nextcloud')
+            ->willReturn('MyCustomCloud');
+
+        $this->assertEquals('MyCustomCloud', $this->template->getName());
+    }
+
+    public function testGetHTMLNameWithDefault(): void {
+        $this->config
+            ->expects($this->once())
+            ->method('getAppValue')
+            ->with('theming', 'name', 'Nextcloud')
+            ->willReturn('Nextcloud');
+
+        $this->assertEquals('Nextcloud', $this->template->getHTMLName());
+    }
+
+    public function testGetHTMLNameWithCustom(): void {
+        $this->config
+            ->expects($this->once())
+            ->method('getAppValue')
+            ->with('theming', 'name', 'Nextcloud')
+            ->willReturn('MyCustomCloud');
+
+        $this->assertEquals('MyCustomCloud', $this->template->getHTMLName());
+    }
+
+    public function testGetTitleWithDefault(): void {
+        $this->config
+            ->expects($this->once())
+            ->method('getAppValue')
+            ->with('theming', 'name', 'Nextcloud')
+            ->willReturn('Nextcloud');
+
+        $this->assertEquals('Nextcloud', $this->template->getTitle());
+    }
+
+    public function testGetTitleWithCustom(): void {
+        $this->config
+            ->expects($this->once())
+            ->method('getAppValue')
+            ->with('theming', 'name', 'Nextcloud')
+            ->willReturn('MyCustomCloud');
+
+        $this->assertEquals('MyCustomCloud', $this->template->getTitle());
+    }
+
+
+    public function testGetEntityWithDefault(): void {
+        $this->config
+            ->expects($this->once())
+            ->method('getAppValue')
+            ->with('theming', 'name', 'Nextcloud')
+            ->willReturn('Nextcloud');
+
+        $this->assertEquals('Nextcloud', $this->template->getEntity());
+    }
+
+    public function testGetEntityWithCustom(): void {
+        $this->config
+            ->expects($this->once())
+            ->method('getAppValue')
+            ->with('theming', 'name', 'Nextcloud')
+            ->willReturn('MyCustomCloud');
+
+        $this->assertEquals('MyCustomCloud', $this->template->getEntity());
+    }
+
+    public function testGetBaseUrlWithDefault(): void {
+        $this->config
+            ->expects($this->once())
+            ->method('getAppValue')
+            ->with('theming', 'url', $this->defaults->getBaseUrl())
+            ->willReturn($this->defaults->getBaseUrl());
+
+        $this->assertEquals($this->defaults->getBaseUrl(), $this->template->getBaseUrl());
+    }
+
+    public function testGetBaseUrlWithCustom(): void {
+        $this->config
+            ->expects($this->once())
+            ->method('getAppValue')
+            ->with('theming', 'url', $this->defaults->getBaseUrl())
+            ->willReturn('https://example.com/');
+
+        $this->assertEquals('https://example.com/', $this->template->getBaseUrl());
+    }
+
+    public static function legalUrlProvider(): array {
+        return [
+            [''],
+            ['https://example.com/legal.html'],
+        ];
+    }
+
+    #[\PHPUnit\Framework\Attributes\DataProvider('legalUrlProvider')]
+    public function testGetImprintURL(string $imprintUrl): void {
+        $this->config
+            ->expects($this->once())
+            ->method('getAppValue')
+            ->with('theming', 'imprintUrl', '')
+            ->willReturn($imprintUrl);
+
+        $this->assertEquals($imprintUrl, $this->template->getImprintUrl());
+    }
+
+    #[\PHPUnit\Framework\Attributes\DataProvider('legalUrlProvider')]
+    public function testGetPrivacyURL(string $privacyUrl): void {
+        $this->config
+            ->expects($this->once())
+            ->method('getAppValue')
+            ->with('theming', 'privacyUrl', '')
+            ->willReturn($privacyUrl);
+
+        $this->assertEquals($privacyUrl, $this->template->getPrivacyUrl());
+    }
+
+    public function testGetSloganWithDefault(): void {
+        $this->config
+            ->expects($this->once())
+            ->method('getAppValue')
+            ->with('theming', 'slogan', $this->defaults->getSlogan())
+            ->willReturn($this->defaults->getSlogan());
+
+        $this->assertEquals($this->defaults->getSlogan(), $this->template->getSlogan());
+    }
+
+    public function testGetSloganWithCustom(): void {
+        $this->config
+            ->expects($this->once())
+            ->method('getAppValue')
+            ->with('theming', 'slogan', $this->defaults->getSlogan())
+            ->willReturn('My custom Slogan');
+
+        $this->assertEquals('My custom Slogan', $this->template->getSlogan());
+    }
+
+    public function testGetShortFooter(): void {
+        $this->config
+            ->expects($this->exactly(5))
+            ->method('getAppValue')
+            ->willReturnMap([
+                ['theming', 'url', $this->defaults->getBaseUrl(), 'url'],
+                ['theming', 'name', 'Nextcloud', 'Name'],
+                ['theming', 'slogan', $this->defaults->getSlogan(), 'Slogan'],
+                ['theming', 'imprintUrl', '', ''],
+                ['theming', 'privacyUrl', '', ''],
+            ]);
+
+        $this->assertEquals('<a href="url" target="_blank" rel="noreferrer noopener" class="entity-name">Name</a> – Slogan', $this->template->getShortFooter());
+    }
+
+    public function testGetShortFooterEmptyUrl(): void {
+        $this->navigationManager->expects($this->once())->method('getAll')->with(INavigationManager::TYPE_GUEST)->willReturn([]);
+        $this->config
+            ->expects($this->exactly(5))
+            ->method('getAppValue')
+            ->willReturnMap([
+                ['theming', 'url', $this->defaults->getBaseUrl(), ''],
+                ['theming', 'name', 'Nextcloud', 'Name'],
+                ['theming', 'slogan', $this->defaults->getSlogan(), 'Slogan'],
+                ['theming', 'imprintUrl', '', ''],
+                ['theming', 'privacyUrl', '', ''],
+            ]);
+
+        $this->assertEquals('<span class="entity-name">Name</span> – Slogan', $this->template->getShortFooter());
+    }
+
+    public function testGetShortFooterEmptySlogan(): void {
+        $this->navigationManager->expects($this->once())->method('getAll')->with(INavigationManager::TYPE_GUEST)->willReturn([]);
+        $this->config
+            ->expects($this->exactly(5))
+            ->method('getAppValue')
+            ->willReturnMap([
+                ['theming', 'url', $this->defaults->getBaseUrl(), 'url'],
+                ['theming', 'name', 'Nextcloud', 'Name'],
+                ['theming', 'slogan', $this->defaults->getSlogan(), ''],
+                ['theming', 'imprintUrl', '', ''],
+                ['theming', 'privacyUrl', '', ''],
+            ]);
+
+        $this->assertEquals('<a href="url" target="_blank" rel="noreferrer noopener" class="entity-name">Name</a>', $this->template->getShortFooter());
+    }
+
+    public function testGetShortFooterImprint(): void {
+        $this->navigationManager->expects($this->once())->method('getAll')->with(INavigationManager::TYPE_GUEST)->willReturn([]);
+        $this->config
+            ->expects($this->exactly(5))
+            ->method('getAppValue')
+            ->willReturnMap([
+                ['theming', 'url', $this->defaults->getBaseUrl(), 'url'],
+                ['theming', 'name', 'Nextcloud', 'Name'],
+                ['theming', 'slogan', $this->defaults->getSlogan(), 'Slogan'],
+                ['theming', 'imprintUrl', '', 'https://example.com/imprint'],
+                ['theming', 'privacyUrl', '', ''],
+            ]);
+
+        $this->l10n
+            ->expects($this->any())
+            ->method('t')
+            ->willReturnArgument(0);
+
+        $this->assertEquals('<a href="url" target="_blank" rel="noreferrer noopener" class="entity-name">Name</a> – Slogan<br/><span class="footer__legal-links"><a href="https://example.com/imprint" class="legal" target="_blank" rel="noreferrer noopener">Legal notice</a></span>', $this->template->getShortFooter());
+    }
+
+    public function testGetShortFooterPrivacy(): void {
+        $this->navigationManager->expects($this->once())->method('getAll')->with(INavigationManager::TYPE_GUEST)->willReturn([]);
+        $this->config
+            ->expects($this->exactly(5))
+            ->method('getAppValue')
+            ->willReturnMap([
+                ['theming', 'url', $this->defaults->getBaseUrl(), 'url'],
+                ['theming', 'name', 'Nextcloud', 'Name'],
+                ['theming', 'slogan', $this->defaults->getSlogan(), 'Slogan'],
+                ['theming', 'imprintUrl', '', ''],
+                ['theming', 'privacyUrl', '', 'https://example.com/privacy'],
+            ]);
+
+        $this->l10n
+            ->expects($this->any())
+            ->method('t')
+            ->willReturnArgument(0);
+
+        $this->assertEquals('<a href="url" target="_blank" rel="noreferrer noopener" class="entity-name">Name</a> – Slogan<br/><span class="footer__legal-links"><a href="https://example.com/privacy" class="legal" target="_blank" rel="noreferrer noopener">Privacy policy</a></span>', $this->template->getShortFooter());
+    }
+
+    public function testGetShortFooterAllLegalLinks(): void {
+        $this->navigationManager->expects($this->once())->method('getAll')->with(INavigationManager::TYPE_GUEST)->willReturn([]);
+        $this->config
+            ->expects($this->exactly(5))
+            ->method('getAppValue')
+            ->willReturnMap([
+                ['theming', 'url', $this->defaults->getBaseUrl(), 'url'],
+                ['theming', 'name', 'Nextcloud', 'Name'],
+                ['theming', 'slogan', $this->defaults->getSlogan(), 'Slogan'],
+                ['theming', 'imprintUrl', '', 'https://example.com/imprint'],
+                ['theming', 'privacyUrl', '', 'https://example.com/privacy'],
+            ]);
+
+        $this->l10n
+            ->expects($this->any())
+            ->method('t')
+            ->willReturnArgument(0);
+
+        $this->assertEquals('<a href="url" target="_blank" rel="noreferrer noopener" class="entity-name">Name</a> – Slogan<br/><span class="footer__legal-links"><a href="https://example.com/imprint" class="legal" target="_blank" rel="noreferrer noopener">Legal notice</a> · <a href="https://example.com/privacy" class="legal" target="_blank" rel="noreferrer noopener">Privacy policy</a></span>', $this->template->getShortFooter());
+    }
+
+    public static function invalidLegalUrlProvider(): array {
+        return [
+            ['example.com/legal'],  # missing scheme
+            ['https:///legal'],     # missing host
+        ];
+    }
+
+    #[\PHPUnit\Framework\Attributes\DataProvider('invalidLegalUrlProvider')]
+    public function testGetShortFooterInvalidImprint(string $invalidImprintUrl): void {
+        $this->navigationManager->expects($this->once())->method('getAll')->with(INavigationManager::TYPE_GUEST)->willReturn([]);
+        $this->config
+            ->expects($this->exactly(5))
+            ->method('getAppValue')
+            ->willReturnMap([
+                ['theming', 'url', $this->defaults->getBaseUrl(), 'url'],
+                ['theming', 'name', 'Nextcloud', 'Name'],
+                ['theming', 'slogan', $this->defaults->getSlogan(), 'Slogan'],
+                ['theming', 'imprintUrl', '', $invalidImprintUrl],
+                ['theming', 'privacyUrl', '', ''],
+            ]);
+
+        $this->assertEquals('<a href="url" target="_blank" rel="noreferrer noopener" class="entity-name">Name</a> – Slogan', $this->template->getShortFooter());
+    }
+
+    #[\PHPUnit\Framework\Attributes\DataProvider('invalidLegalUrlProvider')]
+    public function testGetShortFooterInvalidPrivacy(string $invalidPrivacyUrl): void {
+        $this->navigationManager->expects($this->once())->method('getAll')->with(INavigationManager::TYPE_GUEST)->willReturn([]);
+        $this->config
+            ->expects($this->exactly(5))
+            ->method('getAppValue')
+            ->willReturnMap([
+                ['theming', 'url', $this->defaults->getBaseUrl(), 'url'],
+                ['theming', 'name', 'Nextcloud', 'Name'],
+                ['theming', 'slogan', $this->defaults->getSlogan(), 'Slogan'],
+                ['theming', 'imprintUrl', '', ''],
+                ['theming', 'privacyUrl', '', $invalidPrivacyUrl],
+            ]);
+
+        $this->assertEquals('<a href="url" target="_blank" rel="noreferrer noopener" class="entity-name">Name</a> – Slogan', $this->template->getShortFooter());
+    }
+
+    public function testGetColorPrimaryWithDefault(): void {
+        $this->appConfig
+            ->expects(self::once())
+            ->method('getValueBool')
+            ->with('theming', 'disable-user-theming')
+            ->willReturn(false);
+        $this->appConfig
+            ->expects(self::once())
+            ->method('getValueString')
+            ->with('theming', 'primary_color', '')
+            ->willReturn($this->defaults->getColorPrimary());
+
+        $this->assertEquals($this->defaults->getColorPrimary(), $this->template->getColorPrimary());
+    }
+
+    public function testGetColorPrimaryWithCustom(): void {
+        $this->appConfig
+            ->expects(self::once())
+            ->method('getValueBool')
+            ->with('theming', 'disable-user-theming')
+            ->willReturn(false);
+        $this->appConfig
+            ->expects(self::once())
+            ->method('getValueString')
+            ->with('theming', 'primary_color', '')
+            ->willReturn('#fff');
+
+        $this->assertEquals('#fff', $this->template->getColorPrimary());
+    }
+
+    public static function dataGetColorPrimary(): array {
+        return [
+            'with fallback default' => [
+                'disableTheming' => false,
+                'primaryColor' => '',
+                'userPrimaryColor' => '',
+                'expected' => BackgroundService::DEFAULT_COLOR,
+            ],
+            'with custom admin primary' => [
+                'disableTheming' => false,
+                'primaryColor' => '#aaa',
+                'userPrimaryColor' => '',
+                'expected' => '#aaa',
+            ],
+            'with custom invalid admin primary' => [
+                'disableTheming' => false,
+                'primaryColor' => 'invalid',
+                'userPrimaryColor' => '',
+                'expected' => BackgroundService::DEFAULT_COLOR,
+            ],
+            'with custom invalid user primary' => [
+                'disableTheming' => false,
+                'primaryColor' => '',
+                'userPrimaryColor' => 'invalid-name',
+                'expected' => BackgroundService::DEFAULT_COLOR,
+            ],
+            'with custom user primary' => [
+                'disableTheming' => false,
+                'primaryColor' => '',
+                'userPrimaryColor' => '#bbb',
+                'expected' => '#bbb',
+            ],
+            'with disabled user theming primary' => [
+                'disableTheming' => true,
+                'primaryColor' => '#aaa',
+                'userPrimaryColor' => '#bbb',
+                'expected' => '#aaa',
+            ],
+        ];
+    }
+
+    #[\PHPUnit\Framework\Attributes\DataProvider('dataGetColorPrimary')]
+    public function testGetColorPrimary(bool $disableTheming, string $primaryColor, string $userPrimaryColor, string $expected): void {
+        $user = $this->createMock(IUser::class);
+        $this->userSession->expects($this->any())
+            ->method('getUser')
+            ->willReturn($user);
+        $user->expects($this->any())
+            ->method('getUID')
+            ->willReturn('user');
+        $this->appConfig
+            ->expects(self::any())
+            ->method('getValueBool')
+            ->with('theming', 'disable-user-theming')
+            ->willReturn($disableTheming);
+        $this->appConfig
+            ->expects(self::any())
+            ->method('getValueString')
+            ->with('theming', 'primary_color', '')
+            ->willReturn($primaryColor);
+        $this->config
+            ->expects($this->any())
+            ->method('getUserValue')
+            ->with('user', 'theming', 'primary_color', '')
+            ->willReturn($userPrimaryColor);
+
+        $this->assertEquals($expected, $this->template->getColorPrimary());
+    }
+
+    public function testSet(): void {
+        $this->config
+            ->expects($this->once())
+            ->method('setAppValue')
+            ->with('theming', 'cachebuster', 16);
+        $this->appConfig
+            ->expects($this->once())
+            ->method('setValueString')
+            ->with('theming', 'MySetting', 'MyValue');
+        $this->config
+            ->expects($this->once())
+            ->method('getAppValue')
+            ->with('theming', 'cachebuster', '0')
+            ->willReturn('15');
+        $this->cacheFactory
+            ->expects($this->exactly(2))
+            ->method('createDistributed')
+            ->willReturnMap([
+                ['theming-', $this->cache],
+                ['imagePath', $this->cache],
+            ]);
+        $this->cache
+            ->expects($this->any())
+            ->method('clear')
+            ->with('');
+        $this->template->set('MySetting', 'MyValue');
+    }
+
+    public function testUndoName(): void {
+        $this->config
+            ->expects($this->once())
+            ->method('deleteAppValue')
+            ->with('theming', 'name');
+        $this->config
+            ->expects($this->exactly(2))
+            ->method('getAppValue')
+            ->willReturnMap([
+                ['theming', 'cachebuster', '0', '15'],
+                ['theming', 'name', 'Nextcloud', 'Nextcloud'],
+            ]);
+        $this->config
+            ->expects($this->once())
+            ->method('setAppValue')
+            ->with('theming', 'cachebuster', 16);
+
+        $this->assertSame('Nextcloud', $this->template->undo('name'));
+    }
+
+    public function testUndoBaseUrl(): void {
+        $this->config
+            ->expects($this->once())
+            ->method('deleteAppValue')
+            ->with('theming', 'url');
+        $this->config
+            ->expects($this->exactly(2))
+            ->method('getAppValue')
+            ->willReturnMap([
+                ['theming', 'cachebuster', '0', '15'],
+                ['theming', 'url', $this->defaults->getBaseUrl(), $this->defaults->getBaseUrl()],
+            ]);
+        $this->config
+            ->expects($this->once())
+            ->method('setAppValue')
+            ->with('theming', 'cachebuster', 16);
+
+        $this->assertSame($this->defaults->getBaseUrl(), $this->template->undo('url'));
+    }
+
+    public function testUndoSlogan(): void {
+        $this->config
+            ->expects($this->once())
+            ->method('deleteAppValue')
+            ->with('theming', 'slogan');
+        $this->config
+            ->expects($this->exactly(2))
+            ->method('getAppValue')
+            ->willReturnMap([
+                ['theming', 'cachebuster', '0', '15'],
+                ['theming', 'slogan', $this->defaults->getSlogan(), $this->defaults->getSlogan()],
+            ]);
+        $this->config
+            ->expects($this->once())
+            ->method('setAppValue')
+            ->with('theming', 'cachebuster', 16);
+
+        $this->assertSame($this->defaults->getSlogan(), $this->template->undo('slogan'));
+    }
+
+    public function testUndoPrimaryColor(): void {
+        $this->config
+            ->expects($this->once())
+            ->method('deleteAppValue')
+            ->with('theming', 'primary_color');
+        $this->config
+            ->expects($this->once())
+            ->method('getAppValue')
+            ->with('theming', 'cachebuster', '0')
+            ->willReturn('15');
+        $this->config
+            ->expects($this->once())
+            ->method('setAppValue')
+            ->with('theming', 'cachebuster', 16);
+
+        $this->assertSame($this->defaults->getColorPrimary(), $this->template->undo('primary_color'));
+    }
+
+    public function testUndoDefaultAction(): void {
+        $this->config
+            ->expects($this->once())
+            ->method('deleteAppValue')
+            ->with('theming', 'defaultitem');
+        $this->config
+            ->expects($this->once())
+            ->method('getAppValue')
+            ->with('theming', 'cachebuster', '0')
+            ->willReturn('15');
+        $this->config
+            ->expects($this->once())
+            ->method('setAppValue')
+            ->with('theming', 'cachebuster', 16);
+
+        $this->assertSame('', $this->template->undo('defaultitem'));
+    }
+
+    public function testGetBackground(): void {
+        $this->imageManager
+            ->expects($this->once())
+            ->method('getImageUrl')
+            ->with('background')
+            ->willReturn('custom-background?v=0');
+        $this->assertEquals('custom-background?v=0', $this->template->getBackground());
+    }
+
+    private function getLogoHelper($withName, $useSvg) {
+        $this->imageManager->expects($this->any())
+            ->method('getImage')
+            ->with('logo')
+            ->willThrowException(new NotFoundException());
+        $this->config
+            ->expects($this->exactly(2))
+            ->method('getAppValue')
+            ->willReturnMap([
+                ['theming', 'logoMime', '', ''],
+                ['theming', 'cachebuster', '0', '0'],
+            ]);
+        $this->urlGenerator->expects($this->once())
+            ->method('imagePath')
+            ->with('core', $withName)
+            ->willReturn('core-logo');
+        $this->assertEquals('core-logo?v=0', $this->template->getLogo($useSvg));
+    }
+
+    public function testGetLogoDefaultWithSvg(): void {
+        $this->getLogoHelper('logo/logo.svg', true);
+    }
+
+    public function testGetLogoDefaultWithoutSvg(): void {
+        $this->getLogoHelper('logo/logo.png', false);
+    }
+
+    public function testGetLogoCustom(): void {
+        $this->config
+            ->expects($this->exactly(2))
+            ->method('getAppValue')
+            ->willReturnMap([
+                ['theming', 'logoMime', '', 'image/svg+xml'],
+                ['theming', 'cachebuster', '0', '0'],
+            ]);
+        $this->urlGenerator->expects($this->once())
+            ->method('linkToRoute')
+            ->with('theming.Theming.getImage')
+            ->willReturn('custom-logo?v=0');
+        $this->assertEquals('custom-logo' . '?v=0', $this->template->getLogo());
+    }
+
+    public function testGetScssVariablesCached(): void {
+        $this->config->expects($this->any())->method('getAppValue')->with('theming', 'cachebuster', '0')->willReturn('1');
+        $this->cacheFactory->expects($this->once())
+            ->method('createDistributed')
+            ->with('theming-1-')
+            ->willReturn($this->cache);
+        $this->cache->expects($this->once())->method('get')->with('getScssVariables')->willReturn(['foo' => 'bar']);
+        $this->assertEquals(['foo' => 'bar'], $this->template->getScssVariables());
+    }
+
+    public function testGetScssVariables(): void {
+        $this->config
+            ->expects($this->any())
+            ->method('getAppValue')
+            ->willReturnMap([
+                ['theming', 'cachebuster', '0', '0'],
+                ['theming', 'logoMime', '', 'jpeg'],
+                ['theming', 'backgroundMime', '', 'jpeg'],
+                ['theming', 'logoheaderMime', '', 'jpeg'],
+                ['theming', 'faviconMime', '', 'jpeg'],
+            ]);
+
+        $this->appConfig
+            ->expects(self::atLeastOnce())
+            ->method('getValueString')
+            ->willReturnMap([
+                ['theming', 'primary_color', '', false, $this->defaults->getColorPrimary()],
+                ['theming', 'primary_color', $this->defaults->getColorPrimary(), false, $this->defaults->getColorPrimary()],
+            ]);
+
+        $this->util->expects($this->any())->method('invertTextColor')->with($this->defaults->getColorPrimary())->willReturn(false);
+        $this->util->expects($this->any())->method('elementColor')->with($this->defaults->getColorPrimary())->willReturn('#aaaaaa');
+        $this->cacheFactory->expects($this->once())
+            ->method('createDistributed')
+            ->with('theming-0-')
+            ->willReturn($this->cache);
+        $this->cache->expects($this->once())->method('get')->with('getScssVariables')->willReturn(null);
+        $this->imageManager->expects($this->exactly(4))
+            ->method('getImageUrl')
+            ->willReturnMap([
+                ['logo', 'custom-logo?v=0'],
+                ['logoheader', 'custom-logoheader?v=0'],
+                ['favicon', 'custom-favicon?v=0'],
+                ['background', 'custom-background?v=0'],
+            ]);
+
+        $expected = [
+            'theming-cachebuster' => '\'0\'',
+            'theming-logo-mime' => '\'jpeg\'',
+            'theming-background-mime' => '\'jpeg\'',
+            'image-logo' => "url('custom-logo?v=0')",
+            'image-login-background' => "url('custom-background?v=0')",
+            'color-primary' => $this->defaults->getColorPrimary(),
+            'color-primary-text' => '#ffffff',
+            'image-login-plain' => 'false',
+            'color-primary-element' => '#aaaaaa',
+            'theming-logoheader-mime' => '\'jpeg\'',
+            'theming-favicon-mime' => '\'jpeg\'',
+            'image-logoheader' => "url('custom-logoheader?v=0')",
+            'image-favicon' => "url('custom-favicon?v=0')",
+            'has-legal-links' => 'false'
+        ];
+        $this->assertEquals($expected, $this->template->getScssVariables());
+    }
+
+    public function testGetDefaultAndroidURL(): void {
+        $this->config
+            ->expects($this->once())
+            ->method('getAppValue')
+            ->with('theming', 'AndroidClientUrl', 'https://play.google.com/store/apps/details?id=com.nextcloud.client')
+            ->willReturn('https://play.google.com/store/apps/details?id=com.nextcloud.client');
+
+        $this->assertEquals('https://play.google.com/store/apps/details?id=com.nextcloud.client', $this->template->getAndroidClientUrl());
+    }
+
+    public function testGetCustomAndroidURL(): void {
+        $this->config
+            ->expects($this->once())
+            ->method('getAppValue')
+            ->with('theming', 'AndroidClientUrl', 'https://play.google.com/store/apps/details?id=com.nextcloud.client')
+            ->willReturn('https://play.google.com/store/apps/details?id=com.mycloud.client');
+
+        $this->assertEquals('https://play.google.com/store/apps/details?id=com.mycloud.client', $this->template->getAndroidClientUrl());
+    }
+
+    public function testGetDefaultiOSURL(): void {
+        $this->config
+            ->expects($this->once())
+            ->method('getAppValue')
+            ->with('theming', 'iOSClientUrl', 'https://geo.itunes.apple.com/us/app/nextcloud/id1125420102?mt=8')
+            ->willReturn('https://geo.itunes.apple.com/us/app/nextcloud/id1125420102?mt=8');
+
+        $this->assertEquals('https://geo.itunes.apple.com/us/app/nextcloud/id1125420102?mt=8', $this->template->getiOSClientUrl());
+    }
+
+    public function testGetCustomiOSURL(): void {
+        $this->config
+            ->expects($this->once())
+            ->method('getAppValue')
+            ->with('theming', 'iOSClientUrl', 'https://geo.itunes.apple.com/us/app/nextcloud/id1125420102?mt=8')
+            ->willReturn('https://geo.itunes.apple.com/us/app/nextcloud/id1234567890?mt=8');
+
+        $this->assertEquals('https://geo.itunes.apple.com/us/app/nextcloud/id1234567890?mt=8', $this->template->getiOSClientUrl());
+    }
+
+    public function testGetDefaultiTunesAppId(): void {
+        $this->config
+            ->expects($this->once())
+            ->method('getAppValue')
+            ->with('theming', 'iTunesAppId', '1125420102')
+            ->willReturn('1125420102');
+
+        $this->assertEquals('1125420102', $this->template->getiTunesAppId());
+    }
+
+    public function testGetCustomiTunesAppId(): void {
+        $this->config
+            ->expects($this->once())
+            ->method('getAppValue')
+            ->with('theming', 'iTunesAppId', '1125420102')
+            ->willReturn('1234567890');
+
+        $this->assertEquals('1234567890', $this->template->getiTunesAppId());
+    }
+
+    public static function dataReplaceImagePath(): array {
+        return [
+            ['core', 'test.png', false],
+            ['core', 'manifest.json'],
+            ['core', 'favicon.ico'],
+            ['core', 'favicon-touch.png']
+        ];
+    }
+
+    #[\PHPUnit\Framework\Attributes\DataProvider('dataReplaceImagePath')]
+    public function testReplaceImagePath(string $app, string $image, string|bool $result = 'themingRoute?v=1234abcd'): void {
+        $this->cache->expects($this->any())
+            ->method('get')
+            ->with('shouldReplaceIcons')
+            ->willReturn(true);
+        $this->config
+            ->expects($this->any())
+            ->method('getAppValue')
+            ->with('theming', 'cachebuster', '0')
+            ->willReturn('0');
+        $this->urlGenerator
+            ->expects($this->any())
+            ->method('linkToRoute')
+            ->willReturn('themingRoute');
+        if ($result) {
+            $this->util
+                ->expects($this->once())
+                ->method('getCacheBuster')
+                ->willReturn('1234abcd');
+        }
+        $this->assertEquals($result, $this->template->replaceImagePath($app, $image));
+    }
 }

Spacing +5 added lines, -5 removed lines

@@ -29,7 +29,7 @@ 
 	private IAppConfig&MockObject $appConfig;
 	private IConfig&MockObject $config;
 	private \OC_Defaults $defaults;
-	private IL10N|MockObject $l10n;
+	private IL10N | MockObject $l10n;
 	private IUserSession&MockObject $userSession;
 	private IURLGenerator&MockObject $urlGenerator;
 	private ICacheFactory&MockObject $cacheFactory;
@@ -337,8 +337,8 @@ 
 
 	public static function invalidLegalUrlProvider(): array {
 		return [
-			['example.com/legal'],  # missing scheme
-			['https:///legal'],     # missing host
+			['example.com/legal'], # missing scheme
+			['https:///legal'], # missing host
 		];
 	}
 
@@ -647,7 +647,7 @@ 
 			->method('linkToRoute')
 			->with('theming.Theming.getImage')
 			->willReturn('custom-logo?v=0');
-		$this->assertEquals('custom-logo' . '?v=0', $this->template->getLogo());
+		$this->assertEquals('custom-logo'.'?v=0', $this->template->getLogo());
 	}
 
 	public function testGetScssVariablesCached(): void {
@@ -785,7 +785,7 @@ 
 	}
 
 	#[\PHPUnit\Framework\Attributes\DataProvider('dataReplaceImagePath')]
-	public function testReplaceImagePath(string $app, string $image, string|bool $result = 'themingRoute?v=1234abcd'): void {
+	public function testReplaceImagePath(string $app, string $image, string | bool $result = 'themingRoute?v=1234abcd'): void {
 		$this->cache->expects($this->any())
 			->method('get')
 			->with('shouldReplaceIcons')