Completed
Pull Request — master (#9345)
by Björn
41:51 queued 19:17
created
apps/federatedfilesharing/lib/ocm/CloudFederationProviderFiles.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -121,7 +121,7 @@
 block discarded – undo
121 121
 	 * share received from another server
122 122
 	 *
123 123
 	 * @param ICloudFederationShare $share
124
-	 * @return string provider specific unique ID of the share
124
+	 * @return integer provider specific unique ID of the share
125 125
 	 *
126 126
 	 * @throws ProviderCouldNotAddShareException
127 127
 	 * @throws \OCP\AppFramework\QueryException
Please login to merge, or discard this patch.
lib/private/Share20/ProviderFactory.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -32,7 +32,6 @@
 block discarded – undo
32 32
 use OCA\FederatedFileSharing\AddressHandler;
33 33
 use OCA\FederatedFileSharing\FederatedShareProvider;
34 34
 use OCA\FederatedFileSharing\Notifications;
35
-use OCA\FederatedFileSharing\OCM\CloudFederationProvider;
36 35
 use OCA\FederatedFileSharing\TokenHandler;
37 36
 use OCA\ShareByMail\Settings\SettingsManager;
38 37
 use OCA\ShareByMail\ShareByMailProvider;
Please login to merge, or discard this patch.
apps/federatedfilesharing/lib/Notifications.php 1 patch
Doc Comments   +7 added lines, -3 removed lines patch added patch discarded remove patch
@@ -156,7 +156,7 @@  discard block
 block discarded – undo
156 156
 	 * @param string $remote url
157 157
 	 * @param int $id share id
158 158
 	 * @param string $token
159
-	 * @return bool
159
+	 * @return boolean|null
160 160
 	 */
161 161
 	public function sendRemoteUnShare($remote, $id, $token) {
162 162
 		$this->sendUpdateToRemote($remote, $id, $token, 'unshare');
@@ -168,7 +168,7 @@  discard block
 block discarded – undo
168 168
 	 * @param string $remote url
169 169
 	 * @param int $id share id
170 170
 	 * @param string $token
171
-	 * @return bool
171
+	 * @return boolean|null
172 172
 	 */
173 173
 	public function sendRevokeShare($remote, $id, $token) {
174 174
 		$this->sendUpdateToRemote($remote, $id, $token, 'revoke');
@@ -181,7 +181,7 @@  discard block
 block discarded – undo
181 181
 	 * @param int $remoteId
182 182
 	 * @param string $token
183 183
 	 * @param int $permissions
184
-	 * @return bool
184
+	 * @return boolean|null
185 185
 	 */
186 186
 	public function sendPermissionChange($remote, $remoteId, $token, $permissions) {
187 187
 		$this->sendUpdateToRemote($remote, $remoteId, $token, 'permissions', ['permissions' => $permissions]);
@@ -342,6 +342,10 @@  discard block
 block discarded – undo
342 342
 		return '';
343 343
 	}
344 344
 
345
+	/**
346
+	 * @param string $remoteDomain
347
+	 * @param string $urlSuffix
348
+	 */
345 349
 	protected function sendNotificationToOCMEndPoint($remoteDomain, $urlSuffix, $fields) {
346 350
 		$client = $this->httpClientService->newClient();
347 351
 		switch ($urlSuffix) {
Please login to merge, or discard this patch.
lib/private/Federation/CloudFederationProviderManager.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -23,7 +23,6 @@
 block discarded – undo
23 23
 namespace OC\Federation;
24 24
 
25 25
 use OCP\App\IAppManager;
26
-use OCP\Federation\Exceptions\ProviderAlreadyExistsException;
27 26
 use OCP\Federation\Exceptions\ProviderDoesNotExistsException;
28 27
 use OCP\Federation\ICloudFederationNotification;
29 28
 use OCP\Federation\ICloudFederationProvider;
Please login to merge, or discard this patch.