nextcloud / server

apps/theming/lib/Themes/DefaultTheme.php

Indentation +172 added lines, -172 removed lines

@@ -36,176 +36,176 @@
 use OCP\IUserSession;
 
 class DefaultTheme implements ITheme {
-	use CommonThemeTrait;
-
-	public Util $util;
-	public ThemingDefaults $themingDefaults;
-	public IUserSession $userSession;
-	public IURLGenerator $urlGenerator;
-	public ImageManager $imageManager;
-	public IConfig $config;
-	public IL10N $l;
-	public IAppManager $appManager;
-
-	public string $defaultPrimaryColor;
-	public string $primaryColor;
-
-	public function __construct(Util $util,
-								ThemingDefaults $themingDefaults,
-								IUserSession $userSession,
-								IURLGenerator $urlGenerator,
-								ImageManager $imageManager,
-								IConfig $config,
-								IL10N $l,
-								IAppManager $appManager) {
-		$this->util = $util;
-		$this->themingDefaults = $themingDefaults;
-		$this->userSession = $userSession;
-		$this->urlGenerator = $urlGenerator;
-		$this->imageManager = $imageManager;
-		$this->config = $config;
-		$this->l = $l;
-		$this->appManager = $appManager;
-
-		$this->defaultPrimaryColor = $this->themingDefaults->getDefaultColorPrimary();
-		$this->primaryColor = $this->themingDefaults->getColorPrimary();
-
-		// Override default defaultPrimaryColor if set to improve accessibility
-		if ($this->primaryColor === BackgroundService::DEFAULT_COLOR) {
-			$this->primaryColor = BackgroundService::DEFAULT_ACCESSIBLE_COLOR;
-		}
-	}
-
-	public function getId(): string {
-		return 'default';
-	}
-
-	public function getType(): int {
-		return ITheme::TYPE_THEME;
-	}
-
-	public function getTitle(): string {
-		return $this->l->t('System default theme');
-	}
-
-	public function getEnableLabel(): string {
-		return $this->l->t('Enable the system default');
-	}
-
-	public function getDescription(): string {
-		return $this->l->t('Using the default system appearance.');
-	}
-
-	public function getMediaQuery(): string {
-		return '';
-	}
-
-	public function getCSSVariables(): array {
-		$colorMainText = '#222222';
-		$colorMainTextRgb = join(',', $this->util->hexToRGB($colorMainText));
-		$colorTextMaxcontrast = $this->util->lighten($colorMainText, 33);
-		$colorMainBackground = '#ffffff';
-		$colorMainBackgroundRGB = join(',', $this->util->hexToRGB($colorMainBackground));
-		$colorBoxShadow = $this->util->darken($colorMainBackground, 70);
-		$colorBoxShadowRGB = join(',', $this->util->hexToRGB($colorBoxShadow));
-
-		$variables = [
-			'--color-main-background' => $colorMainBackground,
-			'--color-main-background-rgb' => $colorMainBackgroundRGB,
-			'--color-main-background-translucent' => 'rgba(var(--color-main-background-rgb), .97)',
-			'--color-main-background-blur' => 'rgba(var(--color-main-background-rgb), .8)',
-			'--filter-background-blur' => 'blur(25px)',
-
-			// to use like this: background-image: linear-gradient(0, var('--gradient-main-background));
-			'--gradient-main-background' => 'var(--color-main-background) 0%, var(--color-main-background-translucent) 85%, transparent 100%',
-
-			// used for different active/hover/focus/disabled states
-			'--color-background-hover' => $this->util->darken($colorMainBackground, 4),
-			'--color-background-dark' => $this->util->darken($colorMainBackground, 7),
-			'--color-background-darker' => $this->util->darken($colorMainBackground, 14),
-
-			'--color-placeholder-light' => $this->util->darken($colorMainBackground, 10),
-			'--color-placeholder-dark' => $this->util->darken($colorMainBackground, 20),
-
-			// max contrast for WCAG compliance
-			'--color-main-text' => $colorMainText,
-			'--color-text-maxcontrast' => $colorTextMaxcontrast,
-			'--color-text-maxcontrast-default' => $colorTextMaxcontrast,
-			'--color-text-maxcontrast-background-blur' => $this->util->darken($colorTextMaxcontrast, 7),
-			'--color-text-light' => $colorMainText,
-			'--color-text-lighter' => $this->util->lighten($colorMainText, 33),
-
-			'--color-scrollbar' => 'rgba(' . $colorMainTextRgb . ', .15)',
-
-			// info/warning/success feedback colours
-			'--color-error' => '#e9322d',
-			'--color-error-rgb' => join(',', $this->util->hexToRGB('#e9322d')),
-			'--color-error-hover' => $this->util->mix('#e9322d', $colorMainBackground, 60),
-			'--color-warning' => '#eca700',
-			'--color-warning-rgb' => join(',', $this->util->hexToRGB('#eca700')),
-			'--color-warning-hover' => $this->util->mix('#eca700', $colorMainBackground, 60),
-			'--color-success' => '#46ba61',
-			'--color-success-rgb' => join(',', $this->util->hexToRGB('#46ba61')),
-			'--color-success-hover' => $this->util->mix('#46ba61', $colorMainBackground, 60),
-
-			// used for the icon loading animation
-			'--color-loading-light' => '#cccccc',
-			'--color-loading-dark' => '#444444',
-
-			'--color-box-shadow-rgb' => $colorBoxShadowRGB,
-			'--color-box-shadow' => "rgba(var(--color-box-shadow-rgb), 0.5)",
-
-			'--color-border' => $this->util->darken($colorMainBackground, 7),
-			'--color-border-dark' => $this->util->darken($colorMainBackground, 14),
-
-			'--font-face' => "-apple-system, BlinkMacSystemFont, 'Segoe UI', Roboto, Oxygen-Sans, Cantarell, Ubuntu, 'Helvetica Neue', Arial, sans-serif, 'Noto Color Emoji', 'Apple Color Emoji', 'Segoe UI Emoji', 'Segoe UI Symbol'",
-			'--default-font-size' => '15px',
-
-			// TODO: support "(prefers-reduced-motion)"
-			'--animation-quick' => '100ms',
-			'--animation-slow' => '300ms',
-
-			// Default variables --------------------------------------------
-			'--border-radius' => '3px',
-			'--border-radius-large' => '10px',
-			// pill-style button, value is large so big buttons also have correct roundness
-			'--border-radius-pill' => '100px',
-
-			'--default-clickable-area' => '44px',
-			'--default-line-height' => '24px',
-			'--default-grid-baseline' => '4px',
-
-			// various structure data
-			'--header-height' => '50px',
-			'--navigation-width' => '300px',
-			'--sidebar-min-width' => '300px',
-			'--sidebar-max-width' => '500px',
-			'--list-min-width' => '200px',
-			'--list-max-width' => '300px',
-			'--header-menu-item-height' => '44px',
-			'--header-menu-profile-item-height' => '66px',
-
-			// mobile. Keep in sync with core/js/js.js
-			'--breakpoint-mobile' => '1024px',
-			'--background-invert-if-dark' => 'no',
-			'--background-invert-if-bright' => 'invert(100%)',
-			'--background-image-invert-if-bright' => 'no',
-
-			// Default last fallback values
-			'--image-background' => "url('" . $this->urlGenerator->imagePath('core', 'app-background.jpg') . "')",
-			'--image-background-default' => "url('" . $this->urlGenerator->imagePath('core', 'app-background.jpg') . "')",
-			'--color-background-plain' => $this->defaultPrimaryColor,
-		];
-
-		// Primary variables
-		$variables = array_merge($variables, $this->generatePrimaryVariables($colorMainBackground, $colorMainText));
-		$variables = array_merge($variables, $this->generateGlobalBackgroundVariables());
-		$variables = array_merge($variables, $this->generateUserBackgroundVariables());
-
-		return $variables;
-	}
-
-	public function getCustomCss(): string {
-		return '';
-	}
+    use CommonThemeTrait;
+
+    public Util $util;
+    public ThemingDefaults $themingDefaults;
+    public IUserSession $userSession;
+    public IURLGenerator $urlGenerator;
+    public ImageManager $imageManager;
+    public IConfig $config;
+    public IL10N $l;
+    public IAppManager $appManager;
+
+    public string $defaultPrimaryColor;
+    public string $primaryColor;
+
+    public function __construct(Util $util,
+                                ThemingDefaults $themingDefaults,
+                                IUserSession $userSession,
+                                IURLGenerator $urlGenerator,
+                                ImageManager $imageManager,
+                                IConfig $config,
+                                IL10N $l,
+                                IAppManager $appManager) {
+        $this->util = $util;
+        $this->themingDefaults = $themingDefaults;
+        $this->userSession = $userSession;
+        $this->urlGenerator = $urlGenerator;
+        $this->imageManager = $imageManager;
+        $this->config = $config;
+        $this->l = $l;
+        $this->appManager = $appManager;
+
+        $this->defaultPrimaryColor = $this->themingDefaults->getDefaultColorPrimary();
+        $this->primaryColor = $this->themingDefaults->getColorPrimary();
+
+        // Override default defaultPrimaryColor if set to improve accessibility
+        if ($this->primaryColor === BackgroundService::DEFAULT_COLOR) {
+            $this->primaryColor = BackgroundService::DEFAULT_ACCESSIBLE_COLOR;
+        }
+    }
+
+    public function getId(): string {
+        return 'default';
+    }
+
+    public function getType(): int {
+        return ITheme::TYPE_THEME;
+    }
+
+    public function getTitle(): string {
+        return $this->l->t('System default theme');
+    }
+
+    public function getEnableLabel(): string {
+        return $this->l->t('Enable the system default');
+    }
+
+    public function getDescription(): string {
+        return $this->l->t('Using the default system appearance.');
+    }
+
+    public function getMediaQuery(): string {
+        return '';
+    }
+
+    public function getCSSVariables(): array {
+        $colorMainText = '#222222';
+        $colorMainTextRgb = join(',', $this->util->hexToRGB($colorMainText));
+        $colorTextMaxcontrast = $this->util->lighten($colorMainText, 33);
+        $colorMainBackground = '#ffffff';
+        $colorMainBackgroundRGB = join(',', $this->util->hexToRGB($colorMainBackground));
+        $colorBoxShadow = $this->util->darken($colorMainBackground, 70);
+        $colorBoxShadowRGB = join(',', $this->util->hexToRGB($colorBoxShadow));
+
+        $variables = [
+            '--color-main-background' => $colorMainBackground,
+            '--color-main-background-rgb' => $colorMainBackgroundRGB,
+            '--color-main-background-translucent' => 'rgba(var(--color-main-background-rgb), .97)',
+            '--color-main-background-blur' => 'rgba(var(--color-main-background-rgb), .8)',
+            '--filter-background-blur' => 'blur(25px)',
+
+            // to use like this: background-image: linear-gradient(0, var('--gradient-main-background));
+            '--gradient-main-background' => 'var(--color-main-background) 0%, var(--color-main-background-translucent) 85%, transparent 100%',
+
+            // used for different active/hover/focus/disabled states
+            '--color-background-hover' => $this->util->darken($colorMainBackground, 4),
+            '--color-background-dark' => $this->util->darken($colorMainBackground, 7),
+            '--color-background-darker' => $this->util->darken($colorMainBackground, 14),
+
+            '--color-placeholder-light' => $this->util->darken($colorMainBackground, 10),
+            '--color-placeholder-dark' => $this->util->darken($colorMainBackground, 20),
+
+            // max contrast for WCAG compliance
+            '--color-main-text' => $colorMainText,
+            '--color-text-maxcontrast' => $colorTextMaxcontrast,
+            '--color-text-maxcontrast-default' => $colorTextMaxcontrast,
+            '--color-text-maxcontrast-background-blur' => $this->util->darken($colorTextMaxcontrast, 7),
+            '--color-text-light' => $colorMainText,
+            '--color-text-lighter' => $this->util->lighten($colorMainText, 33),
+
+            '--color-scrollbar' => 'rgba(' . $colorMainTextRgb . ', .15)',
+
+            // info/warning/success feedback colours
+            '--color-error' => '#e9322d',
+            '--color-error-rgb' => join(',', $this->util->hexToRGB('#e9322d')),
+            '--color-error-hover' => $this->util->mix('#e9322d', $colorMainBackground, 60),
+            '--color-warning' => '#eca700',
+            '--color-warning-rgb' => join(',', $this->util->hexToRGB('#eca700')),
+            '--color-warning-hover' => $this->util->mix('#eca700', $colorMainBackground, 60),
+            '--color-success' => '#46ba61',
+            '--color-success-rgb' => join(',', $this->util->hexToRGB('#46ba61')),
+            '--color-success-hover' => $this->util->mix('#46ba61', $colorMainBackground, 60),
+
+            // used for the icon loading animation
+            '--color-loading-light' => '#cccccc',
+            '--color-loading-dark' => '#444444',
+
+            '--color-box-shadow-rgb' => $colorBoxShadowRGB,
+            '--color-box-shadow' => "rgba(var(--color-box-shadow-rgb), 0.5)",
+
+            '--color-border' => $this->util->darken($colorMainBackground, 7),
+            '--color-border-dark' => $this->util->darken($colorMainBackground, 14),
+
+            '--font-face' => "-apple-system, BlinkMacSystemFont, 'Segoe UI', Roboto, Oxygen-Sans, Cantarell, Ubuntu, 'Helvetica Neue', Arial, sans-serif, 'Noto Color Emoji', 'Apple Color Emoji', 'Segoe UI Emoji', 'Segoe UI Symbol'",
+            '--default-font-size' => '15px',
+
+            // TODO: support "(prefers-reduced-motion)"
+            '--animation-quick' => '100ms',
+            '--animation-slow' => '300ms',
+
+            // Default variables --------------------------------------------
+            '--border-radius' => '3px',
+            '--border-radius-large' => '10px',
+            // pill-style button, value is large so big buttons also have correct roundness
+            '--border-radius-pill' => '100px',
+
+            '--default-clickable-area' => '44px',
+            '--default-line-height' => '24px',
+            '--default-grid-baseline' => '4px',
+
+            // various structure data
+            '--header-height' => '50px',
+            '--navigation-width' => '300px',
+            '--sidebar-min-width' => '300px',
+            '--sidebar-max-width' => '500px',
+            '--list-min-width' => '200px',
+            '--list-max-width' => '300px',
+            '--header-menu-item-height' => '44px',
+            '--header-menu-profile-item-height' => '66px',
+
+            // mobile. Keep in sync with core/js/js.js
+            '--breakpoint-mobile' => '1024px',
+            '--background-invert-if-dark' => 'no',
+            '--background-invert-if-bright' => 'invert(100%)',
+            '--background-image-invert-if-bright' => 'no',
+
+            // Default last fallback values
+            '--image-background' => "url('" . $this->urlGenerator->imagePath('core', 'app-background.jpg') . "')",
+            '--image-background-default' => "url('" . $this->urlGenerator->imagePath('core', 'app-background.jpg') . "')",
+            '--color-background-plain' => $this->defaultPrimaryColor,
+        ];
+
+        // Primary variables
+        $variables = array_merge($variables, $this->generatePrimaryVariables($colorMainBackground, $colorMainText));
+        $variables = array_merge($variables, $this->generateGlobalBackgroundVariables());
+        $variables = array_merge($variables, $this->generateUserBackgroundVariables());
+
+        return $variables;
+    }
+
+    public function getCustomCss(): string {
+        return '';
+    }
 }

