@@ -339,7 +339,7 @@ discard block |
||
339 | 339 | * @param IUser $user |
340 | 340 | * @param array $loginDetails |
341 | 341 | * @param bool $regenerateSessionId |
342 | - * @return true returns true if login successful or an exception otherwise |
|
342 | + * @return boolean returns true if login successful or an exception otherwise |
|
343 | 343 | * @throws LoginException |
344 | 344 | */ |
345 | 345 | public function completeLogin(IUser $user, array $loginDetails, $regenerateSessionId = true) { |
@@ -446,6 +446,9 @@ discard block |
||
446 | 446 | return $this->config->getSystemValue('token_auth_enforced', false); |
447 | 447 | } |
448 | 448 | |
449 | + /** |
|
450 | + * @param string $username |
|
451 | + */ |
|
449 | 452 | protected function isTwoFactorEnforced($username) { |
450 | 453 | Util::emitHook( |
451 | 454 | '\OCA\Files_Sharing\API\Server2Server', |
@@ -85,6 +85,9 @@ discard block |
||
85 | 85 | return $path === '.'; |
86 | 86 | } |
87 | 87 | |
88 | + /** |
|
89 | + * @param string $path |
|
90 | + */ |
|
88 | 91 | private function cleanKey($path) { |
89 | 92 | if ($this->isRoot($path)) { |
90 | 93 | return '/'; |
@@ -108,7 +111,7 @@ discard block |
||
108 | 111 | } |
109 | 112 | |
110 | 113 | /** |
111 | - * @param $key |
|
114 | + * @param string $key |
|
112 | 115 | * @return Result|boolean |
113 | 116 | */ |
114 | 117 | private function headObject($key) { |
@@ -234,6 +237,9 @@ discard block |
||
234 | 237 | } |
235 | 238 | } |
236 | 239 | |
240 | + /** |
|
241 | + * @param string $path |
|
242 | + */ |
|
237 | 243 | private function batchDelete($path = null) { |
238 | 244 | $params = array( |
239 | 245 | 'Bucket' => $this->bucket |
@@ -579,6 +585,9 @@ discard block |
||
579 | 585 | return $this->id; |
580 | 586 | } |
581 | 587 | |
588 | + /** |
|
589 | + * @param string $path |
|
590 | + */ |
|
582 | 591 | public function writeBack($tmpFile, $path) { |
583 | 592 | try { |
584 | 593 | $source = fopen($tmpFile, 'r'); |
@@ -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 { |