fix logo not correctly inverted

apps/theming/lib/Themes/CommonThemeTrait.php

@@ -84,7 +84,7 @@ protected function generatePrimaryVariables(string $colorMainBackground, string
 	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');
+		$hasCustomLogoHeader = $this->util->isLogoThemed();
 
 		$variables = [];
 

apps/theming/lib/Util.php

@@ -41,18 +41,13 @@ class Util {
 	private IConfig $config;
 	private IAppManager $appManager;
 	private IAppData $appData;
+	private ImageManager $imageManager;
 
-	/**
-	 * Util constructor.
-	 *
-	 * @param IConfig $config
-	 * @param IAppManager $appManager
-	 * @param IAppData $appData
-	 */
-	public function __construct(IConfig $config, IAppManager $appManager, IAppData $appData) {
+	public function __construct(IConfig $config, IAppManager $appManager, IAppData $appData, ImageManager $imageManager) {
 		$this->config = $config;
 		$this->appManager = $appManager;
 		$this->appData = $appData;
+		$this->imageManager = $imageManager;
 	}
 
 	/**
@@ -266,4 +261,9 @@ public function isBackgroundThemed() {
 		$backgroundLogo = $this->config->getAppValue('theming', 'backgroundMime', '');
 		return $backgroundLogo !== '' && $backgroundLogo !== 'backgroundColor';
 	}
+
+	public function isLogoThemed() {
+		return $this->imageManager->hasImage('logo')
+			|| $this->imageManager->hasImage('logoheader');
+	}
 }

apps/theming/tests/CapabilitiesTest.php

@@ -29,6 +29,7 @@
 namespace OCA\Theming\Tests;
 
 use OCA\Theming\Capabilities;
+use OCA\Theming\ImageManager;
 use OCA\Theming\ThemingDefaults;
 use OCA\Theming\Util;
 use OCP\App\IAppManager;
@@ -173,7 +174,7 @@ public function testGetCapabilities($name, $url, $slogan, $color, $textColor, $l
 			->method('getTextColorPrimary')
 			->willReturn($textColor);
 
-		$util = new Util($this->config, $this->createMock(IAppManager::class), $this->createMock(IAppData::class));
+		$util = new Util($this->config, $this->createMock(IAppManager::class), $this->createMock(IAppData::class), $this->createMock(ImageManager::class));
 		$this->util->expects($this->exactly(3))
 			->method('elementColor')
 			->with($color)

apps/theming/tests/IconBuilderTest.php

@@ -63,7 +63,7 @@ protected function setUp(): void {
 		$this->themingDefaults = $this->createMock(ThemingDefaults::class);
 		$this->appManager = $this->createMock(IAppManager::class);
 		$this->imageManager = $this->createMock(ImageManager::class);
-		$this->util = new Util($this->config, $this->appManager, $this->appData);
+		$this->util = new Util($this->config, $this->appManager, $this->appData, $this->imageManager);
 		$this->iconBuilder = new IconBuilder($this->themingDefaults, $this->util, $this->imageManager);
 	}
 

apps/theming/tests/Themes/DefaultThemeTest.php

@@ -66,7 +66,8 @@ protected function setUp(): void {
 		$util = new Util(
 			$this->config,
 			$this->appManager,
-			$this->createMock(IAppData::class)
+			$this->createMock(IAppData::class),
+			$this->imageManager
 		);
 
 		$this->themingDefaults

apps/theming/tests/Themes/DyslexiaFontTest.php

@@ -68,8 +68,9 @@ protected function setUp(): void {
 
 		$util = new Util(
 			$this->config,
-			$this->createMock(AppManager::class),
-			$this->createMock(IAppData::class)
+			$this->appManager,
+			$this->createMock(IAppData::class),
+			$this->imageManager
 		);
 
 		$userSession = $this->createMock(IUserSession::class);

apps/theming/tests/UtilTest.php

@@ -27,6 +27,7 @@
  */
 namespace OCA\Theming\Tests;
 
+use OCA\Theming\ImageManager;
 use OCA\Theming\Util;
 use OCP\App\IAppManager;
 use OCP\Files\IAppData;
@@ -46,13 +47,16 @@ class UtilTest extends TestCase {
 	protected $appData;
 	/** @var IAppManager */
 	protected $appManager;
+	/** @var ImageManager */
+	protected $imageManager;
 
 	protected function setUp(): void {
 		parent::setUp();
 		$this->config = $this->createMock(IConfig::class);
 		$this->appData = $this->createMock(IAppData::class);
 		$this->appManager = $this->createMock(IAppManager::class);
-		$this->util = new Util($this->config, $this->appManager, $this->appData);
+		$this->imageManager = $this->createMock(ImageManager::class);
+		$this->util = new Util($this->config, $this->appManager, $this->appData, $this->imageManager);
 	}
 
 	public function dataInvertTextColor() {

core/css/header.scss

@@ -172,6 +172,8 @@
 		left: 12px;
 		top: 1px;
 		bottom: 1px;
+		// Invert if not customized and background is bright
+		filter: var(--image-logoheader-custom, var(--background-image-invert-if-bright));
 	}
 
 	.header-appname-container {

lib/private/Server.php

@@ -1209,21 +1209,22 @@ public function __construct($webRoot, \OC\Config $config) {
 			}
 
 			if ($classExists && $c->get(\OCP\IConfig::class)->getSystemValue('installed', false) && $c->get(IAppManager::class)->isInstalled('theming') && $c->getTrustedDomainHelper()->isTrustedDomain($c->getRequest()->getInsecureServerHost())) {
+				$imageManager = new ImageManager(
+					$c->get(\OCP\IConfig::class),
+					$c->getAppDataDir('theming'),
+					$c->get(IURLGenerator::class),
+					$this->get(ICacheFactory::class),
+					$this->get(ILogger::class),
+					$this->get(ITempManager::class)
+				);
 				return new ThemingDefaults(
 					$c->get(\OCP\IConfig::class),
 					$c->getL10N('theming'),
 					$c->get(IUserSession::class),
 					$c->get(IURLGenerator::class),
 					$c->get(ICacheFactory::class),
-					new Util($c->get(\OCP\IConfig::class), $this->get(IAppManager::class), $c->getAppDataDir('theming')),
-					new ImageManager(
-						$c->get(\OCP\IConfig::class),
-						$c->getAppDataDir('theming'),
-						$c->get(IURLGenerator::class),
-						$this->get(ICacheFactory::class),
-						$this->get(ILogger::class),
-						$this->get(ITempManager::class)
-					),
+					new Util($c->get(\OCP\IConfig::class), $this->get(IAppManager::class), $c->getAppDataDir('theming'), $imageManager),
+					$imageManager,
 					$c->get(IAppManager::class),
 					$c->get(INavigationManager::class)
 				);