style: update codestyle for coding-standard 1.2.3

Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>

24 files changed, 79 insertions, 79 deletions

diff --git a/apps/files/lib/Activity/Provider.php b/apps/files/lib/Activity/Provider.php
index 5c574337933..aa3ef062a16 100644
--- a/apps/files/lib/Activity/Provider.php
+++ b/apps/files/lib/Activity/Provider.php
@@ -351,7 +351,7 @@ class Provider implements IProvider {
 	protected function getFile($parameter, ?IEvent $event = null) {
 		if (is_array($parameter)) {
 			$path = reset($parameter);
-			$id = (string) key($parameter);
+			$id = (string)key($parameter);
 		} elseif ($event !== null) {
 			// Legacy from before ownCloud 8.2
 			$path = $parameter;
diff --git a/apps/files/lib/Activity/Settings/FavoriteAction.php b/apps/files/lib/Activity/Settings/FavoriteAction.php
index 583edd5d9bc..b572a9546e0 100644
--- a/apps/files/lib/Activity/Settings/FavoriteAction.php
+++ b/apps/files/lib/Activity/Settings/FavoriteAction.php
@@ -24,8 +24,8 @@ class FavoriteAction extends FileActivitySettings {
 
 	/**
 	 * @return int whether the filter should be rather on the top or bottom of
-	 * the admin section. The filters are arranged in ascending order of the
-	 * priority values. It is required to return a value between 0 and 100.
+	 *             the admin section. The filters are arranged in ascending order of the
+	 *             priority values. It is required to return a value between 0 and 100.
 	 * @since 11.0.0
 	 */
 	public function getPriority() {
diff --git a/apps/files/lib/Activity/Settings/FileChanged.php b/apps/files/lib/Activity/Settings/FileChanged.php
index dfd6183e20b..5af87456550 100644
--- a/apps/files/lib/Activity/Settings/FileChanged.php
+++ b/apps/files/lib/Activity/Settings/FileChanged.php
@@ -24,8 +24,8 @@ class FileChanged extends FileActivitySettings {
 
 	/**
 	 * @return int whether the filter should be rather on the top or bottom of
-	 * the admin section. The filters are arranged in ascending order of the
-	 * priority values. It is required to return a value between 0 and 100.
+	 *             the admin section. The filters are arranged in ascending order of the
+	 *             priority values. It is required to return a value between 0 and 100.
 	 * @since 11.0.0
 	 */
 	public function getPriority() {
diff --git a/apps/files/lib/Activity/Settings/FileFavoriteChanged.php b/apps/files/lib/Activity/Settings/FileFavoriteChanged.php
index f1412331d30..e4ac572f3f3 100644
--- a/apps/files/lib/Activity/Settings/FileFavoriteChanged.php
+++ b/apps/files/lib/Activity/Settings/FileFavoriteChanged.php
@@ -24,8 +24,8 @@ class FileFavoriteChanged extends FileActivitySettings {
 
 	/**
 	 * @return int whether the filter should be rather on the top or bottom of
-	 * the admin section. The filters are arranged in ascending order of the
-	 * priority values. It is required to return a value between 0 and 100.
+	 *             the admin section. The filters are arranged in ascending order of the
+	 *             priority values. It is required to return a value between 0 and 100.
 	 * @since 11.0.0
 	 */
 	public function getPriority() {
diff --git a/apps/files/lib/Collaboration/Resources/ResourceProvider.php b/apps/files/lib/Collaboration/Resources/ResourceProvider.php
index 439d3baf7f6..0ac54afad7d 100644
--- a/apps/files/lib/Collaboration/Resources/ResourceProvider.php
+++ b/apps/files/lib/Collaboration/Resources/ResourceProvider.php
@@ -39,13 +39,13 @@ class ResourceProvider implements IProvider {
 	}
 
 	private function getNode(IResource $resource): ?Node {
-		if (isset($this->nodes[(int) $resource->getId()])) {
-			return $this->nodes[(int) $resource->getId()];
+		if (isset($this->nodes[(int)$resource->getId()])) {
+			return $this->nodes[(int)$resource->getId()];
 		}
-		$node = $this->rootFolder->getFirstNodeById((int) $resource->getId());
+		$node = $this->rootFolder->getFirstNodeById((int)$resource->getId());
 		if ($node) {
-			$this->nodes[(int) $resource->getId()] = $node;
-			return $this->nodes[(int) $resource->getId()];
+			$this->nodes[(int)$resource->getId()] = $node;
+			return $this->nodes[(int)$resource->getId()];
 		}
 		return null;
 	}
@@ -56,8 +56,8 @@ class ResourceProvider implements IProvider {
 	 * @since 16.0.0
 	 */
 	public function getResourceRichObject(IResource $resource): array {
-		if (isset($this->nodes[(int) $resource->getId()])) {
-			$node = $this->nodes[(int) $resource->getId()]->getPath();
+		if (isset($this->nodes[(int)$resource->getId()])) {
+			$node = $this->nodes[(int)$resource->getId()]->getPath();
 		} else {
 			$node = $this->getNode($resource);
 		}
@@ -95,10 +95,10 @@ class ResourceProvider implements IProvider {
 		}
 
 		$userFolder = $this->rootFolder->getUserFolder($user->getUID());
-		$node = $userFolder->getById((int) $resource->getId());
+		$node = $userFolder->getById((int)$resource->getId());
 
 		if ($node) {
-			$this->nodes[(int) $resource->getId()] = $node;
+			$this->nodes[(int)$resource->getId()] = $node;
 			return true;
 		}
 
diff --git a/apps/files/lib/Command/Copy.php b/apps/files/lib/Command/Copy.php
index 9981497232f..e51a1689907 100644
--- a/apps/files/lib/Command/Copy.php
+++ b/apps/files/lib/Command/Copy.php
@@ -30,10 +30,10 @@ class Copy extends Command {
 		$this
 			->setName('files:copy')
 			->setDescription('Copy a file or folder')
-			->addArgument('source', InputArgument::REQUIRED, "Source file id or path")
-			->addArgument('target', InputArgument::REQUIRED, "Target path")
+			->addArgument('source', InputArgument::REQUIRED, 'Source file id or path')
+			->addArgument('target', InputArgument::REQUIRED, 'Target path')
 			->addOption('force', 'f', InputOption::VALUE_NONE, "Don't ask for confirmation and don't output any warnings")
-			->addOption('no-target-directory', 'T', InputOption::VALUE_NONE, "When target path is folder, overwrite the folder instead of copying into the folder");
+			->addOption('no-target-directory', 'T', InputOption::VALUE_NONE, 'When target path is folder, overwrite the folder instead of copying into the folder');
 	}
 
 	public function execute(InputInterface $input, OutputInterface $output): int {
@@ -98,7 +98,7 @@ class Copy extends Command {
 				/** @var QuestionHelper $helper */
 				$helper = $this->getHelper('question');
 
-				$question = new ConfirmationQuestion("<info>" . $targetInput . "</info> already exists, overwrite? [y/N] ", false);
+				$question = new ConfirmationQuestion('<info>' . $targetInput . '</info> already exists, overwrite? [y/N] ', false);
 				if (!$helper->ask($input, $output, $question)) {
 					return 1;
 				}
diff --git a/apps/files/lib/Command/Delete.php b/apps/files/lib/Command/Delete.php
index 32aa52aed2f..f59b636079b 100644
--- a/apps/files/lib/Command/Delete.php
+++ b/apps/files/lib/Command/Delete.php
@@ -30,7 +30,7 @@ class Delete extends Command {
 		$this
 			->setName('files:delete')
 			->setDescription('Delete a file or folder')
-			->addArgument('file', InputArgument::REQUIRED, "File id or path")
+			->addArgument('file', InputArgument::REQUIRED, 'File id or path')
 			->addOption('force', 'f', InputOption::VALUE_NONE, "Don't ask for configuration and don't output any warnings");
 	}
 
@@ -59,30 +59,30 @@ class Delete extends Command {
 					return self::SUCCESS;
 				} else {
 					$node = $storage->getShare()->getNode();
-					$output->writeln("");
+					$output->writeln('');
 				}
 			}
 
 			$filesByUsers = $this->fileUtils->getFilesByUser($node);
 			if (count($filesByUsers) > 1) {
-				$output->writeln("Warning: the provided file is accessible by more than one user");
-				$output->writeln("  all of the following users will lose access to the file when deleted:");
-				$output->writeln("");
+				$output->writeln('Warning: the provided file is accessible by more than one user');
+				$output->writeln('  all of the following users will lose access to the file when deleted:');
+				$output->writeln('');
 				foreach ($filesByUsers as $user => $filesByUser) {
-					$output->writeln($user . ":");
+					$output->writeln($user . ':');
 					foreach($filesByUser as $file) {
-						$output->writeln("  - " . $file->getPath());
+						$output->writeln('  - ' . $file->getPath());
 					}
 				}
-				$output->writeln("");
+				$output->writeln('');
 			}
 
 			if ($node instanceof Folder) {
 				$maybeContents = " and all it's contents";
 			} else {
-				$maybeContents = "";
+				$maybeContents = '';
 			}
-			$question = new ConfirmationQuestion("Delete " . $node->getPath() . $maybeContents . "? [y/N] ", false);
+			$question = new ConfirmationQuestion('Delete ' . $node->getPath() . $maybeContents . '? [y/N] ', false);
 			$deleteConfirmed = $helper->ask($input, $output, $question);
 		}
 
@@ -90,7 +90,7 @@ class Delete extends Command {
 			if ($node->isDeletable()) {
 				$node->delete();
 			} else {
-				$output->writeln("<error>File cannot be deleted, insufficient permissions.</error>");
+				$output->writeln('<error>File cannot be deleted, insufficient permissions.</error>');
 			}
 		}
 
diff --git a/apps/files/lib/Command/DeleteOrphanedFiles.php b/apps/files/lib/Command/DeleteOrphanedFiles.php
index b7101c07258..50031da4f63 100644
--- a/apps/files/lib/Command/DeleteOrphanedFiles.php
+++ b/apps/files/lib/Command/DeleteOrphanedFiles.php
@@ -53,7 +53,7 @@ class DeleteOrphanedFiles extends Command {
 			$result = $query->execute();
 			while ($row = $result->fetch()) {
 				$deletedInLastChunk++;
-				$deletedEntries += $deleteQuery->setParameter('objectid', (int) $row['fileid'])
+				$deletedEntries += $deleteQuery->setParameter('objectid', (int)$row['fileid'])
 					->execute();
 			}
 			$result->closeCursor();
@@ -120,7 +120,7 @@ class DeleteOrphanedFiles extends Command {
 			$result = $query->execute();
 			while ($row = $result->fetch()) {
 				$deletedInLastChunk++;
-				$deletedEntries += $deleteQuery->setParameter('storageid', (int) $row['storage_id'])
+				$deletedEntries += $deleteQuery->setParameter('storageid', (int)$row['storage_id'])
 					->execute();
 			}
 			$result->closeCursor();
diff --git a/apps/files/lib/Command/Get.php b/apps/files/lib/Command/Get.php
index 0d901fe8250..60e028f615e 100644
--- a/apps/files/lib/Command/Get.php
+++ b/apps/files/lib/Command/Get.php
@@ -26,8 +26,8 @@ class Get extends Command {
 		$this
 			->setName('files:get')
 			->setDescription('Get the contents of a file')
-			->addArgument('file', InputArgument::REQUIRED, "Source file id or Nextcloud path")
-			->addArgument('output', InputArgument::OPTIONAL, "Target local file to output to, defaults to STDOUT");
+			->addArgument('file', InputArgument::REQUIRED, 'Source file id or Nextcloud path')
+			->addArgument('output', InputArgument::OPTIONAL, 'Target local file to output to, defaults to STDOUT');
 	}
 
 	public function execute(InputInterface $input, OutputInterface $output): int {
@@ -48,7 +48,7 @@ class Get extends Command {
 		$isTTY = stream_isatty(STDOUT);
 		if ($outputName === null && $isTTY && $node->getMimePart() !== 'text') {
 			$output->writeln([
-				"<error>Warning: Binary output can mess up your terminal</error>",
+				'<error>Warning: Binary output can mess up your terminal</error>',
 				"         Use <info>occ files:get $fileInput -</info> to output it to the terminal anyway",
 				"         Or <info>occ files:get $fileInput <FILE></info> to save to a file instead"
 			]);
diff --git a/apps/files/lib/Command/Move.php b/apps/files/lib/Command/Move.php
index dd1cd1aae76..cd9e56f8e29 100644
--- a/apps/files/lib/Command/Move.php
+++ b/apps/files/lib/Command/Move.php
@@ -31,8 +31,8 @@ class Move extends Command {
 		$this
 			->setName('files:move')
 			->setDescription('Move a file or folder')
-			->addArgument('source', InputArgument::REQUIRED, "Source file id or path")
-			->addArgument('target', InputArgument::REQUIRED, "Target path")
+			->addArgument('source', InputArgument::REQUIRED, 'Source file id or path')
+			->addArgument('target', InputArgument::REQUIRED, 'Target path')
 			->addOption('force', 'f', InputOption::VALUE_NONE, "Don't ask for configuration and don't output any warnings");
 	}
 
@@ -88,7 +88,7 @@ class Move extends Command {
 				/** @var QuestionHelper $helper */
 				$helper = $this->getHelper('question');
 
-				$question = new ConfirmationQuestion("<info>" . $targetInput . "</info> already exists, overwrite? [y/N] ", false);
+				$question = new ConfirmationQuestion('<info>' . $targetInput . '</info> already exists, overwrite? [y/N] ', false);
 				if (!$helper->ask($input, $output, $question)) {
 					return 1;
 				}
diff --git a/apps/files/lib/Command/Object/Delete.php b/apps/files/lib/Command/Object/Delete.php
index f9fc3a58229..07613ecc616 100644
--- a/apps/files/lib/Command/Object/Delete.php
+++ b/apps/files/lib/Command/Object/Delete.php
@@ -27,13 +27,13 @@ class Delete extends Command {
 		$this
 			->setName('files:object:delete')
 			->setDescription('Delete an object from the object store')
-			->addArgument('object', InputArgument::REQUIRED, "Object to delete")
+			->addArgument('object', InputArgument::REQUIRED, 'Object to delete')
 			->addOption('bucket', 'b', InputOption::VALUE_REQUIRED, "Bucket to delete the object from, only required in cases where it can't be determined from the config");
 	}
 
 	public function execute(InputInterface $input, OutputInterface $output): int {
 		$object = $input->getArgument('object');
-		$objectStore = $this->objectUtils->getObjectStore($input->getOption("bucket"), $output);
+		$objectStore = $this->objectUtils->getObjectStore($input->getOption('bucket'), $output);
 		if (!$objectStore) {
 			return -1;
 		}
@@ -41,7 +41,7 @@ class Delete extends Command {
 		if ($fileId = $this->objectUtils->objectExistsInDb($object)) {
 			$output->writeln("<error>Warning, object $object belongs to an existing file, deleting the object will lead to unexpected behavior if not replaced</error>");
 			$output->writeln("  Note: use <info>occ files:delete $fileId</info> to delete the file cleanly or <info>occ info:file $fileId</info> for more information about the file");
-			$output->writeln("");
+			$output->writeln('');
 		}
 
 		if (!$objectStore->objectExists($object)) {
diff --git a/apps/files/lib/Command/Object/Get.php b/apps/files/lib/Command/Object/Get.php
index 054278211f6..c32de020c5a 100644
--- a/apps/files/lib/Command/Object/Get.php
+++ b/apps/files/lib/Command/Object/Get.php
@@ -25,15 +25,15 @@ class Get extends Command {
 		$this
 			->setName('files:object:get')
 			->setDescription('Get the contents of an object')
-			->addArgument('object', InputArgument::REQUIRED, "Object to get")
-			->addArgument('output', InputArgument::REQUIRED, "Target local file to output to, use - for STDOUT")
+			->addArgument('object', InputArgument::REQUIRED, 'Object to get')
+			->addArgument('output', InputArgument::REQUIRED, 'Target local file to output to, use - for STDOUT')
 			->addOption('bucket', 'b', InputOption::VALUE_REQUIRED, "Bucket to get the object from, only required in cases where it can't be determined from the config");
 	}
 
 	public function execute(InputInterface $input, OutputInterface $output): int {
 		$object = $input->getArgument('object');
 		$outputName = $input->getArgument('output');
-		$objectStore = $this->objectUtils->getObjectStore($input->getOption("bucket"), $output);
+		$objectStore = $this->objectUtils->getObjectStore($input->getOption('bucket'), $output);
 		if (!$objectStore) {
 			return self::FAILURE;
 		}
diff --git a/apps/files/lib/Command/Object/ObjectUtil.php b/apps/files/lib/Command/Object/ObjectUtil.php
index 7021c815490..c4ab59608fb 100644
--- a/apps/files/lib/Command/Object/ObjectUtil.php
+++ b/apps/files/lib/Command/Object/ObjectUtil.php
@@ -41,19 +41,19 @@ class ObjectUtil {
 	public function getObjectStore(?string $bucket, OutputInterface $output): ?IObjectStore {
 		$config = $this->getObjectStoreConfig();
 		if (!$config) {
-			$output->writeln("<error>Instance is not using primary object store</error>");
+			$output->writeln('<error>Instance is not using primary object store</error>');
 			return null;
 		}
 		if ($config['multibucket'] && !$bucket) {
-			$output->writeln("<error>--bucket option required</error> because <info>multi bucket</info> is enabled.");
+			$output->writeln('<error>--bucket option required</error> because <info>multi bucket</info> is enabled.');
 			return null;
 		}
 
 		if (!isset($config['arguments'])) {
-			throw new \Exception("no arguments configured for object store configuration");
+			throw new \Exception('no arguments configured for object store configuration');
 		}
 		if (!isset($config['class'])) {
-			throw new \Exception("no class configured for object store configuration");
+			throw new \Exception('no class configured for object store configuration');
 		}
 
 		if ($bucket) {
@@ -65,7 +65,7 @@ class ObjectUtil {
 
 		$store = new $config['class']($config['arguments']);
 		if (!$store instanceof IObjectStore) {
-			throw new \Exception("configured object store class is not an object store implementation");
+			throw new \Exception('configured object store class is not an object store implementation');
 		}
 		return $store;
 	}
diff --git a/apps/files/lib/Command/Object/Put.php b/apps/files/lib/Command/Object/Put.php
index 34e4ceb854b..8516eb51183 100644
--- a/apps/files/lib/Command/Object/Put.php
+++ b/apps/files/lib/Command/Object/Put.php
@@ -29,8 +29,8 @@ class Put extends Command {
 		$this
 			->setName('files:object:put')
 			->setDescription('Write a file to the object store')
-			->addArgument('input', InputArgument::REQUIRED, "Source local path, use - to read from STDIN")
-			->addArgument('object', InputArgument::REQUIRED, "Object to write")
+			->addArgument('input', InputArgument::REQUIRED, 'Source local path, use - to read from STDIN')
+			->addArgument('object', InputArgument::REQUIRED, 'Object to write')
 			->addOption('bucket', 'b', InputOption::VALUE_REQUIRED, "Bucket where to store the object, only required in cases where it can't be determined from the config");
 		;
 	}
@@ -38,7 +38,7 @@ class Put extends Command {
 	public function execute(InputInterface $input, OutputInterface $output): int {
 		$object = $input->getArgument('object');
 		$inputName = (string)$input->getArgument('input');
-		$objectStore = $this->objectUtils->getObjectStore($input->getOption("bucket"), $output);
+		$objectStore = $this->objectUtils->getObjectStore($input->getOption('bucket'), $output);
 		if (!$objectStore) {
 			return -1;
 		}
@@ -46,11 +46,11 @@ class Put extends Command {
 		if ($fileId = $this->objectUtils->objectExistsInDb($object)) {
 			$output->writeln("<error>Warning, object $object belongs to an existing file, overwriting the object contents can lead to unexpected behavior.</error>");
 			$output->writeln("You can use <info>occ files:put $inputName $fileId</info> to write to the file safely.");
-			$output->writeln("");
+			$output->writeln('');
 
 			/** @var QuestionHelper $helper */
 			$helper = $this->getHelper('question');
-			$question = new ConfirmationQuestion("Write to the object anyway? [y/N] ", false);
+			$question = new ConfirmationQuestion('Write to the object anyway? [y/N] ', false);
 			if (!$helper->ask($input, $output, $question)) {
 				return -1;
 			}
diff --git a/apps/files/lib/Command/Put.php b/apps/files/lib/Command/Put.php
index f72fa902346..fd9d75db78c 100644
--- a/apps/files/lib/Command/Put.php
+++ b/apps/files/lib/Command/Put.php
@@ -29,8 +29,8 @@ class Put extends Command {
 		$this
 			->setName('files:put')
 			->setDescription('Write contents of a file')
-			->addArgument('input', InputArgument::REQUIRED, "Source local path, use - to read from STDIN")
-			->addArgument('file', InputArgument::REQUIRED, "Target Nextcloud file path to write to or fileid of existing file");
+			->addArgument('input', InputArgument::REQUIRED, 'Source local path, use - to read from STDIN')
+			->addArgument('file', InputArgument::REQUIRED, 'Target Nextcloud file path to write to or fileid of existing file');
 	}
 
 	public function execute(InputInterface $input, OutputInterface $output): int {
diff --git a/apps/files/lib/Command/RepairTree.php b/apps/files/lib/Command/RepairTree.php
index 99e92741a2e..622ccba48a3 100644
--- a/apps/files/lib/Command/RepairTree.php
+++ b/apps/files/lib/Command/RepairTree.php
@@ -33,7 +33,7 @@ class RepairTree extends Command {
 		$rows = $this->findBrokenTreeBits();
 		$fix = !$input->getOption('dry-run');
 
-		$output->writeln("Found " . count($rows) . " file entries with an invalid path");
+		$output->writeln('Found ' . count($rows) . ' file entries with an invalid path');
 
 		if ($fix) {
 			$this->connection->beginTransaction();
diff --git a/apps/files/lib/Command/ScanAppData.php b/apps/files/lib/Command/ScanAppData.php
index 61f4b6f3ba2..5360d38bdb6 100644
--- a/apps/files/lib/Command/ScanAppData.php
+++ b/apps/files/lib/Command/ScanAppData.php
@@ -210,7 +210,7 @@ class ScanAppData extends Base {
 	}
 
 	protected function reconnectToDatabase(OutputInterface $output): Connection {
-		/** @var Connection $connection*/
+		/** @var Connection $connection */
 		$connection = \OC::$server->get(Connection::class);
 		try {
 			$connection->close();
diff --git a/apps/files/lib/Command/TransferOwnership.php b/apps/files/lib/Command/TransferOwnership.php
index db4c19dc54e..edc73e62c38 100644
--- a/apps/files/lib/Command/TransferOwnership.php
+++ b/apps/files/lib/Command/TransferOwnership.php
@@ -78,12 +78,12 @@ class TransferOwnership extends Command {
 		$destinationUserObject = $this->userManager->get($input->getArgument('destination-user'));
 
 		if (!$sourceUserObject instanceof IUser) {
-			$output->writeln("<error>Unknown source user " . $input->getArgument('source-user') . "</error>");
+			$output->writeln('<error>Unknown source user ' . $input->getArgument('source-user') . '</error>');
 			return self::FAILURE;
 		}
 
 		if (!$destinationUserObject instanceof IUser) {
-			$output->writeln("<error>Unknown destination user " . $input->getArgument('destination-user') . "</error>");
+			$output->writeln('<error>Unknown destination user ' . $input->getArgument('destination-user') . '</error>');
 			return self::FAILURE;
 		}
 
@@ -105,7 +105,7 @@ class TransferOwnership extends Command {
 					}
 					break;
 				default:
-					$output->writeln("<error>Option --transfer-incoming-shares: wrong usage. Transfer aborted.</error>");
+					$output->writeln('<error>Option --transfer-incoming-shares: wrong usage. Transfer aborted.</error>');
 					return self::FAILURE;
 			}
 
@@ -119,7 +119,7 @@ class TransferOwnership extends Command {
 				$includeIncoming
 			);
 		} catch (TransferOwnershipException $e) {
-			$output->writeln("<error>" . $e->getMessage() . "</error>");
+			$output->writeln('<error>' . $e->getMessage() . '</error>');
 			return $e->getCode() !== 0 ? $e->getCode() : self::FAILURE;
 		}
 
diff --git a/apps/files/lib/Controller/ViewController.php b/apps/files/lib/Controller/ViewController.php
index 16d0c3ea1f0..aa9aa81a814 100644
--- a/apps/files/lib/Controller/ViewController.php
+++ b/apps/files/lib/Controller/ViewController.php
@@ -89,7 +89,7 @@ class ViewController extends Controller {
 
 		// This is the entry point from the `/f/{fileid}` URL which is hardcoded in the server.
 		try {
-			return $this->redirectToFile((int) $fileid);
+			return $this->redirectToFile((int)$fileid);
 		} catch (NotFoundException $e) {
 			return new RedirectResponse($this->urlGenerator->linkToRoute('files.view.index', ['fileNotFound' => true]));
 		}
@@ -134,7 +134,7 @@ class ViewController extends Controller {
 	public function index($dir = '', $view = '', $fileid = null, $fileNotFound = false) {
 		if ($fileid !== null && $view !== 'trashbin') {
 			try {
-				return $this->redirectToFileIfInTrashbin((int) $fileid);
+				return $this->redirectToFileIfInTrashbin((int)$fileid);
 			} catch (NotFoundException $e) {
 			}
 		}
@@ -163,14 +163,14 @@ class ViewController extends Controller {
 		// in the correct folder
 		if ($fileid && $dir !== '') {
 			$baseFolder = $this->rootFolder->getUserFolder($userId);
-			$nodes = $baseFolder->getById((int) $fileid);
+			$nodes = $baseFolder->getById((int)$fileid);
 			if (!empty($nodes)) {
 				$nodePath = $baseFolder->getRelativePath($nodes[0]->getPath());
 				$relativePath = $nodePath ? dirname($nodePath) : '';
 				// If the requested path does not contain the file id
 				// or if the requested path is not the file id itself
 				if (count($nodes) === 1 && $relativePath !== $dir && $nodePath !== $dir) {
-					return $this->redirectToFile((int) $fileid);
+					return $this->redirectToFile((int)$fileid);
 				}
 			} else { // fileid does not exist anywhere
 				$fileNotFound = true;
diff --git a/apps/files/lib/Listener/DeclarativeSettingsSetValueEventListener.php b/apps/files/lib/Listener/DeclarativeSettingsSetValueEventListener.php
index 43d01563c26..a3be2b9141e 100644
--- a/apps/files/lib/Listener/DeclarativeSettingsSetValueEventListener.php
+++ b/apps/files/lib/Listener/DeclarativeSettingsSetValueEventListener.php
@@ -32,7 +32,7 @@ class DeclarativeSettingsSetValueEventListener implements IEventListener {
 
 		switch ($event->getFieldId()) {
 			case 'windows_support':
-				$this->service->setFilesWindowsSupport((bool) $event->getValue());
+				$this->service->setFilesWindowsSupport((bool)$event->getValue());
 				$event->stopPropagation();
 				break;
 		}
diff --git a/apps/files/lib/Listener/SyncLivePhotosListener.php b/apps/files/lib/Listener/SyncLivePhotosListener.php
index c74a8370818..049d7319b64 100644
--- a/apps/files/lib/Listener/SyncLivePhotosListener.php
+++ b/apps/files/lib/Listener/SyncLivePhotosListener.php
@@ -100,7 +100,7 @@ class SyncLivePhotosListener implements IEventListener {
 		$peerFileExtension = $peerFile->getExtension();
 		$targetName = $targetFile->getName();
 
-		if (!str_ends_with($targetName, "." . $sourceExtension)) {
+		if (!str_ends_with($targetName, '.' . $sourceExtension)) {
 			throw new AbortedEventException('Cannot change the extension of a Live Photo');
 		}
 
@@ -176,7 +176,7 @@ class SyncLivePhotosListener implements IEventListener {
 				unset($this->pendingDeletion[$peerFile->getId()]);
 				return;
 			} else {
-				throw new AbortedEventException("Cannot delete the video part of a live photo");
+				throw new AbortedEventException('Cannot delete the video part of a live photo');
 			}
 		} else {
 			$this->pendingDeletion[$deletedFile->getId()] = true;
diff --git a/apps/files/lib/Search/FilesSearchProvider.php b/apps/files/lib/Search/FilesSearchProvider.php
index 950c8b14ab6..4dfd9bc0d16 100644
--- a/apps/files/lib/Search/FilesSearchProvider.php
+++ b/apps/files/lib/Search/FilesSearchProvider.php
@@ -172,7 +172,7 @@ class FilesSearchProvider implements IFilteringProvider {
 		return new SearchQuery(
 			new SearchBinaryOperator(SearchBinaryOperator::OPERATOR_AND, $comparisons),
 			$query->getLimit(),
-			(int) $query->getCursor(),
+			(int)$query->getCursor(),
 			$query->getSortOrder() === ISearchQuery::SORT_DATE_DESC
 				? [new SearchOrder(ISearchOrder::DIRECTION_DESCENDING, 'mtime')]
 				: [],
diff --git a/apps/files/lib/Service/OwnershipTransferService.php b/apps/files/lib/Service/OwnershipTransferService.php
index a624df4a924..58595e4f0a9 100644
--- a/apps/files/lib/Service/OwnershipTransferService.php
+++ b/apps/files/lib/Service/OwnershipTransferService.php
@@ -78,7 +78,7 @@ class OwnershipTransferService {
 		// If encryption is on we have to ensure the user has logged in before and that all encryption modules are ready
 		if (($this->encryptionManager->isEnabled() && $destinationUser->getLastLogin() === 0)
 			|| !$this->encryptionManager->isReadyForUser($destinationUid)) {
-			throw new TransferOwnershipException("The target user is not ready to accept files. The user has at least to have logged in once.", 2);
+			throw new TransferOwnershipException('The target user is not ready to accept files. The user has at least to have logged in once.', 2);
 		}
 
 		// setup filesystem
@@ -117,7 +117,7 @@ class OwnershipTransferService {
 		}
 
 		if ($move && !$firstLogin && count($view->getDirectoryContent($finalTarget)) > 0) {
-			throw new TransferOwnershipException("Destination path does not exists or is not empty", 1);
+			throw new TransferOwnershipException('Destination path does not exists or is not empty', 1);
 		}
 
 
@@ -244,10 +244,10 @@ class OwnershipTransferService {
 
 		// no file is allowed to be encrypted
 		if (!empty($encryptedFiles)) {
-			$output->writeln("<error>Some files are encrypted - please decrypt them first.</error>");
+			$output->writeln('<error>Some files are encrypted - please decrypt them first.</error>');
 			foreach ($encryptedFiles as $encryptedFile) {
 				/** @var FileInfo $encryptedFile */
-				$output->writeln("  " . $encryptedFile->getPath());
+				$output->writeln('  ' . $encryptedFile->getPath());
 			}
 			throw new \Exception('Execution terminated.');
 		}
@@ -372,7 +372,7 @@ class OwnershipTransferService {
 			$finalTarget = $finalTarget . '/' . basename($sourcePath);
 		}
 		if ($view->rename($sourcePath, $finalTarget) === false) {
-			throw new TransferOwnershipException("Could not transfer files.", 1);
+			throw new TransferOwnershipException('Could not transfer files.', 1);
 		}
 		if (!is_dir("$sourceUid/files")) {
 			// because the files folder is moved away we need to recreate it
@@ -391,7 +391,7 @@ class OwnershipTransferService {
 		array $shares,
 		OutputInterface $output,
 	):void {
-		$output->writeln("Restoring shares ...");
+		$output->writeln('Restoring shares ...');
 		$progress = new ProgressBar($output, count($shares));
 		$rootFolder = \OCP\Server::get(IRootFolder::class);
 
@@ -459,7 +459,7 @@ class OwnershipTransferService {
 		string $path,
 		string $finalTarget,
 		bool $move): void {
-		$output->writeln("Restoring incoming shares ...");
+		$output->writeln('Restoring incoming shares ...');
 		$progress = new ProgressBar($output, count($sourceShares));
 		$prefix = "$destinationUid/files";
 		$finalShareTarget = '';
diff --git a/apps/files/lib/Service/TagService.php b/apps/files/lib/Service/TagService.php
index 4737938ab52..4bb43145e4a 100644
--- a/apps/files/lib/Service/TagService.php
+++ b/apps/files/lib/Service/TagService.php
@@ -53,7 +53,7 @@ class TagService {
 	 * replace the actual tag selection.
 	 *
 	 * @param string $path path
-	 * @param array  $tags array of tags
+	 * @param array $tags array of tags
 	 * @return array list of tags
 	 * @throws \OCP\Files\NotFoundException if the file does not exist
 	 */