Fix external sharing loading script (#18251)

Fix external sharing loading script

apps/federatedfilesharing/appinfo/app.php

@@ -23,22 +23,6 @@
  *
  */
 
-use OCA\FederatedFileSharing\Notifier;
 use OCA\FederatedFileSharing\AppInfo\Application;
 
-$app = \OC::$server->query(Application::class);
+\OC::$server->query(Application::class);
-$eventDispatcher = \OC::$server->getEventDispatcher();
-
-$manager = \OC::$server->getNotificationManager();
-$manager->registerNotifierService(Notifier::class);
-
-$federatedShareProvider = $app->getFederatedShareProvider();
-
-$eventDispatcher->addListener(
-	'OCA\Files::loadAdditionalScripts',
-	function() use ($federatedShareProvider) {
-		if ($federatedShareProvider->isIncomingServer2serverShareEnabled()) {
-			\OCP\Util::addScript('federatedfilesharing', 'external');
-		}
-	}
-);

apps/federatedfilesharing/js/external.js

@@ -1,4 +1,4 @@
-/*
+/**
  * Copyright (c) 2014 Robin Appelman <icewind@owncloud.com>
  *
  * This file is licensed under the Affero General Public License version 3
@@ -8,14 +8,19 @@
  *
  */
 (function() {
+
+	OCA.Sharing = OCA.Sharing || {}
+
 	/**
 	 * Shows "add external share" dialog.
 	 *
-	 * @param {String} remote remote server URL
+	 * @param {Object} share the share
-	 * @param {String} owner owner name
+	 * @param {String} share.remote remote server URL
-	 * @param {String} name name of the shared folder
+	 * @param {String} share.owner owner name
-	 * @param {String} token authentication token
+	 * @param {String} share.name name of the shared folder
+	 * @param {String} share.token authentication token
 	 * @param {bool} passwordProtected true if the share is password protected
+	 * @param {Function} callback the callback
 	 */
 	OCA.Sharing.showAddExternalDialog = function(share, passwordProtected, callback) {
 		var remote = share.remote;
@@ -109,19 +114,15 @@
 								name: share.name,
 								password: password
 							}
-						).done(
+						).done(function(data) {
-							function(data) {
 							if (data.hasOwnProperty('legacyMount')) {
 								fileList.reload();
 							} else {
 								OC.Notification.showTemporary(data.message);
 							}
-						}
+						}).fail(function(data) {
-						).fail(
-							function(data) {
 							OC.Notification.showTemporary(JSON.parse(data.responseText).message);
-							}
+						});
-						);
 					}
 				};
 
@@ -142,9 +143,7 @@
 		processSharesToConfirm: function() {
 			var fileList = this.filesApp.fileList;
 			// check for new server-to-server shares which need to be approved
-			$.get(OC.generateUrl('/apps/files_sharing/api/externalShares'),
+			$.get(OC.generateUrl('/apps/files_sharing/api/externalShares'), {}, function(shares) {
-			{},
-			function(shares) {
 				var index;
 				for (index = 0; index < shares.length; ++index) {
 					OCA.Sharing.showAddExternalDialog(
@@ -166,12 +165,9 @@
 							}
 						}
 					);
-				}
+				}});
-
-			});
-
 		}
 	};
-})();
+})(OC, OCA);
 
 OC.Plugins.register('OCA.Files.App', OCA.Sharing.ExternalShareDialogPlugin);

apps/federatedfilesharing/lib/AppInfo/Application.php

@@ -32,7 +32,7 @@ use OCA\FederatedFileSharing\AddressHandler;
 use OCA\FederatedFileSharing\Controller\RequestHandlerController;
 use OCA\FederatedFileSharing\FederatedShareProvider;
 use OCA\FederatedFileSharing\Notifications;
-use OCA\FederatedFileSharing\OCM\CloudFederationProvider;
+use OCA\FederatedFileSharing\Notifier;
 use OCA\FederatedFileSharing\OCM\CloudFederationProviderFiles;
 use OCP\AppFramework\App;
 use OCP\GlobalScale\IConfig;
@@ -99,6 +99,23 @@ class Application extends App {
 				$server->getCloudFederationProviderManager()
 			);
 		});
+
+		// register events listeners
+		$eventDispatcher = $server->getEventDispatcher();
+		$manager = $server->getNotificationManager();
+		$federatedShareProvider = $this->getFederatedShareProvider();
+
+		$manager->registerNotifierService(Notifier::class);
+		
+		$eventDispatcher->addListener(
+			'OCA\Files::loadAdditionalScripts',
+			function() use ($federatedShareProvider) {
+				if ($federatedShareProvider->isIncomingServer2serverShareEnabled()) {
+					\OCP\Util::addScript('federatedfilesharing', 'external');
+				}
+			}
+		);
+
 	}
 
 	/**