Completed
Pull Request — master (#1421)
by Maxence
39s
created
lib/Service/SyncService.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -246,7 +246,7 @@  discard block
 block discarded – undo
246 246
 	 * @throws SingleCircleNotFoundException
247 247
 	 */
248 248
 	public function syncNextcloudUser(string $userId): FederatedUser {
249
-		$this->outputService->output('Syncing Nextcloud Account \'' . $userId . '\'', true);
249
+		$this->outputService->output('Syncing Nextcloud Account \''.$userId.'\'', true);
250 250
 
251 251
 		return $this->federatedUserService->getLocalFederatedUser($userId, false, true);
252 252
 	}
@@ -290,10 +290,10 @@  discard block
 block discarded – undo
290 290
 	 * @throws RequestBuilderException
291 291
 	 */
292 292
 	public function syncNextcloudGroup(string $groupId): Circle {
293
-		$this->outputService->output('Syncing Nextcloud Group \'' . $groupId . '\'', true);
293
+		$this->outputService->output('Syncing Nextcloud Group \''.$groupId.'\'', true);
294 294
 
295 295
 		$circle = $this->federatedUserService->getGroupCircle($groupId);
296
-		$members = array_map(function (Member $member): string {
296
+		$members = array_map(function(Member $member): string {
297 297
 			return $member->getSingleId();
298 298
 		}, $this->memberRequest->getMembers($circle->getSingleId()));
299 299
 
@@ -371,7 +371,7 @@  discard block
 block discarded – undo
371 371
 	 */
372 372
 	public function groupDeleted(string $groupId): void {
373 373
 		$circle = new Circle();
374
-		$circle->setName('group:' . $groupId)
374
+		$circle->setName('group:'.$groupId)
375 375
 			   ->setConfig(Circle::CFG_SYSTEM | Circle::CFG_NO_OWNER | Circle::CFG_HIDDEN)
376 376
 			   ->setSource(Member::TYPE_GROUP);
377 377
 
Please login to merge, or discard this patch.
lib/Service/FederatedUserService.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -528,7 +528,7 @@  discard block
 block discarded – undo
528 528
 		if ($check) {
529 529
 			$user = $this->userManager->get($userId);
530 530
 			if ($user === null) {
531
-				throw new FederatedUserNotFoundException('user ' . $userId . ' not found');
531
+				throw new FederatedUserNotFoundException('user '.$userId.' not found');
532 532
 			}
533 533
 			$displayName = $this->userManager->getDisplayName($userId);
534 534
 		} else {
@@ -1043,7 +1043,7 @@  discard block
 block discarded – undo
1043 1043
 			$prefix = ($federatedUser->getUserType() === Member::TYPE_APP) ? 'app'
1044 1044
 				: Member::$TYPE[$federatedUser->getUserType()];
1045 1045
 
1046
-			$circle->setName($prefix . ':' . $federatedUser->getUserId() . ':' . $id)
1046
+			$circle->setName($prefix.':'.$federatedUser->getUserId().':'.$id)
1047 1047
 				   ->setDisplayName($federatedUser->getDisplayName())
1048 1048
 				   ->setSingleId($id)
1049 1049
 				   ->setSource($source);
@@ -1247,7 +1247,7 @@  discard block
 block discarded – undo
1247 1247
 		$owner = $this->getCurrentApp();
1248 1248
 
1249 1249
 		$circle = new Circle();
1250
-		$circle->setName('group:' . $groupId)
1250
+		$circle->setName('group:'.$groupId)
1251 1251
 			   ->setConfig(Circle::CFG_SYSTEM | Circle::CFG_NO_OWNER | Circle::CFG_HIDDEN)
1252 1252
 			   ->setSingleId($this->token(ManagedModel::ID_LENGTH))
1253 1253
 			   ->setSource(Member::TYPE_GROUP);
@@ -1315,8 +1315,8 @@  discard block
 block discarded – undo
1315 1315
 	 * @return string
1316 1316
 	 */
1317 1317
 	private function generateCacheKey(FederatedUser $federatedUser): string {
1318
-		return $federatedUser->getInstance() . '#'
1319
-			   . $federatedUser->getUserType() . '#'
1318
+		return $federatedUser->getInstance().'#'
1319
+			   . $federatedUser->getUserType().'#'
1320 1320
 			   . $federatedUser->getUserId();
1321 1321
 	}
1322 1322
 }
Please login to merge, or discard this patch.