nextcloud / circles

lib/Service/SharingFrameService.php

Spacing +5 added lines, -5 removed lines

@@ -153,7 +153,7 @@ 
 	public function createFrame($circleUniqueId, SharingFrame $frame, $broadcast = null) {
 
 		$this->miscService->log(
-			'Create frame with payload ' . json_encode($frame->getPayload()) . ' as ' . $this->userId, 0
+			'Create frame with payload '.json_encode($frame->getPayload()).' as '.$this->userId, 0
 		);
 		try {
 			$circle = $this->circlesRequest->getCircle($circleUniqueId, $this->userId);
@@ -168,7 +168,7 @@ 
 			$this->initiateShare($circle->getUniqueId(), $frame->getUniqueId());
 		} catch (Exception $e) {
 			$this->miscService->log(
-				'fail to create frame for circle ' . $circleUniqueId . ' - ' . $e->getMessage()
+				'fail to create frame for circle '.$circleUniqueId.' - '.$e->getMessage()
 			);
 			throw $e;
 		}
@@ -318,7 +318,7 @@ 
 			return true;
 		} catch (Exception $e) {
 			$this->miscService->log(
-				'fail to initialise circle share with request ' . json_encode($request) . ' - '
+				'fail to initialise circle share with request '.json_encode($request).' - '
 				. $e->getMessage(), 3
 			);
 			throw $e;
@@ -332,7 +332,7 @@ 
 	 * @return string
 	 */
 	private function generatePayloadDeliveryURL($remote) {
-		return $this->configService->generateRemoteHost($remote) .
+		return $this->configService->generateRemoteHost($remote).
 			   $this->urlGenerator->linkToRoute('circles.Shares.initShareDelivery');
 	}
 
@@ -407,7 +407,7 @@ 
 
 		} catch (Exception $e) {
 			$this->miscService->log(
-				'fail to send frame to ' . $link->getAddress() . ' - ' . $e->getMessage()
+				'fail to send frame to '.$link->getAddress().' - '.$e->getMessage()
 			);
 		}
 	}

lib/Api/v1/Circles.php

Spacing +5 added lines, -5 removed lines

@@ -463,7 +463,7 @@ 
 	 */
 	public static function generateLink($circleUniqueId) {
 		return OC::$server->getURLGenerator()
-						  ->linkToRoute('circles.Navigation.navigate') . '#' . $circleUniqueId;
+						  ->linkToRoute('circles.Navigation.navigate').'#'.$circleUniqueId;
 	}
 
 
@@ -478,7 +478,7 @@ 
 	 */
 	public static function generateAbsoluteLink($circleUniqueId) {
 		return OC::$server->getURLGenerator()
-						  ->linkToRouteAbsolute('circles.Navigation.navigate') . '#' . $circleUniqueId;
+						  ->linkToRouteAbsolute('circles.Navigation.navigate').'#'.$circleUniqueId;
 	}
 
 
@@ -493,8 +493,8 @@ 
 	 */
 	public static function generateRemoteLink(FederatedLink $link) {
 		return OC::$server->getURLGenerator()
-						  ->linkToRoute('circles.Navigation.navigate') . '#' . $link->getUniqueId()
-			   . '-' . $link->getToken();
+						  ->linkToRoute('circles.Navigation.navigate').'#'.$link->getUniqueId()
+			   . '-'.$link->getToken();
 	}
 
 
@@ -505,7 +505,7 @@ 
 	 */
 	public static function generateUserParameter(SharingFrame $frame) {
 		if ($frame->getCloudId() !== null) {
-			$name = $frame->getAuthor() . '@' . $frame->getCloudId();
+			$name = $frame->getAuthor().'@'.$frame->getCloudId();
 		} else {
 			try {
 				$membersService = \OC::$server->query(MembersService::class);

lib/Circles/FileSharingBroadcaster.php

Spacing +12 added lines, -12 removed lines

@@ -132,7 +132,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 {
@@ -325,7 +325,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']);
@@ -418,7 +418,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());
 		}
 	}
 
@@ -436,8 +436,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]);
@@ -476,7 +476,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();
@@ -531,7 +531,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();
 		}
@@ -566,7 +566,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
 		);
@@ -612,7 +612,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());
 		}
 	}
 
@@ -635,7 +635,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 mail with a link to access them.\n",
@@ -677,7 +677,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();
 		}
@@ -722,7 +722,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/Search/GlobalScaleUsers.php

Spacing +1 added lines, -1 removed lines

@@ -95,7 +95,7 @@
 		RequestResultNotJsonException $e
 		) {
 			$this->miscService->log(
-				'Issue while search users from lookup: ' . get_class($e) . ' ' . $e->getMessage()
+				'Issue while search users from lookup: '.get_class($e).' '.$e->getMessage()
 			);
 
 			return [];

lib/Activity/ProviderParser.php

Spacing +4 added lines, -4 removed lines

@@ -137,7 +137,7 @@ 
 				continue;
 			}
 
-			$replace['{' . $k . '}'] = $data[$k]['_parsed'];
+			$replace['{'.$k.'}'] = $data[$k]['_parsed'];
 		}
 
 		$line = strtr($line, $replace);
@@ -301,7 +301,7 @@ 
 		return [
 			'type'    => $member->getTypeName(),
 			'id'      => $member->getUserId(),
-			'name'    => $member->getCachedName() . ' (' . $member->getTypeString() . ')',
+			'name'    => $member->getCachedName().' ('.$member->getTypeString().')',
 			'_parsed' => $member->getCachedName()
 		];
 	}
