Completed
Pull Request — master (#327)
by Maxence
01:35
created
lib/Service/MembersService.php 2 patches
Doc Comments   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -119,7 +119,7 @@  discard block
 block discarded – undo
119 119
 	 * @param $ident
120 120
 	 * @param int $type
121 121
 	 *
122
-	 * @return array
122
+	 * @return Member[]
123 123
 	 * @throws Exception
124 124
 	 */
125 125
 	public function addMember($circleUniqueId, $ident, $type) {
@@ -278,8 +278,8 @@  discard block
 block discarded – undo
278 278
 	/**
279 279
 	 * Verify if a local account is valid.
280 280
 	 *
281
-	 * @param $ident
282
-	 * @param $type
281
+	 * @param string $ident
282
+	 * @param integer $type
283 283
 	 *
284 284
 	 * @throws NoUserException
285 285
 	 */
@@ -300,7 +300,7 @@  discard block
 block discarded – undo
300 300
 	 * Verify if a mail have a valid format.
301 301
 	 *
302 302
 	 * @param $ident
303
-	 * @param $type
303
+	 * @param integer $type
304 304
 	 *
305 305
 	 * @throws EmailAccountInvalidFormatException
306 306
 	 */
@@ -328,7 +328,7 @@  discard block
 block discarded – undo
328 328
 	 * Verify if a contact exist in current user address books.
329 329
 	 *
330 330
 	 * @param $ident
331
-	 * @param $type
331
+	 * @param integer $type
332 332
 	 *
333 333
 	 * @throws NoUserException
334 334
 	 * @throws EmailAccountInvalidFormatException
@@ -417,8 +417,8 @@  discard block
 block discarded – undo
417 417
 	 * Will return any data of a user related to a circle (as a Member). User can be a 'non-member'
418 418
 	 * Viewer needs to be at least Member of the Circle
419 419
 	 *
420
-	 * @param $circleId
421
-	 * @param $userId
420
+	 * @param string $circleId
421
+	 * @param string $userId
422 422
 	 * @param $type
423 423
 	 * @param bool $forceAll
424 424
 	 *
@@ -447,7 +447,7 @@  discard block
 block discarded – undo
447 447
 	 * @param int $type
448 448
 	 * @param int $level
449 449
 	 *
450
-	 * @return array
450
+	 * @return Member[]
451 451
 	 * @throws Exception
452 452
 	 */
453 453
 	public function levelMember($circleUniqueId, $name, $type, $level) {
@@ -478,7 +478,7 @@  discard block
 block discarded – undo
478 478
 	/**
479 479
 	 * @param Circle $circle
480 480
 	 * @param Member $member
481
-	 * @param $level
481
+	 * @param integer $level
482 482
 	 *
483 483
 	 * @throws Exception
484 484
 	 */
@@ -553,9 +553,9 @@  discard block
 block discarded – undo
553 553
 	/**
554 554
 	 * @param string $circleUniqueId
555 555
 	 * @param string $name
556
-	 * @param $type
556
+	 * @param integer $type
557 557
 	 *
558
-	 * @return array
558
+	 * @return Member[]
559 559
 	 * @throws Exception
560 560
 	 */
561 561
 	public function removeMember($circleUniqueId, $name, $type) {
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -344,7 +344,7 @@
 block discarded – undo
344 344
 			);
345 345
 		}
346 346
 
347
-		$tmpContact = $this->userId . ':' . $ident;
347
+		$tmpContact = $this->userId.':'.$ident;
348 348
 		$result = MiscService::getContactData($tmpContact);
349 349
 		if (empty($result)) {
350 350
 			throw new NoUserException($this->l10n->t("This contact is not available"));
Please login to merge, or discard this patch.