nextcloud / server

lib/private/Files/Config/CachedMountFileInfo.php

Spacing +1 added lines, -1 removed lines

@@ -43,6 +43,6 @@
 	}
 
 	public function getPath() {
-		return $this->getMountPoint() . $this->getInternalPath();
+		return $this->getMountPoint().$this->getInternalPath();
 	}
 }

core/Command/Group/Add.php

Spacing +3 added lines, -3 removed lines

@@ -65,7 +65,7 @@ 
 		$gid = $input->getArgument('groupid');
 		$group = $this->groupManager->get($gid);
 		if ($group) {
-			$output->writeln('<error>Group "' . $gid . '" already exists.</error>');
+			$output->writeln('<error>Group "'.$gid.'" already exists.</error>');
 			return 1;
 		} else {
 			$group = $this->groupManager->createGroup($gid);
@@ -73,9 +73,9 @@ 
 				$output->writeln('<error>Could not create group</error>');
 				return 2;
 			}
-			$output->writeln('Created group "' . $group->getGID() . '"');
+			$output->writeln('Created group "'.$group->getGID().'"');
 
-			$displayName = trim((string)$input->getOption('display-name'));
+			$displayName = trim((string) $input->getOption('display-name'));
 			if ($displayName !== '') {
 				$group->setDisplayName($displayName);
 			}

apps/files/lib/BackgroundJob/TransferOwnership.php

Spacing +5 added lines, -5 removed lines

@@ -98,7 +98,7 @@ 
 		$destinationUserObject = $this->userManager->get($destinationUser);
 
 		if (!$sourceUserObject instanceof IUser) {
-			$this->logger->alert('Could not transfer ownership: Unknown source user ' . $sourceUser);
+			$this->logger->alert('Could not transfer ownership: Unknown source user '.$sourceUser);
 			$this->failedNotication($transfer);
 			return;
 		}
@@ -136,7 +136,7 @@ 
 				'targetUser' => $transfer->getTargetUser(),
 				'nodeName' => $transfer->getNodeName(),
 			])
-			->setObject('transfer', (string)$transfer->getId());
+			->setObject('transfer', (string) $transfer->getId());
 		$this->notificationManager->notify($notification);
 
 		// Send notification to source user
@@ -149,7 +149,7 @@ 
 				'targetUser' => $transfer->getTargetUser(),
 				'nodeName' => $transfer->getNodeName(),
 			])
-			->setObject('transfer', (string)$transfer->getId());
+			->setObject('transfer', (string) $transfer->getId());
 		$this->notificationManager->notify($notification);
 	}
 
@@ -164,7 +164,7 @@ 
 				'targetUser' => $transfer->getTargetUser(),
 				'nodeName' => $transfer->getNodeName(),
 			])
-			->setObject('transfer', (string)$transfer->getId());
+			->setObject('transfer', (string) $transfer->getId());
 		$this->notificationManager->notify($notification);
 
 		// Send notification to source user
@@ -177,7 +177,7 @@ 
 				'targetUser' => $transfer->getTargetUser(),
 				'nodeName' => $transfer->getNodeName(),
 			])
-			->setObject('transfer', (string)$transfer->getId());
+			->setObject('transfer', (string) $transfer->getId());
 		$this->notificationManager->notify($notification);
 	}
 }

lib/private/Collaboration/Resources/ProviderManager.php

Spacing +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
 					$this->providerInstances[] = $this->serverContainer->query($provider);
 				} catch (QueryException $e) {
 					$this->logger->logException($e, [
-						'message' => "Could not query resource provider $provider: " . $e->getMessage()
+						'message' => "Could not query resource provider $provider: ".$e->getMessage()
 					]);
 				}
 			}

apps/comments/lib/Search/Result.php

Spacing +1 added lines, -1 removed lines

@@ -105,7 +105,7 @@
 			$suffix = '…';
 		}
 
-		return $prefix . mb_substr($message, $start, $end - $start) . $suffix;
+		return $prefix.mb_substr($message, $start, $end - $start).$suffix;
 	}
 
 }

apps/files/lib/Notification/Notifier.php

Spacing +13 added lines, -13 removed lines

@@ -120,7 +120,7 @@ 
 				$this->urlGenerator->getAbsoluteURL(
 					$this->urlGenerator->linkTo(
 						'',
-						'ocs/v2.php/apps/files/api/v1/transferownership/' . $id
+						'ocs/v2.php/apps/files/api/v1/transferownership/'.$id
 					)
 				),
 				IAction::TYPE_POST
