@@ -149,6 +149,9 @@ |
||
149 | 149 | } |
150 | 150 | |
151 | 151 | |
152 | + /** |
|
153 | + * @param string $display |
|
154 | + */ |
|
152 | 155 | public function setDisplayName($display) { |
153 | 156 | $this->displayName = $display; |
154 | 157 |
@@ -102,9 +102,9 @@ discard block |
||
102 | 102 | /** |
103 | 103 | * @param string $circleUniqueId |
104 | 104 | * @param $ident |
105 | - * @param $type |
|
105 | + * @param integer $type |
|
106 | 106 | * |
107 | - * @return array |
|
107 | + * @return Member[] |
|
108 | 108 | * @throws \Exception |
109 | 109 | */ |
110 | 110 | public function addMember($circleUniqueId, $ident, $type) { |
@@ -167,6 +167,10 @@ discard block |
||
167 | 167 | } |
168 | 168 | } |
169 | 169 | |
170 | + /** |
|
171 | + * @param string $ident |
|
172 | + * @param integer $type |
|
173 | + */ |
|
170 | 174 | private function verifyIdentLocalMember(&$ident, $type) { |
171 | 175 | if ($type !== Member::TYPE_USER) { |
172 | 176 | return; |
@@ -180,6 +184,9 @@ discard block |
||
180 | 184 | } |
181 | 185 | |
182 | 186 | |
187 | + /** |
|
188 | + * @param integer $type |
|
189 | + */ |
|
183 | 190 | private function verifyIdentContact(&$ident, $type) { |
184 | 191 | if ($type !== Member::TYPE_CONTACT) { |
185 | 192 | return; |
@@ -320,7 +327,7 @@ discard block |
||
320 | 327 | * @param int $type |
321 | 328 | * @param int $level |
322 | 329 | * |
323 | - * @return array |
|
330 | + * @return Member[] |
|
324 | 331 | * @throws \Exception |
325 | 332 | */ |
326 | 333 | public function levelMember($circleUniqueId, $name, $type, $level) { |
@@ -359,7 +366,7 @@ discard block |
||
359 | 366 | /** |
360 | 367 | * @param Circle $circle |
361 | 368 | * @param Member $member |
362 | - * @param $level |
|
369 | + * @param integer $level |
|
363 | 370 | * |
364 | 371 | * @throws \Exception |
365 | 372 | */ |
@@ -410,9 +417,9 @@ discard block |
||
410 | 417 | /** |
411 | 418 | * @param string $circleUniqueId |
412 | 419 | * @param string $name |
413 | - * @param $type |
|
420 | + * @param integer $type |
|
414 | 421 | * |
415 | - * @return array |
|
422 | + * @return Member[] |
|
416 | 423 | * @throws \Exception |
417 | 424 | */ |
418 | 425 | public function removeMember($circleUniqueId, $name, $type) { |
@@ -145,7 +145,7 @@ discard block |
||
145 | 145 | } |
146 | 146 | |
147 | 147 | |
148 | - private function addMemberBasedOnItsType(Circle $circle, Member &$member) { |
|
148 | + private function addMemberBasedOnItsType(Circle $circle, Member & $member) { |
|
149 | 149 | $this->addLocalMember($circle, $member); |
150 | 150 | $this->addEmailAddress($member); |
151 | 151 | $this->addContact($member); |
@@ -191,7 +191,7 @@ discard block |
||
191 | 191 | // throw new NoUserException($this->l10n->t("This user does not exist")); |
192 | 192 | // } |
193 | 193 | |
194 | - $ident = $this->userId . ':' . $ident; |
|
194 | + $ident = $this->userId.':'.$ident; |
|
195 | 195 | } |
196 | 196 | |
197 | 197 | |
@@ -363,7 +363,7 @@ discard block |
||
363 | 363 | * |
364 | 364 | * @throws \Exception |
365 | 365 | */ |
366 | - private function editMemberLevel(Circle $circle, Member &$member, $level) { |
|
366 | + private function editMemberLevel(Circle $circle, Member & $member, $level) { |
|
367 | 367 | try { |
368 | 368 | $isMod = $circle->getHigherViewer(); |
369 | 369 | $isMod->hasToBeModerator(); |
@@ -387,7 +387,7 @@ discard block |
||
387 | 387 | * |
388 | 388 | * @throws \Exception |
389 | 389 | */ |
390 | - private function switchOwner(Circle $circle, Member &$member) { |
|
390 | + private function switchOwner(Circle $circle, Member & $member) { |
|
391 | 391 | try { |
392 | 392 | $isMod = $circle->getHigherViewer(); |
393 | 393 | $isMod->hasToBeOwner(); |
@@ -134,7 +134,7 @@ discard block |
||
134 | 134 | /** |
135 | 135 | * @param Member $member |
136 | 136 | */ |
137 | - private function parseMember(Member &$member) { |
|
137 | + private function parseMember(Member & $member) { |
|
138 | 138 | $this->parseMemberFromContact($member); |
139 | 139 | } |
140 | 140 | |
@@ -145,7 +145,7 @@ discard block |
||
145 | 145 | * |
146 | 146 | * @param Member $member |
147 | 147 | */ |
148 | - private function parseMemberFromContact(Member &$member) { |
|
148 | + private function parseMemberFromContact(Member & $member) { |
|
149 | 149 | |
150 | 150 | if ($member->getType() !== Member::TYPE_CONTACT) { |
151 | 151 | return; |
@@ -88,7 +88,7 @@ |
||
88 | 88 | $searcher = \OC::$server->query((string)$container); |
89 | 89 | |
90 | 90 | if (!($searcher instanceof ISearch)) { |
91 | - $this->miscService->log('Search ' . $container . ' is not compatible exception'); |
|
91 | + $this->miscService->log('Search '.$container.' is not compatible exception'); |
|
92 | 92 | continue; |
93 | 93 | } |
94 | 94 |
@@ -379,7 +379,7 @@ discard block |
||
379 | 379 | */ |
380 | 380 | public static function generateLink($circleUniqueId) { |
381 | 381 | return \OC::$server->getURLGenerator() |
382 | - ->linkToRoute('circles.Navigation.navigate') . '#' . $circleUniqueId; |
|
382 | + ->linkToRoute('circles.Navigation.navigate').'#'.$circleUniqueId; |
|
383 | 383 | } |
384 | 384 | |
385 | 385 | |
@@ -394,8 +394,8 @@ discard block |
||
394 | 394 | */ |
395 | 395 | public static function generateRemoteLink(FederatedLink $link) { |
396 | 396 | return \OC::$server->getURLGenerator() |
397 | - ->linkToRoute('circles.Navigation.navigate') . '#' . $link->getUniqueId() |
|
398 | - . '-' . $link->getToken(); |
|
397 | + ->linkToRoute('circles.Navigation.navigate').'#'.$link->getUniqueId() |
|
398 | + . '-'.$link->getToken(); |
|
399 | 399 | } |
400 | 400 | |
401 | 401 | |
@@ -407,7 +407,7 @@ discard block |
||
407 | 407 | public static function generateUserParameter(SharingFrame $frame) { |
408 | 408 | |
409 | 409 | if ($frame->getCloudId() !== null) { |
410 | - $name = $frame->getAuthor() . '@' . $frame->getCloudId(); |
|
410 | + $name = $frame->getAuthor().'@'.$frame->getCloudId(); |
|
411 | 411 | } else { |
412 | 412 | $name = MiscService::getDisplay($frame->getAuthor(), Member::TYPE_USER); |
413 | 413 | } |