Spacing +3 added lines, -3 removed lines

@@ -135,7 +135,7 @@ 
 			'--color-text-light' => $colorMainText,
 			'--color-text-lighter' => $this->util->lighten($colorMainText, 33),
 
-			'--color-scrollbar' => 'rgba(' . $colorMainTextRgb . ', .15)',
+			'--color-scrollbar' => 'rgba('.$colorMainTextRgb.', .15)',
 
 			// info/warning/success feedback colours
 			'--color-error' => '#e9322d',
@@ -192,8 +192,8 @@ 
 			'--background-image-invert-if-bright' => 'no',
 
 			// Default last fallback values
-			'--image-background' => "url('" . $this->urlGenerator->imagePath('core', 'app-background.jpg') . "')",
-			'--image-background-default' => "url('" . $this->urlGenerator->imagePath('core', 'app-background.jpg') . "')",
+			'--image-background' => "url('".$this->urlGenerator->imagePath('core', 'app-background.jpg')."')",
+			'--image-background-default' => "url('".$this->urlGenerator->imagePath('core', 'app-background.jpg')."')",
 			'--color-background-plain' => $this->defaultPrimaryColor,
 		];
 

apps/theming/lib/Themes/CommonThemeTrait.php

Indentation +131 added lines, -131 removed lines