@@ -133,7 +133,7 @@ 
 				$this->urlGenerator->getAbsoluteURL(
 					$this->urlGenerator->linkTo(
 						'',
-						'ocs/v2.php/apps/files/api/v1/transferownership/' . $id
+						'ocs/v2.php/apps/files/api/v1/transferownership/'.$id
 					)
 				),
 				IAction::TYPE_DELETE
@@ -156,7 +156,7 @@ 
 				[
 					'path' => [
 						'type' => 'highlight',
-						'id' => $param['targetUser'] . '::' . $param['nodeName'],
+						'id' => $param['targetUser'].'::'.$param['nodeName'],
 						'name' => $param['nodeName'],
 					]
 				])
@@ -165,7 +165,7 @@ 
 		return $notification;
 	}
 
-	public function handleTransferOwnershipFailedSource(INotification $notification,  string $languageCode): INotification {
+	public function handleTransferOwnershipFailedSource(INotification $notification, string $languageCode): INotification {
 		$l = $this->l10nFactory->get('files', $languageCode);
 		$param = $notification->getSubjectParameters();
 
@@ -177,7 +177,7 @@ 
 				[
 					'path' => [
 						'type' => 'highlight',
-						'id' => $param['targetUser'] . '::' . $param['nodeName'],
+						'id' => $param['targetUser'].'::'.$param['nodeName'],
 						'name' => $param['nodeName'],
 					],
 					'user' => [
@@ -190,7 +190,7 @@ 
 		return $notification;
 	}
 
-	public function handleTransferOwnershipFailedTarget(INotification $notification,  string $languageCode): INotification {
+	public function handleTransferOwnershipFailedTarget(INotification $notification, string $languageCode): INotification {
 		$l = $this->l10nFactory->get('files', $languageCode);
 		$param = $notification->getSubjectParameters();
 
@@ -202,7 +202,7 @@ 
 				[
 					'path' => [
 						'type' => 'highlight',
-						'id' => $param['sourceUser'] . '::' . $param['nodeName'],
+						'id' => $param['sourceUser'].'::'.$param['nodeName'],
 						'name' => $param['nodeName'],
 					],
 					'user' => [
@@ -216,7 +216,7 @@ 
 		return $notification;
 	}
 
-	public function handleTransferOwnershipDoneSource(INotification $notification,  string $languageCode): INotification {
+	public function handleTransferOwnershipDoneSource(INotification $notification, string $languageCode): INotification {
 		$l = $this->l10nFactory->get('files', $languageCode);
 		$param = $notification->getSubjectParameters();
 
@@ -228,7 +228,7 @@ 
 				[
 					'path' => [
 						'type' => 'highlight',
-						'id' => $param['targetUser'] . '::' . $param['nodeName'],
+						'id' => $param['targetUser'].'::'.$param['nodeName'],
 						'name' => $param['nodeName'],
 					],
 					'user' => [
@@ -242,7 +242,7 @@ 
 		return $notification;
 	}
 
-	public function handleTransferOwnershipDoneTarget(INotification $notification,  string $languageCode): INotification {
+	public function handleTransferOwnershipDoneTarget(INotification $notification, string $languageCode): INotification {
 		$l = $this->l10nFactory->get('files', $languageCode);
 		$param = $notification->getSubjectParameters();
 
@@ -254,7 +254,7 @@ 
 				[
 					'path' => [
 						'type' => 'highlight',
-						'id' => $param['sourceUser'] . '::' . $param['nodeName'],
+						'id' => $param['sourceUser'].'::'.$param['nodeName'],
 						'name' => $param['nodeName'],
 					],
 					'user' => [
@@ -275,7 +275,7 @@ 
 
 		// TODO: This should all be moved to a service that also the transferownershipContoller uses.
 		try {
-			$transferOwnership = $this->mapper->getById((int)$notification->getObjectId());
+			$transferOwnership = $this->mapper->getById((int) $notification->getObjectId());
 		} catch (DoesNotExistException $e) {
 			return;
 		}
@@ -289,7 +289,7 @@ 
 				'targetUser' => $transferOwnership->getTargetUser(),
 				'nodeName' => $transferOwnership->getNodeName()
 			])
-			->setObject('transfer', (string)$transferOwnership->getId());
+			->setObject('transfer', (string) $transferOwnership->getId());
 		$this->notificationManager->notify($notification);
 
 		$this->mapper->delete($transferOwnership);

lib/private/Preview/Office.php

Spacing +5 added lines, -5 removed lines

@@ -49,20 +49,20 @@
 
 		$tmpDir = \OC::$server->getTempManager()->getTempBaseDir();
 
-		$defaultParameters = ' -env:UserInstallation=file://' . escapeshellarg($tmpDir . '/owncloud-' . \OC_Util::getInstanceId() . '/') . ' --headless --nologo --nofirststartwizard --invisible --norestore --convert-to png --outdir ';
+		$defaultParameters = ' -env:UserInstallation=file://'.escapeshellarg($tmpDir.'/owncloud-'.\OC_Util::getInstanceId().'/').' --headless --nologo --nofirststartwizard --invisible --norestore --convert-to png --outdir ';
 		$clParameters = \OC::$server->getConfig()->getSystemValue('preview_office_cl_parameters', $defaultParameters);
 
-		$exec = $this->cmd . $clParameters . escapeshellarg($tmpDir) . ' ' . escapeshellarg($absPath);
+		$exec = $this->cmd.$clParameters.escapeshellarg($tmpDir).' '.escapeshellarg($absPath);
 
 		shell_exec($exec);
 
 		//create imagick object from png
 		$pngPreview = null;
 		try {
-			list($dirname, , , $filename) = array_values(pathinfo($absPath));
-			$pngPreview = $tmpDir . '/' . $filename . '.png';
+			list($dirname,,, $filename) = array_values(pathinfo($absPath));
+			$pngPreview = $tmpDir.'/'.$filename.'.png';
 
-			$png = new \imagick($pngPreview . '[0]');
+			$png = new \imagick($pngPreview.'[0]');
 			$png->setImageFormat('jpg');
 		} catch (\Exception $e) {
 			$this->cleanTmpFiles();

lib/public/AppFramework/Http/Template/ExternalShareMenuAction.php

Spacing +10 added lines, -10 removed lines

@@ -66,16 +66,16 @@
 	 * @since 14.0.0
 	 */
 	public function render(): string {
-		return '<li>' .
-			'    <button id="save-external-share" class="icon ' . Util::sanitizeHTML($this->getIcon()) . '" data-protected="false" data-owner-display-name="' . Util::sanitizeHTML($this->displayname) . '" data-owner="' . Util::sanitizeHTML($this->owner) . '" data-name="' . Util::sanitizeHTML($this->shareName) . '">' . Util::sanitizeHTML($this->getLabel()) . '</button>' .
-			'</li>' .
-			'<li id="external-share-menu-item" class="hidden">' .
-			'    <span class="menuitem">' .
-			'        <form class="save-form" action="#">' .
-			'            <input type="text" id="remote_address" placeholder="user@yourNextcloud.org">' .
-			'            <input type="submit" value=" " id="save-button-confirm" class="icon-confirm" disabled="disabled"></button>' .
-			'        </form>' .
-			'    </span>' .
+		return '<li>'.
+			'    <button id="save-external-share" class="icon '.Util::sanitizeHTML($this->getIcon()).'" data-protected="false" data-owner-display-name="'.Util::sanitizeHTML($this->displayname).'" data-owner="'.Util::sanitizeHTML($this->owner).'" data-name="'.Util::sanitizeHTML($this->shareName).'">'.Util::sanitizeHTML($this->getLabel()).'</button>'.
+			'</li>'.
+			'<li id="external-share-menu-item" class="hidden">'.
+			'    <span class="menuitem">'.
+			'        <form class="save-form" action="#">'.
+			'            <input type="text" id="remote_address" placeholder="user@yourNextcloud.org">'.
+			'            <input type="submit" value=" " id="save-button-confirm" class="icon-confirm" disabled="disabled"></button>'.
+			'        </form>'.
+			'    </span>'.
 			'</li>';
 	}
 }

apps/workflowengine/lib/Check/FileMimeType.php

Spacing +1 added lines, -1 removed lines

@@ -155,7 +155,7 @@
 	}
 
 	public function supportedEntities(): array {
-		return [ File::class ];
+		return [File::class];
 	}
 
 	public function isAvailableForScope(int $scope): bool {