nextcloud / server

lib/private/Share20/ProviderFactory.php

Unused Use Statements -1 removed lines

@@ -32,7 +32,6 @@
 use OCA\FederatedFileSharing\AddressHandler;
 use OCA\FederatedFileSharing\FederatedShareProvider;
 use OCA\FederatedFileSharing\Notifications;
-use OCA\FederatedFileSharing\OCM\CloudFederationProvider;
 use OCA\FederatedFileSharing\TokenHandler;
 use OCA\ShareByMail\Settings\SettingsManager;
 use OCA\ShareByMail\ShareByMailProvider;

apps/cloud_federation_api/lib/Controller/RequestHandlerController.php

Doc Comments -1 removed lines

@@ -106,7 +106,6 @@
 	 * @param string $ownerDisplayName display name of the user who shared the item
 	 * @param string $sharedBy provider specific UID of the user who shared the resource
 	 * @param string $sharedByDisplayName display name of the user who shared the resource
-	 * @param string $sharedSecret use to authenticate accross servers
 	 * @param array $protocol (e,.g. ['name' => 'webdav', 'options' => ['username' => 'john', 'permissions' => 31]])
 	 * @param string $shareType ('group' or 'user' share)
 	 * @param $resourceType ('file', 'calendar',...)

apps/files_sharing/lib/External/Manager.php

Doc Comments +5 added lines, -2 removed lines

@@ -324,9 +324,9 @@ 
 	 *
 	 * @param string $remoteDomain
 	 * @param string $token
-	 * @param $remoteId id of the share
+	 * @param integer $remoteId id of the share
 	 * @param string $feedback
-	 * @return mixed
+	 * @return boolean|null
 	 */
 	protected function tryOCMEndPoint($remoteDomain, $token, $remoteId, $feedback) {
 		switch ($feedback) {
@@ -406,6 +406,9 @@ 
 		return $result;
 	}
 
+	/**
+	 * @param string $mountPoint
+	 */
 	public function removeShare($mountPoint) {
 
 		$mountPointObj = $this->mountManager->find($mountPoint);

apps/federatedfilesharing/lib/ocm/CloudFederationProviderFiles.php

Doc Comments +2 added lines, -2 removed lines

@@ -122,7 +122,7 @@ 
 	 * share received from another server
 	 *
 	 * @param ICloudFederationShare $share
-	 * @return string provider specific unique ID of the share
+	 * @return integer provider specific unique ID of the share
 	 *
 	 * @throws ProviderCouldNotAddShareException
 	 * @throws \OCP\AppFramework\QueryException
@@ -265,7 +265,7 @@ 
 	}
 
 	/**
-	 * @param $id
+	 * @param string $id
 	 * @param $notification
 	 * @return bool
 	 * @throws ActionNotSupportedException

Unused Use Statements -2 removed lines

@@ -27,7 +27,6 @@ 
 use OCA\FederatedFileSharing\AddressHandler;
 use OCA\FederatedFileSharing\FederatedShareProvider;
 use OCP\Activity\IManager as IActivityManager;
-use OCP\Activity\IManager;
 use OCP\App\IAppManager;
 use OCP\Federation\Exceptions\ActionNotSupportedException;
 use OCP\Federation\Exceptions\AuthenticationFailedException;
@@ -42,7 +41,6 @@ 
 use OCP\IURLGenerator;
 use OCP\IUserManager;
 use OCP\Notification\IManager as INotificationManager;
-use OCP\Share\Exceptions\ShareNotFound;
 use OCP\Share\IShare;
 use OCP\Util;
 

lib/public/Federation/Exceptions/AuthenticationFailedException.php

Doc Comments -1 removed lines

@@ -28,7 +28,6 @@
 	/**
 	 * BadRequestException constructor.
 	 *
-	 * @param array $missingParameters
 	 */
 	public function __construct() {
 		$l = \OC::$server->getL10N('federation');