@@ -30,135 +30,135 @@
 use OCA\Theming\Util;
 
 trait CommonThemeTrait {
-	public Util $util;
-
-	/**
-	 * Generate primary-related variables
-	 * This is shared between multiple themes because colorMainBackground and colorMainText
-	 * will change in between.
-	 */
-	protected function generatePrimaryVariables(string $colorMainBackground, string $colorMainText): array {
-		$colorPrimaryLight = $this->util->mix($this->primaryColor, $colorMainBackground, -80);
-		$colorPrimaryElement = $this->util->elementColor($this->primaryColor);
-		$colorPrimaryElementDefault = $this->util->elementColor($this->defaultPrimaryColor);
-		$colorPrimaryElementLight = $this->util->mix($colorPrimaryElement, $colorMainBackground, -80);
-
-		// primary related colours
-		return [
-			// invert filter if primary is too bright
-			// to be used for legacy reasons only. Use inline
-			// svg with proper css variable instead or material
-			// design icons.
-			// ⚠️ Using 'no' as a value to make sure we specify an
-			// invalid one with no fallback. 'unset' could here fallback to some
-			// other theme with media queries
-			'--primary-invert-if-bright' => $this->util->invertTextColor($this->primaryColor) ? 'invert(100%)' : 'no',
-
-			'--color-primary' => $this->primaryColor,
-			'--color-primary-default' => $this->defaultPrimaryColor,
-			'--color-primary-text' => $this->util->invertTextColor($this->primaryColor) ? '#000000' : '#ffffff',
-			'--color-primary-hover' => $this->util->mix($this->primaryColor, $colorMainBackground, 60),
-			'--color-primary-light' => $colorPrimaryLight,
-			'--color-primary-light-text' => $this->util->mix($this->primaryColor, $this->util->invertTextColor($colorPrimaryLight) ? '#000000' : '#ffffff', -20),
-			'--color-primary-light-hover' => $this->util->mix($colorPrimaryLight, $colorMainText, 90),
-			'--color-primary-text-dark' => $this->util->darken($this->util->invertTextColor($this->primaryColor) ? '#000000' : '#ffffff', 7),
-
-			// used for buttons, inputs...
-			'--color-primary-element' => $colorPrimaryElement,
-			'--color-primary-element-default-hover' => $this->util->mix($colorPrimaryElementDefault, $colorMainBackground, 60),
-			'--color-primary-element-text' => $this->util->invertTextColor($colorPrimaryElement) ? '#000000' : '#ffffff',
-			'--color-primary-element-hover' => $this->util->mix($colorPrimaryElement, $colorMainBackground, 60),
-			'--color-primary-element-light' => $colorPrimaryElementLight,
-			'--color-primary-element-light-text' => $this->util->mix($colorPrimaryElement, $this->util->invertTextColor($colorPrimaryElementLight) ? '#000000' : '#ffffff', -20),
-			'--color-primary-element-light-hover' => $this->util->mix($colorPrimaryElementLight, $colorMainText, 90),
-			'--color-primary-element-text-dark' => $this->util->darken($this->util->invertTextColor($colorPrimaryElement) ? '#000000' : '#ffffff', 7),
-
-			// to use like this: background-image: var(--gradient-primary-background);
-			'--gradient-primary-background' => 'linear-gradient(40deg, var(--color-primary) 0%, var(--color-primary-hover) 100%)',
-		];
-	}
-
-	/**
-	 * Generate admin theming background-related variables
-	 */
-	protected function generateGlobalBackgroundVariables(): array {
-		$user = $this->userSession->getUser();
-		$backgroundDeleted = $this->config->getAppValue(Application::APP_ID, 'backgroundMime', '') === 'backgroundColor';
-		$hasCustomLogoHeader = $this->imageManager->hasImage('logo') || $this->imageManager->hasImage('logoheader');
-
-		$variables = [];
-
-		// If primary as background has been request or if we have a custom primary colour
-		// let's not define the background image
-		if ($backgroundDeleted) {
-			$variables['--color-background-plain'] = $this->themingDefaults->getColorPrimary();
-			if ($this->themingDefaults->isUserThemingDisabled() || $user === null) {
-				$variables['--image-background-plain'] = 'true';
-			}
-		}
-
-		// Register image variables only if custom-defined
-		foreach (ImageManager::SupportedImageKeys as $image) {
-			if ($this->imageManager->hasImage($image)) {
-				$imageUrl = $this->imageManager->getImageUrl($image);
-				if ($image === 'background') {
-					// If background deleted is set, ignoring variable
-					if ($backgroundDeleted) {
-						$variables['--image-background-default'] = 'no';
-						continue;
-					}
-					$variables['--image-background-size'] = 'cover';
-					$variables['--image-background-default'] = "url('" . $imageUrl . "')";
-				}
-				$variables["--image-$image"] = "url('" . $imageUrl . "')";
-			}
-		}
-
-		if ($hasCustomLogoHeader) {
-			$variables["--image-logoheader-custom"] = 'true';
-		}
-
-		return $variables;
-	}
-
-	/**
-	 * Generate user theming background-related variables
-	 */
-	protected function generateUserBackgroundVariables(): array {
-		$user = $this->userSession->getUser();
-		if ($user !== null
-			&& !$this->themingDefaults->isUserThemingDisabled()
-			&& $this->appManager->isEnabledForUser(Application::APP_ID)) {
-			$themingBackground = $this->config->getUserValue($user->getUID(), Application::APP_ID, 'background', 'default');
-			$currentVersion = (int)$this->config->getUserValue($user->getUID(), Application::APP_ID, 'userCacheBuster', '0');
-
-			// The user uploaded a custom background
-			if ($themingBackground === 'custom') {
-				$cacheBuster = substr(sha1($user->getUID() . '_' . $currentVersion), 0, 8);
-				return [
-					'--image-background' => "url('" . $this->urlGenerator->linkToRouteAbsolute('theming.userTheme.getBackground') . "?v=$cacheBuster')",
-					// TODO: implement primary color from custom background --color-background-plain
-				];
-			}
-
-			// The user picked a shipped background
-			if (isset(BackgroundService::SHIPPED_BACKGROUNDS[$themingBackground])) {
-				return [
-					'--image-background' => "url('" . $this->urlGenerator->linkTo(Application::APP_ID, "/img/background/$themingBackground") . "')",
-					'--color-background-plain' => $this->themingDefaults->getColorPrimary(),
-					'--background-image-invert-if-bright' => BackgroundService::SHIPPED_BACKGROUNDS[$themingBackground]['theming'] ?? null === BackgroundService::THEMING_MODE_DARK ? 'invert(100%)' : 'no',
-				];
-			}
-
-			// The user picked a static colour
-			if (substr($themingBackground, 0, 1) === '#') {
-				return [
-					'--image-background' => 'no',
-					'--color-background-plain' => $this->themingDefaults->getColorPrimary(),
-				];
-			}
-		}
-
-		return [];
-	}
+    public Util $util;
+
+    /**
+     * Generate primary-related variables
+     * This is shared between multiple themes because colorMainBackground and colorMainText
+     * will change in between.
+     */
+    protected function generatePrimaryVariables(string $colorMainBackground, string $colorMainText): array {
+        $colorPrimaryLight = $this->util->mix($this->primaryColor, $colorMainBackground, -80);
+        $colorPrimaryElement = $this->util->elementColor($this->primaryColor);
+        $colorPrimaryElementDefault = $this->util->elementColor($this->defaultPrimaryColor);
+        $colorPrimaryElementLight = $this->util->mix($colorPrimaryElement, $colorMainBackground, -80);
+
+        // primary related colours
+        return [
+            // invert filter if primary is too bright
+            // to be used for legacy reasons only. Use inline
+            // svg with proper css variable instead or material
+            // design icons.
+            // ⚠️ Using 'no' as a value to make sure we specify an
+            // invalid one with no fallback. 'unset' could here fallback to some
+            // other theme with media queries
+            '--primary-invert-if-bright' => $this->util->invertTextColor($this->primaryColor) ? 'invert(100%)' : 'no',
+
+            '--color-primary' => $this->primaryColor,
+            '--color-primary-default' => $this->defaultPrimaryColor,
+            '--color-primary-text' => $this->util->invertTextColor($this->primaryColor) ? '#000000' : '#ffffff',
+            '--color-primary-hover' => $this->util->mix($this->primaryColor, $colorMainBackground, 60),
+            '--color-primary-light' => $colorPrimaryLight,
+            '--color-primary-light-text' => $this->util->mix($this->primaryColor, $this->util->invertTextColor($colorPrimaryLight) ? '#000000' : '#ffffff', -20),
+            '--color-primary-light-hover' => $this->util->mix($colorPrimaryLight, $colorMainText, 90),
+            '--color-primary-text-dark' => $this->util->darken($this->util->invertTextColor($this->primaryColor) ? '#000000' : '#ffffff', 7),
+
+            // used for buttons, inputs...
+            '--color-primary-element' => $colorPrimaryElement,
+            '--color-primary-element-default-hover' => $this->util->mix($colorPrimaryElementDefault, $colorMainBackground, 60),
+            '--color-primary-element-text' => $this->util->invertTextColor($colorPrimaryElement) ? '#000000' : '#ffffff',
+            '--color-primary-element-hover' => $this->util->mix($colorPrimaryElement, $colorMainBackground, 60),
+            '--color-primary-element-light' => $colorPrimaryElementLight,
+            '--color-primary-element-light-text' => $this->util->mix($colorPrimaryElement, $this->util->invertTextColor($colorPrimaryElementLight) ? '#000000' : '#ffffff', -20),
+            '--color-primary-element-light-hover' => $this->util->mix($colorPrimaryElementLight, $colorMainText, 90),
+            '--color-primary-element-text-dark' => $this->util->darken($this->util->invertTextColor($colorPrimaryElement) ? '#000000' : '#ffffff', 7),
+
+            // to use like this: background-image: var(--gradient-primary-background);
+            '--gradient-primary-background' => 'linear-gradient(40deg, var(--color-primary) 0%, var(--color-primary-hover) 100%)',
+        ];
+    }
+
+    /**
+     * Generate admin theming background-related variables
+     */
+    protected function generateGlobalBackgroundVariables(): array {
+        $user = $this->userSession->getUser();
+        $backgroundDeleted = $this->config->getAppValue(Application::APP_ID, 'backgroundMime', '') === 'backgroundColor';
+        $hasCustomLogoHeader = $this->imageManager->hasImage('logo') || $this->imageManager->hasImage('logoheader');
+
+        $variables = [];
+
+        // If primary as background has been request or if we have a custom primary colour
+        // let's not define the background image
+        if ($backgroundDeleted) {
+            $variables['--color-background-plain'] = $this->themingDefaults->getColorPrimary();
+            if ($this->themingDefaults->isUserThemingDisabled() || $user === null) {
+                $variables['--image-background-plain'] = 'true';
+            }
+        }
+
+        // Register image variables only if custom-defined
+        foreach (ImageManager::SupportedImageKeys as $image) {
+            if ($this->imageManager->hasImage($image)) {
+                $imageUrl = $this->imageManager->getImageUrl($image);
+                if ($image === 'background') {
+                    // If background deleted is set, ignoring variable
+                    if ($backgroundDeleted) {
+                        $variables['--image-background-default'] = 'no';
+                        continue;
+                    }
+                    $variables['--image-background-size'] = 'cover';
+                    $variables['--image-background-default'] = "url('" . $imageUrl . "')";
+                }
+                $variables["--image-$image"] = "url('" . $imageUrl . "')";
+            }
+        }
+
+        if ($hasCustomLogoHeader) {
+            $variables["--image-logoheader-custom"] = 'true';
+        }
+
+        return $variables;
+    }
+
+    /**
+     * Generate user theming background-related variables
+     */
+    protected function generateUserBackgroundVariables(): array {
+        $user = $this->userSession->getUser();
+        if ($user !== null
+            && !$this->themingDefaults->isUserThemingDisabled()
+            && $this->appManager->isEnabledForUser(Application::APP_ID)) {
+            $themingBackground = $this->config->getUserValue($user->getUID(), Application::APP_ID, 'background', 'default');
+            $currentVersion = (int)$this->config->getUserValue($user->getUID(), Application::APP_ID, 'userCacheBuster', '0');
+
+            // The user uploaded a custom background
+            if ($themingBackground === 'custom') {
+                $cacheBuster = substr(sha1($user->getUID() . '_' . $currentVersion), 0, 8);
+                return [
+                    '--image-background' => "url('" . $this->urlGenerator->linkToRouteAbsolute('theming.userTheme.getBackground') . "?v=$cacheBuster')",
+                    // TODO: implement primary color from custom background --color-background-plain
+                ];
+            }
+
+            // The user picked a shipped background
+            if (isset(BackgroundService::SHIPPED_BACKGROUNDS[$themingBackground])) {
+                return [
+                    '--image-background' => "url('" . $this->urlGenerator->linkTo(Application::APP_ID, "/img/background/$themingBackground") . "')",
+                    '--color-background-plain' => $this->themingDefaults->getColorPrimary(),
+                    '--background-image-invert-if-bright' => BackgroundService::SHIPPED_BACKGROUNDS[$themingBackground]['theming'] ?? null === BackgroundService::THEMING_MODE_DARK ? 'invert(100%)' : 'no',
+                ];
+            }
+
+            // The user picked a static colour
+            if (substr($themingBackground, 0, 1) === '#') {
+                return [
+                    '--image-background' => 'no',
+                    '--color-background-plain' => $this->themingDefaults->getColorPrimary(),
+                ];
+            }
+        }
+
+        return [];
+    }
 }

