@@ -104,7 +104,7 @@ discard block |
||
104 | 104 | try { |
105 | 105 | $level = Member::parseLevelString($levelString); |
106 | 106 | } catch (ParseMemberLevelException $e) { |
107 | - throw new MemberHelperException('method ' . $levelString . ' not found'); |
|
107 | + throw new MemberHelperException('method '.$levelString.' not found'); |
|
108 | 108 | } |
109 | 109 | |
110 | 110 | $this->mustHaveLevelEqualOrAbove($level); |
@@ -122,11 +122,11 @@ discard block |
||
122 | 122 | try { |
123 | 123 | $level = Member::parseLevelString($levelString); |
124 | 124 | } catch (ParseMemberLevelException $e) { |
125 | - throw new MemberHelperException('method ' . $levelString . ' not found'); |
|
125 | + throw new MemberHelperException('method '.$levelString.' not found'); |
|
126 | 126 | } |
127 | 127 | |
128 | 128 | if ($this->member->getLevel() >= $level) { |
129 | - throw new MemberLevelException('Member cannot be ' . $levelString); |
|
129 | + throw new MemberLevelException('Member cannot be '.$levelString); |
|
130 | 130 | } |
131 | 131 | } |
132 | 132 |
@@ -494,7 +494,6 @@ |
||
494 | 494 | |
495 | 495 | /** |
496 | 496 | * Get shares for a given path |
497 | - |
|
498 | 497 | * @param Node $path |
499 | 498 | * |
500 | 499 | * @return IShare[]|null |
@@ -331,19 +331,19 @@ discard block |
||
331 | 331 | */ |
332 | 332 | private function createShare($share) { |
333 | 333 | $this->miscService->log( |
334 | - 'Creating share (1/4) - type: ' . $share->getShareType() . ' - token: ' |
|
335 | - . $share->getToken() . ' - type: ' . $share->getShareType() . ' - with: ' |
|
336 | - . $share->getSharedWith() . ' - permissions: ' . $share->getPermissions(), 0 |
|
334 | + 'Creating share (1/4) - type: '.$share->getShareType().' - token: ' |
|
335 | + . $share->getToken().' - type: '.$share->getShareType().' - with: ' |
|
336 | + . $share->getSharedWith().' - permissions: '.$share->getPermissions(), 0 |
|
337 | 337 | ); |
338 | 338 | |
339 | 339 | $qb = $this->getBaseInsertSql($share); |
340 | - $this->miscService->log('Share creation (2/4) : ' . json_encode($qb->getSQL()), 0); |
|
340 | + $this->miscService->log('Share creation (2/4) : '.json_encode($qb->getSQL()), 0); |
|
341 | 341 | |
342 | 342 | $result = $qb->execute(); |
343 | - $this->miscService->log('Share creation result (3/4) : ' . json_encode($result), 0); |
|
343 | + $this->miscService->log('Share creation result (3/4) : '.json_encode($result), 0); |
|
344 | 344 | |
345 | 345 | $id = $qb->getLastInsertId(); |
346 | - $this->miscService->log('Share created ID (4/4) : ' . $id, 0); |
|
346 | + $this->miscService->log('Share created ID (4/4) : '.$id, 0); |
|
347 | 347 | |
348 | 348 | try { |
349 | 349 | $share->setId($id); |
@@ -590,7 +590,7 @@ discard block |
||
590 | 590 | \OC::$server->getLogger()->log(3, 'deprecated>getShareByToken'); |
591 | 591 | $qb = $this->dbConnection->getQueryBuilder(); |
592 | 592 | |
593 | - $this->miscService->log("Opening share by token '#" . $token . "'", 0); |
|
593 | + $this->miscService->log("Opening share by token '#".$token."'", 0); |
|
594 | 594 | |
595 | 595 | $cursor = $qb->select('*') |
596 | 596 | ->from('share') |
@@ -614,7 +614,7 @@ discard block |
||
614 | 614 | try { |
615 | 615 | $data = $this->getShareByPersonalToken($token); |
616 | 616 | } catch (Exception $e) { |
617 | - $this->miscService->log("Share '#" . $token . "' not found.", 0); |
|
617 | + $this->miscService->log("Share '#".$token."' not found.", 0); |
|
618 | 618 | throw new ShareNotFound('Share not found', $this->l10n->t('Could not find share')); |
619 | 619 | } |
620 | 620 | } |
@@ -623,7 +623,7 @@ discard block |
||
623 | 623 | $share = $this->createShareObject($data); |
624 | 624 | } catch (InvalidShare $e) { |
625 | 625 | $this->miscService->log( |
626 | - "Share Object '#" . $token . "' not created. " . json_encode($data), 0 |
|
626 | + "Share Object '#".$token."' not created. ".json_encode($data), 0 |
|
627 | 627 | ); |
628 | 628 | throw new ShareNotFound('Share not found', $this->l10n->t('Could not find share')); |
629 | 629 | } |
@@ -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 |
@@ -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 |
@@ -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 |
@@ -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 |
@@ -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 |
@@ -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 |
@@ -159,7 +159,7 @@ discard block |
||
159 | 159 | } catch (Exception $e) { |
160 | 160 | if ($this->pOn) { |
161 | 161 | $message = ($e->getMessage() !== '') ? $e->getMessage() : get_class($e); |
162 | - $this->output->writeln('<error>' . $message . '</error>'); |
|
162 | + $this->output->writeln('<error>'.$message.'</error>'); |
|
163 | 163 | } else { |
164 | 164 | throw $e; |
165 | 165 | } |
@@ -243,7 +243,7 @@ discard block |
||
243 | 243 | */ |
244 | 244 | private function loadConfiguration() { |
245 | 245 | $this->p('Loading configuration'); |
246 | - $configuration = file_get_contents(__DIR__ . '/../../testConfiguration.json'); |
|
246 | + $configuration = file_get_contents(__DIR__.'/../../testConfiguration.json'); |
|
247 | 247 | $this->config = json_decode($configuration, true); |
248 | 248 | $this->r(true, 'testConfiguration.json'); |
249 | 249 | |
@@ -276,22 +276,22 @@ discard block |
||
276 | 276 | $this->r(); |
277 | 277 | |
278 | 278 | foreach ($this->getInstances() as $instance) { |
279 | - $this->p('Creating users on ' . $instance); |
|
279 | + $this->p('Creating users on '.$instance); |
|
280 | 280 | foreach ($this->getConfigArray($instance, 'users') as $userId) { |
281 | 281 | $this->pm($userId); |
282 | 282 | $this->occ( |
283 | - $instance, 'user:add --password-from-env ' . $userId, false, false, |
|
283 | + $instance, 'user:add --password-from-env '.$userId, false, false, |
|
284 | 284 | ['OC_PASS' => 'testtest'] |
285 | 285 | ); |
286 | 286 | } |
287 | 287 | $this->r(); |
288 | 288 | |
289 | 289 | foreach ($this->getConfigArray($instance, 'groups') as $groupId => $users) { |
290 | - $this->p('Creating group <info>' . $groupId . '</info> on <info>' . $instance . '</info>'); |
|
291 | - $this->occ($instance, 'group:add ' . $groupId, false, false); |
|
290 | + $this->p('Creating group <info>'.$groupId.'</info> on <info>'.$instance.'</info>'); |
|
291 | + $this->occ($instance, 'group:add '.$groupId, false, false); |
|
292 | 292 | foreach ($users as $userId) { |
293 | 293 | $this->pm($userId); |
294 | - $this->occ($instance, 'group:adduser ' . $groupId . ' ' . $userId, true, false); |
|
294 | + $this->occ($instance, 'group:adduser '.$groupId.' '.$userId, true, false); |
|
295 | 295 | } |
296 | 296 | $this->r(); |
297 | 297 | } |
@@ -326,7 +326,7 @@ discard block |
||
326 | 326 | foreach ($this->getInstances(true) as $instance) { |
327 | 327 | $this->pm($instance); |
328 | 328 | foreach ($this->getConfigArray($instance, 'config') as $k => $v) { |
329 | - $this->occ($instance, 'config:app:set --value ' . $v . ' circles ' . $k, true, false); |
|
329 | + $this->occ($instance, 'config:app:set --value '.$v.' circles '.$k, true, false); |
|
330 | 330 | } |
331 | 331 | } |
332 | 332 | $this->r(); |
@@ -339,7 +339,7 @@ discard block |
||
339 | 339 | */ |
340 | 340 | private function confirmVersion() { |
341 | 341 | $version = $this->configService->getAppValue('installed_version'); |
342 | - $this->p('Confirming version <info>' . $version . '</info>'); |
|
342 | + $this->p('Confirming version <info>'.$version.'</info>'); |
|
343 | 343 | foreach ($this->getInstances(false) as $instance) { |
344 | 344 | $this->pm($instance); |
345 | 345 | $capabilities = $this->occ($instance, 'circles:check --capabilities'); |
@@ -390,12 +390,12 @@ discard block |
||
390 | 390 | */ |
391 | 391 | private function statusFreshInstances() { |
392 | 392 | foreach ($this->getInstances() as $instanceId) { |
393 | - $this->p('Circles on ' . $instanceId); |
|
393 | + $this->p('Circles on '.$instanceId); |
|
394 | 394 | $result = $this->occ($instanceId, 'circles:manage:list --all'); |
395 | 395 | $expectedSize = sizeof($this->getConfigArray($instanceId, 'groups')) |
396 | 396 | + sizeof($this->getConfigArray($instanceId, 'users')) |
397 | 397 | + 1; |
398 | - $this->r((sizeof($result) === $expectedSize), sizeof($result) . ' circles'); |
|
398 | + $this->r((sizeof($result) === $expectedSize), sizeof($result).' circles'); |
|
399 | 399 | |
400 | 400 | $membersList = $groupsList = []; |
401 | 401 | foreach ($result as $item) { |
@@ -414,12 +414,12 @@ discard block |
||
414 | 414 | $instance = $this->getConfig($instanceId, 'config.frontal_cloud_id'); |
415 | 415 | |
416 | 416 | foreach ($this->getConfigArray($instanceId, 'users') as $userId) { |
417 | - $this->p('Checking Single Circle for <comment>' . $userId . '@' . $instance . '</comment>'); |
|
417 | + $this->p('Checking Single Circle for <comment>'.$userId.'@'.$instance.'</comment>'); |
|
418 | 418 | $circle = $this->getSingleCircleForMember($membersList, $userId, $instance); |
419 | 419 | |
420 | 420 | $compareToOwnerBasedOn = new Circle(); |
421 | 421 | $compareToOwnerBasedOn->setConfig(Circle::CFG_SINGLE) |
422 | - ->setName('user:' . $userId . ':{CIRCLEID}') |
|
422 | + ->setName('user:'.$userId.':{CIRCLEID}') |
|
423 | 423 | ->setDisplayName($userId); |
424 | 424 | |
425 | 425 | $compareToOwner = new Member(); |
@@ -437,7 +437,7 @@ discard block |
||
437 | 437 | $compareTo = new Circle(); |
438 | 438 | $compareTo->setOwner($compareToOwner) |
439 | 439 | ->setConfig(Circle::CFG_SINGLE) |
440 | - ->setName('user:' . $userId . ':{CIRCLEID}') |
|
440 | + ->setName('user:'.$userId.':{CIRCLEID}') |
|
441 | 441 | ->setDisplayName($userId); |
442 | 442 | |
443 | 443 | $this->confirmCircleData($circle, $compareTo); |
@@ -474,8 +474,8 @@ discard block |
||
474 | 474 | $this->r(true, $circle->getSingleId()); |
475 | 475 | |
476 | 476 | foreach ($this->getConfigArray($instanceId, 'groups') as $groupId => $members) { |
477 | - $this->p('Checking Circle for <comment>' . $groupId . '@' . $instance . '</comment>'); |
|
478 | - $circle = $this->getCircleFromList($groupsList, 'group:' . $groupId); |
|
477 | + $this->p('Checking Circle for <comment>'.$groupId.'@'.$instance.'</comment>'); |
|
478 | + $circle = $this->getCircleFromList($groupsList, 'group:'.$groupId); |
|
479 | 479 | |
480 | 480 | $appCircle = $this->getSingleCircleForMember($membersList, 'circles', $instance); |
481 | 481 | $appOwner = $appCircle->getOwner(); |
@@ -499,7 +499,7 @@ discard block |
||
499 | 499 | $compareTo = new Circle(); |
500 | 500 | $compareTo->setOwner($compareToOwner) |
501 | 501 | ->setConfig(Circle::CFG_SYSTEM | Circle::CFG_NO_OWNER | Circle::CFG_HIDDEN) |
502 | - ->setName('group:' . $groupId) |
|
502 | + ->setName('group:'.$groupId) |
|
503 | 503 | ->setDisplayName($groupId); |
504 | 504 | |
505 | 505 | $this->confirmCircleData($circle, $compareTo); |
@@ -516,12 +516,12 @@ discard block |
||
516 | 516 | */ |
517 | 517 | private function createRemoteLink() { |
518 | 518 | foreach ($this->getInstances() as $instanceId) { |
519 | - $this->p('Init remote link from ' . $instanceId); |
|
519 | + $this->p('Init remote link from '.$instanceId); |
|
520 | 520 | $links = $this->getConfigArray($instanceId, 'remote'); |
521 | 521 | foreach ($links as $link => $type) { |
522 | 522 | $remote = $this->getConfig($link, 'config.frontal_cloud_id'); |
523 | - $this->pm($remote . '(' . $type . ')'); |
|
524 | - $this->occ($instanceId, 'circles:remote ' . $remote . ' --type ' . $type . ' --yes'); |
|
523 | + $this->pm($remote.'('.$type.')'); |
|
524 | + $this->occ($instanceId, 'circles:remote '.$remote.' --type '.$type.' --yes'); |
|
525 | 525 | } |
526 | 526 | $this->r(); |
527 | 527 | } |
@@ -535,7 +535,7 @@ discard block |
||
535 | 535 | private function buildingLocalDatabase() { |
536 | 536 | $this->circles = $this->federatedUsers = []; |
537 | 537 | foreach ($this->getInstances() as $instanceId) { |
538 | - $this->p('Retrieving Circles from ' . $instanceId); |
|
538 | + $this->p('Retrieving Circles from '.$instanceId); |
|
539 | 539 | $circles = $this->occ($instanceId, 'circles:manage:list --all'); |
540 | 540 | foreach ($circles as $item) { |
541 | 541 | /** @var Circle $circle */ |
@@ -570,11 +570,11 @@ discard block |
||
570 | 570 | $name = self::$TEST_CIRCLES[0]; |
571 | 571 | $owner = $this->getInstanceUsers($localInstanceId)[1]; |
572 | 572 | $dataCreatedCircle001 = |
573 | - $this->occ($localInstanceId, 'circles:manage:create --type user ' . $owner . ' ' . $name); |
|
573 | + $this->occ($localInstanceId, 'circles:manage:create --type user '.$owner.' '.$name); |
|
574 | 574 | /** @var Circle $createdCircle */ |
575 | 575 | $createdCircle = $this->deserialize($dataCreatedCircle001, Circle::class); |
576 | 576 | $this->circles[$localInstanceId][$createdCircle->getName()] = $createdCircle; |
577 | - $this->r(true, $createdCircle->getSingleId());; |
|
577 | + $this->r(true, $createdCircle->getSingleId()); ; |
|
578 | 578 | |
579 | 579 | $this->p('Comparing data returned at creation'); |
580 | 580 | if ($createdCircle->getSingleId() === '' || $createdCircle->getOwner()->getId() === '') { |
@@ -601,7 +601,7 @@ discard block |
||
601 | 601 | |
602 | 602 | |
603 | 603 | $this->p('Comparing local stored data'); |
604 | - $dataCircle = $this->occ($localInstanceId, 'circle:manage:details ' . $createdCircle->getSingleId()); |
|
604 | + $dataCircle = $this->occ($localInstanceId, 'circle:manage:details '.$createdCircle->getSingleId()); |
|
605 | 605 | |
606 | 606 | /** @var Circle $tmpCircle */ |
607 | 607 | $tmpCircle = $this->deserialize($dataCircle, Circle::class); |
@@ -610,9 +610,9 @@ discard block |
||
610 | 610 | |
611 | 611 | $links = $this->getConfigArray('global-scale-1', 'remote'); |
612 | 612 | foreach ($this->getInstances(false) as $instanceId) { |
613 | - $this->p('Comparing data stored on ' . $instanceId); |
|
613 | + $this->p('Comparing data stored on '.$instanceId); |
|
614 | 614 | $dataCircle = |
615 | - $this->occ($instanceId, 'circle:manage:details ' . $createdCircle->getSingleId(), false); |
|
615 | + $this->occ($instanceId, 'circle:manage:details '.$createdCircle->getSingleId(), false); |
|
616 | 616 | |
617 | 617 | if ($instanceId === $localInstanceId || $links[$instanceId] === 'GlobalScale') { |
618 | 618 | /** @var Circle $tmpCircle */ |
@@ -646,7 +646,7 @@ discard block |
||
646 | 646 | $member = $this->getInstanceUsers($instanceId)[2]; |
647 | 647 | |
648 | 648 | $addedMember = $this->processMemberAdd($instanceId, $circleName, $member, 'user'); |
649 | - $this->r(true, $addedMember->getId());; |
|
649 | + $this->r(true, $addedMember->getId()); ; |
|
650 | 650 | |
651 | 651 | // check test2 |
652 | 652 | } |
@@ -664,15 +664,15 @@ discard block |
||
664 | 664 | $name = self::$TEST_CIRCLES[0]; |
665 | 665 | $circle = $this->getCircleByName($localInstanceId, $name); |
666 | 666 | $userId = $this->getInstanceUsers($localInstanceId)[6]; |
667 | - $userCircle = $this->getCircleByName($localInstanceId, 'user:' . $userId); |
|
667 | + $userCircle = $this->getCircleByName($localInstanceId, 'user:'.$userId); |
|
668 | 668 | $user = $userCircle->getOwner(); |
669 | 669 | $dataAddedMember = |
670 | 670 | $this->occ( |
671 | - $localInstanceId, 'circles:members:add ' . $circle->getSingleId() . ' ' . $user->getSingleId() |
|
671 | + $localInstanceId, 'circles:members:add '.$circle->getSingleId().' '.$user->getSingleId() |
|
672 | 672 | ); |
673 | 673 | /** @var Member $addedMember */ |
674 | 674 | $addedMember = $this->deserialize($dataAddedMember, Member::class); |
675 | - $this->r(true, $addedMember->getId());; |
|
675 | + $this->r(true, $addedMember->getId()); ; |
|
676 | 676 | |
677 | 677 | // check test6 |
678 | 678 | } |
@@ -687,15 +687,15 @@ discard block |
||
687 | 687 | |
688 | 688 | $circle = $this->getCircleByName($localInstanceId, $circleName); |
689 | 689 | $userId = $this->getInstanceUsers($localInstanceId)[6]; |
690 | - $userCircle = $this->getCircleByName($localInstanceId, 'user:' . $userId); |
|
690 | + $userCircle = $this->getCircleByName($localInstanceId, 'user:'.$userId); |
|
691 | 691 | $user = $userCircle->getOwner(); |
692 | 692 | $dataAddedMember = |
693 | 693 | $this->occ( |
694 | - $localInstanceId, 'circles:members:add ' . $circle->getSingleId() . ' ' . $user->getSingleId() |
|
694 | + $localInstanceId, 'circles:members:add '.$circle->getSingleId().' '.$user->getSingleId() |
|
695 | 695 | ); |
696 | 696 | /** @var Member $addedMember */ |
697 | 697 | $addedMember = $this->deserialize($dataAddedMember, Member::class); |
698 | - $this->r(true, $addedMember->getId());; |
|
698 | + $this->r(true, $addedMember->getId()); ; |
|
699 | 699 | } |
700 | 700 | |
701 | 701 | private function levelLocalMemberToModerator() { |
@@ -785,13 +785,13 @@ discard block |
||
785 | 785 | ]; |
786 | 786 | } |
787 | 787 | |
788 | - $this->compare($compareTo->getSingleId(), $circle->getSingleId(), $prefix . '.id', $params); |
|
789 | - $this->compare($compareTo->getName(), $circle->getName(), $prefix . '.name', $params); |
|
788 | + $this->compare($compareTo->getSingleId(), $circle->getSingleId(), $prefix.'.id', $params); |
|
789 | + $this->compare($compareTo->getName(), $circle->getName(), $prefix.'.name', $params); |
|
790 | 790 | $this->compare( |
791 | - $compareTo->getDisplayName(), $circle->getDisplayName(), $prefix . '.displayName', $params |
|
791 | + $compareTo->getDisplayName(), $circle->getDisplayName(), $prefix.'.displayName', $params |
|
792 | 792 | ); |
793 | - $this->compareInt($compareTo->getConfig(), $circle->getConfig(), $prefix . '.config', true); |
|
794 | - $this->compareInt($compareTo->getSource(), $circle->getSource(), $prefix . '.source'); |
|
793 | + $this->compareInt($compareTo->getConfig(), $circle->getConfig(), $prefix.'.config', true); |
|
794 | + $this->compareInt($compareTo->getSource(), $circle->getSource(), $prefix.'.source'); |
|
795 | 795 | |
796 | 796 | if ($compareTo->hasOwner()) { |
797 | 797 | $compareToOwner = $compareTo->getOwner(); |
@@ -801,7 +801,7 @@ discard block |
||
801 | 801 | throw new Exception('empty owner'); |
802 | 802 | } |
803 | 803 | if ($owner->getCircleId() !== $circle->getSingleId()) { |
804 | - throw new Exception($prefix . '.owner.circleId is different than ' . $prefix . '.id'); |
|
804 | + throw new Exception($prefix.'.owner.circleId is different than '.$prefix.'.id'); |
|
805 | 805 | } |
806 | 806 | $this->confirmMemberData($owner, $compareToOwner, 'owner', false, $params); |
807 | 807 | } |
@@ -814,7 +814,7 @@ discard block |
||
814 | 814 | } |
815 | 815 | $initiator = $circle->getInitiator(); |
816 | 816 | if ($initiator->getCircleId() !== $circle->getSingleId()) { |
817 | - throw new Exception($prefix . '.initiator.circleId is different than ' . $prefix . '.id'); |
|
817 | + throw new Exception($prefix.'.initiator.circleId is different than '.$prefix.'.id'); |
|
818 | 818 | } |
819 | 819 | $this->confirmMemberData($initiator, $compareToInitiator, 'owner', false, $params); |
820 | 820 | } |
@@ -841,27 +841,27 @@ discard block |
||
841 | 841 | bool $versa = false, |
842 | 842 | array $params = [] |
843 | 843 | ) { |
844 | - $this->compare($compareTo->getId(), $member->getId(), $prefix . '.id', $params); |
|
845 | - $this->compare($compareTo->getCircleId(), $member->getCircleId(), $prefix . '.circleId', $params); |
|
846 | - $this->compare($compareTo->getSingleId(), $member->getSingleId(), $prefix . '.singleId', $params); |
|
847 | - $this->compare($compareTo->getUserId(), $member->getUserId(), $prefix . '.userId', $params); |
|
844 | + $this->compare($compareTo->getId(), $member->getId(), $prefix.'.id', $params); |
|
845 | + $this->compare($compareTo->getCircleId(), $member->getCircleId(), $prefix.'.circleId', $params); |
|
846 | + $this->compare($compareTo->getSingleId(), $member->getSingleId(), $prefix.'.singleId', $params); |
|
847 | + $this->compare($compareTo->getUserId(), $member->getUserId(), $prefix.'.userId', $params); |
|
848 | 848 | $this->compare( |
849 | - $compareTo->getDisplayName(), $member->getDisplayName(), $prefix . '.displayName', $params |
|
849 | + $compareTo->getDisplayName(), $member->getDisplayName(), $prefix.'.displayName', $params |
|
850 | 850 | ); |
851 | - $this->compareInt($compareTo->getUserType(), $member->getUserType(), $prefix . '.userType'); |
|
852 | - $this->compare($compareTo->getInstance(), $member->getInstance(), $prefix . '.instance', $params); |
|
853 | - $this->compareInt($compareTo->getLevel(), $member->getLevel(), $prefix . '.level', true); |
|
854 | - $this->compare($compareTo->getStatus(), $member->getStatus(), $prefix . '.status', $params); |
|
851 | + $this->compareInt($compareTo->getUserType(), $member->getUserType(), $prefix.'.userType'); |
|
852 | + $this->compare($compareTo->getInstance(), $member->getInstance(), $prefix.'.instance', $params); |
|
853 | + $this->compareInt($compareTo->getLevel(), $member->getLevel(), $prefix.'.level', true); |
|
854 | + $this->compare($compareTo->getStatus(), $member->getStatus(), $prefix.'.status', $params); |
|
855 | 855 | |
856 | 856 | if ($compareTo->hasBasedOn()) { |
857 | 857 | if (!$member->hasBasedOn()) { |
858 | - throw new Exception('empty ' . $prefix . '.basedOn'); |
|
858 | + throw new Exception('empty '.$prefix.'.basedOn'); |
|
859 | 859 | } |
860 | 860 | $basedOn = $member->getBasedOn(); |
861 | 861 | $this->confirmCircleData( |
862 | 862 | $basedOn, |
863 | 863 | $compareTo->getBasedOn(), |
864 | - $prefix . '.basedOn', |
|
864 | + $prefix.'.basedOn', |
|
865 | 865 | false, |
866 | 866 | $params |
867 | 867 | ); |
@@ -880,7 +880,7 @@ discard block |
||
880 | 880 | private function compare(string $expected, string $compare, string $def, array $params) { |
881 | 881 | if ($expected !== '' |
882 | 882 | && $this->feedStringWithParams($expected, $params) !== $compare) { |
883 | - throw new Exception($def . ': ' . $compare . ' (' . $expected . ')'); |
|
883 | + throw new Exception($def.': '.$compare.' ('.$expected.')'); |
|
884 | 884 | } |
885 | 885 | } |
886 | 886 | |
@@ -896,7 +896,7 @@ discard block |
||
896 | 896 | private function compareInt(int $expected, int $compare, string $def, bool $force = false) { |
897 | 897 | if (($expected > 0 || ($force && $expected >= 0)) |
898 | 898 | && $expected !== $compare) { |
899 | - throw new Exception('wrong ' . $def . ': ' . $compare . ' (' . $expected . ')'); |
|
899 | + throw new Exception('wrong '.$def.': '.$compare.' ('.$expected.')'); |
|
900 | 900 | } |
901 | 901 | } |
902 | 902 | |
@@ -932,7 +932,7 @@ discard block |
||
932 | 932 | } |
933 | 933 | } |
934 | 934 | |
935 | - throw new CircleNotFoundException('cannot find ' . $userId . ' in the list of Single Circle'); |
|
935 | + throw new CircleNotFoundException('cannot find '.$userId.' in the list of Single Circle'); |
|
936 | 936 | } |
937 | 937 | |
938 | 938 | |
@@ -950,7 +950,7 @@ discard block |
||
950 | 950 | } |
951 | 951 | |
952 | 952 | throw new CircleNotFoundException( |
953 | - 'cannot extract \'' . $name . '\' from the list of generated Circles' |
|
953 | + 'cannot extract \''.$name.'\' from the list of generated Circles' |
|
954 | 954 | ); |
955 | 955 | } |
956 | 956 | |
@@ -970,7 +970,7 @@ discard block |
||
970 | 970 | } |
971 | 971 | |
972 | 972 | throw new CircleNotFoundException( |
973 | - 'cannot extract \'' . $name . '\' from the list of provided Circles' |
|
973 | + 'cannot extract \''.$name.'\' from the list of provided Circles' |
|
974 | 974 | ); |
975 | 975 | } |
976 | 976 | |
@@ -1015,7 +1015,7 @@ discard block |
||
1015 | 1015 | } |
1016 | 1016 | } |
1017 | 1017 | |
1018 | - throw new ItemNotFoundException($instance . ' not found'); |
|
1018 | + throw new ItemNotFoundException($instance.' not found'); |
|
1019 | 1019 | } |
1020 | 1020 | |
1021 | 1021 | |
@@ -1050,7 +1050,7 @@ discard block |
||
1050 | 1050 | ): ?array { |
1051 | 1051 | $configInstance = $this->getConfigInstance($instance); |
1052 | 1052 | $path = $this->get('path', $configInstance); |
1053 | - $occ = rtrim($path, '/') . '/occ'; |
|
1053 | + $occ = rtrim($path, '/').'/occ'; |
|
1054 | 1054 | |
1055 | 1055 | $command = array_merge([$occ], explode(' ', $cmd)); |
1056 | 1056 | if ($jsonAsOutput) { |
@@ -1060,7 +1060,7 @@ discard block |
||
1060 | 1060 | $process->run(null, $env); |
1061 | 1061 | |
1062 | 1062 | if ($exceptionOnFail && !$process->isSuccessful()) { |
1063 | - throw new Exception(implode(' ', $command) . ' failed'); |
|
1063 | + throw new Exception(implode(' ', $command).' failed'); |
|
1064 | 1064 | } |
1065 | 1065 | |
1066 | 1066 | $output = json_decode($process->getOutput(), true); |
@@ -1083,7 +1083,7 @@ discard block |
||
1083 | 1083 | */ |
1084 | 1084 | private function t(string $title): void { |
1085 | 1085 | $this->output->writeln(''); |
1086 | - $this->output->writeln('<comment>### ' . $title . '</comment>'); |
|
1086 | + $this->output->writeln('<comment>### '.$title.'</comment>'); |
|
1087 | 1087 | $this->output->writeln(''); |
1088 | 1088 | } |
1089 | 1089 | |
@@ -1092,14 +1092,14 @@ discard block |
||
1092 | 1092 | */ |
1093 | 1093 | private function p(string $processing): void { |
1094 | 1094 | $this->pOn = true; |
1095 | - $this->output->write('- ' . $processing . ': '); |
|
1095 | + $this->output->write('- '.$processing.': '); |
|
1096 | 1096 | } |
1097 | 1097 | |
1098 | 1098 | /** |
1099 | 1099 | * @param string $more |
1100 | 1100 | */ |
1101 | 1101 | private function pm(string $more): void { |
1102 | - $this->output->write($more . ' '); |
|
1102 | + $this->output->write($more.' '); |
|
1103 | 1103 | } |
1104 | 1104 | |
1105 | 1105 | /** |
@@ -1109,9 +1109,9 @@ discard block |
||
1109 | 1109 | private function r(bool $result = true, string $info = ''): void { |
1110 | 1110 | $this->pOn = false; |
1111 | 1111 | if ($result) { |
1112 | - $this->output->writeln('<info>' . (($info !== '') ? $info : 'done') . '</info>'); |
|
1112 | + $this->output->writeln('<info>'.(($info !== '') ? $info : 'done').'</info>'); |
|
1113 | 1113 | } else { |
1114 | - $this->output->writeln('<error>' . (($info !== '') ? $info : 'done') . '</error>'); |
|
1114 | + $this->output->writeln('<error>'.(($info !== '') ? $info : 'done').'</error>'); |
|
1115 | 1115 | } |
1116 | 1116 | } |
1117 | 1117 | |
@@ -1133,16 +1133,16 @@ discard block |
||
1133 | 1133 | $dataAddedMember = |
1134 | 1134 | $this->occ( |
1135 | 1135 | $instanceId, |
1136 | - 'circles:members:add ' . $circle->getSingleId() . ' ' . $userId . ' --type ' . $type |
|
1136 | + 'circles:members:add '.$circle->getSingleId().' '.$userId.' --type '.$type |
|
1137 | 1137 | ); |
1138 | 1138 | /** @var Member $addedMember */ |
1139 | 1139 | $addedMember = $this->deserialize($dataAddedMember, Member::class); |
1140 | 1140 | |
1141 | 1141 | |
1142 | - echo 'ADDEDMEMBER: ' . json_encode($addedMember, JSON_PRETTY_PRINT) . "\n"; |
|
1142 | + echo 'ADDEDMEMBER: '.json_encode($addedMember, JSON_PRETTY_PRINT)."\n"; |
|
1143 | 1143 | |
1144 | 1144 | $federatedUser = $this->federatedUsers[$instanceId][$userId]; |
1145 | - echo 'FEDERATEDUER: ' . json_encode($federatedUser, JSON_PRETTY_PRINT) . "\n"; |
|
1145 | + echo 'FEDERATEDUER: '.json_encode($federatedUser, JSON_PRETTY_PRINT)."\n"; |
|
1146 | 1146 | |
1147 | 1147 | return $addedMember; |
1148 | 1148 | } |