[FIX] Resolve 'app already exists' error on app update (#17544)

* fix: resolve 'app already exists' error on app update

The 'Apps' and 'Marketplace' admin pages were using the function
`installApp` from the `AppClientOrchestrator` class instead of `updateApp` to
update apps. As a result, the 'app already exists' error was being thrown and
apps were not being updated.

* fix: remove `install app` handle from apps admin page

As the `install app` handle is only needed in the marketplace page,
there's no need to have it in the apps admin page.

app/apps/client/admin/apps.js

@@ -226,7 +226,7 @@ Template.apps.events({
 		} = instance.state.get('apps').find(({ id }) => id === currentTarget.dataset.id);
 		FlowRouter.go('app-manage', { appId }, { version });
 	},
-	async 'click .js-install, click .js-update'(event, instance) {
+	async 'click .js-update'(event, instance) {
 		event.preventDefault();
 		event.stopPropagation();
 
@@ -240,7 +240,7 @@ Template.apps.events({
 		instance.startAppWorking(app.id);
 
 		try {
-			const { status } = await Apps.installApp(app.id, app.marketplaceVersion);
+			const { status } = await Apps.updateApp(app.id, app.marketplaceVersion);
 			warnStatusChange(app.name, status);
 		} catch (error) {
 			handleAPIError(error);

app/apps/client/admin/marketplace.js

@@ -273,7 +273,7 @@ Template.marketplace.events({
 		} = instance.state.get('apps').find(({ id }) => id === currentTarget.dataset.id);
 		FlowRouter.go('marketplace-app', { appId }, { version: version || marketplaceVersion });
 	},
-	async 'click .js-install, click .js-update'(event, instance) {
+	async 'click .js-install'(event, instance) {
 		event.preventDefault();
 		event.stopPropagation();
 
@@ -284,6 +284,7 @@ Template.marketplace.events({
 		}
 
 		const { currentTarget: button } = event;
+
 		const app = instance.state.get('apps').find(({ id }) => id === button.dataset.id);
 
 		instance.startAppWorking(app.id);
@@ -297,6 +298,31 @@ Template.marketplace.events({
 			instance.stopAppWorking(app.id);
 		}
 	},
+	async 'click .js-update'(event, instance) {
+		event.preventDefault();
+		event.stopPropagation();
+
+		const isLoggedInCloud = await checkCloudLogin();
+		instance.state.set('isLoggedInCloud', isLoggedInCloud);
+		if (!isLoggedInCloud) {
+			return;
+		}
+
+		const { currentTarget: button } = event;
+
+		const app = instance.state.get('apps').find(({ id }) => id === button.dataset.id);
+
+		instance.startAppWorking(app.id);
+
+		try {
+			const { status } = await Apps.updateApp(app.id, app.marketplaceVersion);
+			warnStatusChange(app.name, status);
+		} catch (error) {
+			handleAPIError(error);
+		} finally {
+			instance.stopAppWorking(app.id);
+		}
+	},
 	async 'click .js-purchase'(event, instance) {
 		event.preventDefault();
 		event.stopPropagation();