nextcloud / server

apps/theming/lib/Controller/ThemingController.php

Indentation +310 added lines, -310 removed lines

@@ -65,335 +65,335 @@
  * @package OCA\Theming\Controller
  */
 class ThemingController extends Controller {
-	private ThemingDefaults $themingDefaults;
-	private IL10N $l10n;
-	private IConfig $config;
-	private ITempManager $tempManager;
-	private IAppData $appData;
-	private IURLGenerator $urlGenerator;
-	private IAppManager $appManager;
-	private ImageManager $imageManager;
-	private ThemesService $themesService;
+    private ThemingDefaults $themingDefaults;
+    private IL10N $l10n;
+    private IConfig $config;
+    private ITempManager $tempManager;
+    private IAppData $appData;
+    private IURLGenerator $urlGenerator;
+    private IAppManager $appManager;
+    private ImageManager $imageManager;
+    private ThemesService $themesService;
 
-	public function __construct(
-		$appName,
-		IRequest $request,
-		IConfig $config,
-		ThemingDefaults $themingDefaults,
-		IL10N $l,
-		ITempManager $tempManager,
-		IAppData $appData,
-		IURLGenerator $urlGenerator,
-		IAppManager $appManager,
-		ImageManager $imageManager,
-		ThemesService $themesService
-	) {
-		parent::__construct($appName, $request);
+    public function __construct(
+        $appName,
+        IRequest $request,
+        IConfig $config,
+        ThemingDefaults $themingDefaults,
+        IL10N $l,
+        ITempManager $tempManager,
+        IAppData $appData,
+        IURLGenerator $urlGenerator,
+        IAppManager $appManager,
+        ImageManager $imageManager,
+        ThemesService $themesService
+    ) {
+        parent::__construct($appName, $request);
 
-		$this->themingDefaults = $themingDefaults;
-		$this->l10n = $l;
-		$this->config = $config;
-		$this->tempManager = $tempManager;
-		$this->appData = $appData;
-		$this->urlGenerator = $urlGenerator;
-		$this->appManager = $appManager;
-		$this->imageManager = $imageManager;
-		$this->themesService = $themesService;
-	}
+        $this->themingDefaults = $themingDefaults;
+        $this->l10n = $l;
+        $this->config = $config;
+        $this->tempManager = $tempManager;
+        $this->appData = $appData;
+        $this->urlGenerator = $urlGenerator;
+        $this->appManager = $appManager;
+        $this->imageManager = $imageManager;
+        $this->themesService = $themesService;
+    }
 
-	/**
-	 * @AuthorizedAdminSetting(settings=OCA\Theming\Settings\Admin)
-	 * @param string $setting
-	 * @param string $value
-	 * @return DataResponse
-	 * @throws NotPermittedException
-	 */
-	public function updateStylesheet($setting, $value) {
-		$value = trim($value);
-		$error = null;
-		switch ($setting) {
-			case 'name':
-				if (strlen($value) > 250) {
-					$error = $this->l10n->t('The given name is too long');
-				}
-				break;
-			case 'url':
-				if (strlen($value) > 500) {
-					$error = $this->l10n->t('The given web address is too long');
-				}
-				if (!$this->isValidUrl($value)) {
-					$error = $this->l10n->t('The given web address is not a valid URL');
-				}
-				break;
-			case 'imprintUrl':
-				if (strlen($value) > 500) {
-					$error = $this->l10n->t('The given legal notice address is too long');
-				}
-				if (!$this->isValidUrl($value)) {
-					$error = $this->l10n->t('The given legal notice address is not a valid URL');
-				}
-				break;
-			case 'privacyUrl':
-				if (strlen($value) > 500) {
-					$error = $this->l10n->t('The given privacy policy address is too long');
-				}
-				if (!$this->isValidUrl($value)) {
-					$error = $this->l10n->t('The given privacy policy address is not a valid URL');
-				}
-				break;
-			case 'slogan':
-				if (strlen($value) > 500) {
-					$error = $this->l10n->t('The given slogan is too long');
-				}
-				break;
-			case 'color':
-				if (!preg_match('/^\#([0-9a-f]{3}|[0-9a-f]{6})$/i', $value)) {
-					$error = $this->l10n->t('The given color is invalid');
-				}
-				break;
-		}
-		if ($error !== null) {
-			return new DataResponse([
-				'data' => [
-					'message' => $error,
-				],
-				'status' => 'error'
-			], Http::STATUS_BAD_REQUEST);
-		}
+    /**
+     * @AuthorizedAdminSetting(settings=OCA\Theming\Settings\Admin)
+     * @param string $setting
+     * @param string $value
+     * @return DataResponse
+     * @throws NotPermittedException
+     */
+    public function updateStylesheet($setting, $value) {
+        $value = trim($value);
+        $error = null;
+        switch ($setting) {
+            case 'name':
+                if (strlen($value) > 250) {
+                    $error = $this->l10n->t('The given name is too long');
+                }
+                break;
+            case 'url':
+                if (strlen($value) > 500) {
+                    $error = $this->l10n->t('The given web address is too long');
+                }
+                if (!$this->isValidUrl($value)) {
+                    $error = $this->l10n->t('The given web address is not a valid URL');
+                }
+                break;
+            case 'imprintUrl':
+                if (strlen($value) > 500) {
+                    $error = $this->l10n->t('The given legal notice address is too long');
+                }
+                if (!$this->isValidUrl($value)) {
+                    $error = $this->l10n->t('The given legal notice address is not a valid URL');
+                }
+                break;
+            case 'privacyUrl':
+                if (strlen($value) > 500) {
+                    $error = $this->l10n->t('The given privacy policy address is too long');
+                }
+                if (!$this->isValidUrl($value)) {
+                    $error = $this->l10n->t('The given privacy policy address is not a valid URL');
+                }
+                break;
+            case 'slogan':
+                if (strlen($value) > 500) {
+                    $error = $this->l10n->t('The given slogan is too long');
+                }
+                break;
+            case 'color':
+                if (!preg_match('/^\#([0-9a-f]{3}|[0-9a-f]{6})$/i', $value)) {
+                    $error = $this->l10n->t('The given color is invalid');
+                }
+                break;
+        }
+        if ($error !== null) {
+            return new DataResponse([
+                'data' => [
+                    'message' => $error,
+                ],
+                'status' => 'error'
+            ], Http::STATUS_BAD_REQUEST);
+        }
 
-		$this->themingDefaults->set($setting, $value);
+        $this->themingDefaults->set($setting, $value);
 
-		return new DataResponse([
-			'data' => [
-				'message' => $this->l10n->t('Saved'),
-			],
-			'status' => 'success'
-		]);
-	}
+        return new DataResponse([
+            'data' => [
+                'message' => $this->l10n->t('Saved'),
+            ],
+            'status' => 'success'
+        ]);
+    }
 
-	/**
-	 * Check that a string is a valid http/https url
-	 */
-	private function isValidUrl(string $url): bool {
-		return ((strpos($url, 'http://') === 0 || strpos($url, 'https://') === 0) &&
-			filter_var($url, FILTER_VALIDATE_URL) !== false);
-	}
+    /**
+     * Check that a string is a valid http/https url
+     */
+    private function isValidUrl(string $url): bool {
+        return ((strpos($url, 'http://') === 0 || strpos($url, 'https://') === 0) &&
+            filter_var($url, FILTER_VALIDATE_URL) !== false);
+    }
 
-	/**
-	 * @AuthorizedAdminSetting(settings=OCA\Theming\Settings\Admin)
-	 * @return DataResponse
-	 * @throws NotPermittedException
-	 */
-	public function uploadImage(): DataResponse {
-		$key = $this->request->getParam('key');
-		$image = $this->request->getUploadedFile('image');
-		$error = null;
-		$phpFileUploadErrors = [
-			UPLOAD_ERR_OK => $this->l10n->t('The file was uploaded'),
-			UPLOAD_ERR_INI_SIZE => $this->l10n->t('The uploaded file exceeds the upload_max_filesize directive in php.ini'),
-			UPLOAD_ERR_FORM_SIZE => $this->l10n->t('The uploaded file exceeds the MAX_FILE_SIZE directive that was specified in the HTML form'),
-			UPLOAD_ERR_PARTIAL => $this->l10n->t('The file was only partially uploaded'),
-			UPLOAD_ERR_NO_FILE => $this->l10n->t('No file was uploaded'),
-			UPLOAD_ERR_NO_TMP_DIR => $this->l10n->t('Missing a temporary folder'),
-			UPLOAD_ERR_CANT_WRITE => $this->l10n->t('Could not write file to disk'),
-			UPLOAD_ERR_EXTENSION => $this->l10n->t('A PHP extension stopped the file upload'),
-		];
-		if (empty($image)) {
-			$error = $this->l10n->t('No file uploaded');
-		}
-		if (!empty($image) && array_key_exists('error', $image) && $image['error'] !== UPLOAD_ERR_OK) {
-			$error = $phpFileUploadErrors[$image['error']];
-		}
+    /**
+     * @AuthorizedAdminSetting(settings=OCA\Theming\Settings\Admin)
+     * @return DataResponse
+     * @throws NotPermittedException
+     */
+    public function uploadImage(): DataResponse {
+        $key = $this->request->getParam('key');
+        $image = $this->request->getUploadedFile('image');
+        $error = null;
+        $phpFileUploadErrors = [
+            UPLOAD_ERR_OK => $this->l10n->t('The file was uploaded'),
+            UPLOAD_ERR_INI_SIZE => $this->l10n->t('The uploaded file exceeds the upload_max_filesize directive in php.ini'),
+            UPLOAD_ERR_FORM_SIZE => $this->l10n->t('The uploaded file exceeds the MAX_FILE_SIZE directive that was specified in the HTML form'),
+            UPLOAD_ERR_PARTIAL => $this->l10n->t('The file was only partially uploaded'),
+            UPLOAD_ERR_NO_FILE => $this->l10n->t('No file was uploaded'),
+            UPLOAD_ERR_NO_TMP_DIR => $this->l10n->t('Missing a temporary folder'),
+            UPLOAD_ERR_CANT_WRITE => $this->l10n->t('Could not write file to disk'),
+            UPLOAD_ERR_EXTENSION => $this->l10n->t('A PHP extension stopped the file upload'),
+        ];
+        if (empty($image)) {
+            $error = $this->l10n->t('No file uploaded');
+        }
+        if (!empty($image) && array_key_exists('error', $image) && $image['error'] !== UPLOAD_ERR_OK) {
+            $error = $phpFileUploadErrors[$image['error']];
+        }
 
-		if ($error !== null) {
-			return new DataResponse(
-				[
-					'data' => [
-						'message' => $error
-					],
-					'status' => 'failure',
-				],
-				Http::STATUS_UNPROCESSABLE_ENTITY
-			);
-		}
+        if ($error !== null) {
+            return new DataResponse(
+                [
+                    'data' => [
+                        'message' => $error
+                    ],
+                    'status' => 'failure',
+                ],
+                Http::STATUS_UNPROCESSABLE_ENTITY
+            );
+        }
 
-		try {
-			$mime = $this->imageManager->updateImage($key, $image['tmp_name']);
-			$this->themingDefaults->set($key . 'Mime', $mime);
-		} catch (\Exception $e) {
-			return new DataResponse(
-				[
-					'data' => [
-						'message' => $e->getMessage()
-					],
-					'status' => 'failure',
-				],
-				Http::STATUS_UNPROCESSABLE_ENTITY
-			);
-		}
+        try {
+            $mime = $this->imageManager->updateImage($key, $image['tmp_name']);
+            $this->themingDefaults->set($key . 'Mime', $mime);
+        } catch (\Exception $e) {
+            return new DataResponse(
+                [
+                    'data' => [
+                        'message' => $e->getMessage()
+                    ],
+                    'status' => 'failure',
+                ],
+                Http::STATUS_UNPROCESSABLE_ENTITY
+            );
+        }
 
-		$name = $image['name'];
+        $name = $image['name'];
 
-		return new DataResponse(
-			[
-				'data' =>
-					[
-						'name' => $name,
-						'url' => $this->imageManager->getImageUrl($key),
-						'message' => $this->l10n->t('Saved'),
-					],
-				'status' => 'success'
-			]
-		);
-	}
+        return new DataResponse(
+            [
+                'data' =>
+                    [
+                        'name' => $name,
+                        'url' => $this->imageManager->getImageUrl($key),
+                        'message' => $this->l10n->t('Saved'),
+                    ],
+                'status' => 'success'
+            ]
+        );
+    }
 
-	/**
-	 * Revert setting to default value
-	 * @AuthorizedAdminSetting(settings=OCA\Theming\Settings\Admin)
-	 *
-	 * @param string $setting setting which should be reverted
-	 * @return DataResponse
-	 * @throws NotPermittedException
-	 */
-	public function undo(string $setting): DataResponse {
-		$value = $this->themingDefaults->undo($setting);
+    /**
+     * Revert setting to default value
+     * @AuthorizedAdminSetting(settings=OCA\Theming\Settings\Admin)
+     *
+     * @param string $setting setting which should be reverted
+     * @return DataResponse
+     * @throws NotPermittedException
+     */
+    public function undo(string $setting): DataResponse {
+        $value = $this->themingDefaults->undo($setting);
 
-		return new DataResponse(
-			[
-				'data' =>
-					[
-						'value' => $value,
-						'message' => $this->l10n->t('Saved'),
-					],
-				'status' => 'success'
-			]
-		);
-	}
+        return new DataResponse(
+            [
+                'data' =>
+                    [
+                        'value' => $value,
+                        'message' => $this->l10n->t('Saved'),
+                    ],
+                'status' => 'success'
+            ]
+        );
+    }
 
-	/**
-	 * @PublicPage
-	 * @NoCSRFRequired
-	 * @NoSameSiteCookieRequired
-	 *
-	 * @param string $key
-	 * @param bool $useSvg
-	 * @return FileDisplayResponse|NotFoundResponse
-	 * @throws NotPermittedException
-	 */
-	public function getImage(string $key, bool $useSvg = true) {
-		try {
-			$file = $this->imageManager->getImage($key, $useSvg);
-		} catch (NotFoundException $e) {
-			return new NotFoundResponse();
-		}
+    /**
+     * @PublicPage
+     * @NoCSRFRequired
+     * @NoSameSiteCookieRequired
+     *
+     * @param string $key
+     * @param bool $useSvg
+     * @return FileDisplayResponse|NotFoundResponse
+     * @throws NotPermittedException
+     */
+    public function getImage(string $key, bool $useSvg = true) {
+        try {
+            $file = $this->imageManager->getImage($key, $useSvg);
+        } catch (NotFoundException $e) {
+            return new NotFoundResponse();
+        }
 
-		$response = new FileDisplayResponse($file);
-		$csp = new Http\ContentSecurityPolicy();
-		$csp->allowInlineStyle();
-		$response->setContentSecurityPolicy($csp);
-		$response->cacheFor(3600);
-		$response->addHeader('Content-Type', $this->config->getAppValue($this->appName, $key . 'Mime', ''));
-		$response->addHeader('Content-Disposition', 'attachment; filename="' . $key . '"');
-		if (!$useSvg) {
-			$response->addHeader('Content-Type', 'image/png');
-		} else {
-			$response->addHeader('Content-Type', $this->config->getAppValue($this->appName, $key . 'Mime', ''));
-		}
-		return $response;
-	}
+        $response = new FileDisplayResponse($file);
+        $csp = new Http\ContentSecurityPolicy();
+        $csp->allowInlineStyle();
+        $response->setContentSecurityPolicy($csp);
+        $response->cacheFor(3600);
+        $response->addHeader('Content-Type', $this->config->getAppValue($this->appName, $key . 'Mime', ''));
+        $response->addHeader('Content-Disposition', 'attachment; filename="' . $key . '"');
+        if (!$useSvg) {
+            $response->addHeader('Content-Type', 'image/png');
+        } else {
+            $response->addHeader('Content-Type', $this->config->getAppValue($this->appName, $key . 'Mime', ''));
+        }
+        return $response;
+    }
 
-	/**
-	 * @NoCSRFRequired
-	 * @PublicPage
-	 * @NoSameSiteCookieRequired
-	 * @NoTwoFactorRequired
-	 *
-	 * @return DataDisplayResponse|NotFoundResponse
-	 */
-	public function getThemeStylesheet(string $themeId, bool $plain = false, bool $withCustomCss = false) {
-		$themes = $this->themesService->getThemes();
-		if (!in_array($themeId, array_keys($themes))) {
-			return new NotFoundResponse();
-		}
+    /**
+     * @NoCSRFRequired
+     * @PublicPage
+     * @NoSameSiteCookieRequired
+     * @NoTwoFactorRequired
+     *
+     * @return DataDisplayResponse|NotFoundResponse
+     */
+    public function getThemeStylesheet(string $themeId, bool $plain = false, bool $withCustomCss = false) {
+        $themes = $this->themesService->getThemes();
+        if (!in_array($themeId, array_keys($themes))) {
+            return new NotFoundResponse();
+        }
 
-		$theme = $themes[$themeId];
-		$customCss  = $theme->getCustomCss();
+        $theme = $themes[$themeId];
+        $customCss  = $theme->getCustomCss();
 
-		// Generate variables
-		$variables = '';
-		foreach ($theme->getCSSVariables() as $variable => $value) {
-			$variables .= "$variable:$value; ";
-		};
+        // Generate variables
+        $variables = '';
+        foreach ($theme->getCSSVariables() as $variable => $value) {
+            $variables .= "$variable:$value; ";
+        };
 
-		// If plain is set, the browser decides of the css priority
-		if ($plain) {
-			$css = ":root { $variables } " . $customCss;
-		} else { 
-			// If not set, we'll rely on the body class
-			$compiler = new Compiler();
-			$compiledCss = $compiler->compileString("[data-theme-$themeId] { $variables $customCss }");
-			$css = $compiledCss->getCss();;
-		}
+        // If plain is set, the browser decides of the css priority
+        if ($plain) {
+            $css = ":root { $variables } " . $customCss;
+        } else { 
+            // If not set, we'll rely on the body class
+            $compiler = new Compiler();
+            $compiledCss = $compiler->compileString("[data-theme-$themeId] { $variables $customCss }");
+            $css = $compiledCss->getCss();;
+        }
 
-		try {
-			$response = new DataDisplayResponse($css, Http::STATUS_OK, ['Content-Type' => 'text/css']);
-			$response->cacheFor(86400);
-			return $response;
-		} catch (NotFoundException $e) {
-			return new NotFoundResponse();
-		}
-	}
+        try {
+            $response = new DataDisplayResponse($css, Http::STATUS_OK, ['Content-Type' => 'text/css']);
+            $response->cacheFor(86400);
+            return $response;
+        } catch (NotFoundException $e) {
+            return new NotFoundResponse();
+        }
+    }
 
-	/**
-	 * @NoCSRFRequired
-	 * @PublicPage
-	 *
-	 * @return Http\JSONResponse
-	 */
-	public function getManifest($app) {
-		$cacheBusterValue = $this->config->getAppValue('theming', 'cachebuster', '0');
-		if ($app === 'core' || $app === 'settings') {
-			$name = $this->themingDefaults->getName();
-			$shortName = $this->themingDefaults->getName();
-			$startUrl = $this->urlGenerator->getBaseUrl();
-			$description = $this->themingDefaults->getSlogan();
-		} else {
-			$info = $this->appManager->getAppInfo($app, false, $this->l10n->getLanguageCode());
-			$name = $info['name'] . ' - ' . $this->themingDefaults->getName();
-			$shortName = $info['name'];
-			if (strpos($this->request->getRequestUri(), '/index.php/') !== false) {
-				$startUrl = $this->urlGenerator->getBaseUrl() . '/index.php/apps/' . $app . '/';
-			} else {
-				$startUrl = $this->urlGenerator->getBaseUrl() . '/apps/' . $app . '/';
-			}
-			$description = $info['summary'] ?? '';
-		}
-		$responseJS = [
-			'name' => $name,
-			'short_name' => $shortName,
-			'start_url' => $startUrl,
-			'theme_color' => $this->themingDefaults->getColorPrimary(),
-			'background_color' => $this->themingDefaults->getColorPrimary(),
-			'description' => $description,
-			'icons' =>
-				[
-					[
-						'src' => $this->urlGenerator->linkToRoute('theming.Icon.getTouchIcon',
-								['app' => $app]) . '?v=' . $cacheBusterValue,
-						'type' => 'image/png',
-						'sizes' => '512x512'
-					],
-					[
-						'src' => $this->urlGenerator->linkToRoute('theming.Icon.getFavicon',
-								['app' => $app]) . '?v=' . $cacheBusterValue,
-						'type' => 'image/svg+xml',
-						'sizes' => '16x16'
-					]
-				],
-			'display' => 'standalone'
-		];
-		$response = new Http\JSONResponse($responseJS);
-		$response->cacheFor(3600);
-		return $response;
-	}
+    /**
+     * @NoCSRFRequired
+     * @PublicPage
+     *
+     * @return Http\JSONResponse
+     */
+    public function getManifest($app) {
+        $cacheBusterValue = $this->config->getAppValue('theming', 'cachebuster', '0');
+        if ($app === 'core' || $app === 'settings') {
+            $name = $this->themingDefaults->getName();
+            $shortName = $this->themingDefaults->getName();
+            $startUrl = $this->urlGenerator->getBaseUrl();
+            $description = $this->themingDefaults->getSlogan();
+        } else {
+            $info = $this->appManager->getAppInfo($app, false, $this->l10n->getLanguageCode());
+            $name = $info['name'] . ' - ' . $this->themingDefaults->getName();
+            $shortName = $info['name'];
+            if (strpos($this->request->getRequestUri(), '/index.php/') !== false) {
+                $startUrl = $this->urlGenerator->getBaseUrl() . '/index.php/apps/' . $app . '/';
+            } else {
+                $startUrl = $this->urlGenerator->getBaseUrl() . '/apps/' . $app . '/';
+            }
+            $description = $info['summary'] ?? '';
+        }
+        $responseJS = [
+            'name' => $name,
+            'short_name' => $shortName,
+            'start_url' => $startUrl,
+            'theme_color' => $this->themingDefaults->getColorPrimary(),
+            'background_color' => $this->themingDefaults->getColorPrimary(),
+            'description' => $description,
+            'icons' =>
+                [
+                    [
+                        'src' => $this->urlGenerator->linkToRoute('theming.Icon.getTouchIcon',
+                                ['app' => $app]) . '?v=' . $cacheBusterValue,
+                        'type' => 'image/png',
+                        'sizes' => '512x512'
+                    ],
+                    [
+                        'src' => $this->urlGenerator->linkToRoute('theming.Icon.getFavicon',
+                                ['app' => $app]) . '?v=' . $cacheBusterValue,
+                        'type' => 'image/svg+xml',
+                        'sizes' => '16x16'
+                    ]
+                ],
+            'display' => 'standalone'
+        ];
+        $response = new Http\JSONResponse($responseJS);
+        $response->cacheFor(3600);
+        return $response;
+    }
 }

