chore: sync changes for Apps management migration to settings (#417)

This PR synchronizes backend logic for ExAppFetcher and required changes for Apps management. ![Screen Shot 2024-10-14 at 17 01 48](https://github.com/user-attachments/assets/60667041-92b1-4d9e-a78a-9476a1271d1f) ## TODO - [ ] Merge with the server PR for settings app changes: nextcloud/server#48665 - [x] Remove old UI parts that are not needed anymore --------- Signed-off-by: Andrey Borysenko <[email protected]> Signed-off-by: nextcloud-command <[email protected]> Co-authored-by: nextcloud-command <[email protected]>
nextcloud · Oct 29, 2024 · b8200b7 · b8200b7
1 parent 661716b
commit b8200b7
Show file tree

Hide file tree

Showing 40 changed files with 125 additions and 2,382 deletions.
diff --git a/appinfo/routes.php b/appinfo/routes.php
@@ -27,7 +27,6 @@
 			'requirements' => ['other' => '.+'], 'defaults' => ['other' => '']],
 
 		// ExApps actions
-		['name' => 'ExAppsPage#viewApps', 'url' => '/apps', 'verb' => 'GET' , 'root' => '/apps'],
 		['name' => 'ExAppsPage#listCategories', 'url' => '/apps/categories', 'verb' => 'GET' , 'root' => ''],
 		['name' => 'ExAppsPage#listApps', 'url' => '/apps/list', 'verb' => 'GET' , 'root' => ''],
 		['name' => 'ExAppsPage#enableApp', 'url' => '/apps/enable/{appId}', 'verb' => 'GET' , 'root' => ''],

diff --git a/js/app_api-adminSettings.js b/js/app_api-adminSettings.js
diff --git a/js/app_api-adminSettings.js.map b/js/app_api-adminSettings.js.map
diff --git a/js/app_api-main.js b/js/app_api-main.js
diff --git a/js/app_api-main.js.LICENSE.txt b/js/app_api-main.js.LICENSE.txt
diff --git a/js/app_api-main.js.map b/js/app_api-main.js.map
diff --git a/js/app_api-src_views_Apps_vue.js b/js/app_api-src_views_Apps_vue.js
diff --git a/js/app_api-src_views_Apps_vue.js.map b/js/app_api-src_views_Apps_vue.js.map
diff --git a/js/app_api-vendors-node_modules_nextcloud_dialogs_dist_chunks_FilePicker-CDU2njOJ_mjs.js b/js/app_api-vendors-node_modules_nextcloud_dialogs_dist_chunks_FilePicker-CDU2njOJ_mjs.js
diff --git a/js/app_api-vendors-node_modules_nextcloud_dialogs_dist_chunks_FilePicker-CDU2njOJ_mjs.js.map b/js/app_api-vendors-node_modules_nextcloud_dialogs_dist_chunks_FilePicker-CDU2njOJ_mjs.js.map
diff --git a/...odules_vue-material-design-icons_Alert_vue-node_modules_vue-material-design-ico-20a3e2.js b/...odules_vue-material-design-icons_Alert_vue-node_modules_vue-material-design-ico-20a3e2.js
diff --git a/...aterial-design-icons_Alert_vue-node_modules_vue-material-design-ico-20a3e2.js.LICENSE.txt b/...aterial-design-icons_Alert_vue-node_modules_vue-material-design-ico-20a3e2.js.LICENSE.txt
diff --git a/...es_vue-material-design-icons_Alert_vue-node_modules_vue-material-design-ico-20a3e2.js.map b/...es_vue-material-design-icons_Alert_vue-node_modules_vue-material-design-ico-20a3e2.js.map
diff --git a/lib/AppInfo/Application.php b/lib/AppInfo/Application.php
@@ -36,12 +36,6 @@
 use OCP\Files\Events\Node\NodeTouchedEvent;
 use OCP\Files\Events\Node\NodeWrittenEvent;
 use OCP\IConfig;
-use OCP\IGroupManager;
-use OCP\IL10N;
-use OCP\INavigationManager;
-use OCP\IURLGenerator;
-use OCP\IUser;
-use OCP\IUserSession;
 use OCP\SabrePluginEvent;
 use OCP\Settings\Events\DeclarativeSettingsGetValueEvent;
 use OCP\Settings\Events\DeclarativeSettingsRegisterFormEvent;
@@ -110,9 +104,7 @@ public function register(IRegistrationContext $context): void {
 	}
 
 	public function boot(IBootContext $context): void {
-		$server = $context->getServerContainer();
 		try {
-			$context->injectFn($this->registerExAppsManagementNavigation(...));
 			$context->injectFn($this->registerExAppsMenuEntries(...));
 		} catch (NotFoundExceptionInterface|ContainerExceptionInterface|Throwable) {
 		}
@@ -127,35 +119,6 @@ public function registerDavAuth(): void {
 		});
 	}
 
