Completed
Pull Request — master (#9018)
by Björn
21:08 queued 01:12
created
lib/private/User/Session.php 1 patch
Doc Comments   +4 added lines, -1 removed lines patch added patch discarded remove patch
@@ -339,7 +339,7 @@  discard block
 block discarded – undo
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
 block discarded – undo
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',
Please login to merge, or discard this patch.
apps/files_external/lib/Lib/Storage/AmazonS3.php 1 patch
Doc Comments   +10 added lines, -1 removed lines patch added patch discarded remove patch
@@ -85,6 +85,9 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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');
Please login to merge, or discard this patch.
lib/private/Comments/Manager.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -213,7 +213,7 @@
 block discarded – undo
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;
Please login to merge, or discard this patch.
lib/private/Share20/Manager.php 1 patch
Doc Comments   +5 added lines, -2 removed lines patch added patch discarded remove patch
@@ -575,7 +575,7 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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()) {
Please login to merge, or discard this patch.
apps/files/lib/Activity/Provider.php 1 patch
Doc Comments   +6 added lines patch added patch discarded remove patch
@@ -112,6 +112,9 @@  discard block
 block discarded – undo
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
 block discarded – undo
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) {
Please login to merge, or discard this patch.
apps/files_external/lib/Lib/Storage/Swift.php 1 patch
Doc Comments   +4 added lines, -1 removed lines patch added patch discarded remove patch
@@ -83,7 +83,7 @@  discard block
 block discarded – undo
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
 block discarded – undo
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);
Please login to merge, or discard this patch.
lib/private/legacy/app.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -269,7 +269,7 @@
 block discarded – undo
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 {
Please login to merge, or discard this patch.
lib/private/AppFramework/Utility/ControllerMethodReflector.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -40,7 +40,7 @@
 block discarded – undo
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){
Please login to merge, or discard this patch.
lib/private/Session/Internal.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -166,7 +166,7 @@
 block discarded – undo
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 {
Please login to merge, or discard this patch.