@@ -213,7 +213,7 @@ |
||
213 | 213 | /** |
214 | 214 | * removes an entry from the comments run time cache |
215 | 215 | * |
216 | - * @param mixed $id the comment's id |
|
216 | + * @param string $id the comment's id |
|
217 | 217 | */ |
218 | 218 | protected function uncache($id) { |
219 | 219 | $id = (string)$id; |
@@ -575,7 +575,7 @@ discard block |
||
575 | 575 | * Share a path |
576 | 576 | * |
577 | 577 | * @param \OCP\Share\IShare $share |
578 | - * @return Share The share object |
|
578 | + * @return Share\IShare The share object |
|
579 | 579 | * @throws \Exception |
580 | 580 | * |
581 | 581 | * TODO: handle link share permissions or check them |
@@ -1161,7 +1161,7 @@ discard block |
||
1161 | 1161 | * Get the share by token possible with password |
1162 | 1162 | * |
1163 | 1163 | * @param string $token |
1164 | - * @return Share |
|
1164 | + * @return Share\IShare |
|
1165 | 1165 | * |
1166 | 1166 | * @throws ShareNotFound |
1167 | 1167 | */ |
@@ -1223,6 +1223,9 @@ discard block |
||
1223 | 1223 | return $share; |
1224 | 1224 | } |
1225 | 1225 | |
1226 | + /** |
|
1227 | + * @param Share\IShare $share |
|
1228 | + */ |
|
1226 | 1229 | protected function checkExpireDate($share) { |
1227 | 1230 | if ($share->getExpirationDate() !== null && |
1228 | 1231 | $share->getExpirationDate() <= new \DateTime()) { |
@@ -112,6 +112,9 @@ discard block |
||
112 | 112 | return $this->parseLongVersion($event, $previousEvent); |
113 | 113 | } |
114 | 114 | |
115 | + /** |
|
116 | + * @param string $icon |
|
117 | + */ |
|
115 | 118 | protected function setIcon(IEvent $event, $icon) { |
116 | 119 | if ($this->activityManager->getRequirePNG()) { |
117 | 120 | $event->setIcon($this->url->getAbsoluteURL($this->url->imagePath('files', $icon . '.png'))); |
@@ -252,6 +255,9 @@ discard block |
||
252 | 255 | return $event; |
253 | 256 | } |
254 | 257 | |
258 | + /** |
|
259 | + * @param string $subject |
|
260 | + */ |
|
255 | 261 | protected function setSubjects(IEvent $event, $subject, array $parameters) { |
256 | 262 | $placeholders = $replacements = []; |
257 | 263 | foreach ($parameters as $placeholder => $parameter) { |
@@ -83,7 +83,7 @@ discard block |
||
83 | 83 | |
84 | 84 | /** |
85 | 85 | * @param string $path |
86 | - * @return mixed|string |
|
86 | + * @return string |
|
87 | 87 | */ |
88 | 88 | private function normalizePath(string $path) { |
89 | 89 | $path = trim($path, '/'); |
@@ -570,6 +570,9 @@ discard block |
||
570 | 570 | return $this->container; |
571 | 571 | } |
572 | 572 | |
573 | + /** |
|
574 | + * @param string $path |
|
575 | + */ |
|
573 | 576 | public function writeBack($tmpFile, $path) { |
574 | 577 | $fileData = fopen($tmpFile, 'r'); |
575 | 578 | $this->objectStore->writeObject($path, $fileData); |
@@ -269,7 +269,7 @@ |
||
269 | 269 | * check if an app is of a specific type |
270 | 270 | * |
271 | 271 | * @param string $app |
272 | - * @param array $types |
|
272 | + * @param string[] $types |
|
273 | 273 | * @return bool |
274 | 274 | */ |
275 | 275 | public static function isType(string $app, array $types): bool { |
@@ -40,7 +40,7 @@ |
||
40 | 40 | private $parameters = []; |
41 | 41 | |
42 | 42 | /** |
43 | - * @param object $object an object or classname |
|
43 | + * @param \OCP\AppFramework\Controller $object an object or classname |
|
44 | 44 | * @param string $method the method which we want to inspect |
45 | 45 | */ |
46 | 46 | public function reflect($object, string $method){ |
@@ -166,7 +166,7 @@ |
||
166 | 166 | * @param array $parameters |
167 | 167 | * @param bool $silence whether to suppress warnings |
168 | 168 | * @throws \ErrorException via trapError |
169 | - * @return mixed |
|
169 | + * @return string |
|
170 | 170 | */ |
171 | 171 | private function invoke(string $functionName, array $parameters = [], bool $silence = false) { |
172 | 172 | try { |
@@ -44,6 +44,12 @@ discard block |
||
44 | 44 | |
45 | 45 | class Color { |
46 | 46 | public $r, $g, $b; |
47 | + |
|
48 | + /** |
|
49 | + * @param integer $r |
|
50 | + * @param integer $g |
|
51 | + * @param integer $b |
|
52 | + */ |
|
47 | 53 | public function __construct($r, $g, $b) { |
48 | 54 | $this->r = $r; |
49 | 55 | $this->g = $g; |
@@ -304,6 +310,7 @@ discard block |
||
304 | 310 | * Calculate steps between two Colors |
305 | 311 | * @param object Color $steps start color |
306 | 312 | * @param object Color $ends end color |
313 | + * @param integer $steps |
|
307 | 314 | * @return array [r,g,b] steps for each color to go from $steps to $ends |
308 | 315 | */ |
309 | 316 | private function stepCalc($steps, $ends) { |
@@ -315,8 +322,9 @@ discard block |
||
315 | 322 | } |
316 | 323 | /** |
317 | 324 | * Convert a string to an integer evenly |
318 | - * @param string $hash the text to parse |
|
319 | - * @param int $maximum the maximum range |
|
325 | + * @param integer $steps |
|
326 | + * @param Color $color1 |
|
327 | + * @param Color $color2 |
|
320 | 328 | * @return int between 0 and $maximum |
321 | 329 | */ |
322 | 330 | private function mixPalette($steps, $color1, $color2) { |
@@ -506,7 +506,7 @@ discard block |
||
506 | 506 | |
507 | 507 | /** |
508 | 508 | * returns the internal Nextcloud name for the given LDAP DN of the user, false on DN outside of search DN or failure |
509 | - * @param string $dn the dn of the user object |
|
509 | + * @param string $fdn the dn of the user object |
|
510 | 510 | * @param string $ldapName optional, the display name of the object |
511 | 511 | * @return string|false with with the name to use in Nextcloud |
512 | 512 | */ |
@@ -805,7 +805,7 @@ discard block |
||
805 | 805 | * the login filter. |
806 | 806 | * |
807 | 807 | * @param string $loginName |
808 | - * @param array $attributes optional, list of attributes to read |
|
808 | + * @param string[] $attributes optional, list of attributes to read |
|
809 | 809 | * @return array |
810 | 810 | */ |
811 | 811 | public function fetchUsersByLoginName($loginName, $attributes = array('dn')) { |
@@ -889,7 +889,7 @@ discard block |
||
889 | 889 | |
890 | 890 | /** |
891 | 891 | * @param string $filter |
892 | - * @param string|string[] $attr |
|
892 | + * @param string[] $attr |
|
893 | 893 | * @param int $limit |
894 | 894 | * @param int $offset |
895 | 895 | * @return array |
@@ -937,7 +937,7 @@ discard block |
||
937 | 937 | |
938 | 938 | /** |
939 | 939 | * @param string $filter |
940 | - * @param string|string[] $attr |
|
940 | + * @param string[] $attr |
|
941 | 941 | * @param int $limit |
942 | 942 | * @param int $offset |
943 | 943 | * @return false|int |
@@ -1036,7 +1036,7 @@ discard block |
||
1036 | 1036 | /** |
1037 | 1037 | * retrieved. Results will according to the order in the array. |
1038 | 1038 | * |
1039 | - * @param $filter |
|
1039 | + * @param string $filter |
|
1040 | 1040 | * @param $base |
1041 | 1041 | * @param string[]|string|null $attr |
1042 | 1042 | * @param int $limit optional, maximum results to be counted |
@@ -1331,6 +1331,7 @@ discard block |
||
1331 | 1331 | * escapes (user provided) parts for LDAP filter |
1332 | 1332 | * @param string $input, the provided value |
1333 | 1333 | * @param bool $allowAsterisk whether in * at the beginning should be preserved |
1334 | + * @param string $input |
|
1334 | 1335 | * @return string the escaped string |
1335 | 1336 | */ |
1336 | 1337 | public function escapeFilterPart($input, $allowAsterisk = false) { |