-	/**
-	 * Register ExApps management navigation entry right after default Apps management link.
-	 *
-	 * @throws ContainerExceptionInterface
-	 * @throws NotFoundExceptionInterface
-	 */
-	private function registerExAppsManagementNavigation(IUserSession $userSession): void {
-		$container = $this->getContainer();
-		/** @var IGroupManager $groupManager */
-		$groupManager = $container->get(IGroupManager::class);
-		/** @var IUser $user */
-		$user = $userSession->getUser();
-		if ($groupManager->isInGroup($user->getUID(), 'admin')) {
-			$container->get(INavigationManager::class)->add(function () use ($container) {
-				$urlGenerator = $container->get(IURLGenerator::class);
-				$l10n = $container->get(IL10N::class);
-				return [
-					'id' => self::APP_ID,
-					'type' => 'settings',
-					'order' => 6,
-					'href' => $urlGenerator->linkToRoute('app_api.ExAppsPage.viewApps'),
-					'icon' => $urlGenerator->imagePath('app_api', 'app-dark.svg'),
-					'target' => '_blank',
-					'name' => $l10n->t('External Apps'),
-				];
-			});
-		}
-	}
-
 	private function registerExAppsMenuEntries(): void {
 		$container = $this->getContainer();
 		$menuEntryService = $container->get(TopMenuService::class);

diff --git a/lib/Controller/ExAppsPageController.php b/lib/Controller/ExAppsPageController.php
@@ -22,11 +22,8 @@
 use OCP\AppFramework\Http;
 use OCP\AppFramework\Http\Attribute\NoCSRFRequired;
 use OCP\AppFramework\Http\Attribute\PasswordConfirmationRequired;
-use OCP\AppFramework\Http\ContentSecurityPolicy;
 use OCP\AppFramework\Http\DataDownloadResponse;
 use OCP\AppFramework\Http\JSONResponse;
-use OCP\AppFramework\Http\TemplateResponse;
-use OCP\AppFramework\Services\IInitialState;
 use OCP\IConfig;
 use OCP\IL10N;
 use OCP\IRequest;
@@ -37,89 +34,22 @@
  * ExApps actions controller similar to default one with project-specific changes and additions
  */
 class ExAppsPageController extends Controller {
-	private IInitialState $initialStateService;
-	private IConfig $config;
-	private AppAPIService $service;
-	private DaemonConfigService $daemonConfigService;
-	private DockerActions $dockerActions;
-	private CategoryFetcher $categoryFetcher;
-	private IFactory $l10nFactory;
-	private ExAppFetcher $exAppFetcher;
-	private IL10N $l10n;
-	private LoggerInterface $logger;
-	private IAppManager $appManager;
 
 	public function __construct(
 		IRequest $request,
-		IConfig $config,
-		IInitialState $initialStateService,
-		AppAPIService $service,
-		DaemonConfigService $daemonConfigService,
-		DockerActions $dockerActions,
-		CategoryFetcher $categoryFetcher,
-		IFactory $l10nFactory,
-		ExAppFetcher $exAppFetcher,
-		IL10N $l10n,
-		LoggerInterface $logger,
-		IAppManager $appManager,
+		private readonly IConfig $config,
+		private readonly AppAPIService $service,
+		private readonly DaemonConfigService $daemonConfigService,
+		private readonly DockerActions $dockerActions,
+		private readonly CategoryFetcher $categoryFetcher,
+		private readonly IFactory $l10nFactory,
+		private readonly ExAppFetcher $exAppFetcher,
+		private readonly IL10N $l10n,
+		private readonly LoggerInterface $logger,
+		private readonly IAppManager $appManager,
 		private readonly ExAppService $exAppService,
 	) {
 		parent::__construct(Application::APP_ID, $request);
-
-		$this->initialStateService = $initialStateService;
-		$this->config = $config;
-		$this->service = $service;
-		$this->daemonConfigService = $daemonConfigService;
-		$this->dockerActions = $dockerActions;
-		$this->categoryFetcher = $categoryFetcher;
-		$this->l10nFactory = $l10nFactory;
-		$this->l10n = $l10n;
-		$this->exAppFetcher = $exAppFetcher;
-		$this->logger = $logger;
-		$this->appManager = $appManager;
-	}
-
-	#[NoCSRFRequired]
-	public function viewApps(): TemplateResponse {
-		$defaultDaemonConfigName = $this->config->getAppValue(Application::APP_ID, 'default_daemon_config');
-
-		$appInitialData = [
-			'appstoreEnabled' => $this->config->getSystemValueBool('appstoreenabled', true),
-			'updateCount' => count($this->getExAppsWithUpdates()),
-		];
-
-		if ($defaultDaemonConfigName !== '') {
-			$daemonConfig = $this->daemonConfigService->getDaemonConfigByName($defaultDaemonConfigName);
-			if ($daemonConfig !== null) {
-				$this->dockerActions->initGuzzleClient($daemonConfig);
-				$daemonConfigAccessible = $this->dockerActions->ping($this->dockerActions->buildDockerUrl($daemonConfig));
-				$appInitialData['daemon_config_accessible'] = $daemonConfigAccessible;
-				$appInitialData['default_daemon_config'] = $daemonConfig->jsonSerialize();
-				unset($appInitialData['default_daemon_config']['deploy_config']['haproxy_password']); // do not expose password
-				if (!$daemonConfigAccessible) {
-					$this->logger->error(sprintf('Deploy daemon "%s" is not accessible by Nextcloud. Please verify its configuration', $daemonConfig->getName()));
-				}
-			}
-		}
-
-		$this->initialStateService->provideInitialState('apps', $appInitialData);
-
-		$templateResponse = new TemplateResponse(Application::APP_ID, 'main');
-		$policy = new ContentSecurityPolicy();
-		$policy->addAllowedImageDomain('https://usercontent.apps.nextcloud.com');
-		$templateResponse->setContentSecurityPolicy($policy);
-
-		return $templateResponse;
-	}
-
-	private function getExAppsWithUpdates(): array {
-		$apps = $this->exAppFetcher->get();
-		$appsWithUpdates = array_filter($apps, function (array $app) {
-			$exApp = $this->exAppService->getExApp($app['id']);
-			$newestVersion = $app['releases'][0]['version'];
-			return $exApp !== null && isset($app['releases'][0]['version']) && version_compare($newestVersion, $exApp->getVersion(), '>');
-		});
-		return array_values($appsWithUpdates);
 	}
 
 	/**
@@ -195,6 +125,7 @@ private function getAppsForCategory(string $requestedCategory = ''): array {
 
 			$formattedApps[] = [
 				'id' => $app['id'],
+				'app_api' => true,
 				'installed' => $exApp !== null, // if ExApp registered then it's assumed that it was already deployed (installed)
 				'appstore' => true,
 				'name' => $app['translations'][$currentLanguage]['name'] ?? $app['translations']['en']['name'],
@@ -228,6 +159,7 @@ private function getAppsForCategory(string $requestedCategory = ''): array {
 				'removable' => $existsLocally,
 				'active' => $exApp !== null && $exApp->getEnabled() === 1,
 				'needsDownload' => !$existsLocally,
+				'groups' => [],
 				'fromAppStore' => true,
 				'appstoreData' => $app,
 				'daemon' => $daemon,
@@ -242,7 +174,7 @@ private function getAppsForCategory(string $requestedCategory = ''): array {
 	#[NoCSRFRequired]
 	public function listApps(): JSONResponse {
 		$apps = $this->getAppsForCategory('');
-		$appsWithUpdate = $this->getExAppsWithUpdates();
+		$appsWithUpdate = $this->exAppFetcher->getExAppsWithUpdates();
 
 		$exApps = $this->exAppService->getExAppsList('all');
 		$dependencyAnalyzer = new DependencyAnalyzer(new Platform($this->config), $this->l10n);
@@ -325,12 +257,14 @@ private function buildLocalAppsList(array $apps, array $exApps): array {
 
 				$formattedLocalApps[] = [
 					'id' => $app['id'],
+					'app_api' => true,
 					'appstore' => false,
 					'installed' => true,
 					'name' => $exApp->getName(),
-					'description' => '',
+					'description' => $this->l10n->t('This app is not installed from the AppStore. No extra information available. Only enable/disable and remove actions are allowed.'),
 					'summary' => '',
 					'license' => '',
+					'licence' => '',
 					'author' => '',
 					'shipped' => false,
 					'version' => $exApp->getVersion(),
@@ -357,6 +291,7 @@ private function buildLocalAppsList(array $apps, array $exApps): array {
 					'removable' => true, // to allow "remove" command for manual-install
 					'active' => $exApp->getEnabled() === 1,
 					'needsDownload' => false,
+					'groups' => [],
 					'fromAppStore' => false,
 					'appstoreData' => $app,
 					'daemon' => $daemon,
@@ -391,7 +326,7 @@ public function enableApp(string $appId): JSONResponse {
 			return new JSONResponse([]);
 		}
 
-		$appsWithUpdate = $this->getExAppsWithUpdates();
+		$appsWithUpdate = $this->exAppFetcher->getExAppsWithUpdates();
 		$appIdsWithUpdate = array_map(function (array $appWithUpdate) {
 			return $appWithUpdate['id'];
 		}, $appsWithUpdate);
@@ -426,7 +361,7 @@ public function disableApp(string $appId): JSONResponse {
 	#[PasswordConfirmationRequired]
 	#[NoCSRFRequired]
 	public function updateApp(string $appId): JSONResponse {
-		$appsWithUpdate = $this->getExAppsWithUpdates();
+		$appsWithUpdate = $this->exAppFetcher->getExAppsWithUpdates();
 		$appIdsWithUpdate = array_map(function (array $appWithUpdate) {
 			return $appWithUpdate['id'];
 		}, $appsWithUpdate);