@@ -337,8 +337,8 @@ 
 		return [
 			'type'    => 'circle',
 			'id'      => $link->getUniqueId(),
-			'name'    => $link->getToken() . '@' . $link->getAddress(),
-			'_parsed' => $link->getToken() . '@' . $link->getAddress()
+			'name'    => $link->getToken().'@'.$link->getAddress(),
+			'_parsed' => $link->getToken().'@'.$link->getAddress()
 		];
 	}
 

lib/AppInfo/Application.php

Spacing +1 added lines, -1 removed lines

@@ -59,7 +59,7 @@
 use Throwable;
 
 
-require_once __DIR__ . '/../../vendor/autoload.php';
+require_once __DIR__.'/../../vendor/autoload.php';
 
 
 /**

lib/Db/SharesRequest.php

Spacing +1 added lines, -1 removed lines

@@ -105,7 +105,7 @@
 		$expr = $qb->expr();
 
 		$this->limitToShareType($qb, self::SHARE_TYPE);
-		$qb->andWhere($expr->isNull($this->default_select_alias . '.parent'));
+		$qb->andWhere($expr->isNull($this->default_select_alias.'.parent'));
 
 		$shares = [];
 		$cursor = $qb->execute();

lib/Service/ConfigService.php

Spacing +6 added lines, -6 removed lines

@@ -251,7 +251,7 @@ 
 		if ((!$this->isNonSSLLinksAllowed() || strpos($remote, 'http://') !== 0)
 			&& strpos($remote, 'https://') !== 0
 		) {
-			$remote = 'https://' . $remote;
+			$remote = 'https://'.$remote;
 		}
 
 		return rtrim($remote, '/');
@@ -509,7 +509,7 @@ 
 				return false;
 			}
 
-			throw new GSStatusException('GS and lookup are not configured : ' . $lookup . ', ' . $enabled);
+			throw new GSStatusException('GS and lookup are not configured : '.$lookup.', '.$enabled);
 		}
 
 		$clef = $this->config->getSystemValue('gss.jwt.key', '');
@@ -573,7 +573,7 @@ 
 			}
 
 			if (array_key_exists('port', $local)) {
-				return $local['host'] . ':' . $local['port'];
+				return $local['host'].':'.$local['port'];
 			} else {
 				return $local['host'];
 			}
@@ -595,13 +595,13 @@ 
 	 * @return string
 	 */
 	public function getRemotePath(string $route = 'circles.Navigation.navigate', array $args = []): string {
-		$base = $this->getAppValue(self::LOCAL_CLOUD_SCHEME) . '://' . $this->getLocalInstance();
+		$base = $this->getAppValue(self::LOCAL_CLOUD_SCHEME).'://'.$this->getLocalInstance();
 
 		if ($route === '') {
 			return $base;
 		}
 
-		return $base . $this->urlGenerator->linkToRoute($route, $args);
+		return $base.$this->urlGenerator->linkToRoute($route, $args);
 	}
 
 	/**
@@ -702,7 +702,7 @@ 
 			$ncBase = substr($ncBase, 0, -strlen($forcedPath));
 		}
 
-		return rtrim($ncBase, '/') . $link;
+		return rtrim($ncBase, '/').$link;
 	}
 
 }

lib/Command/CirclesTest.php

Spacing +8 added lines, -8 removed lines

@@ -141,9 +141,9 @@ 
 		$test->setAsync(true);
 		$token = $this->gsUpstreamService->newEvent($test);
 
-		$output->writeln('- Async request is sent, now waiting ' . $this->delay . ' seconds');
+		$output->writeln('- Async request is sent, now waiting '.$this->delay.' seconds');
 		sleep($this->delay);
-		$output->writeln('- Pause is over, checking results for ' . $token);
+		$output->writeln('- Pause is over, checking results for '.$token);
 
 		$wrappers = $this->gsUpstreamService->getEventsByToken($token);
 
@@ -155,7 +155,7 @@ 
 
 		$localLooksGood = false;
 		foreach ($instances as $instance) {
-			$output->write($instance . ' ');
+			$output->write($instance.' ');
 			if (array_key_exists($instance, $result)
 				&& $result[$instance]->getResult()
 									 ->gInt('status') === 1) {
@@ -192,7 +192,7 @@ 
 		$this->configService->configureRequest($request, $route, $args);
 		$request->setFollowLocation(false);
 
-		$o->write('- ' . $type . ' request on ' . $request->getCompleteUrl() . ': ');
+		$o->write('- '.$type.' request on '.$request->getCompleteUrl().': ');
 		$this->doRequest($request);
 
 		$color = 'error';
@@ -201,7 +201,7 @@ 
 			$color = 'info';
 		}
 
-		$o->writeln('<' . $color . '>' . $result->getStatusCode() . '</' . $color . '>');
+		$o->writeln('<'.$color.'>'.$result->getStatusCode().'</'.$color.'>');
 
 		if ($result->getStatusCode() === 200) {
 			return true;
@@ -223,7 +223,7 @@ 
 
 		$output->writeln('');
 		$output->writeln(
-			'The address <info>' . $address . '</info> seems to reach your local Nextcloud.'
+			'The address <info>'.$address.'</info> seems to reach your local Nextcloud.'
 		);
 
 		$helper = $this->getHelper('question');
@@ -240,8 +240,8 @@ 
 
 		$this->configService->setAppValue(ConfigService::FORCE_NC_BASE, $address);
 		$output->writeln(
-			'New configuration <info>' . Application::APP_ID . '.' . ConfigService::FORCE_NC_BASE . '=\''
-			. $address . '\'</info> stored in database'
+			'New configuration <info>'.Application::APP_ID.'.'.ConfigService::FORCE_NC_BASE.'=\''
+			. $address.'\'</info> stored in database'
 		);
 	}