@@ -52,7 +52,6 @@ |
||
52 | 52 | * @param string $user the uid of the current user |
53 | 53 | * @param bool $isAdmin whether the current users is an admin |
54 | 54 | * @param IGroupManager $groupManager |
55 | - * @param IUserManager $userManager |
|
56 | 55 | * @param IUserSession $userSession |
57 | 56 | */ |
58 | 57 | public function __construct( |
@@ -354,6 +354,7 @@ discard block |
||
354 | 354 | * Calculate steps between two Colors |
355 | 355 | * @param object Color $steps start color |
356 | 356 | * @param object Color $ends end color |
357 | + * @param integer $steps |
|
357 | 358 | * @return array [r,g,b] steps for each color to go from $steps to $ends |
358 | 359 | */ |
359 | 360 | private function stepCalc($steps, $ends) { |
@@ -366,8 +367,9 @@ discard block |
||
366 | 367 | |
367 | 368 | /** |
368 | 369 | * Convert a string to an integer evenly |
369 | - * @param string $hash the text to parse |
|
370 | - * @param int $maximum the maximum range |
|
370 | + * @param integer $steps |
|
371 | + * @param Color $color1 |
|
372 | + * @param Color $color2 |
|
371 | 373 | * @return int between 0 and $maximum |
372 | 374 | */ |
373 | 375 | private function mixPalette($steps, $color1, $color2) { |
@@ -304,6 +304,7 @@ discard block |
||
304 | 304 | * @param array $params the parameters of the sql query |
305 | 305 | * @param int $limit the maximum number of rows |
306 | 306 | * @param int $offset from which row we want to start |
307 | + * @param string $msg |
|
307 | 308 | * @return string formatted error message string |
308 | 309 | * @since 9.1.0 |
309 | 310 | * @deprecated 14.0.0 Move over to QBMapper |
@@ -359,7 +360,7 @@ discard block |
||
359 | 360 | * Returns an db result and throws exceptions when there are more or less |
360 | 361 | * results |
361 | 362 | * @param string $sql the sql query |
362 | - * @param array $params the parameters of the sql query |
|
363 | + * @param string[] $params the parameters of the sql query |
|
363 | 364 | * @param int $limit the maximum number of rows |
364 | 365 | * @param int $offset from which row we want to start |
365 | 366 | * @throws DoesNotExistException if the item does not exist |
@@ -380,7 +380,6 @@ |
||
380 | 380 | /** |
381 | 381 | * get the number of disabled users in a group |
382 | 382 | * |
383 | - * @param string $search |
|
384 | 383 | * @return int|bool |
385 | 384 | */ |
386 | 385 | public function countDisabledInGroup(string $gid): int { |
@@ -164,8 +164,8 @@ |
||
164 | 164 | } |
165 | 165 | |
166 | 166 | /** |
167 | - * @param array $states |
|
168 | - * @param IProvider $providers |
|
167 | + * @param string[] $states |
|
168 | + * @param IProvider[] $providers |
|
169 | 169 | */ |
170 | 170 | private function isProviderMissing(array $states, array $providers): bool { |
171 | 171 | $indexed = []; |
@@ -47,7 +47,7 @@ |
||
47 | 47 | |
48 | 48 | /** |
49 | 49 | * @throws \Sabre\DAV\Exception\Forbidden |
50 | - * @return bool |
|
50 | + * @return false|null |
|
51 | 51 | */ |
52 | 52 | public function handleAnonymousOptions(RequestInterface $request, ResponseInterface $response) { |
53 | 53 | if ($request->getMethod() === 'OPTIONS' && $request->getPath() === '') { |
@@ -238,7 +238,6 @@ discard block |
||
238 | 238 | /** |
239 | 239 | * Get all available apps in a category |
240 | 240 | * |
241 | - * @param string $category |
|
242 | 241 | * @return JSONResponse |
243 | 242 | * @throws \Exception |
244 | 243 | */ |
@@ -415,7 +414,7 @@ discard block |
||
415 | 414 | * apps will be enabled for specific groups only if $groups is defined |
416 | 415 | * |
417 | 416 | * @PasswordConfirmationRequired |
418 | - * @param array $appIds |
|
417 | + * @param string[] $appIds |
|
419 | 418 | * @param array $groups |
420 | 419 | * @return JSONResponse |
421 | 420 | */ |
@@ -479,7 +478,7 @@ discard block |
||
479 | 478 | /** |
480 | 479 | * @PasswordConfirmationRequired |
481 | 480 | * |
482 | - * @param array $appIds |
|
481 | + * @param string[] $appIds |
|
483 | 482 | * @return JSONResponse |
484 | 483 | */ |
485 | 484 | public function disableApps(array $appIds): JSONResponse { |
@@ -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) { |
@@ -576,7 +576,7 @@ discard block |
||
576 | 576 | * Share a path |
577 | 577 | * |
578 | 578 | * @param \OCP\Share\IShare $share |
579 | - * @return Share The share object |
|
579 | + * @return IShare The share object |
|
580 | 580 | * @throws \Exception |
581 | 581 | * |
582 | 582 | * TODO: handle link share permissions or check them |
@@ -1169,7 +1169,7 @@ discard block |
||
1169 | 1169 | * Get the share by token possible with password |
1170 | 1170 | * |
1171 | 1171 | * @param string $token |
1172 | - * @return Share |
|
1172 | + * @return IShare |
|
1173 | 1173 | * |
1174 | 1174 | * @throws ShareNotFound |
1175 | 1175 | */ |
@@ -1231,6 +1231,9 @@ discard block |
||
1231 | 1231 | return $share; |
1232 | 1232 | } |
1233 | 1233 | |
1234 | + /** |
|
1235 | + * @param IShare $share |
|
1236 | + */ |
|
1234 | 1237 | protected function checkExpireDate($share) { |
1235 | 1238 | if ($share->getExpirationDate() !== null && |
1236 | 1239 | $share->getExpirationDate() <= new \DateTime()) { |