@@ -450,7 +450,7 @@ |
||
450 | 450 | /** |
451 | 451 | * translate Nextcloud permissions to OCM Permissions |
452 | 452 | * |
453 | - * @param $ncPermissions |
|
453 | + * @param integer $ncPermissions |
|
454 | 454 | * @return array |
455 | 455 | */ |
456 | 456 | protected function ncPermissions2ocmPermissions($ncPermissions) { |
@@ -182,7 +182,7 @@ |
||
182 | 182 | |
183 | 183 | /** |
184 | 184 | * @param string $dir |
185 | - * @param $recursive |
|
185 | + * @param boolean $recursive |
|
186 | 186 | * @param callable|null $mountFilter |
187 | 187 | * @throws ForbiddenException |
188 | 188 | * @throws NotFoundException |
@@ -150,7 +150,7 @@ discard block |
||
150 | 150 | * share received from another server |
151 | 151 | * |
152 | 152 | * @param ICloudFederationShare $share |
153 | - * @return string provider specific unique ID of the share |
|
153 | + * @return integer provider specific unique ID of the share |
|
154 | 154 | * |
155 | 155 | * @throws ProviderCouldNotAddShareException |
156 | 156 | * @throws \OCP\AppFramework\QueryException |
@@ -325,11 +325,11 @@ discard block |
||
325 | 325 | /** |
326 | 326 | * notify user about new federated share |
327 | 327 | * |
328 | - * @param $shareWith |
|
329 | - * @param $shareId |
|
330 | - * @param $ownerFederatedId |
|
331 | - * @param $sharedByFederatedId |
|
332 | - * @param $name |
|
328 | + * @param string $shareWith |
|
329 | + * @param integer $shareId |
|
330 | + * @param string $ownerFederatedId |
|
331 | + * @param string $sharedByFederatedId |
|
332 | + * @param string $name |
|
333 | 333 | */ |
334 | 334 | private function notifyAboutNewShare($shareWith, $shareId, $ownerFederatedId, $sharedByFederatedId, $name) { |
335 | 335 | $notification = $this->notificationManager->createNotification(); |
@@ -821,7 +821,7 @@ discard block |
||
821 | 821 | /** |
822 | 822 | * get the supported share types, e.g. "user", "group", etc. |
823 | 823 | * |
824 | - * @return array |
|
824 | + * @return string[] |
|
825 | 825 | * |
826 | 826 | * @since 14.0.0 |
827 | 827 | */ |
@@ -210,12 +210,12 @@ discard block |
||
210 | 210 | * @param $password |
211 | 211 | * @param $name |
212 | 212 | * @param $owner |
213 | - * @param $user |
|
213 | + * @param string $user |
|
214 | 214 | * @param $mountPoint |
215 | 215 | * @param $hash |
216 | - * @param $accepted |
|
216 | + * @param integer $accepted |
|
217 | 217 | * @param $remoteId |
218 | - * @param $parent |
|
218 | + * @param integer $parent |
|
219 | 219 | * @param $shareType |
220 | 220 | * @return bool |
221 | 221 | */ |
@@ -409,7 +409,7 @@ discard block |
||
409 | 409 | * |
410 | 410 | * @param string $remoteDomain |
411 | 411 | * @param string $token |
412 | - * @param $remoteId id of the share |
|
412 | + * @param integer $remoteId id of the share |
|
413 | 413 | * @param string $feedback |
414 | 414 | * @return bool |
415 | 415 | */ |
@@ -506,6 +506,9 @@ discard block |
||
506 | 506 | return $result; |
507 | 507 | } |
508 | 508 | |
509 | + /** |
|
510 | + * @param string $mountPoint |
|
511 | + */ |
|
509 | 512 | public function removeShare($mountPoint) { |
510 | 513 | |
511 | 514 | $mountPointObj = $this->mountManager->find($mountPoint); |
@@ -295,8 +295,8 @@ |
||
295 | 295 | * |
296 | 296 | * @NoAdminRequired |
297 | 297 | * |
298 | - * @param bool $show |
|
299 | - * @param bool $key the key of the folder |
|
298 | + * @param integer $show |
|
299 | + * @param string|boolean $key the key of the folder |
|
300 | 300 | * |
301 | 301 | * @return Response |
302 | 302 | */ |
@@ -335,7 +335,7 @@ |
||
335 | 335 | * @param string $shareWith |
336 | 336 | * @param string $publicUpload |
337 | 337 | * @param string $password |
338 | - * @param bool $sendPasswordByTalk |
|
338 | + * @param string $sendPasswordByTalk |
|
339 | 339 | * @param string $expireDate |
340 | 340 | * |
341 | 341 | * @return DataResponse |
@@ -307,10 +307,10 @@ discard block |
||
307 | 307 | * publish activity if a file/folder was shared by mail |
308 | 308 | * |
309 | 309 | * @param $subject |
310 | - * @param $parameters |
|
311 | - * @param $affectedUser |
|
310 | + * @param string[] $parameters |
|
311 | + * @param string $affectedUser |
|
312 | 312 | * @param $fileId |
313 | - * @param $filePath |
|
313 | + * @param string $filePath |
|
314 | 314 | */ |
315 | 315 | protected function publishActivity($subject, $parameters, $affectedUser, $fileId, $filePath) { |
316 | 316 | $event = $this->activityManager->generateEvent(); |
@@ -1067,7 +1067,7 @@ discard block |
||
1067 | 1067 | /** |
1068 | 1068 | * get database row of a give share |
1069 | 1069 | * |
1070 | - * @param $id |
|
1070 | + * @param integer $id |
|
1071 | 1071 | * @return array |
1072 | 1072 | * @throws ShareNotFound |
1073 | 1073 | */ |
@@ -202,7 +202,7 @@ |
||
202 | 202 | * @param string $displayName |
203 | 203 | * @param string $email |
204 | 204 | * @param array $groups |
205 | - * @param array $subadmins |
|
205 | + * @param array $subadmin |
|
206 | 206 | * @param string $quota |
207 | 207 | * @param string $language |
208 | 208 | * @return DataResponse |
@@ -89,6 +89,9 @@ discard block |
||
89 | 89 | return $path === '.'; |
90 | 90 | } |
91 | 91 | |
92 | + /** |
|
93 | + * @param string $path |
|
94 | + */ |
|
92 | 95 | private function cleanKey($path) { |
93 | 96 | if ($this->isRoot($path)) { |
94 | 97 | return '/'; |
@@ -240,6 +243,9 @@ discard block |
||
240 | 243 | } |
241 | 244 | } |
242 | 245 | |
246 | + /** |
|
247 | + * @param string $path |
|
248 | + */ |
|
243 | 249 | private function batchDelete($path = null) { |
244 | 250 | $params = array( |
245 | 251 | 'Bucket' => $this->bucket |
@@ -352,7 +358,7 @@ discard block |
||
352 | 358 | * When the information is already present (e.g. opendir has been called before) |
353 | 359 | * this value is return. Otherwise a headObject is emitted. |
354 | 360 | * |
355 | - * @param $path |
|
361 | + * @param string $path |
|
356 | 362 | * @return int|mixed |
357 | 363 | */ |
358 | 364 | private function getContentLength($path) { |
@@ -374,8 +380,8 @@ discard block |
||
374 | 380 | * When the information is already present (e.g. opendir has been called before) |
375 | 381 | * this value is return. Otherwise a headObject is emitted. |
376 | 382 | * |
377 | - * @param $path |
|
378 | - * @return mixed|string |
|
383 | + * @param string $path |
|
384 | + * @return string |
|
379 | 385 | */ |
380 | 386 | private function getLastModified($path) { |
381 | 387 | if (isset($this->filesCache[$path])) { |
@@ -633,6 +639,9 @@ discard block |
||
633 | 639 | return $this->id; |
634 | 640 | } |
635 | 641 | |
642 | + /** |
|
643 | + * @param string $path |
|
644 | + */ |
|
636 | 645 | public function writeBack($tmpFile, $path) { |
637 | 646 | try { |
638 | 647 | $source = fopen($tmpFile, 'r'); |