Spacing +6 added lines, -6 removed lines

@@ -108,9 +108,9 @@ 
 						continue;
 					}
 					$variables['--image-background-size'] = 'cover';
-					$variables['--image-background-default'] = "url('" . $imageUrl . "')";
+					$variables['--image-background-default'] = "url('".$imageUrl."')";
 				}
-				$variables["--image-$image"] = "url('" . $imageUrl . "')";
+				$variables["--image-$image"] = "url('".$imageUrl."')";
 			}
 		}
 
@@ -130,13 +130,13 @@ 
 			&& !$this->themingDefaults->isUserThemingDisabled()
 			&& $this->appManager->isEnabledForUser(Application::APP_ID)) {
 			$themingBackground = $this->config->getUserValue($user->getUID(), Application::APP_ID, 'background', 'default');
-			$currentVersion = (int)$this->config->getUserValue($user->getUID(), Application::APP_ID, 'userCacheBuster', '0');
+			$currentVersion = (int) $this->config->getUserValue($user->getUID(), Application::APP_ID, 'userCacheBuster', '0');
 
 			// The user uploaded a custom background
 			if ($themingBackground === 'custom') {
-				$cacheBuster = substr(sha1($user->getUID() . '_' . $currentVersion), 0, 8);
+				$cacheBuster = substr(sha1($user->getUID().'_'.$currentVersion), 0, 8);
 				return [
-					'--image-background' => "url('" . $this->urlGenerator->linkToRouteAbsolute('theming.userTheme.getBackground') . "?v=$cacheBuster')",
+					'--image-background' => "url('".$this->urlGenerator->linkToRouteAbsolute('theming.userTheme.getBackground')."?v=$cacheBuster')",
 					// TODO: implement primary color from custom background --color-background-plain
 				];
 			}
@@ -144,7 +144,7 @@ 
 			// The user picked a shipped background
 			if (isset(BackgroundService::SHIPPED_BACKGROUNDS[$themingBackground])) {
 				return [
-					'--image-background' => "url('" . $this->urlGenerator->linkTo(Application::APP_ID, "/img/background/$themingBackground") . "')",
+					'--image-background' => "url('".$this->urlGenerator->linkTo(Application::APP_ID, "/img/background/$themingBackground")."')",
 					'--color-background-plain' => $this->themingDefaults->getColorPrimary(),
 					'--background-image-invert-if-bright' => BackgroundService::SHIPPED_BACKGROUNDS[$themingBackground]['theming'] ?? null === BackgroundService::THEMING_MODE_DARK ? 'invert(100%)' : 'no',
 				];

apps/theming/lib/Settings/Admin.php

Indentation +78 added lines, -78 removed lines

@@ -39,91 +39,91 @@
 use OCP\Util;
 
 class Admin implements IDelegatedSettings {
-	private string $appName;
-	private IConfig $config;
-	private IL10N $l;
-	private ThemingDefaults $themingDefaults;
-	private IInitialState $initialState;
-	private IURLGenerator $urlGenerator;
-	private ImageManager $imageManager;
+    private string $appName;
+    private IConfig $config;
+    private IL10N $l;
+    private ThemingDefaults $themingDefaults;
+    private IInitialState $initialState;
+    private IURLGenerator $urlGenerator;
+    private ImageManager $imageManager;
 
-	public function __construct(string $appName,
-								IConfig $config,
-								IL10N $l,
-								ThemingDefaults $themingDefaults,
-								IInitialState $initialState,
-								IURLGenerator $urlGenerator,
-								ImageManager $imageManager) {
-		$this->appName = $appName;
-		$this->config = $config;
-		$this->l = $l;
-		$this->themingDefaults = $themingDefaults;
-		$this->initialState = $initialState;
-		$this->urlGenerator = $urlGenerator;
-		$this->imageManager = $imageManager;
-	}
+    public function __construct(string $appName,
+                                IConfig $config,
+                                IL10N $l,
+                                ThemingDefaults $themingDefaults,
+                                IInitialState $initialState,
+                                IURLGenerator $urlGenerator,
+                                ImageManager $imageManager) {
+        $this->appName = $appName;
+        $this->config = $config;
+        $this->l = $l;
+        $this->themingDefaults = $themingDefaults;
+        $this->initialState = $initialState;
+        $this->urlGenerator = $urlGenerator;
+        $this->imageManager = $imageManager;
+    }
 
-	/**
-	 * @return TemplateResponse
-	 */
-	public function getForm(): TemplateResponse {
-		$themable = true;
-		$errorMessage = '';
-		$theme = $this->config->getSystemValue('theme', '');
-		if ($theme !== '') {
-			$themable = false;
-			$errorMessage = $this->l->t('You are already using a custom theme. Theming app settings might be overwritten by that.');
-		}
+    /**
+     * @return TemplateResponse
+     */
+    public function getForm(): TemplateResponse {
+        $themable = true;
+        $errorMessage = '';
+        $theme = $this->config->getSystemValue('theme', '');
+        if ($theme !== '') {
+            $themable = false;
+            $errorMessage = $this->l->t('You are already using a custom theme. Theming app settings might be overwritten by that.');
+        }
 
-		$this->initialState->provideInitialState('adminThemingParameters', [
-			'isThemable' => $themable,
-			'notThemableErrorMessage' => $errorMessage,
-			'name' => $this->themingDefaults->getEntity(),
-			'url' => $this->themingDefaults->getBaseUrl(),
-			'slogan' => $this->themingDefaults->getSlogan(),
-			'color' => $this->themingDefaults->getDefaultColorPrimary(),
-			'logoMime' => $this->config->getAppValue(Application::APP_ID, 'logoMime', ''),
-			'backgroundMime' => $this->config->getAppValue(Application::APP_ID, 'backgroundMime', ''),
-			'logoheaderMime' => $this->config->getAppValue(Application::APP_ID, 'logoheaderMime', ''),
-			'faviconMime' => $this->config->getAppValue(Application::APP_ID, 'faviconMime', ''),
-			'legalNoticeUrl' => $this->themingDefaults->getImprintUrl(),
-			'privacyPolicyUrl' => $this->themingDefaults->getPrivacyUrl(),
-			'docUrl' => $this->urlGenerator->linkToDocs('admin-theming'),
-			'docUrlIcons' => $this->urlGenerator->linkToDocs('admin-theming-icons'),
-			'canThemeIcons' => $this->imageManager->shouldReplaceIcons(),
-			'userThemingDisabled' => $this->themingDefaults->isUserThemingDisabled(),
-		]);
+        $this->initialState->provideInitialState('adminThemingParameters', [
+            'isThemable' => $themable,
+            'notThemableErrorMessage' => $errorMessage,
+            'name' => $this->themingDefaults->getEntity(),
+            'url' => $this->themingDefaults->getBaseUrl(),
+            'slogan' => $this->themingDefaults->getSlogan(),
+            'color' => $this->themingDefaults->getDefaultColorPrimary(),
+            'logoMime' => $this->config->getAppValue(Application::APP_ID, 'logoMime', ''),
+            'backgroundMime' => $this->config->getAppValue(Application::APP_ID, 'backgroundMime', ''),
+            'logoheaderMime' => $this->config->getAppValue(Application::APP_ID, 'logoheaderMime', ''),
+            'faviconMime' => $this->config->getAppValue(Application::APP_ID, 'faviconMime', ''),
+            'legalNoticeUrl' => $this->themingDefaults->getImprintUrl(),
+            'privacyPolicyUrl' => $this->themingDefaults->getPrivacyUrl(),
+            'docUrl' => $this->urlGenerator->linkToDocs('admin-theming'),
+            'docUrlIcons' => $this->urlGenerator->linkToDocs('admin-theming-icons'),
+            'canThemeIcons' => $this->imageManager->shouldReplaceIcons(),
+            'userThemingDisabled' => $this->themingDefaults->isUserThemingDisabled(),
+        ]);
 
-		Util::addScript($this->appName, 'admin-theming');
+        Util::addScript($this->appName, 'admin-theming');
 
-		return new TemplateResponse($this->appName, 'settings-admin');
-	}
+        return new TemplateResponse($this->appName, 'settings-admin');
+    }
 
-	/**
-	 * @return string the section ID, e.g. 'sharing'
-	 */
-	public function getSection(): string {
-		return $this->appName;
-	}
+    /**
+     * @return string the section ID, e.g. 'sharing'
+     */
+    public function getSection(): string {
+        return $this->appName;
+    }
 
-	/**
-	 * @return int whether the form should be rather on the top or bottom of
-	 * the admin section. The forms are arranged in ascending order of the
-	 * priority values. It is required to return a value between 0 and 100.
-	 *
-	 * E.g.: 70
-	 */
-	public function getPriority(): int {
-		return 5;
-	}
+    /**
+     * @return int whether the form should be rather on the top or bottom of
+     * the admin section. The forms are arranged in ascending order of the
+     * priority values. It is required to return a value between 0 and 100.
+     *
+     * E.g.: 70
+     */
+    public function getPriority(): int {
+        return 5;
+    }
 
-	public function getName(): ?string {
-		return null; // Only one setting in this section
-	}
+    public function getName(): ?string {
+        return null; // Only one setting in this section
+    }
 
-	public function getAuthorizedAppConfig(): array {
-		return [
-			$this->appName => '/.*/',
-		];
-	}
+    public function getAuthorizedAppConfig(): array {
+        return [
+            $this->appName => '/.*/',
+        ];
+    }
 }

apps/theming/lib/Settings/Personal.php

Indentation +57 added lines, -57 removed lines

@@ -36,70 +36,70 @@
 
 class Personal implements ISettings {
 
-	protected string $appName;
-	private IConfig $config;
-	private ThemesService $themesService;
-	private IInitialState $initialStateService;
-	private ThemingDefaults $themingDefaults;
+    protected string $appName;
+    private IConfig $config;
+    private ThemesService $themesService;
+    private IInitialState $initialStateService;
+    private ThemingDefaults $themingDefaults;
 
-	public function __construct(string $appName,
-								IConfig $config,
-								ThemesService $themesService,
-								IInitialState $initialStateService,
-								ThemingDefaults $themingDefaults) {
-		$this->appName = $appName;
-		$this->config = $config;
-		$this->themesService = $themesService;
-		$this->initialStateService = $initialStateService;
-		$this->themingDefaults = $themingDefaults;
-	}
+    public function __construct(string $appName,
+                                IConfig $config,
+                                ThemesService $themesService,
+                                IInitialState $initialStateService,
+                                ThemingDefaults $themingDefaults) {
+        $this->appName = $appName;
+        $this->config = $config;
+        $this->themesService = $themesService;
+        $this->initialStateService = $initialStateService;
+        $this->themingDefaults = $themingDefaults;
+    }
 
-	public function getForm(): TemplateResponse {
-		$enforcedTheme = $this->config->getSystemValueString('enforce_theme', '');
+    public function getForm(): TemplateResponse {
+        $enforcedTheme = $this->config->getSystemValueString('enforce_theme', '');
 
-		$themes = array_map(function($theme) {
-			return [
-				'id' => $theme->getId(),
-				'type' => $theme->getType(),
-				'title' => $theme->getTitle(),
-				'enableLabel' => $theme->getEnableLabel(),
-				'description' => $theme->getDescription(),
-				'enabled' => $this->themesService->isEnabled($theme),
-			];
-		}, $this->themesService->getThemes());
+        $themes = array_map(function($theme) {
+            return [
+                'id' => $theme->getId(),
+                'type' => $theme->getType(),
+                'title' => $theme->getTitle(),
+                'enableLabel' => $theme->getEnableLabel(),
+                'description' => $theme->getDescription(),
+                'enabled' => $this->themesService->isEnabled($theme),
+            ];
+        }, $this->themesService->getThemes());
 
-		if ($enforcedTheme !== '') {
-			$themes = array_filter($themes, function($theme) use ($enforcedTheme) {
-				return $theme['type'] !== ITheme::TYPE_THEME || $theme['id'] === $enforcedTheme;
-			});
-		}
+        if ($enforcedTheme !== '') {
+            $themes = array_filter($themes, function($theme) use ($enforcedTheme) {
+                return $theme['type'] !== ITheme::TYPE_THEME || $theme['id'] === $enforcedTheme;
+            });
+        }
 
-		$this->initialStateService->provideInitialState('themes', array_values($themes));
-		$this->initialStateService->provideInitialState('enforceTheme', $enforcedTheme);
-		$this->initialStateService->provideInitialState('isUserThemingDisabled', $this->themingDefaults->isUserThemingDisabled());
+        $this->initialStateService->provideInitialState('themes', array_values($themes));
+        $this->initialStateService->provideInitialState('enforceTheme', $enforcedTheme);
+        $this->initialStateService->provideInitialState('isUserThemingDisabled', $this->themingDefaults->isUserThemingDisabled());
 
-		Util::addScript($this->appName, 'personal-theming');
+        Util::addScript($this->appName, 'personal-theming');
 
-		return new TemplateResponse($this->appName, 'settings-personal');
-	}
+        return new TemplateResponse($this->appName, 'settings-personal');
+    }
 
-	/**
-	 * @return string the section ID, e.g. 'sharing'
-	 * @since 9.1
-	 */
-	public function getSection(): string {
-		return $this->appName;
-	}
+    /**
+     * @return string the section ID, e.g. 'sharing'
+     * @since 9.1
+     */
+    public function getSection(): string {
+        return $this->appName;
+    }
 
-	/**
-	 * @return int whether the form should be rather on the top or bottom of
-	 * the admin section. The forms are arranged in ascending order of the
-	 * priority values. It is required to return a value between 0 and 100.
-	 *
-	 * E.g.: 70
-	 * @since 9.1
-	 */
-	public function getPriority(): int {
-		return 40;
-	}
+    /**
+     * @return int whether the form should be rather on the top or bottom of
+     * the admin section. The forms are arranged in ascending order of the
+     * priority values. It is required to return a value between 0 and 100.
+     *
+     * E.g.: 70
+     * @since 9.1
+     */
+    public function getPriority(): int {
+        return 40;
+    }
 }

apps/theming/lib/ThemingDefaults.php

Indentation +450 added lines, -450 removed lines

@@ -55,454 +55,454 @@
 
 class ThemingDefaults extends \OC_Defaults {
 
-	private IConfig $config;
-	private IL10N $l;
-	private ImageManager $imageManager;
-	private IUserSession $userSession;
-	private IURLGenerator $urlGenerator;
-	private ICacheFactory $cacheFactory;
-	private Util $util;
-	private IAppManager $appManager;
-	private INavigationManager $navigationManager;
-
-	private string $name;
-	private string $title;
-	private string $entity;
-	private string $productName;
-	private string $url;
-	private string $color;
-
-	private string $iTunesAppId;
-	private string $iOSClientUrl;
-	private string $AndroidClientUrl;
-	private string $FDroidClientUrl;
-
-	/**
-	 * ThemingDefaults constructor.
-	 *
-	 * @param IConfig $config
-	 * @param IL10N $l
-	 * @param ImageManager $imageManager
-	 * @param IUserSession $userSession
-	 * @param IURLGenerator $urlGenerator
-	 * @param ICacheFactory $cacheFactory
-	 * @param Util $util
-	 * @param IAppManager $appManager
-	 */
-	public function __construct(IConfig $config,
-								IL10N $l,
-								IUserSession $userSession,
-								IURLGenerator $urlGenerator,
-								ICacheFactory $cacheFactory,
-								Util $util,
-								ImageManager $imageManager,
-								IAppManager $appManager,
-								INavigationManager $navigationManager
-	) {
-		parent::__construct();
-		$this->config = $config;
-		$this->l = $l;
-		$this->imageManager = $imageManager;
-		$this->userSession = $userSession;
-		$this->urlGenerator = $urlGenerator;
-		$this->cacheFactory = $cacheFactory;
-		$this->util = $util;
-		$this->appManager = $appManager;
-		$this->navigationManager = $navigationManager;
-
-		$this->name = parent::getName();
-		$this->title = parent::getTitle();
-		$this->entity = parent::getEntity();
-		$this->productName = parent::getProductName();
-		$this->url = parent::getBaseUrl();
-		$this->color = parent::getColorPrimary();
-		$this->iTunesAppId = parent::getiTunesAppId();
-		$this->iOSClientUrl = parent::getiOSClientUrl();
-		$this->AndroidClientUrl = parent::getAndroidClientUrl();
-		$this->FDroidClientUrl = parent::getFDroidClientUrl();
-	}
-
-	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 getShortFooter() {
-		$slogan = $this->getSlogan();
-		$baseUrl = $this->getBaseUrl();
-		if ($baseUrl !== '') {
-			$footer = '<a href="' . $baseUrl . '" target="_blank"' .
-				' rel="noreferrer noopener" class="entity-name">' . $this->getEntity() . '</a>';
-		} else {
-			$footer = '<span class="entity-name">' .$this->getEntity() . '</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/>' . $legalLinks;
-		}
-
-		return $footer;
-	}
-
-	/**
-	 * Color that is used for the header as well as for mail headers
-	 */
-	public function getColorPrimary(): string {
-		$user = $this->userSession->getUser();
-
-		// admin-defined primary color
-		$defaultColor = $this->getDefaultColorPrimary();
-
-		if ($this->isUserThemingDisabled()) {
-			return $defaultColor;
-		}
-
-		// user-defined primary color
-		$themingBackground = '';
-		if (!empty($user)) {
-			$themingBackground = $this->config->getUserValue($user->getUID(), Application::APP_ID, 'background', '');
-			// If the user selected the default background
-			if ($themingBackground === '') {
-				return BackgroundService::DEFAULT_COLOR;
-			}
-
-			// If the user selected a specific colour
-			if (preg_match('/^\#([0-9a-f]{3}|[0-9a-f]{6})$/i', $themingBackground)) {
-				return $themingBackground;
-			}
-
-			// if the user-selected background is a background reference
-			if (isset(BackgroundService::SHIPPED_BACKGROUNDS[$themingBackground]['primary_color'])) {
-				return BackgroundService::SHIPPED_BACKGROUNDS[$themingBackground]['primary_color'];
-			}
-		}
-
-		// If the default color is not valid, return the default background one
-		if (!preg_match('/^\#([0-9a-f]{3}|[0-9a-f]{6})$/i', $defaultColor)) {
-			return BackgroundService::DEFAULT_COLOR;
-		}
-
-		// Finally, return the system global primary color
-		return $defaultColor;
-	}
-
-	/**
-	 * Return the default color primary
-	 */
-	public function getDefaultColorPrimary(): string {
-		$color = $this->config->getAppValue(Application::APP_ID, 'color');
-		if (!preg_match('/^\#([0-9a-f]{3}|[0-9a-f]{6})$/i', $color)) {
-			$color = '#0082c9';
-		}
-		return $color;
-	}
-
-	/**
-	 * 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
-	 *
-	 * @return string
-	 */
-	public function getBackground(): string {
-		return $this->imageManager->getImageUrl('background');
-	}
-
-	/**
-	 * @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
-	 */
-	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->config->getAppValue('theming', '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 (strpos($image, 'filetypes/') === 0 && file_exists(\OC::$SERVERROOT . '/core/img/' . $image)) {
-			$route = $this->urlGenerator->linkToRoute('theming.Icon.getThemedIcon', ['app' => $app, 'image' => $image]);
-		}
-
-		if ($route) {
-			return $route . '?v=' . $cacheBusterValue;
-		}
-
-		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) {
-		$this->config->setAppValue('theming', $setting, $value);
-		$this->increaseCacheBuster();
-	}
-
-	/**
-	 * Revert settings to the default value
-	 *
-	 * @param string $setting setting which should be reverted
-	 * @return string default value
-	 */
-	public function undo($setting) {
-		$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 'color':
-				$returnValue = $this->getColorPrimary();
-				break;
-			case 'logo':
-			case 'logoheader':
-			case 'background':
-			case 'favicon':
-				$this->imageManager->delete($setting);
-				break;
-		}
-
-		return $returnValue;
-	}
-
-	/**
-	 * Color of text in the header and primary buttons
-	 *
-	 * @return string
-	 */
-	public function getTextColorPrimary() {
-		return $this->util->invertTextColor($this->getColorPrimary()) ? '#000000' : '#ffffff';
-	}
-
-	/**
-	 * Has the admin disabled user customization
-	 */
-	public function isUserThemingDisabled(): bool {
-		return $this->config->getAppValue('theming', 'disable-user-theming', 'no') === 'yes';
-	}
+    private IConfig $config;
+    private IL10N $l;
+    private ImageManager $imageManager;
+    private IUserSession $userSession;
+    private IURLGenerator $urlGenerator;
+    private ICacheFactory $cacheFactory;
+    private Util $util;
+    private IAppManager $appManager;
+    private INavigationManager $navigationManager;
+
+    private string $name;
+    private string $title;
+    private string $entity;
+    private string $productName;
+    private string $url;
+    private string $color;
+
+    private string $iTunesAppId;
+    private string $iOSClientUrl;
+    private string $AndroidClientUrl;
+    private string $FDroidClientUrl;
+
+    /**
+     * ThemingDefaults constructor.
+     *
+     * @param IConfig $config
+     * @param IL10N $l
+     * @param ImageManager $imageManager
+     * @param IUserSession $userSession
+     * @param IURLGenerator $urlGenerator
+     * @param ICacheFactory $cacheFactory
+     * @param Util $util
+     * @param IAppManager $appManager
+     */
+    public function __construct(IConfig $config,
+                                IL10N $l,
+                                IUserSession $userSession,
+                                IURLGenerator $urlGenerator,
+                                ICacheFactory $cacheFactory,
+                                Util $util,
+                                ImageManager $imageManager,
+                                IAppManager $appManager,
+                                INavigationManager $navigationManager
+    ) {
+        parent::__construct();
+        $this->config = $config;
+        $this->l = $l;
+        $this->imageManager = $imageManager;
+        $this->userSession = $userSession;
+        $this->urlGenerator = $urlGenerator;
+        $this->cacheFactory = $cacheFactory;
+        $this->util = $util;
+        $this->appManager = $appManager;
+        $this->navigationManager = $navigationManager;
+
+        $this->name = parent::getName();
+        $this->title = parent::getTitle();
+        $this->entity = parent::getEntity();
+        $this->productName = parent::getProductName();
+        $this->url = parent::getBaseUrl();
+        $this->color = parent::getColorPrimary();
+        $this->iTunesAppId = parent::getiTunesAppId();
+        $this->iOSClientUrl = parent::getiOSClientUrl();
+        $this->AndroidClientUrl = parent::getAndroidClientUrl();
+        $this->FDroidClientUrl = parent::getFDroidClientUrl();
+    }
+
+    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 getShortFooter() {
+        $slogan = $this->getSlogan();
+        $baseUrl = $this->getBaseUrl();
+        if ($baseUrl !== '') {
+            $footer = '<a href="' . $baseUrl . '" target="_blank"' .
+                ' rel="noreferrer noopener" class="entity-name">' . $this->getEntity() . '</a>';
+        } else {
+            $footer = '<span class="entity-name">' .$this->getEntity() . '</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/>' . $legalLinks;
+        }
+
+        return $footer;
+    }
+
+    /**
+     * Color that is used for the header as well as for mail headers
+     */
+    public function getColorPrimary(): string {
+        $user = $this->userSession->getUser();
+
+        // admin-defined primary color
+        $defaultColor = $this->getDefaultColorPrimary();
+
+        if ($this->isUserThemingDisabled()) {
+            return $defaultColor;
+        }
+
+        // user-defined primary color
+        $themingBackground = '';
+        if (!empty($user)) {
+            $themingBackground = $this->config->getUserValue($user->getUID(), Application::APP_ID, 'background', '');
+            // If the user selected the default background
+            if ($themingBackground === '') {
+                return BackgroundService::DEFAULT_COLOR;
+            }
+
+            // If the user selected a specific colour
+            if (preg_match('/^\#([0-9a-f]{3}|[0-9a-f]{6})$/i', $themingBackground)) {
+                return $themingBackground;
+            }
+
+            // if the user-selected background is a background reference
+            if (isset(BackgroundService::SHIPPED_BACKGROUNDS[$themingBackground]['primary_color'])) {
+                return BackgroundService::SHIPPED_BACKGROUNDS[$themingBackground]['primary_color'];
+            }
+        }
+
+        // If the default color is not valid, return the default background one
+        if (!preg_match('/^\#([0-9a-f]{3}|[0-9a-f]{6})$/i', $defaultColor)) {
+            return BackgroundService::DEFAULT_COLOR;
+        }
+
+        // Finally, return the system global primary color
+        return $defaultColor;
+    }
+
+    /**
+     * Return the default color primary
+     */
+    public function getDefaultColorPrimary(): string {
+        $color = $this->config->getAppValue(Application::APP_ID, 'color');
+        if (!preg_match('/^\#([0-9a-f]{3}|[0-9a-f]{6})$/i', $color)) {
+            $color = '#0082c9';
+        }
+        return $color;
+    }
+
+    /**
+     * 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
+     *
+     * @return string
+     */
+    public function getBackground(): string {
+        return $this->imageManager->getImageUrl('background');
+    }
+
+    /**
+     * @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
+     */
+    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->config->getAppValue('theming', '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 (strpos($image, 'filetypes/') === 0 && file_exists(\OC::$SERVERROOT . '/core/img/' . $image)) {
+            $route = $this->urlGenerator->linkToRoute('theming.Icon.getThemedIcon', ['app' => $app, 'image' => $image]);
+        }
+
+        if ($route) {
+            return $route . '?v=' . $cacheBusterValue;
+        }
+
+        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) {
+        $this->config->setAppValue('theming', $setting, $value);
+        $this->increaseCacheBuster();
+    }
+
+    /**
+     * Revert settings to the default value
+     *
+     * @param string $setting setting which should be reverted
+     * @return string default value
+     */
+    public function undo($setting) {
+        $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 'color':
+                $returnValue = $this->getColorPrimary();
+                break;
+            case 'logo':
+            case 'logoheader':
+            case 'background':
+            case 'favicon':
+                $this->imageManager->delete($setting);
+                break;
+        }
+
+        return $returnValue;
+    }
+
+    /**
+     * Color of text in the header and primary buttons
+     *
+     * @return string
+     */
+    public function getTextColorPrimary() {
+        return $this->util->invertTextColor($this->getColorPrimary()) ? '#000000' : '#ffffff';
+    }
+
+    /**
+     * Has the admin disabled user customization
+     */
+    public function isUserThemingDisabled(): bool {
+        return $this->config->getAppValue('theming', 'disable-user-theming', 'no') === 'yes';
+    }
 }

