improve webmanifest

Signed-off-by: szaimen <[email protected]>

Author: szaimen

apps/theming/lib/Controller/ThemingController.php

@@ -372,9 +372,29 @@ public function getStylesheet() {
 	 */
 	public function getManifest($app) {
 		$cacheBusterValue = $this->config->getAppValue('theming', 'cachebuster', '0');
+		if ($app === 'core') {
+			$name = $this->themingDefaults->getName();
+			$short_name = $this->themingDefaults->getName();
+			$start_url = $this->urlGenerator->getBaseUrl();
+			$description = $this->themingDefaults->getSlogan();
+		} else {
+			$info = $this->appManager->getAppInfo($app);
+			$name = $info['name'] . ' - ' . $this->themingDefaults->getName();
+			$short_name = $info['name'];
+			if (strpos($this->request->getRequestUri(), '/index.php/') !== false) {
+				$start_url = $this->urlGenerator->getBaseUrl() . '/index.php' . $this->urlGenerator->linkTo($app, '');
+			} else {
+				$start_url = $this->urlGenerator->getBaseUrl() . $this->urlGenerator->linkTo($app, '');
+			}
+			$description = $info['summary'];
+		}
 		$responseJS = [
-			'name' => $this->themingDefaults->getName(),
-			'start_url' => $this->urlGenerator->getBaseUrl(),
+			'name' => $name,
+			'short_name' => $short_name,
+			'start_url' => $start_url,
+			'theme_color' => $this->themingDefaults->getColorPrimary(),
+			'background_color' => $this->themingDefaults->getColorPrimary(),
+			'description' => $description,
 			'icons' =>
 				[
 					[

apps/theming/lib/ThemingDefaults.php

@@ -384,7 +384,7 @@ public function replaceImagePath($app, $image) {
 				}
 			} catch (AppPathNotFoundException $e) {
 			}
-			$route = $this->urlGenerator->linkToRoute('theming.Theming.getManifest');
+			$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]);

apps/theming/tests/Controller/ThemingControllerTest.php

@@ -823,7 +823,10 @@ public function testGetManifest() {
 						'sizes' => '16x16'
 					]
 				],
-			'display' => 'standalone'
+			'display' => 'standalone',
+			'short_name' => 'Nextcloud',
+			'background_color' => null,
+			'description' => null
 		]);
 		$response->cacheFor(3600);
 		$this->assertEquals($response, $this->themingController->getManifest('core'));