nextcloud / server

core/Command/App/Disable.php

Spacing +2 added lines, -2 removed lines

@@ -72,14 +72,14 @@
 
 	private function disableApp(string $appId, OutputInterface $output): void {
 		if ($this->appManager->isInstalled($appId) === false) {
-			$output->writeln('No such app enabled: ' . $appId);
+			$output->writeln('No such app enabled: '.$appId);
 			return;
 		}
 
 		try {
 			$this->appManager->disableApp($appId);
 			$appVersion = \OC_App::getAppVersion($appId);
-			$output->writeln($appId . ' ' . $appVersion . ' disabled');
+			$output->writeln($appId.' '.$appVersion.' disabled');
 		} catch (\Exception $e) {
 			$output->writeln($e->getMessage());
 			$this->exitCode = 2;

core/Command/App/Enable.php

Spacing +6 added lines, -6 removed lines

@@ -102,12 +102,12 @@ 
 	 * @param OutputInterface $output
 	 */
 	private function enableApp(string $appId, array $groupIds, bool $forceEnable, OutputInterface $output): void {
-		$groupNames = array_map(function (IGroup $group) {
+		$groupNames = array_map(function(IGroup $group) {
 			return $group->getDisplayName();
 		}, $groupIds);
 
 		if ($this->appManager->isInstalled($appId) && $groupIds === []) {
-			$output->writeln($appId . ' already enabled');
+			$output->writeln($appId.' already enabled');
 			return;
 		}
 
@@ -124,13 +124,13 @@ 
 
 			if ($groupIds === []) {
 				$this->appManager->enableApp($appId, $forceEnable);
-				$output->writeln($appId . ' ' . $appVersion . ' enabled');
+				$output->writeln($appId.' '.$appVersion.' enabled');
 			} else {
 				$this->appManager->enableAppForGroups($appId, $groupIds, $forceEnable);
-				$output->writeln($appId . ' ' . $appVersion . ' enabled for groups: ' . implode(', ', $groupNames));
+				$output->writeln($appId.' '.$appVersion.' enabled for groups: '.implode(', ', $groupNames));
 			}
 		} catch (AppPathNotFoundException $e) {
-			$output->writeln($appId . ' not found');
+			$output->writeln($appId.' not found');
 			$this->exitCode = 1;
 		} catch (\Exception $e) {
 			$output->writeln($e->getMessage());
@@ -160,7 +160,7 @@ 
 	 */
 	public function completeOptionValues($optionName, CompletionContext $context) {
 		if ($optionName === 'groups') {
-			return array_map(function (IGroup $group) {
+			return array_map(function(IGroup $group) {
 				return $group->getGID();
 			}, $this->groupManager->search($context->getCurrentWord()));
 		}

core/Command/App/Remove.php

Spacing +7 added lines, -7 removed lines

@@ -80,14 +80,14 @@ 
 
 		// Check if the app is installed
 		if (!\OC_App::getAppPath($appId)) {
-			$output->writeln($appId . ' is not installed');
+			$output->writeln($appId.' is not installed');
 			return 1;
 		}
 
 		// Removing shipped apps is not possible, therefore we pre-check that
 		// before trying to remove it
 		if ($this->manager->isShipped($appId)) {
-			$output->writeln($appId . ' could not be removed as it is a shipped app');
+			$output->writeln($appId.' could not be removed as it is a shipped app');
 			return 1;
 		}
 
@@ -96,9 +96,9 @@ 
 		if (!$input->getOption('keep-data')) {
 			try {
 				$this->manager->disableApp($appId);
-				$output->writeln($appId . ' disabled');
+				$output->writeln($appId.' disabled');
 			} catch (Throwable $e) {
-				$output->writeln('<error>Error: ' . $e->getMessage() . '</error>');
+				$output->writeln('<error>Error: '.$e->getMessage().'</error>');
 				$this->logger->logException($e, [
 					'app' => 'CLI',
 					'level' => ILogger::ERROR
@@ -111,7 +111,7 @@ 
 		try {
 			$result = $this->installer->removeApp($appId);
 		} catch (Throwable $e) {
-			$output->writeln('<error>Error: ' . $e->getMessage() . '</error>');
+			$output->writeln('<error>Error: '.$e->getMessage().'</error>');
 			$this->logger->logException($e, [
 				'app' => 'CLI',
 				'level' => ILogger::ERROR
@@ -120,12 +120,12 @@ 
 		}
 
 		if ($result === false) {
-			$output->writeln($appId . ' could not be removed');
+			$output->writeln($appId.' could not be removed');
 			return 1;
 		}
 
 		$appVersion = \OC_App::getAppVersion($appId);
-		$output->writeln($appId . ' ' . $appVersion . ' removed');
+		$output->writeln($appId.' '.$appVersion.' removed');
 
 		return 0;
 	}

lib/private/Authentication/TwoFactorAuth/Db/ProviderUserAssignmentDao.php

Spacing +2 added lines, -2 removed lines

@@ -60,7 +60,7 @@ 
 		$result = $query->execute();
 		$providers = [];
 		foreach ($result->fetchAll() as $row) {
-			$providers[$row['provider_id']] = 1 === (int)$row['enabled'];
+			$providers[$row['provider_id']] = 1 === (int) $row['enabled'];
 		}
 		$result->closeCursor();
 
@@ -114,7 +114,7 @@ 
 			->where($qb2->expr()->eq('uid', $qb2->createNamedParameter($uid)));
 		$deleteQuery->execute();
 
-		return array_map(function (array $row) {
+		return array_map(function(array $row) {
 			return [
 				'provider_id' => $row['provider_id'],
 				'uid' => $row['uid'],

lib/private/Search.php

Spacing +3 added lines, -3 removed lines

@@ -52,7 +52,7 @@ 
 		$this->initProviders();
 		$results = [];
 		foreach ($this->providers as $provider) {
-			if (! $provider->providesResultsFor($inApps)) {
+			if (!$provider->providesResultsFor($inApps)) {
 				continue;
 			}
 			if ($provider instanceof PagedProvider) {
@@ -87,7 +87,7 @@ 
 	public function removeProvider($provider) {
 		$this->registeredProviders = array_filter(
 			$this->registeredProviders,
-			function ($element) use ($provider) {
+			function($element) use ($provider) {
 				return ($element['class'] != $provider);
 			}
 		);
@@ -108,7 +108,7 @@ 
 	 * Create instances of all the registered search providers
 	 */
 	private function initProviders() {
-		if (! empty($this->providers)) {
+		if (!empty($this->providers)) {
 			return;
 		}
 		foreach ($this->registeredProviders as $provider) {

lib/private/Files/Config/MountProviderCollection.php

Spacing +10 added lines, -10 removed lines

@@ -83,13 +83,13 @@ 
 	 */
 	public function getMountsForUser(IUser $user) {
 		$loader = $this->loader;
-		$mounts = array_map(function (IMountProvider $provider) use ($user, $loader) {
+		$mounts = array_map(function(IMountProvider $provider) use ($user, $loader) {
 			return $provider->getMountsForUser($user, $loader);
 		}, $this->providers);
-		$mounts = array_filter($mounts, function ($result) {
+		$mounts = array_filter($mounts, function($result) {
 			return is_array($result);
 		});
-		$mounts = array_reduce($mounts, function (array $mounts, array $providerMounts) {
+		$mounts = array_reduce($mounts, function(array $mounts, array $providerMounts) {
 			return array_merge($mounts, $providerMounts);
 		}, []);
 		return $this->filterMounts($user, $mounts);
@@ -99,10 +99,10 @@ 
 		// shared mount provider gets to go last since it needs to know existing files
 		// to check for name collisions
 		$firstMounts = [];
-		$firstProviders = array_filter($this->providers, function (IMountProvider $provider) {
+		$firstProviders = array_filter($this->providers, function(IMountProvider $provider) {
 			return (get_class($provider) !== 'OCA\Files_Sharing\MountProvider');
 		});
-		$lastProviders = array_filter($this->providers, function (IMountProvider $provider) {
+		$lastProviders = array_filter($this->providers, function(IMountProvider $provider) {
 			return (get_class($provider) === 'OCA\Files_Sharing\MountProvider');
 		});
 		foreach ($firstProviders as $provider) {
@@ -140,11 +140,11 @@ 
 		$providers = array_reverse($this->homeProviders); // call the latest registered provider first to give apps an opportunity to overwrite builtin
 		foreach ($providers as $homeProvider) {
 			if ($mount = $homeProvider->getHomeMountForUser($user, $this->loader)) {
-				$mount->setMountPoint('/' . $user->getUID()); //make sure the mountpoint is what we expect
+				$mount->setMountPoint('/'.$user->getUID()); //make sure the mountpoint is what we expect
 				return $mount;
 			}
 		}
-		throw new \Exception('No home storage configured for user ' . $user);
+		throw new \Exception('No home storage configured for user '.$user);
 	}
 
 	/**
@@ -163,7 +163,7 @@ 
 	}
 
 	private function filterMounts(IUser $user, array $mountPoints) {
-		return array_filter($mountPoints, function (IMountPoint $mountPoint) use ($user) {
+		return array_filter($mountPoints, function(IMountPoint $mountPoint) use ($user) {
 			foreach ($this->mountFilters as $filter) {
 				if ($filter($mountPoint, $user) === false) {
 					return false;
@@ -215,10 +215,10 @@ 
 	 */
 	public function getRootMounts(): array {
 		$loader = $this->loader;
-		$mounts = array_map(function (IRootMountProvider $provider) use ($loader) {
+		$mounts = array_map(function(IRootMountProvider $provider) use ($loader) {
 			return $provider->getRootMounts($loader);
 		}, $this->rootProviders);
-		$mounts = array_reduce($mounts, function (array $mounts, array $providerMounts) {
+		$mounts = array_reduce($mounts, function(array $mounts, array $providerMounts) {
 			return array_merge($mounts, $providerMounts);
 		}, []);
 		return $mounts;

apps/files_external/lib/Lib/Auth/Password/UserProvided.php

Spacing +1 added lines, -1 removed lines

@@ -61,7 +61,7 @@
 	}
 
 	private function getCredentialsIdentifier($storageId) {
-		return self::CREDENTIALS_IDENTIFIER_PREFIX . $storageId;
+		return self::CREDENTIALS_IDENTIFIER_PREFIX.$storageId;
 	}
 
 	public function saveBackendOptions(IUser $user, $mountId, array $options) {

apps/files_sharing/lib/ShareBackend/File.php

Spacing +4 added lines, -4 removed lines

@@ -97,7 +97,7 @@ 
 	 */
 	public function generateTarget($itemSource, $shareWith, $exclude = null) {
 		$shareFolder = \OCA\Files_Sharing\Helper::getShareFolder();
-		$target = \OC\Files\Filesystem::normalizePath($shareFolder . '/' . basename($itemSource));
+		$target = \OC\Files\Filesystem::normalizePath($shareFolder.'/'.basename($itemSource));
 
 		// for group shares we return the target right away
 		if ($shareWith === false) {
@@ -105,13 +105,13 @@ 
 		}
 
 		\OC\Files\Filesystem::initMountPoints($shareWith);
-		$view = new \OC\Files\View('/' . $shareWith . '/files');
+		$view = new \OC\Files\View('/'.$shareWith.'/files');
 
 		if (!$view->is_dir($shareFolder)) {
 			$dir = '';
 			$subdirs = explode('/', $shareFolder);
 			foreach ($subdirs as $subdir) {
-				$dir = $dir . '/' . $subdir;
+				$dir = $dir.'/'.$subdir;
 				if (!$view->is_dir($dir)) {
 					$view->mkdir($dir);
 				}
@@ -249,7 +249,7 @@ 
 		if ($share['item_type'] === 'folder' && $target !== '') {
 			// note: in case of ext storage mount points the path might be empty
 			// which would cause a leading slash to appear
-			$share['path'] = ltrim($share['path'] . '/' . $target, '/');
+			$share['path'] = ltrim($share['path'].'/'.$target, '/');
 		}
 		return self::resolveReshares($share);
 	}

lib/private/SystemTag/SystemTagManager.php

Spacing +7 added lines, -7 removed lines

@@ -141,14 +141,14 @@ 
 			->from(self::TAG_TABLE);
 
 		if (!\is_null($visibilityFilter)) {
-			$query->andWhere($query->expr()->eq('visibility', $query->createNamedParameter((int)$visibilityFilter)));
+			$query->andWhere($query->expr()->eq('visibility', $query->createNamedParameter((int) $visibilityFilter)));
 		}
 
 		if (!empty($nameSearchPattern)) {
 			$query->andWhere(
 				$query->expr()->like(
 					'name',
-					$query->createNamedParameter('%' . $this->connection->escapeLikeParameter($nameSearchPattern). '%')
+					$query->createNamedParameter('%'.$this->connection->escapeLikeParameter($nameSearchPattern).'%')
 				)
 			);
 		}
@@ -182,7 +182,7 @@ 
 		$result->closeCursor();
 		if (!$row) {
 			throw new TagNotFoundException(
-				'Tag ("' . $tagName . '", '. $userVisible . ', ' . $userAssignable . ') does not exist'
+				'Tag ("'.$tagName.'", '.$userVisible.', '.$userAssignable.') does not exist'
 			);
 		}
 
@@ -205,7 +205,7 @@ 
 			$query->execute();
 		} catch (UniqueConstraintViolationException $e) {
 			throw new TagAlreadyExistsException(
-				'Tag ("' . $tagName . '", '. $userVisible . ', ' . $userAssignable . ') already exists',
+				'Tag ("'.$tagName.'", '.$userVisible.', '.$userAssignable.') already exists',
 				0,
 				$e
 			);
@@ -214,7 +214,7 @@ 
 		$tagId = $query->getLastInsertId();
 
 		$tag = new SystemTag(
-			(string)$tagId,
+			(string) $tagId,
 			$tagName,
 			$userVisible,
 			$userAssignable
@@ -266,7 +266,7 @@ 
 			}
 		} catch (UniqueConstraintViolationException $e) {
 			throw new TagAlreadyExistsException(
-				'Tag ("' . $newName . '", '. $userVisible . ', ' . $userAssignable . ') already exists',
+				'Tag ("'.$newName.'", '.$userVisible.', '.$userAssignable.') already exists',
 				0,
 				$e
 			);
@@ -373,7 +373,7 @@ 
 	}
 
 	private function createSystemTagFromRow($row) {
-		return new SystemTag((string)$row['id'], $row['name'], (bool)$row['visibility'], (bool)$row['editable']);
+		return new SystemTag((string) $row['id'], $row['name'], (bool) $row['visibility'], (bool) $row['editable']);
 	}
 
 	/**