@@ -92,7 +92,7 @@ discard block |
||
92 | 92 | $prefix = '[Example Event] (ExampleMembershipsRemoved) '; |
93 | 93 | |
94 | 94 | $memberships = array_map( |
95 | - function (Membership $membership) { |
|
95 | + function(Membership $membership) { |
|
96 | 96 | $inheritance = ($membership->getInheritanceDepth() > 1) ? |
97 | 97 | 'an inherited member' : 'a direct member'; |
98 | 98 | try { |
@@ -100,15 +100,15 @@ discard block |
||
100 | 100 | } catch (Exception $e) { |
101 | 101 | $this->e($e); |
102 | 102 | |
103 | - return $membership->getSingleId() . ' is not ' . $inheritance . ' of ' |
|
104 | - . $membership->getCircleId() . ' anymore'; |
|
103 | + return $membership->getSingleId().' is not '.$inheritance.' of ' |
|
104 | + . $membership->getCircleId().' anymore'; |
|
105 | 105 | } |
106 | 106 | |
107 | - return $federatedUser->getUserId() . ' (' . Member::$TYPE[$federatedUser->getUserType()] |
|
108 | - . ') is not ' . $inheritance . ' of ' . $membership->getCircleId() . ' anymore'; |
|
107 | + return $federatedUser->getUserId().' ('.Member::$TYPE[$federatedUser->getUserType()] |
|
108 | + . ') is not '.$inheritance.' of '.$membership->getCircleId().' anymore'; |
|
109 | 109 | }, $event->getMemberships() |
110 | 110 | ); |
111 | 111 | |
112 | - $this->log(3, $prefix . implode('. ', $memberships)); |
|
112 | + $this->log(3, $prefix.implode('. ', $memberships)); |
|
113 | 113 | } |
114 | 114 | } |
@@ -92,7 +92,7 @@ discard block |
||
92 | 92 | $prefix = '[Example Event] (ExampleMembershipsCreated) '; |
93 | 93 | |
94 | 94 | $memberships = array_map( |
95 | - function (Membership $membership) { |
|
95 | + function(Membership $membership) { |
|
96 | 96 | $inheritance = ($membership->getInheritanceDepth() > 1) ? |
97 | 97 | 'an inherited member' : 'a direct member'; |
98 | 98 | try { |
@@ -100,15 +100,15 @@ discard block |
||
100 | 100 | } catch (Exception $e) { |
101 | 101 | $this->e($e); |
102 | 102 | |
103 | - return $membership->getSingleId() . ' is now ' . $inheritance . ' of ' |
|
103 | + return $membership->getSingleId().' is now '.$inheritance.' of ' |
|
104 | 104 | . $membership->getCircleId(); |
105 | 105 | } |
106 | 106 | |
107 | - return $federatedUser->getUserId() . ' (' . Member::$TYPE[$federatedUser->getUserType()] |
|
108 | - . ') is now ' . $inheritance . ' of ' . $membership->getCircleId(); |
|
107 | + return $federatedUser->getUserId().' ('.Member::$TYPE[$federatedUser->getUserType()] |
|
108 | + . ') is now '.$inheritance.' of '.$membership->getCircleId(); |
|
109 | 109 | }, $event->getMemberships() |
110 | 110 | ); |
111 | 111 | |
112 | - $this->log(3, $prefix . implode('. ', $memberships)); |
|
112 | + $this->log(3, $prefix.implode('. ', $memberships)); |
|
113 | 113 | } |
114 | 114 | } |
@@ -152,7 +152,7 @@ discard block |
||
152 | 152 | return $mountPoint; |
153 | 153 | } |
154 | 154 | |
155 | - return '/' . $userId . '/files/' . ltrim($mountPoint, '/'); |
|
155 | + return '/'.$userId.'/files/'.ltrim($mountPoint, '/'); |
|
156 | 156 | } |
157 | 157 | |
158 | 158 | /** |
@@ -291,7 +291,7 @@ discard block |
||
291 | 291 | public function toMount(string $userId, string $protocol = 'https'): array { |
292 | 292 | return [ |
293 | 293 | 'owner' => $this->getOwner(), |
294 | - 'remote' => $protocol . '://' . $this->getInstance(), |
|
294 | + 'remote' => $protocol.'://'.$this->getInstance(), |
|
295 | 295 | 'token' => $this->getToken(), |
296 | 296 | 'share_token' => $this->getToken(), |
297 | 297 | 'password' => $this->getPassword(), |
@@ -409,12 +409,12 @@ discard block |
||
409 | 409 | throw new UnknownInterfaceException('misconfigured scheme'); |
410 | 410 | } |
411 | 411 | |
412 | - $base = $scheme . '://' . $this->getCloudInstance($interface) . $path; |
|
412 | + $base = $scheme.'://'.$this->getCloudInstance($interface).$path; |
|
413 | 413 | if ($route === '') { |
414 | 414 | return $base; |
415 | 415 | } |
416 | 416 | |
417 | - return $base . $this->configService->linkToRoute($route, $args); |
|
417 | + return $base.$this->configService->linkToRoute($route, $args); |
|
418 | 418 | } |
419 | 419 | |
420 | 420 | |
@@ -447,7 +447,7 @@ discard block |
||
447 | 447 | return $this->configService->getLoopbackPath($route, $args); |
448 | 448 | } |
449 | 449 | |
450 | - return rtrim($base, '/') . $this->configService->linkToRoute($route, $args); |
|
450 | + return rtrim($base, '/').$this->configService->linkToRoute($route, $args); |
|
451 | 451 | } |
452 | 452 | |
453 | 453 |
@@ -239,7 +239,7 @@ discard block |
||
239 | 239 | $done = []; |
240 | 240 | foreach ($circles as $circle) { |
241 | 241 | $this->outputService->output( |
242 | - 'Caching memberships for Members of \'' . $circle->getDisplayName() . '\'', |
|
242 | + 'Caching memberships for Members of \''.$circle->getDisplayName().'\'', |
|
243 | 243 | true |
244 | 244 | ); |
245 | 245 | |
@@ -287,7 +287,7 @@ discard block |
||
287 | 287 | try { |
288 | 288 | $data = new SimpleDataStore($row); |
289 | 289 | $this->outputService->output( |
290 | - 'Migrating Circle \'' . $data->g('name') . '\' (' . $data->g('unique_id') . ')', |
|
290 | + 'Migrating Circle \''.$data->g('name').'\' ('.$data->g('unique_id').')', |
|
291 | 291 | true |
292 | 292 | ); |
293 | 293 | |
@@ -391,7 +391,7 @@ discard block |
||
391 | 391 | try { |
392 | 392 | $data = new SimpleDataStore($row); |
393 | 393 | $this->outputService->output( |
394 | - 'Migrating Member \'' . $data->g('user_id') . '\' from \'' . $data->g('circle_id') |
|
394 | + 'Migrating Member \''.$data->g('user_id').'\' from \''.$data->g('circle_id') |
|
395 | 395 | . '\'', |
396 | 396 | true |
397 | 397 | ); |
@@ -506,8 +506,8 @@ discard block |
||
506 | 506 | $federatedUser = |
507 | 507 | $this->federatedUserService->getLocalFederatedUser($data->g('share_with')); |
508 | 508 | $this->outputService->output( |
509 | - 'Migrating child share #' . $data->gInt('id') . ' owner: ' . $data->g('share_with') |
|
510 | - . ' -> ' . $federatedUser->getSingleId(), |
|
509 | + 'Migrating child share #'.$data->gInt('id').' owner: '.$data->g('share_with') |
|
510 | + . ' -> '.$federatedUser->getSingleId(), |
|
511 | 511 | true |
512 | 512 | ); |
513 | 513 | |
@@ -578,7 +578,7 @@ discard block |
||
578 | 578 | $addressBook = |
579 | 579 | $this->contactService->getAddressBoxById($cm, $this->get('addressbook-key', $entry)); |
580 | 580 | |
581 | - $member->setUserId($userId . '/' . $addressBook->getUri() . '/' . $contactId); |
|
581 | + $member->setUserId($userId.'/'.$addressBook->getUri().'/'.$contactId); |
|
582 | 582 | } |
583 | 583 | } |
584 | 584 | |
@@ -613,7 +613,7 @@ discard block |
||
613 | 613 | try { |
614 | 614 | $data = new SimpleDataStore($row); |
615 | 615 | $this->outputService->output( |
616 | - 'Migrating ShareToken \'' . $data->g('token') . '\' for \'' . $data->g('user_id') |
|
616 | + 'Migrating ShareToken \''.$data->g('token').'\' for \''.$data->g('user_id') |
|
617 | 617 | . '\'', |
618 | 618 | true |
619 | 619 | ); |
@@ -250,7 +250,7 @@ discard block |
||
250 | 250 | */ |
251 | 251 | private function getChildrenMembers(string $id, array &$knownIds = []): array { |
252 | 252 | $singleIds = array_map( |
253 | - function (Member $item): string { |
|
253 | + function(Member $item): string { |
|
254 | 254 | return $item->getSingleId(); |
255 | 255 | }, $this->memberRequest->getMembers($id) |
256 | 256 | ); |
@@ -274,7 +274,7 @@ discard block |
||
274 | 274 | */ |
275 | 275 | private function getChildrenMemberships(string $id, array &$knownIds = []): array { |
276 | 276 | $singleIds = array_map( |
277 | - function (Membership $item): string { |
|
277 | + function(Membership $item): string { |
|
278 | 278 | return $item->getSingleId(); |
279 | 279 | }, $this->membershipRequest->getInherited($id) |
280 | 280 | ); |
@@ -320,7 +320,7 @@ discard block |
||
320 | 320 | */ |
321 | 321 | private function removeDeprecatedMemberships(array $memberships, array $known): array { |
322 | 322 | $circleIds = array_map( |
323 | - function (Membership $membership): string { |
|
323 | + function(Membership $membership): string { |
|
324 | 324 | return $membership->getCircleId(); |
325 | 325 | }, $memberships |
326 | 326 | ); |
@@ -117,7 +117,7 @@ discard block |
||
117 | 117 | '<error>WARNING! You are about to delete all data related to the Circles App!</error>' |
118 | 118 | ); |
119 | 119 | $question = new ConfirmationQuestion( |
120 | - '<comment>Do you really want to ' . $action . ' Circles ?</comment> (y/N) ', false, |
|
120 | + '<comment>Do you really want to '.$action.' Circles ?</comment> (y/N) ', false, |
|
121 | 121 | '/^(y|Y)/i' |
122 | 122 | ); |
123 | 123 | |
@@ -132,7 +132,7 @@ discard block |
||
132 | 132 | $output->writeln('<error>WARNING! This operation is not reversible.</error>'); |
133 | 133 | |
134 | 134 | $question = new Question( |
135 | - '<comment>Please confirm this destructive operation by typing \'' . $action |
|
135 | + '<comment>Please confirm this destructive operation by typing \''.$action |
|
136 | 136 | . '\'</comment>: ', '' |
137 | 137 | ); |
138 | 138 | |
@@ -149,7 +149,7 @@ discard block |
||
149 | 149 | $this->coreRequestBuilder->uninstall(); |
150 | 150 | } |
151 | 151 | |
152 | - $output->writeln('<info>' . $action . ' done</info>'); |
|
152 | + $output->writeln('<info>'.$action.' done</info>'); |
|
153 | 153 | |
154 | 154 | return 0; |
155 | 155 | } |
@@ -5,7 +5,7 @@ |
||
5 | 5 | require_once __DIR__.'/../../../lib/base.php'; |
6 | 6 | require_once __DIR__.'/../vendor/autoload.php'; |
7 | 7 | |
8 | -\OC::$composerAutoloader->addPsr4('Tests\\', OC::$SERVERROOT . '/tests/unit/', true); |
|
8 | +\OC::$composerAutoloader->addPsr4('Tests\\', OC::$SERVERROOT.'/tests/unit/', true); |
|
9 | 9 | |
10 | 10 | \OC_App::loadApp('circles'); |
11 | 11 |
@@ -125,7 +125,7 @@ |
||
125 | 125 | } catch (FederatedItemException $e) { |
126 | 126 | if ($input->getOption('status-code')) { |
127 | 127 | throw new FederatedItemException( |
128 | - ' [' . get_class($e) . ', ' . $e->getStatus() . ']' . "\n" . $e->getMessage() |
|
128 | + ' ['.get_class($e).', '.$e->getStatus().']'."\n".$e->getMessage() |
|
129 | 129 | ); |
130 | 130 | } |
131 | 131 |