nextcloud / server

apps/provisioning_api/lib/AppInfo/Application.php

Spacing +2 added lines, -2 removed lines

@@ -60,7 +60,7 @@ 
 	public function register(IRegistrationContext $context): void {
 		$context->registerEventListener(UserDeletedEvent::class, UserDeletedListener::class);
 
-		$context->registerService(NewUserMailHelper::class, function (ContainerInterface $c) {
+		$context->registerService(NewUserMailHelper::class, function(ContainerInterface $c) {
 			return new NewUserMailHelper(
 				$c->get(Defaults::class),
 				$c->get(IURLGenerator::class),
@@ -73,7 +73,7 @@ 
 				Util::getDefaultEmailAddress('no-reply')
 			);
 		});
-		$context->registerService(ProvisioningApiMiddleware::class, function (ContainerInterface $c) {
+		$context->registerService(ProvisioningApiMiddleware::class, function(ContainerInterface $c) {
 			$user = $c->get(IUserManager::class)->get($c->get('UserId'));
 			$isAdmin = false;
 			$isSubAdmin = false;

apps/user_ldap/lib/Command/ResetUser.php

Spacing +1 added lines, -1 removed lines

@@ -103,7 +103,7 @@
 			if (isset($pluginManagerSuppressed)) {
 				$this->pluginManager->setSuppressDeletion($pluginManagerSuppressed);
 			}
-			$output->writeln('<error>' . $e->getMessage() . '</error>');
+			$output->writeln('<error>'.$e->getMessage().'</error>');
 			return 1;
 		}
 		$output->writeln('<error>Error while resetting user</error>');

lib/private/Collaboration/Collaborators/UserPlugin.php

Spacing +3 added lines, -3 removed lines

@@ -124,7 +124,7 @@ 
 						}
 					}
 
-					uasort($users, function ($a, $b) {
+					uasort($users, function($a, $b) {
 						/**
 						 * @var \OC\User\User $a
 						 * @var \OC\User\User $b
@@ -169,7 +169,7 @@ 
 					'message' => $userStatus->getMessage(),
 					'icon' => $userStatus->getIcon(),
 					'clearAt' => $userStatus->getClearAt()
-						? (int)$userStatus->getClearAt()->format('U')
+						? (int) $userStatus->getClearAt()->format('U')
 						: null,
 				];
 			}
@@ -253,7 +253,7 @@ 
 							'message' => $userStatus->getMessage(),
 							'icon' => $userStatus->getIcon(),
 							'clearAt' => $userStatus->getClearAt()
-								? (int)$userStatus->getClearAt()->format('U')
+								? (int) $userStatus->getClearAt()->format('U')
 								: null,
 						];
 					}

lib/private/DB/OracleMigrator.php

Spacing +22 added lines, -22 removed lines

@@ -78,7 +78,7 @@ 
 		return new Index(
 		//TODO migrate existing uppercase indexes, then $this->connection->quoteIdentifier($index->getName()),
 			$index->getName(),
-			array_map(function ($columnName) {
+			array_map(function($columnName) {
 				return $this->connection->quoteIdentifier($columnName);
 			}, $index->getColumns()),
 			$index->isUnique(),
@@ -97,11 +97,11 @@ 
 	 */
 	protected function quoteForeignKeyConstraint($fkc) {
 		return new ForeignKeyConstraint(
-			array_map(function ($columnName) {
+			array_map(function($columnName) {
 				return $this->connection->quoteIdentifier($columnName);
 			}, $fkc->getLocalColumns()),
 			$this->connection->quoteIdentifier($fkc->getForeignTableName()),
-			array_map(function ($columnName) {
+			array_map(function($columnName) {
 				return $this->connection->quoteIdentifier($columnName);
 			}, $fkc->getForeignColumns()),
 			$fkc->getName(),
@@ -119,24 +119,24 @@ 
 		$schemaDiff = parent::getDiff($targetSchema, $connection);
 
 		// oracle forces us to quote the identifiers
-		$schemaDiff->newTables = array_map(function (Table $table) {
+		$schemaDiff->newTables = array_map(function(Table $table) {
 			return new Table(
 				$this->connection->quoteIdentifier($table->getName()),
-				array_map(function (Column $column) {
+				array_map(function(Column $column) {
 					return $this->quoteColumn($column);
 				}, $table->getColumns()),
-				array_map(function (Index $index) {
+				array_map(function(Index $index) {
 					return $this->quoteIndex($index);
 				}, $table->getIndexes()),
 				[],
-				array_map(function (ForeignKeyConstraint $fck) {
+				array_map(function(ForeignKeyConstraint $fck) {
 					return $this->quoteForeignKeyConstraint($fck);
 				}, $table->getForeignKeys()),
 				$table->getOptions()
 			);
 		}, $schemaDiff->newTables);
 
-		$schemaDiff->removedTables = array_map(function (Table $table) {
+		$schemaDiff->removedTables = array_map(function(Table $table) {
 			return new Table(
 				$this->connection->quoteIdentifier($table->getName()),
 				$table->getColumns(),
@@ -150,7 +150,7 @@ 
 		foreach ($schemaDiff->changedTables as $tableDiff) {
 			$tableDiff->name = $this->connection->quoteIdentifier($tableDiff->name);
 
-			$tableDiff->addedColumns = array_map(function (Column $column) {
+			$tableDiff->addedColumns = array_map(function(Column $column) {
 				return $this->quoteColumn($column);
 			}, $tableDiff->addedColumns);
 
@@ -160,43 +160,43 @@ 
 				$column->changedProperties = array_diff($column->changedProperties, ['autoincrement', 'unsigned']);
 			}
 			// remove columns that no longer have changed (because autoincrement and unsigned are not supported)
-			$tableDiff->changedColumns = array_filter($tableDiff->changedColumns, function (ColumnDiff $column) {
+			$tableDiff->changedColumns = array_filter($tableDiff->changedColumns, function(ColumnDiff $column) {
 				return count($column->changedProperties) > 0;
 			});
 
-			$tableDiff->removedColumns = array_map(function (Column $column) {
+			$tableDiff->removedColumns = array_map(function(Column $column) {
 				return $this->quoteColumn($column);
 			}, $tableDiff->removedColumns);
 
-			$tableDiff->renamedColumns = array_map(function (Column $column) {
+			$tableDiff->renamedColumns = array_map(function(Column $column) {
 				return $this->quoteColumn($column);
 			}, $tableDiff->renamedColumns);
 
-			$tableDiff->addedIndexes = array_map(function (Index $index) {
+			$tableDiff->addedIndexes = array_map(function(Index $index) {
 				return $this->quoteIndex($index);
 			}, $tableDiff->addedIndexes);
 
-			$tableDiff->changedIndexes = array_map(function (Index $index) {
+			$tableDiff->changedIndexes = array_map(function(Index $index) {
 				return $this->quoteIndex($index);
 			}, $tableDiff->changedIndexes);
 
-			$tableDiff->removedIndexes = array_map(function (Index $index) {
+			$tableDiff->removedIndexes = array_map(function(Index $index) {
 				return $this->quoteIndex($index);
 			}, $tableDiff->removedIndexes);
 
-			$tableDiff->renamedIndexes = array_map(function (Index $index) {
+			$tableDiff->renamedIndexes = array_map(function(Index $index) {
 				return $this->quoteIndex($index);
 			}, $tableDiff->renamedIndexes);
 
-			$tableDiff->addedForeignKeys = array_map(function (ForeignKeyConstraint $fkc) {
+			$tableDiff->addedForeignKeys = array_map(function(ForeignKeyConstraint $fkc) {
 				return $this->quoteForeignKeyConstraint($fkc);
 			}, $tableDiff->addedForeignKeys);
 
-			$tableDiff->changedForeignKeys = array_map(function (ForeignKeyConstraint $fkc) {
+			$tableDiff->changedForeignKeys = array_map(function(ForeignKeyConstraint $fkc) {
 				return $this->quoteForeignKeyConstraint($fkc);
 			}, $tableDiff->changedForeignKeys);
 
-			$tableDiff->removedForeignKeys = array_map(function (ForeignKeyConstraint $fkc) {
+			$tableDiff->removedForeignKeys = array_map(function(ForeignKeyConstraint $fkc) {
 				return $this->quoteForeignKeyConstraint($fkc);
 			}, $tableDiff->removedForeignKeys);
 		}
@@ -210,15 +210,15 @@ 
 	 */
 	protected function convertStatementToScript($statement) {
 		if (substr($statement, -1) === ';') {
-			return $statement . PHP_EOL . '/' . PHP_EOL;
+			return $statement.PHP_EOL.'/'.PHP_EOL;
 		}
-		$script = $statement . ';';
+		$script = $statement.';';
 		$script .= PHP_EOL;
 		$script .= PHP_EOL;
 		return $script;
 	}
 
 	protected function getFilterExpression() {
-		return '/^"' . preg_quote($this->config->getSystemValue('dbtableprefix', 'oc_')) . '/';
+		return '/^"'.preg_quote($this->config->getSystemValue('dbtableprefix', 'oc_')).'/';
 	}
 }

apps/files_sharing/lib/Command/CleanupRemoteStorages.php

Spacing +5 added lines, -5 removed lines

@@ -69,11 +69,11 @@ 
 	public function execute(InputInterface $input, OutputInterface $output): int {
 		$remoteStorages = $this->getRemoteStorages();
 
-		$output->writeln(count($remoteStorages) . ' remote storage(s) need(s) to be checked');
+		$output->writeln(count($remoteStorages).' remote storage(s) need(s) to be checked');
 
 		$remoteShareIds = $this->getRemoteShareIds();
 
-		$output->writeln(count($remoteShareIds) . ' remote share(s) exist');
+		$output->writeln(count($remoteShareIds).' remote share(s) exist');
 
 		foreach ($remoteShareIds as $id => $remoteShareId) {
 			if (isset($remoteStorages[$remoteShareId])) {
@@ -151,13 +151,13 @@ 
 			->where($queryBuilder->expr()->like(
 				'id',
 				// match all 'shared::' + 32 characters storages
-				$queryBuilder->createNamedParameter($this->connection->escapeLikeParameter('shared::') . str_repeat('_', 32)),
+				$queryBuilder->createNamedParameter($this->connection->escapeLikeParameter('shared::').str_repeat('_', 32)),
 				IQueryBuilder::PARAM_STR)
 			)
 			->andWhere($queryBuilder->expr()->notLike(
 				'id',
 				// but not the ones starting with a '/', they are for normal shares
-				$queryBuilder->createNamedParameter($this->connection->escapeLikeParameter('shared::/') . '%'),
+				$queryBuilder->createNamedParameter($this->connection->escapeLikeParameter('shared::/').'%'),
 				IQueryBuilder::PARAM_STR)
 			)->orderBy('numeric_id');
 		$query = $queryBuilder->execute();
@@ -183,7 +183,7 @@ 
 			$cloudId = $this->cloudIdManager->getCloudId($row['owner'], $row['remote']);
 			$remote = $cloudId->getRemote();
 
-			$remoteShareIds[$row['id']] = 'shared::' . md5($row['share_token'] . '@' . $remote);
+			$remoteShareIds[$row['id']] = 'shared::'.md5($row['share_token'].'@'.$remote);
 		}
 
 		return $remoteShareIds;

lib/private/Encryption/Update.php

Spacing +7 added lines, -7 removed lines

@@ -90,7 +90,7 @@ 
 			if ($params['itemType'] === 'file' || $params['itemType'] === 'folder') {
 				$path = Filesystem::getPath($params['fileSource']);
 				[$owner, $ownerPath] = $this->getOwnerPath($path);
-				$absPath = '/' . $owner . '/files/' . $ownerPath;
+				$absPath = '/'.$owner.'/files/'.$ownerPath;
 				$this->update($absPath);
 			}
 		}
@@ -106,7 +106,7 @@ 
 			if ($params['itemType'] === 'file' || $params['itemType'] === 'folder') {
 				$path = Filesystem::getPath($params['fileSource']);
 				[$owner, $ownerPath] = $this->getOwnerPath($path);
-				$absPath = '/' . $owner . '/files/' . $ownerPath;
+				$absPath = '/'.$owner.'/files/'.$ownerPath;
 				$this->update($absPath);
 			}
 		}
@@ -120,7 +120,7 @@ 
 	 */
 	public function postRestore($params) {
 		if ($this->encryptionManager->isEnabled()) {
-			$path = Filesystem::normalizePath('/' . $this->uid . '/files/' . $params['filePath']);
+			$path = Filesystem::normalizePath('/'.$this->uid.'/files/'.$params['filePath']);
 			$this->update($path);
 		}
 	}
@@ -139,7 +139,7 @@ 
 			dirname($source) !== dirname($target)
 		) {
 			[$owner, $ownerPath] = $this->getOwnerPath($target);
-			$absPath = '/' . $owner . '/files/' . $ownerPath;
+			$absPath = '/'.$owner.'/files/'.$ownerPath;
 			$this->update($absPath);
 		}
 	}
@@ -154,10 +154,10 @@ 
 	protected function getOwnerPath($path) {
 		$info = Filesystem::getFileInfo($path);
 		$owner = Filesystem::getOwner($path);
-		$view = new View('/' . $owner . '/files');
+		$view = new View('/'.$owner.'/files');
 		$path = $view->getPath($info->getId());
 		if ($path === null) {
-			throw new InvalidArgumentException('No file found for ' . $info->getId());
+			throw new InvalidArgumentException('No file found for '.$info->getId());
 		}
 
 		return [$owner, $path];
@@ -193,7 +193,7 @@ 
 				$encryptionModule->update($file, $this->uid, $usersSharing);
 			} catch (GenericEncryptionException $e) {
 				// If the update of an individual file fails e.g. due to a corrupt key we should continue the operation and just log the failure
-				$this->logger->error('Failed to update encryption module for ' . $this->uid . ' ' . $file, [ 'exception' => $e ]);
+				$this->logger->error('Failed to update encryption module for '.$this->uid.' '.$file, ['exception' => $e]);
 			}
 		}
 	}

core/Command/Preview/ResetRenderedTexts.php

Spacing +6 added lines, -6 removed lines

@@ -95,7 +95,7 @@ 
 		$avatarsToDeleteCount = 0;
 
 		foreach ($this->getAvatarsToDelete() as [$userId, $avatar]) {
-			$output->writeln('Deleting avatar for ' . $userId, OutputInterface::VERBOSITY_VERBOSE);
+			$output->writeln('Deleting avatar for '.$userId, OutputInterface::VERBOSITY_VERBOSE);
 
 			$avatarsToDeleteCount++;
 
@@ -112,7 +112,7 @@ 
 			}
 		}
 
-		$output->writeln('Deleted ' . $avatarsToDeleteCount . ' avatars');
+		$output->writeln('Deleted '.$avatarsToDeleteCount.' avatars');
 		$output->writeln('');
 	}
 
@@ -130,7 +130,7 @@ 
 		$previewsToDeleteCount = 0;
 
 		foreach ($this->getPreviewsToDelete() as ['name' => $previewFileId, 'path' => $filePath]) {
-			$output->writeln('Deleting previews for ' . $filePath, OutputInterface::VERBOSITY_VERBOSE);
+			$output->writeln('Deleting previews for '.$filePath, OutputInterface::VERBOSITY_VERBOSE);
 
 			$previewsToDeleteCount++;
 
@@ -139,7 +139,7 @@ 
 			}
 
 			try {
-				$preview = $this->previewFolder->getFolder((string)$previewFileId);
+				$preview = $this->previewFolder->getFolder((string) $previewFileId);
 				$preview->delete();
 			} catch (NotFoundException $e) {
 				// continue
@@ -148,7 +148,7 @@ 
 			}
 		}
 
-		$output->writeln('Deleted ' . $previewsToDeleteCount . ' previews');
+		$output->writeln('Deleted '.$previewsToDeleteCount.' previews');
 	}
 
 	// Copy pasted and adjusted from
@@ -171,7 +171,7 @@ 
 		 * We take the md5 of the name (fileid) and split the first 7 chars. That way
 		 * there are not a gazillion files in the root of the preview appdata.
 		 */
-		$like = $this->connection->escapeLikeParameter($data['path']) . '/_/_/_/_/_/_/_/%';
+		$like = $this->connection->escapeLikeParameter($data['path']).'/_/_/_/_/_/_/_/%';
 
 		$qb = $this->connection->getQueryBuilder();
 		$qb->select('a.name', 'b.path')

lib/private/Search/Provider/File.php

Spacing +3 added lines, -3 removed lines

@@ -65,9 +65,9 @@
 		}
 		$userFolder = $rootFolder->getUserFolder($user->getUID());
 		$fileQuery = new SearchQuery(
-			new SearchComparison(ISearchComparison::COMPARE_LIKE, 'name', '%' . $query . '%'),
-			(int)$limit,
-			(int)$offset,
+			new SearchComparison(ISearchComparison::COMPARE_LIKE, 'name', '%'.$query.'%'),
+			(int) $limit,
+			(int) $offset,
 			[
 				new SearchOrder(ISearchOrder::DIRECTION_DESCENDING, 'mtime'),
 			],

apps/files/lib/Search/FilesSearchProvider.php

Spacing +4 added lines, -4 removed lines

@@ -103,9 +103,9 @@ 
 	public function search(IUser $user, ISearchQuery $query): SearchResult {
 		$userFolder = $this->rootFolder->getUserFolder($user->getUID());
 		$fileQuery = new SearchQuery(
-			new SearchComparison(ISearchComparison::COMPARE_LIKE, 'name', '%' . $query->getTerm() . '%'),
+			new SearchComparison(ISearchComparison::COMPARE_LIKE, 'name', '%'.$query->getTerm().'%'),
 			$query->getLimit(),
-			(int)$query->getCursor(),
+			(int) $query->getCursor(),
 			$query->getSortOrder() === ISearchQuery::SORT_DATE_DESC ? [
 				new SearchOrder(ISearchOrder::DIRECTION_DESCENDING, 'mtime'),
 			] : [],
@@ -114,7 +114,7 @@ 
 
 		return SearchResult::paginated(
 			$this->l10n->t('Files'),
-			array_map(function (Node $result) use ($userFolder) {
+			array_map(function(Node $result) use ($userFolder) {
 				// Generate thumbnail url
 				$thumbnailUrl = $this->urlGenerator->linkToRouteAbsolute('core.Preview.getPreviewByFileId', ['x' => 32, 'y' => 32, 'fileId' => $result->getId()]);
 				$path = $userFolder->getRelativePath($result->getPath());
@@ -133,7 +133,7 @@ 
 					$this->urlGenerator->getAbsoluteURL($link),
 					$result->getMimetype() === FileInfo::MIMETYPE_FOLDER ? 'icon-folder' : $this->mimeTypeDetector->mimeTypeIcon($result->getMimetype())
 				);
-				$searchResultEntry->addAttribute('fileId', (string)$result->getId());
+				$searchResultEntry->addAttribute('fileId', (string) $result->getId());
 				$searchResultEntry->addAttribute('path', $path);
 				return $searchResultEntry;
 			}, $userFolder->search($fileQuery)),