Spacing +12 added lines, -12 removed lines

@@ -219,7 +219,7 @@ 
 
 		try {
 			$mime = $this->imageManager->updateImage($key, $image['tmp_name']);
-			$this->themingDefaults->set($key . 'Mime', $mime);
+			$this->themingDefaults->set($key.'Mime', $mime);
 		} catch (\Exception $e) {
 			return new DataResponse(
 				[
@@ -292,12 +292,12 @@ 
 		$csp->allowInlineStyle();
 		$response->setContentSecurityPolicy($csp);
 		$response->cacheFor(3600);
-		$response->addHeader('Content-Type', $this->config->getAppValue($this->appName, $key . 'Mime', ''));
-		$response->addHeader('Content-Disposition', 'attachment; filename="' . $key . '"');
+		$response->addHeader('Content-Type', $this->config->getAppValue($this->appName, $key.'Mime', ''));
+		$response->addHeader('Content-Disposition', 'attachment; filename="'.$key.'"');
 		if (!$useSvg) {
 			$response->addHeader('Content-Type', 'image/png');
 		} else {
-			$response->addHeader('Content-Type', $this->config->getAppValue($this->appName, $key . 'Mime', ''));
+			$response->addHeader('Content-Type', $this->config->getAppValue($this->appName, $key.'Mime', ''));
 		}
 		return $response;
 	}
@@ -317,7 +317,7 @@ 
 		}
 
 		$theme = $themes[$themeId];
-		$customCss  = $theme->getCustomCss();
+		$customCss = $theme->getCustomCss();
 
 		// Generate variables
 		$variables = '';
@@ -327,12 +327,12 @@ 
 
 		// If plain is set, the browser decides of the css priority
 		if ($plain) {
-			$css = ":root { $variables } " . $customCss;
+			$css = ":root { $variables } ".$customCss;
 		} else { 
 			// If not set, we'll rely on the body class
 			$compiler = new Compiler();
 			$compiledCss = $compiler->compileString("[data-theme-$themeId] { $variables $customCss }");
-			$css = $compiledCss->getCss();;
+			$css = $compiledCss->getCss(); ;
 		}
 
 		try {
@@ -359,12 +359,12 @@ 
 			$description = $this->themingDefaults->getSlogan();
 		} else {
 			$info = $this->appManager->getAppInfo($app, false, $this->l10n->getLanguageCode());
-			$name = $info['name'] . ' - ' . $this->themingDefaults->getName();
+			$name = $info['name'].' - '.$this->themingDefaults->getName();
 			$shortName = $info['name'];
 			if (strpos($this->request->getRequestUri(), '/index.php/') !== false) {
-				$startUrl = $this->urlGenerator->getBaseUrl() . '/index.php/apps/' . $app . '/';
+				$startUrl = $this->urlGenerator->getBaseUrl().'/index.php/apps/'.$app.'/';
 			} else {
-				$startUrl = $this->urlGenerator->getBaseUrl() . '/apps/' . $app . '/';
+				$startUrl = $this->urlGenerator->getBaseUrl().'/apps/'.$app.'/';
 			}
 			$description = $info['summary'] ?? '';
 		}
@@ -379,13 +379,13 @@ 
 				[
 					[
 						'src' => $this->urlGenerator->linkToRoute('theming.Icon.getTouchIcon',
-								['app' => $app]) . '?v=' . $cacheBusterValue,
+								['app' => $app]).'?v='.$cacheBusterValue,
 						'type' => 'image/png',
 						'sizes' => '512x512'
 					],
 					[
 						'src' => $this->urlGenerator->linkToRoute('theming.Icon.getFavicon',
-								['app' => $app]) . '?v=' . $cacheBusterValue,
+								['app' => $app]).'?v='.$cacheBusterValue,
 						'type' => 'image/svg+xml',
 						'sizes' => '16x16'
 					]