apps/theming/lib/ImageManager.php

Indentation +266 added lines, -266 removed lines

@@ -45,270 +45,270 @@
 use OCP\IURLGenerator;
 
 class ImageManager {
-	public const SupportedImageKeys = ['background', 'logo', 'logoheader', 'favicon'];
-
-	/** @var IConfig */
-	private $config;
-	/** @var IAppData */
-	private $appData;
-	/** @var IURLGenerator */
-	private $urlGenerator;
-	/** @var ICacheFactory */
-	private $cacheFactory;
-	/** @var ILogger */
-	private $logger;
-	/** @var ITempManager */
-	private $tempManager;
-
-	public function __construct(IConfig $config,
-								IAppData $appData,
-								IURLGenerator $urlGenerator,
-								ICacheFactory $cacheFactory,
-								ILogger $logger,
-								ITempManager $tempManager) {
-		$this->config = $config;
-		$this->urlGenerator = $urlGenerator;
-		$this->cacheFactory = $cacheFactory;
-		$this->logger = $logger;
-		$this->tempManager = $tempManager;
-		$this->appData = $appData;
-	}
-
-	public function getImageUrl(string $key, bool $useSvg = true): string {
-		$cacheBusterCounter = $this->config->getAppValue('theming', 'cachebuster', '0');
-		if ($this->hasImage($key)) {
-			return $this->urlGenerator->linkToRoute('theming.Theming.getImage', [ 'key' => $key ]) . '?v=' . $cacheBusterCounter;
-		}
-
-		switch ($key) {
-			case 'logo':
-			case 'logoheader':
-			case 'favicon':
-				return $this->urlGenerator->imagePath('core', 'logo/logo.png') . '?v=' . $cacheBusterCounter;
-			case 'background':
-				return $this->urlGenerator->imagePath('core', 'background.png') . '?v=' . $cacheBusterCounter;
-		}
-		return '';
-	}
-
-	public function getImageUrlAbsolute(string $key, bool $useSvg = true): string {
-		return $this->urlGenerator->getAbsoluteURL($this->getImageUrl($key, $useSvg));
-	}
-
-	/**
-	 * @param string $key
-	 * @param bool $useSvg
-	 * @return ISimpleFile
-	 * @throws NotFoundException
-	 * @throws NotPermittedException
-	 */
-	public function getImage(string $key, bool $useSvg = true): ISimpleFile {
-		$logo = $this->config->getAppValue('theming', $key . 'Mime', '');
-		$folder = $this->getRootFolder()->getFolder('images');
-
-		if ($logo === '' || !$folder->fileExists($key)) {
-			throw new NotFoundException();
-		}
-
-		if (!$useSvg && $this->shouldReplaceIcons()) {
-			if (!$folder->fileExists($key . '.png')) {
-				try {
-					$finalIconFile = new \Imagick();
-					$finalIconFile->setBackgroundColor('none');
-					$finalIconFile->readImageBlob($folder->getFile($key)->getContent());
-					$finalIconFile->setImageFormat('png32');
-					$pngFile = $folder->newFile($key . '.png');
-					$pngFile->putContent($finalIconFile->getImageBlob());
-					return $pngFile;
-				} catch (\ImagickException $e) {
-					$this->logger->info('The image was requested to be no SVG file, but converting it to PNG failed: ' . $e->getMessage());
-				}
-			} else {
-				return $folder->getFile($key . '.png');
-			}
-		}
-
-		return $folder->getFile($key);
-	}
-
-	public function hasImage(string $key): bool {
-		$mimeSetting = $this->config->getAppValue('theming', $key . 'Mime', '');
-		return $mimeSetting !== '';
-	}
-
-	/**
-	 * Get folder for current theming files
-	 *
-	 * @return ISimpleFolder
-	 * @throws NotPermittedException
-	 */
-	public function getCacheFolder(): ISimpleFolder {
-		$cacheBusterValue = $this->config->getAppValue('theming', 'cachebuster', '0');
-		try {
-			$folder = $this->getRootFolder()->getFolder($cacheBusterValue);
-		} catch (NotFoundException $e) {
-			$folder = $this->getRootFolder()->newFolder($cacheBusterValue);
-			$this->cleanup();
-		}
-		return $folder;
-	}
-
-	/**
-	 * Get a file from AppData
-	 *
-	 * @param string $filename
-	 * @throws NotFoundException
-	 * @return \OCP\Files\SimpleFS\ISimpleFile
-	 * @throws NotPermittedException
-	 */
-	public function getCachedImage(string $filename): ISimpleFile {
-		$currentFolder = $this->getCacheFolder();
-		return $currentFolder->getFile($filename);
-	}
-
-	/**
-	 * Store a file for theming in AppData
-	 *
-	 * @param string $filename
-	 * @param string $data
-	 * @return \OCP\Files\SimpleFS\ISimpleFile
-	 * @throws NotFoundException
-	 * @throws NotPermittedException
-	 */
-	public function setCachedImage(string $filename, string $data): ISimpleFile {
-		$currentFolder = $this->getCacheFolder();
-		if ($currentFolder->fileExists($filename)) {
-			$file = $currentFolder->getFile($filename);
-		} else {
-			$file = $currentFolder->newFile($filename);
-		}
-		$file->putContent($data);
-		return $file;
-	}
-
-	public function delete(string $key): void {
-		/* ignore exceptions, since we don't want to fail hard if something goes wrong during cleanup */
-		try {
-			$file = $this->getRootFolder()->getFolder('images')->getFile($key);
-			$file->delete();
-		} catch (NotFoundException $e) {
-		} catch (NotPermittedException $e) {
-		}
-		try {
-			$file = $this->getRootFolder()->getFolder('images')->getFile($key . '.png');
-			$file->delete();
-		} catch (NotFoundException $e) {
-		} catch (NotPermittedException $e) {
-		}
-	}
-
-	public function updateImage(string $key, string $tmpFile): string {
-		$this->delete($key);
-
-		try {
-			$folder = $this->getRootFolder()->getFolder('images');
-		} catch (NotFoundException $e) {
-			$folder = $this->getRootFolder()->newFolder('images');
-		}
-
-		$target = $folder->newFile($key);
-		$supportedFormats = $this->getSupportedUploadImageFormats($key);
-		$detectedMimeType = mime_content_type($tmpFile);
-		if (!in_array($detectedMimeType, $supportedFormats, true)) {
-			throw new \Exception('Unsupported image type');
-		}
-
-		if ($key === 'background' && strpos($detectedMimeType, 'image/svg') === false && strpos($detectedMimeType, 'image/gif') === false) {
-			// Optimize the image since some people may upload images that will be
-			// either to big or are not progressive rendering.
-			$newImage = @imagecreatefromstring(file_get_contents($tmpFile));
-
-			// Preserve transparency
-			imagesavealpha($newImage, true);
-			imagealphablending($newImage, true);
-
-			$tmpFile = $this->tempManager->getTemporaryFile();
-			$newWidth = (int)(imagesx($newImage) < 4096 ? imagesx($newImage) : 4096);
-			$newHeight = (int)(imagesy($newImage) / (imagesx($newImage) / $newWidth));
-			$outputImage = imagescale($newImage, $newWidth, $newHeight);
-
-			imageinterlace($outputImage, 1);
-			imagepng($outputImage, $tmpFile, 8);
-			imagedestroy($outputImage);
-
-			$target->putContent(file_get_contents($tmpFile));
-		} else {
-			$target->putContent(file_get_contents($tmpFile));
-		}
-
-		return $detectedMimeType;
-	}
-
-	/**
-	 * Returns a list of supported mime types for image uploads.
-	 * "favicon" images are only allowed to be SVG when imagemagick with SVG support is available.
-	 *
-	 * @param string $key The image key, e.g. "favicon"
-	 * @return string[]
-	 */
-	private function getSupportedUploadImageFormats(string $key): array {
-		$supportedFormats = ['image/jpeg', 'image/png', 'image/gif', 'image/webp'];
-
-		if ($key !== 'favicon' || $this->shouldReplaceIcons() === true) {
-			$supportedFormats[] = 'image/svg+xml';
-			$supportedFormats[] = 'image/svg';
-		}
-
-		if ($key === 'favicon') {
-			$supportedFormats[] = 'image/x-icon';
-			$supportedFormats[] = 'image/vnd.microsoft.icon';
-		}
-
-		return $supportedFormats;
-	}
-
-	/**
-	 * remove cached files that are not required any longer
-	 *
-	 * @throws NotPermittedException
-	 * @throws NotFoundException
-	 */
-	public function cleanup() {
-		$currentFolder = $this->getCacheFolder();
-		$folders = $this->getRootFolder()->getDirectoryListing();
-		foreach ($folders as $folder) {
-			if ($folder->getName() !== 'images' && $folder->getName() !== $currentFolder->getName()) {
-				$folder->delete();
-			}
-		}
-	}
-
-	/**
-	 * Check if Imagemagick is enabled and if SVG is supported
-	 * otherwise we can't render custom icons
-	 *
-	 * @return bool
-	 */
-	public function shouldReplaceIcons() {
-		$cache = $this->cacheFactory->createDistributed('theming-' . $this->urlGenerator->getBaseUrl());
-		if ($value = $cache->get('shouldReplaceIcons')) {
-			return (bool)$value;
-		}
-		$value = false;
-		if (extension_loaded('imagick')) {
-			if (count(\Imagick::queryFormats('SVG')) >= 1) {
-				$value = true;
-			}
-		}
-		$cache->set('shouldReplaceIcons', $value);
-		return $value;
-	}
-
-	private function getRootFolder(): ISimpleFolder {
-		try {
-			return $this->appData->getFolder('global');
-		} catch (NotFoundException $e) {
-			return $this->appData->newFolder('global');
-		}
-	}
+    public const SupportedImageKeys = ['background', 'logo', 'logoheader', 'favicon'];
+
+    /** @var IConfig */
+    private $config;
+    /** @var IAppData */
+    private $appData;
+    /** @var IURLGenerator */
+    private $urlGenerator;
+    /** @var ICacheFactory */
+    private $cacheFactory;
+    /** @var ILogger */
+    private $logger;
+    /** @var ITempManager */
+    private $tempManager;
+
+    public function __construct(IConfig $config,
+                                IAppData $appData,
+                                IURLGenerator $urlGenerator,
+                                ICacheFactory $cacheFactory,
+                                ILogger $logger,
+                                ITempManager $tempManager) {
+        $this->config = $config;
+        $this->urlGenerator = $urlGenerator;
+        $this->cacheFactory = $cacheFactory;
+        $this->logger = $logger;
+        $this->tempManager = $tempManager;
+        $this->appData = $appData;
+    }
+
+    public function getImageUrl(string $key, bool $useSvg = true): string {
+        $cacheBusterCounter = $this->config->getAppValue('theming', 'cachebuster', '0');
+        if ($this->hasImage($key)) {
+            return $this->urlGenerator->linkToRoute('theming.Theming.getImage', [ 'key' => $key ]) . '?v=' . $cacheBusterCounter;
+        }
+
+        switch ($key) {
+            case 'logo':
+            case 'logoheader':
+            case 'favicon':
+                return $this->urlGenerator->imagePath('core', 'logo/logo.png') . '?v=' . $cacheBusterCounter;
+            case 'background':
+                return $this->urlGenerator->imagePath('core', 'background.png') . '?v=' . $cacheBusterCounter;
+        }
+        return '';
+    }
+
+    public function getImageUrlAbsolute(string $key, bool $useSvg = true): string {
+        return $this->urlGenerator->getAbsoluteURL($this->getImageUrl($key, $useSvg));
+    }
+
+    /**
+     * @param string $key
+     * @param bool $useSvg
+     * @return ISimpleFile
+     * @throws NotFoundException
+     * @throws NotPermittedException
+     */
+    public function getImage(string $key, bool $useSvg = true): ISimpleFile {
+        $logo = $this->config->getAppValue('theming', $key . 'Mime', '');
+        $folder = $this->getRootFolder()->getFolder('images');
+
+        if ($logo === '' || !$folder->fileExists($key)) {
+            throw new NotFoundException();
+        }
+
+        if (!$useSvg && $this->shouldReplaceIcons()) {
+            if (!$folder->fileExists($key . '.png')) {
+                try {
+                    $finalIconFile = new \Imagick();
+                    $finalIconFile->setBackgroundColor('none');
+                    $finalIconFile->readImageBlob($folder->getFile($key)->getContent());
+                    $finalIconFile->setImageFormat('png32');
+                    $pngFile = $folder->newFile($key . '.png');
+                    $pngFile->putContent($finalIconFile->getImageBlob());
+                    return $pngFile;
+                } catch (\ImagickException $e) {
+                    $this->logger->info('The image was requested to be no SVG file, but converting it to PNG failed: ' . $e->getMessage());
+                }
+            } else {
+                return $folder->getFile($key . '.png');
+            }
+        }
+
+        return $folder->getFile($key);
+    }
+
+    public function hasImage(string $key): bool {
+        $mimeSetting = $this->config->getAppValue('theming', $key . 'Mime', '');
+        return $mimeSetting !== '';
+    }
+
+    /**
+     * Get folder for current theming files
+     *
+     * @return ISimpleFolder
+     * @throws NotPermittedException
+     */
+    public function getCacheFolder(): ISimpleFolder {
+        $cacheBusterValue = $this->config->getAppValue('theming', 'cachebuster', '0');
+        try {
+            $folder = $this->getRootFolder()->getFolder($cacheBusterValue);
+        } catch (NotFoundException $e) {
+            $folder = $this->getRootFolder()->newFolder($cacheBusterValue);
+            $this->cleanup();
+        }
+        return $folder;
+    }
+
+    /**
+     * Get a file from AppData
+     *
+     * @param string $filename
+     * @throws NotFoundException
+     * @return \OCP\Files\SimpleFS\ISimpleFile
+     * @throws NotPermittedException
+     */
+    public function getCachedImage(string $filename): ISimpleFile {
+        $currentFolder = $this->getCacheFolder();
+        return $currentFolder->getFile($filename);
+    }
+
+    /**
+     * Store a file for theming in AppData
+     *
+     * @param string $filename
+     * @param string $data
+     * @return \OCP\Files\SimpleFS\ISimpleFile
+     * @throws NotFoundException
+     * @throws NotPermittedException
+     */
+    public function setCachedImage(string $filename, string $data): ISimpleFile {
+        $currentFolder = $this->getCacheFolder();
+        if ($currentFolder->fileExists($filename)) {
+            $file = $currentFolder->getFile($filename);
+        } else {
+            $file = $currentFolder->newFile($filename);
+        }
+        $file->putContent($data);
+        return $file;
+    }
+
+    public function delete(string $key): void {
+        /* ignore exceptions, since we don't want to fail hard if something goes wrong during cleanup */
+        try {
+            $file = $this->getRootFolder()->getFolder('images')->getFile($key);
+            $file->delete();
+        } catch (NotFoundException $e) {
+        } catch (NotPermittedException $e) {
+        }
+        try {
+            $file = $this->getRootFolder()->getFolder('images')->getFile($key . '.png');
+            $file->delete();
+        } catch (NotFoundException $e) {
+        } catch (NotPermittedException $e) {
+        }
+    }
+
+    public function updateImage(string $key, string $tmpFile): string {
+        $this->delete($key);
+
+        try {
+            $folder = $this->getRootFolder()->getFolder('images');
+        } catch (NotFoundException $e) {
+            $folder = $this->getRootFolder()->newFolder('images');
+        }
+
+        $target = $folder->newFile($key);
+        $supportedFormats = $this->getSupportedUploadImageFormats($key);
+        $detectedMimeType = mime_content_type($tmpFile);
+        if (!in_array($detectedMimeType, $supportedFormats, true)) {
+            throw new \Exception('Unsupported image type');
+        }
+
+        if ($key === 'background' && strpos($detectedMimeType, 'image/svg') === false && strpos($detectedMimeType, 'image/gif') === false) {
+            // Optimize the image since some people may upload images that will be
+            // either to big or are not progressive rendering.
+            $newImage = @imagecreatefromstring(file_get_contents($tmpFile));
+
+            // Preserve transparency
+            imagesavealpha($newImage, true);
+            imagealphablending($newImage, true);
+
+            $tmpFile = $this->tempManager->getTemporaryFile();
+            $newWidth = (int)(imagesx($newImage) < 4096 ? imagesx($newImage) : 4096);
+            $newHeight = (int)(imagesy($newImage) / (imagesx($newImage) / $newWidth));
+            $outputImage = imagescale($newImage, $newWidth, $newHeight);
+
+            imageinterlace($outputImage, 1);
+            imagepng($outputImage, $tmpFile, 8);
+            imagedestroy($outputImage);
+
+            $target->putContent(file_get_contents($tmpFile));
+        } else {
+            $target->putContent(file_get_contents($tmpFile));
+        }
+
+        return $detectedMimeType;
+    }
+
+    /**
+     * Returns a list of supported mime types for image uploads.
+     * "favicon" images are only allowed to be SVG when imagemagick with SVG support is available.
+     *
+     * @param string $key The image key, e.g. "favicon"
+     * @return string[]
+     */
+    private function getSupportedUploadImageFormats(string $key): array {
+        $supportedFormats = ['image/jpeg', 'image/png', 'image/gif', 'image/webp'];
+
+        if ($key !== 'favicon' || $this->shouldReplaceIcons() === true) {
+            $supportedFormats[] = 'image/svg+xml';
+            $supportedFormats[] = 'image/svg';
+        }
+
+        if ($key === 'favicon') {
+            $supportedFormats[] = 'image/x-icon';
+            $supportedFormats[] = 'image/vnd.microsoft.icon';
+        }
+
+        return $supportedFormats;
+    }
+
+    /**
+     * remove cached files that are not required any longer
+     *
+     * @throws NotPermittedException
+     * @throws NotFoundException
+     */
+    public function cleanup() {
+        $currentFolder = $this->getCacheFolder();
+        $folders = $this->getRootFolder()->getDirectoryListing();
+        foreach ($folders as $folder) {
+            if ($folder->getName() !== 'images' && $folder->getName() !== $currentFolder->getName()) {
+                $folder->delete();
+            }
+        }
+    }
+
+    /**
+     * Check if Imagemagick is enabled and if SVG is supported
+     * otherwise we can't render custom icons
+     *
+     * @return bool
+     */
+    public function shouldReplaceIcons() {
+        $cache = $this->cacheFactory->createDistributed('theming-' . $this->urlGenerator->getBaseUrl());
+        if ($value = $cache->get('shouldReplaceIcons')) {
+            return (bool)$value;
+        }
+        $value = false;
+        if (extension_loaded('imagick')) {
+            if (count(\Imagick::queryFormats('SVG')) >= 1) {
+                $value = true;
+            }
+        }
+        $cache->set('shouldReplaceIcons', $value);
+        return $value;
+    }
+
+    private function getRootFolder(): ISimpleFolder {
+        try {
+            return $this->appData->getFolder('global');
+        } catch (NotFoundException $e) {
+            return $this->appData->newFolder('global');
+        }
+    }
 }

