@@ -320,7 +320,7 @@ |
||
320 | 320 | |
321 | 321 | /** |
322 | 322 | * @param IJob $job |
323 | - * @param $timeTaken |
|
323 | + * @param integer $timeTaken |
|
324 | 324 | */ |
325 | 325 | public function setExecutionTime(IJob $job, $timeTaken) { |
326 | 326 | $query = $this->connection->getQueryBuilder(); |
@@ -431,7 +431,7 @@ |
||
431 | 431 | } |
432 | 432 | |
433 | 433 | /** |
434 | - * @param array $disabledApps |
|
434 | + * @param string[] $disabledApps |
|
435 | 435 | * @throws \Exception |
436 | 436 | */ |
437 | 437 | private function upgradeAppStoreApps(array $disabledApps) { |
@@ -48,6 +48,9 @@ discard block |
||
48 | 48 | $this->stack = $stack; |
49 | 49 | } |
50 | 50 | |
51 | + /** |
|
52 | + * @param double $time |
|
53 | + */ |
|
51 | 54 | public function end($time) { |
52 | 55 | $this->end = $time; |
53 | 56 | } |
@@ -67,7 +70,7 @@ discard block |
||
67 | 70 | } |
68 | 71 | |
69 | 72 | /** |
70 | - * @return float |
|
73 | + * @return integer |
|
71 | 74 | */ |
72 | 75 | public function getStart() { |
73 | 76 | return $this->start; |
@@ -448,7 +448,7 @@ |
||
448 | 448 | * |
449 | 449 | * @param string $path source path |
450 | 450 | * @param string $destination destination path |
451 | - * @return bool|void false to stop handling, void to skip this handler |
|
451 | + * @return null|false false to stop handling, void to skip this handler |
|
452 | 452 | */ |
453 | 453 | public function beforeMoveFutureFile($path, $destination) { |
454 | 454 | $sourceNode = $this->tree->getNodeForPath($path); |
@@ -71,7 +71,7 @@ discard block |
||
71 | 71 | /** |
72 | 72 | * get color for on-page elements: |
73 | 73 | * theme color by default, grey if theme color is to bright |
74 | - * @param $color |
|
74 | + * @param string $color |
|
75 | 75 | * @return string |
76 | 76 | */ |
77 | 77 | public function elementColor($color) { |
@@ -114,7 +114,7 @@ discard block |
||
114 | 114 | |
115 | 115 | /** |
116 | 116 | * @param $app string app name |
117 | - * @return string|ISimpleFile path to app icon / file of logo |
|
117 | + * @return string path to app icon / file of logo |
|
118 | 118 | */ |
119 | 119 | public function getAppIcon($app) { |
120 | 120 | $app = str_replace(array('\0', '/', '\\', '..'), '', $app); |
@@ -190,8 +190,8 @@ discard block |
||
190 | 190 | /** |
191 | 191 | * replace default color with a custom one |
192 | 192 | * |
193 | - * @param $svg string content of a svg file |
|
194 | - * @param $color string color to match |
|
193 | + * @param string $svg string content of a svg file |
|
194 | + * @param string $color string color to match |
|
195 | 195 | * @return string |
196 | 196 | */ |
197 | 197 | public function colorizeSvg($svg, $color) { |
@@ -265,6 +265,9 @@ discard block |
||
265 | 265 | return $this->mapper->has($table, $className); |
266 | 266 | } |
267 | 267 | |
268 | + /** |
|
269 | + * @param string $type |
|
270 | + */ |
|
268 | 271 | private function setupSettingsEntry($settingsClassName, $type) { |
269 | 272 | if (!class_exists($settingsClassName)) { |
270 | 273 | $this->log->debug('Could not find ' . $type . ' section class ' . $settingsClassName); |
@@ -294,6 +297,9 @@ discard block |
||
294 | 297 | } |
295 | 298 | } |
296 | 299 | |
300 | + /** |
|
301 | + * @param string $type |
|
302 | + */ |
|
297 | 303 | private function getSectionTableForType($type) { |
298 | 304 | if($type === 'admin') { |
299 | 305 | return Mapper::TABLE_ADMIN_SECTIONS; |
@@ -491,7 +491,7 @@ discard block |
||
491 | 491 | |
492 | 492 | /** |
493 | 493 | * returns the internal Nextcloud name for the given LDAP DN of the user, false on DN outside of search DN or failure |
494 | - * @param string $dn the dn of the user object |
|
494 | + * @param string $fdn the dn of the user object |
|
495 | 495 | * @param string $ldapName optional, the display name of the object |
496 | 496 | * @return string|false with with the name to use in Nextcloud |
497 | 497 | */ |
@@ -508,7 +508,7 @@ discard block |
||
508 | 508 | |
509 | 509 | /** |
510 | 510 | * returns an internal Nextcloud name for the given LDAP DN, false on DN outside of search DN |
511 | - * @param string $dn the dn of the user object |
|
511 | + * @param string $fdn the dn of the user object |
|
512 | 512 | * @param string $ldapName optional, the display name of the object |
513 | 513 | * @param bool $isUser optional, whether it is a user object (otherwise group assumed) |
514 | 514 | * @return string|false with with the name to use in Nextcloud |
@@ -771,7 +771,7 @@ discard block |
||
771 | 771 | * the login filter. |
772 | 772 | * |
773 | 773 | * @param string $loginName |
774 | - * @param array $attributes optional, list of attributes to read |
|
774 | + * @param string[] $attributes optional, list of attributes to read |
|
775 | 775 | * @return array |
776 | 776 | */ |
777 | 777 | public function fetchUsersByLoginName($loginName, $attributes = array('dn')) { |
@@ -844,7 +844,7 @@ discard block |
||
844 | 844 | |
845 | 845 | /** |
846 | 846 | * @param string $filter |
847 | - * @param string|string[] $attr |
|
847 | + * @param string[] $attr |
|
848 | 848 | * @param int $limit |
849 | 849 | * @param int $offset |
850 | 850 | * @return array |
@@ -892,7 +892,7 @@ discard block |
||
892 | 892 | |
893 | 893 | /** |
894 | 894 | * @param string $filter |
895 | - * @param string|string[] $attr |
|
895 | + * @param string[] $attr |
|
896 | 896 | * @param int $limit |
897 | 897 | * @param int $offset |
898 | 898 | * @return false|int |
@@ -989,6 +989,7 @@ discard block |
||
989 | 989 | * retrieved. Results will according to the order in the array. |
990 | 990 | * @param int $limit optional, maximum results to be counted |
991 | 991 | * @param int $offset optional, a starting point |
992 | + * @param string $filter |
|
992 | 993 | * @return array|false array with the search result as first value and pagedSearchOK as |
993 | 994 | * second | false if not successful |
994 | 995 | * @throws \OC\ServerNotAvailableException |
@@ -1240,7 +1241,7 @@ discard block |
||
1240 | 1241 | |
1241 | 1242 | /** |
1242 | 1243 | * @param string $name |
1243 | - * @return bool|mixed|string |
|
1244 | + * @return string |
|
1244 | 1245 | */ |
1245 | 1246 | public function sanitizeUsername($name) { |
1246 | 1247 | if($this->connection->ldapIgnoreNamingRules) { |
@@ -1264,6 +1265,7 @@ discard block |
||
1264 | 1265 | * escapes (user provided) parts for LDAP filter |
1265 | 1266 | * @param string $input, the provided value |
1266 | 1267 | * @param bool $allowAsterisk whether in * at the beginning should be preserved |
1268 | + * @param string $input |
|
1267 | 1269 | * @return string the escaped string |
1268 | 1270 | */ |
1269 | 1271 | public function escapeFilterPart($input, $allowAsterisk = false) { |
@@ -86,6 +86,9 @@ discard block |
||
86 | 86 | return $path === '.'; |
87 | 87 | } |
88 | 88 | |
89 | + /** |
|
90 | + * @param string $path |
|
91 | + */ |
|
89 | 92 | private function cleanKey($path) { |
90 | 93 | if ($this->isRoot($path)) { |
91 | 94 | return '/'; |
@@ -109,7 +112,7 @@ discard block |
||
109 | 112 | } |
110 | 113 | |
111 | 114 | /** |
112 | - * @param $key |
|
115 | + * @param string $key |
|
113 | 116 | * @return Result|boolean |
114 | 117 | */ |
115 | 118 | private function headObject($key) { |
@@ -236,6 +239,9 @@ discard block |
||
236 | 239 | return false; |
237 | 240 | } |
238 | 241 | |
242 | + /** |
|
243 | + * @param string $path |
|
244 | + */ |
|
239 | 245 | private function batchDelete($path = null) { |
240 | 246 | $params = array( |
241 | 247 | 'Bucket' => $this->bucket |
@@ -581,6 +587,9 @@ discard block |
||
581 | 587 | return $this->id; |
582 | 588 | } |
583 | 589 | |
590 | + /** |
|
591 | + * @param string $path |
|
592 | + */ |
|
584 | 593 | public function writeBack($tmpFile, $path) { |
585 | 594 | try { |
586 | 595 | $source = fopen($tmpFile, 'r'); |
@@ -164,7 +164,7 @@ |
||
164 | 164 | * @param array $parameters |
165 | 165 | * @param bool $silence whether to suppress warnings |
166 | 166 | * @throws \ErrorException via trapError |
167 | - * @return mixed |
|
167 | + * @return string |
|
168 | 168 | */ |
169 | 169 | private function invoke($functionName, array $parameters = [], $silence = false) { |
170 | 170 | try { |