nextcloud / circles

lib/Handlers/WebfingerHandler.php

Spacing +1 added lines, -1 removed lines

@@ -87,7 +87,7 @@
 				'version' => $this->configService->getAppValue('installed_version'),
 				'api' => Application::APP_API
 			];
-		} catch (UnknownInterfaceException|SignatoryException $e) {
+		} catch (UnknownInterfaceException | SignatoryException $e) {
 			return $response;
 		}
 

lib/Dashboard/TeamDashboardWidget.php

Spacing +1 added lines, -1 removed lines

@@ -76,7 +76,7 @@
 				->setItemsLimit($limit)
 				->setItemsOffset($since ? (int)$since : 0);
 
-			$circles = array_map(function (Circle $circle) {
+			$circles = array_map(function(Circle $circle) {
 				return new WidgetItem(
 					$circle->getDisplayName(),
 					'',

lib/FileSharingTeamResourceProvider.php

Spacing +6 added lines, -6 removed lines

@@ -59,18 +59,18 @@ 
 		}
 
 		$shares = $this->shareByCircleProvider->getSharesToCircle($teamId);
-		usort($shares, function ($a, $b) {
+		usort($shares, function($a, $b) {
 			return (int)($b->getItemType() === 'folder') - (int)($a->getItemType() === 'folder');
 		});
-		return array_map(function (ShareWrapper $shareWrapper) {
+		return array_map(function(ShareWrapper $shareWrapper) {
 			$isFolder = $shareWrapper->getItemType() === 'folder';
 			return new TeamResource(
 				$this,
 				(string)$shareWrapper->getFileSource(),
 				basename($shareWrapper->getFileTarget()),
-				$this->urlGenerator->getAbsoluteURL('/index.php/f/' . $shareWrapper->getFileSource()),
+				$this->urlGenerator->getAbsoluteURL('/index.php/f/'.$shareWrapper->getFileSource()),
 				iconSvg: $isFolder ? '<svg xmlns="http://www.w3.org/2000/svg" id="mdi-folder" viewBox="0 0 24 24"><path d="M10,4H4C2.89,4 2,4.89 2,6V18A2,2 0 0,0 4,20H20A2,2 0 0,0 22,18V8C22,6.89 21.1,6 20,6H12L10,4Z" /></svg>' : null,
-				iconURL: !$isFolder ?
+				iconURL : !$isFolder ?
 					$this->urlGenerator->linkToRouteAbsolute('core.preview.getPreviewByFileId', ['fileId' => $shareWrapper->getFileSource(), 'mimeFallback' => true, ])
 					: null,
 			);
@@ -82,7 +82,7 @@ 
 			return false;
 		}
 
-		return count(array_filter($this->getSharedWith($teamId), function (TeamResource $resource) use ($resourceId) {
+		return count(array_filter($this->getSharedWith($teamId), function(TeamResource $resource) use ($resourceId) {
 			return $resource->getId() === $resourceId;
 		})) !== 0;
 	}
@@ -94,7 +94,7 @@ 
 
 		$shares = $this->shareByCircleProvider->getSharesByFileId((int)$resourceId);
 
-		return array_map(function ($share) {
+		return array_map(function($share) {
 			return $share->getSharedWith();
 		}, $shares);
 	}

lib/Activity/ProviderParser.php

Spacing +1 added lines, -1 removed lines

@@ -125,7 +125,7 @@
 				continue;
 			}
 
-			$replace['{' . $k . '}'] = $data[$k]['_parsed'];
+			$replace['{'.$k.'}'] = $data[$k]['_parsed'];
 		}
 
 		$event->setParsedSubject(strtr($line, $replace));

lib/ShareByCircleProvider.php

Spacing +5 added lines, -5 removed lines

@@ -442,7 +442,7 @@ 
 
 		return array_filter(
 			array_map(
-				function (ShareWrapper $wrapper) {
+				function(ShareWrapper $wrapper) {
 					return $wrapper->getShare($this->rootFolder, $this->userManager, $this->urlGenerator);
 				}, $wrappedShares
 			)
@@ -494,7 +494,7 @@ 
 
 		return array_filter(
 			array_map(
-				function (ShareWrapper $wrapper) {
+				function(ShareWrapper $wrapper) {
 					return $wrapper->getShare($this->rootFolder, $this->userManager, $this->urlGenerator);
 				}, $wrappedShares
 			)
@@ -543,7 +543,7 @@ 
 
 		return array_filter(
 			array_map(
-				function (ShareWrapper $wrapper) {
+				function(ShareWrapper $wrapper) {
 					return $wrapper->getShare(
 						$this->rootFolder, $this->userManager, $this->urlGenerator, true
 					);
@@ -574,7 +574,7 @@ 
 
 		$share = $wrappedShare->getShare($this->rootFolder, $this->userManager, $this->urlGenerator);
 		if ($share->getPassword() !== '') {
-			$this->logger->notice('share is protected by a password, hash: ' . $share->getPassword());
+			$this->logger->notice('share is protected by a password, hash: '.$share->getPassword());
 		}
 
 		return $share;
@@ -684,7 +684,7 @@ 
 							];
 						} else {
 							// we only store temp value, as token is unknown at this point
-							$remote[$member->getUserid() . '@' . $member->getInstance()] = [
+							$remote[$member->getUserid().'@'.$member->getInstance()] = [
 								'node_id' => $share->getFileSource(),
 								'shareId' => $share->getId(),
 								'memberId' => $member->getId(),

lib/Circles/FileSharingBroadcaster.php

Spacing +13 added lines, -13 removed lines

@@ -114,7 +114,7 @@ 
 			$this->miscService = OC::$server->query(MiscService::class);
 		} catch (QueryException $e) {
 			OC::$server->getLogger()
-				->log(1, 'Circles: cannot init FileSharingBroadcaster - ' . $e->getMessage());
+				->log(1, 'Circles: cannot init FileSharingBroadcaster - '.$e->getMessage());
 		}
 
 		try {
@@ -259,7 +259,7 @@ 
 
 		$allShares = $this->fileSharesRequest->getSharesForCircle($member->getCircleId());
 		$knownShares = array_map(
-			function (SharesToken $shareToken) {
+			function(SharesToken $shareToken) {
 				return $shareToken->getShareId();
 			},
 			$this->tokensRequest->getTokensFromMember($member)
@@ -306,7 +306,7 @@ 
 	 * @throws IllegalIDChangeException
 	 */
 	private function generateShare($data): IShare {
-		$this->logger->log(0, 'Regenerate shares from payload: ' . json_encode($data));
+		$this->logger->log(0, 'Regenerate shares from payload: '.json_encode($data));
 
 		$share = new Share($this->rootFolder, $this->userManager);
 		$share->setId($data['id']);
@@ -399,7 +399,7 @@ 
 			$this->sendPasswordByMail($share, $displayName, $email, $password);
 		} catch (Exception $e) {
 			OC::$server->getLogger()
-				->log(1, 'Circles::sharedByMail - mail were not sent: ' . $e->getMessage());
+				->log(1, 'Circles::sharedByMail - mail were not sent: '.$e->getMessage());
 		}
 	}
 
@@ -417,8 +417,8 @@ 
 		$message = $this->mailer->createMessage();
 
 		$this->logger->log(
-			0, "Sending mail to circle '" . $circleName . "': " . $email . ' file: ' . $fileName
-			   . ' - link: ' . $link
+			0, "Sending mail to circle '".$circleName."': ".$email.' file: '.$fileName
+			   . ' - link: '.$link
 		);
 
 		$subject = $this->l10n->t('%s shared »%s« with you.', [$author, $fileName]);
@@ -456,7 +456,7 @@ 
 
 		$message = $this->mailer->createMessage();
 
-		$this->logger->log(0, "Sending password mail to circle '" . $circleName . "': " . $email);
+		$this->logger->log(0, "Sending password mail to circle '".$circleName."': ".$email);
 
 		$filename = $share->getNode()
 			->getName();
@@ -511,7 +511,7 @@ 
 		$message->setFrom([\OCP\Util::getDefaultEmailAddress($instanceName) => $senderName]);
 		if ($initiatorEmailAddress !== null) {
 			$message->setReplyTo([$initiatorEmailAddress => $initiatorDisplayName]);
-			$emailTemplate->addFooter($instanceName . ' - ' . $this->defaults->getSlogan());
+			$emailTemplate->addFooter($instanceName.' - '.$this->defaults->getSlogan());
 		} else {
 			$emailTemplate->addFooter();
 		}
@@ -546,7 +546,7 @@ 
 		$emailTemplate->addHeader();
 		$emailTemplate->addHeading($subject, false);
 		$emailTemplate->addBodyText(
-			htmlspecialchars($text) . '<br>' . htmlspecialchars(
+			htmlspecialchars($text).'<br>'.htmlspecialchars(
 				$this->l10n->t('Click the button below to open it.')
 			), $text
 		);
@@ -592,7 +592,7 @@ 
 			$this->sendMailExistingShares($template, $author->getCachedName(), $recipient);
 			$this->sendPasswordExistingShares($author, $recipient, $password);
 		} catch (Exception $e) {
-			$this->logger->log(2, 'Failed to send mail about existing share ' . $e->getMessage());
+			$this->logger->log(2, 'Failed to send mail about existing share '.$e->getMessage());
 		}
 	}
 
@@ -615,7 +615,7 @@ 
 		$authorName = ($authorUser instanceof IUser) ? $authorUser->getDisplayName() : $author;
 		$authorEmail = ($authorUser instanceof IUser) ? $authorUser->getEMailAddress() : null;
 
-		$this->logger->log(0, "Sending password mail about existing files to '" . $email . "'");
+		$this->logger->log(0, "Sending password mail about existing files to '".$email."'");
 
 		$plainBodyPart = $this->l10n->t(
 			"%1\$s shared multiple files with you.\nYou should have already received a separate email with a link to access them.\n",
@@ -657,7 +657,7 @@ 
 		$message->setFrom([\OCP\Util::getDefaultEmailAddress($instanceName) => $senderName]);
 		if ($authorEmail !== null) {
 			$message->setReplyTo([$authorEmail => $authorName]);
-			$emailTemplate->addFooter($instanceName . ' - ' . $this->defaults->getSlogan());
+			$emailTemplate->addFooter($instanceName.' - '.$this->defaults->getSlogan());
 		} else {
 			$emailTemplate->addFooter();
 		}
@@ -702,7 +702,7 @@ 
 	 */
 	protected function generateMailExitingShares($author, $circleName) {
 		$this->logger->log(
-			0, "Generating mail about existing share mail from '" . $author . "' in "
+			0, "Generating mail about existing share mail from '".$author."' in "
 			   . $circleName
 		);
 

lib/Tools/ActivityPub/NCSignature.php

Spacing +9 added lines, -9 removed lines

@@ -196,8 +196,8 @@ 
 			throw new SignatureException('missing elements in \'headers\'');
 		}
 
-		$target = strtolower($request->getMethod()) . ' ' . $request->getRequestUri();
-		$estimated = ['(request-target): ' . $target];
+		$target = strtolower($request->getMethod()).' '.$request->getRequestUri();
+		$estimated = ['(request-target): '.$target];
 
 		foreach ($headers as $key) {
 			$value = $request->getHeader($key);
@@ -208,7 +208,7 @@ 
 				throw new SignatureException('empty elements in \'headers\'');
 			}
 
-			$estimated[] = $key . ': ' . $value;
+			$estimated[] = $key.': '.$value;
 		}
 		$signedRequest->setClearSignature(implode("\n", $estimated));
 	}
@@ -280,7 +280,7 @@ 
 		$request = $signedRequest->getOutgoingRequest();
 
 		$data = new SimpleDataStore();
-		$data->s('(request-target)', NCRequest::method($request->getType()) . ' ' . $request->getPath())
+		$data->s('(request-target)', NCRequest::method($request->getType()).' '.$request->getPath())
 			->sInt('content-length', strlen($signedRequest->getBody()))
 			->s('date', gmdate($this->dateHeader))
 			->s('digest', $signedRequest->getDigest())
@@ -299,7 +299,7 @@ 
 		foreach ($data->keys() as $element) {
 			try {
 				$value = $data->gItem($element);
-				$signing[] = $element . ': ' . $value;
+				$signing[] = $element.': '.$value;
 				if ($element !== '(request-target)') {
 					$signedRequest->getOutgoingRequest()->addHeader($element, $value);
 				}
@@ -332,10 +332,10 @@ 
 		$headers = array_diff($signedRequest->getSignatureHeader()->keys(), ['(request-target)']);
 		$signatory = $signedRequest->getSignatory();
 		$signatureElements = [
-			'keyId="' . $signatory->getKeyId() . '"',
-			'algorithm="' . $this->getChosenEncryption($signatory) . '"',
-			'headers="' . implode(' ', $headers) . '"',
-			'signature="' . $signedRequest->getSignedSignature() . '"'
+			'keyId="'.$signatory->getKeyId().'"',
+			'algorithm="'.$this->getChosenEncryption($signatory).'"',
+			'headers="'.implode(' ', $headers).'"',
+			'signature="'.$signedRequest->getSignedSignature().'"'
 		];
 
 		$signedRequest->getOutgoingRequest()->addHeader('Signature', implode(',', $signatureElements));

lib/Tools/Model/NCSignedRequest.php

Spacing +1 added lines, -1 removed lines

@@ -115,7 +115,7 @@
 	 */
 	public function setBody(string $body): self {
 		$this->body = $body;
-		$this->setDigest('SHA-256=' . base64_encode(hash('sha256', utf8_encode($body), true)));
+		$this->setDigest('SHA-256='.base64_encode(hash('sha256', utf8_encode($body), true)));
 
 		return $this;
 	}

lib/Command/MembersList.php

Spacing +16 added lines, -17 removed lines

@@ -167,11 +167,11 @@ 
 			$this->federatedUserService->commandLineInitiator($initiator, $initiatorType, $circleId, true);
 			$circle = $this->circleService->getCircle($circleId);
 
-			$output->writeln('<info>Name</info>: ' . $circle->getName());
+			$output->writeln('<info>Name</info>: '.$circle->getName());
 			$owner = $circle->getOwner();
-			$output->writeln('<info>Owner</info>: ' . $owner->getUserId() . '@' . $owner->getInstance());
+			$output->writeln('<info>Owner</info>: '.$owner->getUserId().'@'.$owner->getInstance());
 			$type = implode(', ', Circle::getCircleFlags($circle, Circle::FLAGS_LONG));
-			$output->writeln('<info>Config</info>: ' . $type);
+			$output->writeln('<info>Config</info>: '.$type);
 			$output->writeln(' ');
 
 			$tree = new TreeNode(null, new SimpleDataStore(['circle' => $circle]));
@@ -236,8 +236,7 @@ 
 						$member,
 						$this->input->getOption('display-name')
 					),
-					($level > 0) ? Member::$DEF_LEVEL[$level] :
-						'(' . strtolower($member->getStatus()) . ')',
+					($level > 0) ? Member::$DEF_LEVEL[$level] : '('.strtolower($member->getStatus()).')',
 					($member->hasInvitedBy()) ? $this->configService->displayFederatedUser(
 						$member->getInvitedBy(),
 						$this->input->getOption('display-name')
@@ -327,7 +326,7 @@ 
 							$circle = $this->remoteService->getCircleFromInstance(
 								$member->getSingleId(), $member->getInstance(), $data
 							);
-						} catch (CircleNotFoundException|RemoteInstanceException $e) {
+						} catch (CircleNotFoundException | RemoteInstanceException $e) {
 						}
 					} else {
 						$this->federatedUserService->commandLineInitiator(
@@ -403,14 +402,14 @@ 
 				$member = $data->gObj('member', Member::class);
 
 				if ($lineNumber === 1) {
-					$line .= '<info>' . $member->getSingleId() . '</info>';
+					$line .= '<info>'.$member->getSingleId().'</info>';
 					if (!$this->configService->isLocalInstance($member->getInstance())) {
-						$line .= '@' . $member->getInstance();
+						$line .= '@'.$member->getInstance();
 					}
-					$line .= ' (' . Member::$DEF_LEVEL[$member->getLevel()] . ')';
+					$line .= ' ('.Member::$DEF_LEVEL[$member->getLevel()].')';
 
-					$line .= ' <info>MemberId</info>: ' . $member->getId();
-					$line .= ' <info>Name</info>: ' . $this->configService->displayFederatedUser(
+					$line .= ' <info>MemberId</info>: '.$member->getId();
+					$line .= ' <info>Name</info>: '.$this->configService->displayFederatedUser(
 						$member,
 						$this->input->getOption('display-name')
 					);
@@ -418,7 +417,7 @@ 
 						$line .= ' <info>Source</info>: '
 								 . Circle::$DEF_SOURCE[$member->getBasedOn()->getSource()];
 					} else {
-						$line .= ' <info>Type</info>: ' . Member::$TYPE[$member->getUserType()];
+						$line .= ' <info>Type</info>: '.Member::$TYPE[$member->getUserType()];
 					}
 				}
 
@@ -431,21 +430,21 @@ 
 						return $line;
 					}
 					$owner = $circle->getOwner();
-					$line .= '<info>Owner</info>: ' . $owner->getUserId() . '@' . $owner->getInstance();
+					$line .= '<info>Owner</info>: '.$owner->getUserId().'@'.$owner->getInstance();
 					$type = implode(', ', Circle::getCircleFlags($circle, Circle::FLAGS_LONG));
-					$line .= ($type === '') ? '' : ' <info>Config</info>: ' . $type;
+					$line .= ($type === '') ? '' : ' <info>Config</info>: '.$type;
 				}
 			} else {
 				if ($lineNumber === 1 && !is_null($circle)) {
-					$line .= '<info>' . $circle->getSingleId() . '</info>';
+					$line .= '<info>'.$circle->getSingleId().'</info>';
 					if (!$this->configService->isLocalInstance($circle->getInstance())) {
-						$line .= '@' . $circle->getInstance();
+						$line .= '@'.$circle->getInstance();
 					}
 				}
 			}
 
 			return $line;
-		} catch (InvalidItemException|ItemNotFoundException|UnknownTypeException $e) {
+		} catch (InvalidItemException | ItemNotFoundException | UnknownTypeException $e) {
 		}
 
 		return '';