Spacing +14 added lines, -14 removed lines

@@ -77,16 +77,16 @@ 
 	public function getImageUrl(string $key, bool $useSvg = true): string {
 		$cacheBusterCounter = $this->config->getAppValue('theming', 'cachebuster', '0');
 		if ($this->hasImage($key)) {
-			return $this->urlGenerator->linkToRoute('theming.Theming.getImage', [ 'key' => $key ]) . '?v=' . $cacheBusterCounter;
+			return $this->urlGenerator->linkToRoute('theming.Theming.getImage', ['key' => $key]).'?v='.$cacheBusterCounter;
 		}
 
 		switch ($key) {
 			case 'logo':
 			case 'logoheader':
 			case 'favicon':
-				return $this->urlGenerator->imagePath('core', 'logo/logo.png') . '?v=' . $cacheBusterCounter;
+				return $this->urlGenerator->imagePath('core', 'logo/logo.png').'?v='.$cacheBusterCounter;
 			case 'background':
-				return $this->urlGenerator->imagePath('core', 'background.png') . '?v=' . $cacheBusterCounter;
+				return $this->urlGenerator->imagePath('core', 'background.png').'?v='.$cacheBusterCounter;
 		}
 		return '';
 	}
@@ -103,7 +103,7 @@ 
 	 * @throws NotPermittedException
 	 */
 	public function getImage(string $key, bool $useSvg = true): ISimpleFile {
-		$logo = $this->config->getAppValue('theming', $key . 'Mime', '');
+		$logo = $this->config->getAppValue('theming', $key.'Mime', '');
 		$folder = $this->getRootFolder()->getFolder('images');
 
 		if ($logo === '' || !$folder->fileExists($key)) {
@@ -111,20 +111,20 @@ 
 		}
 
 		if (!$useSvg && $this->shouldReplaceIcons()) {
-			if (!$folder->fileExists($key . '.png')) {
+			if (!$folder->fileExists($key.'.png')) {
 				try {
 					$finalIconFile = new \Imagick();
 					$finalIconFile->setBackgroundColor('none');
 					$finalIconFile->readImageBlob($folder->getFile($key)->getContent());
 					$finalIconFile->setImageFormat('png32');
-					$pngFile = $folder->newFile($key . '.png');
+					$pngFile = $folder->newFile($key.'.png');
 					$pngFile->putContent($finalIconFile->getImageBlob());
 					return $pngFile;
 				} catch (\ImagickException $e) {
-					$this->logger->info('The image was requested to be no SVG file, but converting it to PNG failed: ' . $e->getMessage());
+					$this->logger->info('The image was requested to be no SVG file, but converting it to PNG failed: '.$e->getMessage());
 				}
 			} else {
-				return $folder->getFile($key . '.png');
+				return $folder->getFile($key.'.png');
 			}
 		}
 
@@ -132,7 +132,7 @@ 
 	}
 
 	public function hasImage(string $key): bool {
-		$mimeSetting = $this->config->getAppValue('theming', $key . 'Mime', '');
+		$mimeSetting = $this->config->getAppValue('theming', $key.'Mime', '');
 		return $mimeSetting !== '';
 	}
 
@@ -195,7 +195,7 @@ 
 		} catch (NotPermittedException $e) {
 		}
 		try {
-			$file = $this->getRootFolder()->getFolder('images')->getFile($key . '.png');
+			$file = $this->getRootFolder()->getFolder('images')->getFile($key.'.png');
 			$file->delete();
 		} catch (NotFoundException $e) {
 		} catch (NotPermittedException $e) {
@@ -228,8 +228,8 @@ 
 			imagealphablending($newImage, true);
 
 			$tmpFile = $this->tempManager->getTemporaryFile();
-			$newWidth = (int)(imagesx($newImage) < 4096 ? imagesx($newImage) : 4096);
-			$newHeight = (int)(imagesy($newImage) / (imagesx($newImage) / $newWidth));
+			$newWidth = (int) (imagesx($newImage) < 4096 ? imagesx($newImage) : 4096);
+			$newHeight = (int) (imagesy($newImage) / (imagesx($newImage) / $newWidth));
 			$outputImage = imagescale($newImage, $newWidth, $newHeight);
 
 			imageinterlace($outputImage, 1);
@@ -290,9 +290,9 @@ 
 	 * @return bool
 	 */
 	public function shouldReplaceIcons() {
-		$cache = $this->cacheFactory->createDistributed('theming-' . $this->urlGenerator->getBaseUrl());
+		$cache = $this->cacheFactory->createDistributed('theming-'.$this->urlGenerator->getBaseUrl());
 		if ($value = $cache->get('shouldReplaceIcons')) {
-			return (bool)$value;
+			return (bool) $value;
 		}
 		$value = false;
 		if (extension_loaded('imagick')) {