From 8fdb9e1c3d073593d565b8073b0e0fc13d1210c4 Mon Sep 17 00:00:00 2001 From: Git'Fellow <12234510+solracsf@users.noreply.github.com> Date: Fri, 5 Jun 2026 10:34:44 +0200 Subject: [PATCH] fix(theming): read cachebuster as int in manifest endpoint Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> --- apps/theming/lib/Controller/ThemingController.php | 2 +- apps/theming/tests/Controller/ThemingControllerTest.php | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/apps/theming/lib/Controller/ThemingController.php b/apps/theming/lib/Controller/ThemingController.php index bbd598a2c288a..7491c47cb706c 100644 --- a/apps/theming/lib/Controller/ThemingController.php +++ b/apps/theming/lib/Controller/ThemingController.php @@ -456,7 +456,7 @@ public function getThemeStylesheet(string $themeId, bool $plain = false, bool $w #[BruteForceProtection(action: 'manifest')] #[OpenAPI(scope: OpenAPI::SCOPE_DEFAULT)] public function getManifest(string $app): JSONResponse { - $cacheBusterValue = $this->appConfig->getAppValueString('cachebuster', '0'); + $cacheBusterValue = (string)$this->appConfig->getAppValueInt('cachebuster'); if ($app === 'core' || $app === 'settings') { $name = $this->themingDefaults->getName(); $shortName = $this->themingDefaults->getName(); diff --git a/apps/theming/tests/Controller/ThemingControllerTest.php b/apps/theming/tests/Controller/ThemingControllerTest.php index b452b0e85811e..c28dcee1ae6aa 100644 --- a/apps/theming/tests/Controller/ThemingControllerTest.php +++ b/apps/theming/tests/Controller/ThemingControllerTest.php @@ -736,9 +736,9 @@ public static function dataGetManifest(): array { public function testGetManifest(bool $standalone): void { $this->appConfig ->expects($this->once()) - ->method('getAppValueString') - ->with('cachebuster', '0') - ->willReturn('0'); + ->method('getAppValueInt') + ->with('cachebuster') + ->willReturn(0); $this->themingDefaults ->expects($this->any()) ->method('getName')