nextcloud / server

lib/public/Files/Storage/IStorage.php

Spacing +1 added lines, -1 removed lines

@@ -322,7 +322,7 @@
 	 * @return array{url: ?string, expiration: ?int}|false
 	 * @since 33.0.0
 	 */
-	public function getDirectDownloadById(string $fileId): array|false;
+	public function getDirectDownloadById(string $fileId): array | false;
 
 	/**
 	 * @return void

lib/private/Lockdown/Filesystem/NullStorage.php

Spacing +6 added lines, -6 removed lines

@@ -45,7 +45,7 @@ 
 		throw new \OC\ForbiddenException('This request is not allowed to access the filesystem');
 	}
 
-	public function filetype(string $path): string|false {
+	public function filetype(string $path): string | false {
 		return ($path === '') ? 'dir' : false;
 	}
 
@@ -81,7 +81,7 @@ 
 		return $path === '';
 	}
 
-	public function filemtime(string $path): int|false {
+	public function filemtime(string $path): int | false {
 		return ($path === '') ? time() : false;
 	}
 
@@ -125,7 +125,7 @@ 
 		throw new \OC\ForbiddenException('This request is not allowed to access the filesystem');
 	}
 
-	public function getLocalFile(string $path): string|false {
+	public function getLocalFile(string $path): string | false {
 		return false;
 	}
 
@@ -141,11 +141,11 @@ 
 		return false;
 	}
 
-	public function getDirectDownload(string $path): array|false {
+	public function getDirectDownload(string $path): array | false {
 		return false;
 	}
 
-	public function getDirectDownloadById(string $fileId): array|false {
+	public function getDirectDownloadById(string $fileId): array | false {
 		return false;
 	}
 
@@ -161,7 +161,7 @@ 
 		return true;
 	}
 
-	public function getOwner(string $path): string|false {
+	public function getOwner(string $path): string | false {
 		return false;
 	}
 

lib/private/Files/ObjectStore/S3ObjectTrait.php

Spacing +9 added lines, -9 removed lines

@@ -40,11 +40,11 @@ 
 	 * @since 7.0.0
 	 */
 	public function readObject($urn) {
-		$fh = SeekableHttpStream::open(function ($range) use ($urn) {
+		$fh = SeekableHttpStream::open(function($range) use ($urn) {
 			$command = $this->getConnection()->getCommand('GetObject', [
 				'Bucket' => $this->bucket,
 				'Key' => $urn,
-				'Range' => 'bytes=' . $range,
+				'Range' => 'bytes='.$range,
 			] + $this->getSSECParameters());
 			$request = \Aws\serialize($command);
 			$headers = [];
@@ -84,9 +84,9 @@ 
 		$result = [];
 		foreach ($metadata as $key => $value) {
 			if (mb_check_encoding($value, 'ASCII')) {
-				$result['x-amz-meta-' . $key] = $value;
+				$result['x-amz-meta-'.$key] = $value;
 			} else {
-				$result['x-amz-meta-' . $key] = 'base64:' . base64_encode($value);
+				$result['x-amz-meta-'.$key] = 'base64:'.base64_encode($value);
 			}
 		}
 		return $result;
@@ -157,12 +157,12 @@ 
 					'Metadata' => $this->buildS3Metadata($metaData),
 					'StorageClass' => $this->storageClass,
 				] + $this->getSSECParameters(),
-				'before_upload' => function (Command $command) use (&$totalWritten) {
+				'before_upload' => function(Command $command) use (&$totalWritten) {
 					$totalWritten += $command['ContentLength'];
 				},
-				'before_complete' => function ($_command) use (&$totalWritten, $size, &$uploader, &$attempts) {
+				'before_complete' => function($_command) use (&$totalWritten, $size, &$uploader, &$attempts) {
 					if ($size !== null && $totalWritten != $size) {
-						$e = new \Exception('Incomplete multi part upload, expected ' . $size . ' bytes, wrote ' . $totalWritten);
+						$e = new \Exception('Incomplete multi part upload, expected '.$size.' bytes, wrote '.$totalWritten);
 						throw new MultipartUploadException($uploader->getState(), $e);
 					}
 				},
@@ -275,7 +275,7 @@ 
 			'Key' => $from,
 		] + $this->getSSECParameters());
 
-		$size = (int)($sourceMetadata->get('Size') ?? $sourceMetadata->get('ContentLength'));
+		$size = (int) ($sourceMetadata->get('Size') ?? $sourceMetadata->get('ContentLength'));
 
 		if ($this->useMultipartCopy && $size > $this->copySizeLimit) {
 			$copy = new MultipartCopy($this->getConnection(), [
@@ -308,7 +308,7 @@ 
 		}
 
 		try {
-			return (string)$this->getConnection()->createPresignedRequest($command, $expiration, [
+			return (string) $this->getConnection()->createPresignedRequest($command, $expiration, [
 				'signPayload' => true,
 			])->getUri();
 		} catch (AwsException) {

lib/private/Files/ObjectStore/ObjectStoreStorage.php

Spacing +30 added lines, -30 removed lines

@@ -57,17 +57,17 @@ 
 			throw new \Exception('missing IObjectStore instance');
 		}
 		if (isset($parameters['storageid'])) {
-			$this->id = 'object::store:' . $parameters['storageid'];
+			$this->id = 'object::store:'.$parameters['storageid'];
 		} else {
-			$this->id = 'object::store:' . $this->objectStore->getStorageId();
+			$this->id = 'object::store:'.$this->objectStore->getStorageId();
 		}
 		if (isset($parameters['objectPrefix'])) {
 			$this->objectPrefix = $parameters['objectPrefix'];
 		}
 		if (isset($parameters['validateWrites'])) {
-			$this->validateWrites = (bool)$parameters['validateWrites'];
+			$this->validateWrites = (bool) $parameters['validateWrites'];
 		}
-		$this->handleCopiesAsOwned = (bool)($parameters['handleCopiesAsOwned'] ?? false);
+		$this->handleCopiesAsOwned = (bool) ($parameters['handleCopiesAsOwned'] ?? false);
 		if (isset($parameters['totalSizeLimit'])) {
 			$this->totalSizeLimit = $parameters['totalSizeLimit'];
 		}
@@ -204,7 +204,7 @@ 
 		} catch (\Exception $ex) {
 			if ($ex->getCode() !== 404) {
 				$this->logger->error(
-					'Could not delete object ' . $this->getURN($entry->getId()) . ' for ' . $entry->getPath(),
+					'Could not delete object '.$this->getURN($entry->getId()).' for '.$entry->getPath(),
 					[
 						'app' => 'objectstore',
 						'exception' => $ex,
@@ -220,7 +220,7 @@ 
 		return true;
 	}
 
-	public function stat(string $path): array|false {
+	public function stat(string $path): array | false {
 		$path = $this->normalizePath($path);
 		$cacheEntry = $this->getCache()->get($path);
 		if ($cacheEntry instanceof CacheEntry) {
@@ -255,7 +255,7 @@ 
 	 * @return string the unified resource name used to identify the object
 	 */
 	public function getURN(int $fileId): string {
-		return $this->objectPrefix . $fileId;
+		return $this->objectPrefix.$fileId;
 	}
 
 	public function opendir(string $path) {
@@ -275,7 +275,7 @@ 
 		}
 	}
 
-	public function filetype(string $path): string|false {
+	public function filetype(string $path): string | false {
 		$path = $this->normalizePath($path);
 		$stat = $this->stat($path);
 		if ($stat) {
@@ -316,12 +316,12 @@ 
 						$streamStat = fstat($handle);
 						$actualSize = $streamStat['size'] ?? -1;
 						if ($actualSize > -1 && $actualSize !== $filesize) {
-							$this->getCache()->update((int)$stat['fileid'], ['size' => $actualSize]);
+							$this->getCache()->update((int) $stat['fileid'], ['size' => $actualSize]);
 						}
 						return $handle;
 					} catch (NotFoundException $e) {
 						$this->logger->error(
-							'Could not get object ' . $this->getURN($stat['fileid']) . ' for file ' . $path,
+							'Could not get object '.$this->getURN($stat['fileid']).' for file '.$path,
 							[
 								'app' => 'objectstore',
 								'exception' => $e,
@@ -330,7 +330,7 @@ 
 						throw $e;
 					} catch (\Exception $e) {
 						$this->logger->error(
-							'Could not get object ' . $this->getURN($stat['fileid']) . ' for file ' . $path,
+							'Could not get object '.$this->getURN($stat['fileid']).' for file '.$path,
 							[
 								'app' => 'objectstore',
 								'exception' => $e,
@@ -354,7 +354,7 @@ 
 
 				$tmpFile = \OC::$server->getTempManager()->getTemporaryFile($ext);
 				$handle = fopen($tmpFile, $mode);
-				return CallbackWrapper::wrap($handle, null, null, function () use ($path, $tmpFile) {
+				return CallbackWrapper::wrap($handle, null, null, function() use ($path, $tmpFile) {
 					$this->writeBack($tmpFile, $path);
 					unlink($tmpFile);
 				});
@@ -372,7 +372,7 @@ 
 					file_put_contents($tmpFile, $source);
 				}
 				$handle = fopen($tmpFile, $mode);
-				return CallbackWrapper::wrap($handle, null, null, function () use ($path, $tmpFile) {
+				return CallbackWrapper::wrap($handle, null, null, function() use ($path, $tmpFile) {
 					$this->writeBack($tmpFile, $path);
 					unlink($tmpFile);
 				});
@@ -382,7 +382,7 @@ 
 
 	public function file_exists(string $path): bool {
 		$path = $this->normalizePath($path);
-		return (bool)$this->stat($path);
+		return (bool) $this->stat($path);
 	}
 
 	public function rename(string $source, string $target): bool {
@@ -394,7 +394,7 @@ 
 		return true;
 	}
 
-	public function getMimeType(string $path): string|false {
+	public function getMimeType(string $path): string | false {
 		$path = $this->normalizePath($path);
 		return parent::getMimeType($path);
 	}
@@ -423,7 +423,7 @@ 
 				$this->file_put_contents($path, ' ');
 			} catch (\Exception $ex) {
 				$this->logger->error(
-					'Could not create object for ' . $path,
+					'Could not create object for '.$path,
 					[
 						'app' => 'objectstore',
 						'exception' => $ex,
@@ -472,7 +472,7 @@ 
 		}
 		// update stat with new data
 		$mTime = time();
-		$stat['size'] = (int)$size;
+		$stat['size'] = (int) $size;
 		$stat['mtime'] = $mTime;
 		$stat['storage_mtime'] = $mTime;
 
@@ -492,7 +492,7 @@ 
 		$stat['checksum'] = '';
 
 		$exists = $this->getCache()->inCache($path);
-		$uploadPath = $exists ? $path : $path . '.part';
+		$uploadPath = $exists ? $path : $path.'.part';
 
 		if ($exists) {
 			$fileId = $stat['fileid'];
@@ -509,7 +509,7 @@ 
 			//upload to object storage
 
 			$totalWritten = 0;
-			$countStream = CountWrapper::wrap($stream, function ($writtenSize) use ($fileId, $size, $exists, &$totalWritten) {
+			$countStream = CountWrapper::wrap($stream, function($writtenSize) use ($fileId, $size, $exists, &$totalWritten) {
 				if (is_null($size) && !$exists) {
 					$this->getCache()->update($fileId, [
 						'size' => $writtenSize,
@@ -536,7 +536,7 @@ 
 				 */
 				$this->getCache()->remove($uploadPath);
 				$this->logger->error(
-					'Could not create object ' . $urn . ' for ' . $path,
+					'Could not create object '.$urn.' for '.$path,
 					[
 						'app' => 'objectstore',
 						'exception' => $ex,
@@ -544,7 +544,7 @@ 
 				);
 			} else {
 				$this->logger->error(
-					'Could not update object ' . $urn . ' for ' . $path,
+					'Could not update object '.$urn.' for '.$path,
 					[
 						'app' => 'objectstore',
 						'exception' => $ex,
@@ -708,7 +708,7 @@ 
 			$this->mkdir($to, false, ['size' => $sourceEntry->getSize()]);
 
 			foreach ($sourceCache->getFolderContentsById($sourceEntry->getId()) as $child) {
-				$this->copyInner($sourceCache, $child, $to . '/' . $child->getName());
+				$this->copyInner($sourceCache, $child, $to.'/'.$child->getName());
 			}
 		} else {
 			$this->copyFile($sourceEntry, $to);
@@ -767,7 +767,7 @@ 
 		$cacheEntry = $this->getCache()->get($targetPath);
 		$urn = $this->getURN($cacheEntry->getId());
 
-		$result = $this->objectStore->uploadMultipartPart($urn, $writeToken, (int)$chunkId, $data, $size);
+		$result = $this->objectStore->uploadMultipartPart($urn, $writeToken, (int) $chunkId, $data, $size);
 
 		$parts[$chunkId] = [
 			'PartNumber' => $chunkId,
@@ -795,10 +795,10 @@ 
 				$stat['mimetype'] = $this->getMimeType($targetPath);
 				$this->getCache()->update($stat['fileid'], $stat);
 			}
-		} catch (S3MultipartUploadException|S3Exception $e) {
+		} catch (S3MultipartUploadException | S3Exception $e) {
 			$this->objectStore->abortMultipartUpload($urn, $writeToken);
 			$this->logger->error(
-				'Could not complete multipart upload ' . $urn . ' with uploadId ' . $writeToken,
+				'Could not complete multipart upload '.$urn.' with uploadId '.$writeToken,
 				[
 					'app' => 'objectstore',
 					'exception' => $e,
@@ -822,7 +822,7 @@ 
 		$this->preserveCacheItemsOnDelete = $preserve;
 	}
 
-	public function free_space(string $path): int|float|false {
+	public function free_space(string $path): int | float | false {
 		if ($this->totalSizeLimit === null) {
 			return FileInfo::SPACE_UNLIMITED;
 		}
@@ -847,20 +847,20 @@ 
 	}
 
 	#[Override]
-	public function getDirectDownloadById(string $fileId): array|false {
+	public function getDirectDownloadById(string $fileId): array | false {
 		$expiration = new \DateTimeImmutable('+60 minutes');
-		$url = $this->objectStore->preSignedUrl($this->getURN((int)$fileId), $expiration);
+		$url = $this->objectStore->preSignedUrl($this->getURN((int) $fileId), $expiration);
 		return $url ? ['url' => $url, 'expiration' => $expiration->getTimestamp()] : false;
 	}
 
 	#[Override]
-	public function getDirectDownload(string $path): array|false {
+	public function getDirectDownload(string $path): array | false {
 		$path = $this->normalizePath($path);
 		$cacheEntry = $this->getCache()->get($path);
 
 		if (!$cacheEntry || $cacheEntry->getMimeType() === FileInfo::MIMETYPE_FOLDER) {
 			return false;
 		}
-		return $this->getDirectDownloadById((string)$cacheEntry->getId());
+		return $this->getDirectDownloadById((string) $cacheEntry->getId());
 	}
 }

lib/private/Files/ObjectStore/Swift.php

Spacing +1 added lines, -1 removed lines

@@ -131,7 +131,7 @@
 
 	public function copyObject($from, $to) {
 		$this->getContainer()->getObject($from)->copy([
-			'destination' => $this->getContainer()->name . '/' . $to
+			'destination' => $this->getContainer()->name.'/'.$to
 		]);
 	}
 	public function preSignedUrl(string $urn, \DateTimeInterface $expiration): ?string {

lib/private/Files/Storage/Wrapper/Wrapper.php

Spacing +15 added lines, -15 removed lines

@@ -44,7 +44,7 @@ 
 
 	public function getWrapperStorage(): Storage {
 		if (!$this->storage) {
-			$message = 'storage wrapper ' . get_class($this) . " doesn't have a wrapped storage set";
+			$message = 'storage wrapper '.get_class($this)." doesn't have a wrapped storage set";
 			$logger = Server::get(LoggerInterface::class);
 			$logger->error($message);
 			$this->storage = new FailedStorage(['exception' => new \Exception($message)]);
@@ -76,15 +76,15 @@ 
 		return $this->getWrapperStorage()->is_file($path);
 	}
 
-	public function stat(string $path): array|false {
+	public function stat(string $path): array | false {
 		return $this->getWrapperStorage()->stat($path);
 	}
 
-	public function filetype(string $path): string|false {
+	public function filetype(string $path): string | false {
 		return $this->getWrapperStorage()->filetype($path);
 	}
 
-	public function filesize(string $path): int|float|false {
+	public function filesize(string $path): int | float | false {
 		return $this->getWrapperStorage()->filesize($path);
 	}
 
@@ -116,15 +116,15 @@ 
 		return $this->getWrapperStorage()->file_exists($path);
 	}
 
-	public function filemtime(string $path): int|false {
+	public function filemtime(string $path): int | false {
 		return $this->getWrapperStorage()->filemtime($path);
 	}
 
-	public function file_get_contents(string $path): string|false {
+	public function file_get_contents(string $path): string | false {
 		return $this->getWrapperStorage()->file_get_contents($path);
 	}
 
-	public function file_put_contents(string $path, mixed $data): int|float|false {
+	public function file_put_contents(string $path, mixed $data): int | float | false {
 		return $this->getWrapperStorage()->file_put_contents($path, $data);
 	}
 
@@ -144,15 +144,15 @@ 
 		return $this->getWrapperStorage()->fopen($path, $mode);
 	}
 
-	public function getMimeType(string $path): string|false {
+	public function getMimeType(string $path): string | false {
 		return $this->getWrapperStorage()->getMimeType($path);
 	}
 
-	public function hash(string $type, string $path, bool $raw = false): string|false {
+	public function hash(string $type, string $path, bool $raw = false): string | false {
 		return $this->getWrapperStorage()->hash($type, $path, $raw);
 	}
 
-	public function free_space(string $path): int|float|false {
+	public function free_space(string $path): int | float | false {
 		return $this->getWrapperStorage()->free_space($path);
 	}
 
@@ -160,7 +160,7 @@ 
 		return $this->getWrapperStorage()->touch($path, $mtime);
 	}
 
-	public function getLocalFile(string $path): string|false {
+	public function getLocalFile(string $path): string | false {
 		return $this->getWrapperStorage()->getLocalFile($path);
 	}
 
@@ -182,7 +182,7 @@ 
 		return $this->getWrapperStorage()->getScanner($path, $storage);
 	}
 
-	public function getOwner(string $path): string|false {
+	public function getOwner(string $path): string | false {
 		return $this->getWrapperStorage()->getOwner($path);
 	}
 
@@ -211,7 +211,7 @@ 
 		return $this->getWrapperStorage()->getStorageCache();
 	}
 
-	public function getETag(string $path): string|false {
+	public function getETag(string $path): string | false {
 		return $this->getWrapperStorage()->getETag($path);
 	}
 
@@ -260,12 +260,12 @@ 
 	}
 
 	#[Override]
-	public function getDirectDownload(string $path): array|false {
+	public function getDirectDownload(string $path): array | false {
 		return $this->getWrapperStorage()->getDirectDownload($path);
 	}
 
 	#[Override]
-	public function getDirectDownloadById(string $fileId): array|false {
+	public function getDirectDownloadById(string $fileId): array | false {
 		return $this->getWrapperStorage()->getDirectDownloadById($fileId);
 	}
 

lib/private/Files/Storage/Wrapper/Availability.php

Spacing +14 added lines, -14 removed lines

@@ -107,15 +107,15 @@ 
 		return $this->handleAvailability('is_file', $path);
 	}
 
-	public function stat(string $path): array|false {
+	public function stat(string $path): array | false {
 		return $this->handleAvailability('stat', $path);
 	}
 
-	public function filetype(string $path): string|false {
+	public function filetype(string $path): string | false {
 		return $this->handleAvailability('filetype', $path);
 	}
 
-	public function filesize(string $path): int|float|false {
+	public function filesize(string $path): int | float | false {
 		return $this->handleAvailability('filesize', $path);
 	}
 
@@ -150,15 +150,15 @@ 
 		return $this->handleAvailability('file_exists', $path);
 	}
 
-	public function filemtime(string $path): int|false {
+	public function filemtime(string $path): int | false {
 		return $this->handleAvailability('filemtime', $path);
 	}
 
-	public function file_get_contents(string $path): string|false {
+	public function file_get_contents(string $path): string | false {
 		return $this->handleAvailability('file_get_contents', $path);
 	}
 
-	public function file_put_contents(string $path, mixed $data): int|float|false {
+	public function file_put_contents(string $path, mixed $data): int | float | false {
 		return $this->handleAvailability('file_put_contents', $path, $data);
 	}
 
@@ -178,15 +178,15 @@ 
 		return $this->handleAvailability('fopen', $path, $mode);
 	}
 
-	public function getMimeType(string $path): string|false {
+	public function getMimeType(string $path): string | false {
 		return $this->handleAvailability('getMimeType', $path);
 	}
 
-	public function hash(string $type, string $path, bool $raw = false): string|false {
+	public function hash(string $type, string $path, bool $raw = false): string | false {
 		return $this->handleAvailability('hash', $type, $path, $raw);
 	}
 
-	public function free_space(string $path): int|float|false {
+	public function free_space(string $path): int | float | false {
 		return $this->handleAvailability('free_space', $path);
 	}
 
@@ -194,7 +194,7 @@ 
 		return $this->handleAvailability('touch', $path, $mtime);
 	}
 
-	public function getLocalFile(string $path): string|false {
+	public function getLocalFile(string $path): string | false {
 		return $this->handleAvailability('getLocalFile', $path);
 	}
 
@@ -211,7 +211,7 @@ 
 		}
 	}
 
-	public function getOwner(string $path): string|false {
+	public function getOwner(string $path): string | false {
 		try {
 			return parent::getOwner($path);
 		} catch (StorageNotAvailableException $e) {
@@ -220,15 +220,15 @@ 
 		}
 	}
 
-	public function getETag(string $path): string|false {
+	public function getETag(string $path): string | false {
 		return $this->handleAvailability('getETag', $path);
 	}
 
-	public function getDirectDownload(string $path): array|false {
+	public function getDirectDownload(string $path): array | false {
 		return $this->handleAvailability('getDirectDownload', $path);
 	}
 
-	public function getDirectDownloadById(string $fileId): array|false {
+	public function getDirectDownloadById(string $fileId): array | false {
 		return $this->handleAvailability('getDirectDownloadById', $fileId);
 	}
 

lib/private/Files/Storage/Common.php

Spacing +33 added lines, -33 removed lines

@@ -91,7 +91,7 @@ 
 		return $this->filetype($path) === 'file';
 	}
 
-	public function filesize(string $path): int|float|false {
+	public function filesize(string $path): int | float | false {
 		if ($this->is_dir($path)) {
 			return 0; //by definition
 		} else {
@@ -152,7 +152,7 @@ 
 		return $permissions;
 	}
 
-	public function filemtime(string $path): int|false {
+	public function filemtime(string $path): int | false {
 		$stat = $this->stat($path);
 		if (isset($stat['mtime']) && $stat['mtime'] > 0) {
 			return $stat['mtime'];
@@ -161,7 +161,7 @@ 
 		}
 	}
 
-	public function file_get_contents(string $path): string|false {
+	public function file_get_contents(string $path): string | false {
 		$handle = $this->fopen($path, 'r');
 		if (!$handle) {
 			return false;
@@ -171,7 +171,7 @@ 
 		return $data;
 	}
 
-	public function file_put_contents(string $path, mixed $data): int|float|false {
+	public function file_put_contents(string $path, mixed $data): int | float | false {
 		$handle = $this->fopen($path, 'w');
 		if (!$handle) {
 			return false;
@@ -196,7 +196,7 @@ 
 			$this->mkdir($target);
 			while (($file = readdir($dir)) !== false) {
 				if (!Filesystem::isIgnoredDir($file)) {
-					if (!$this->copy($source . '/' . $file, $target . '/' . $file)) {
+					if (!$this->copy($source.'/'.$file, $target.'/'.$file)) {
 						closedir($dir);
 						return false;
 					}
@@ -216,7 +216,7 @@ 
 		}
 	}
 
-	public function getMimeType(string $path): string|false {
+	public function getMimeType(string $path): string | false {
 		if ($this->is_dir($path)) {
 			return 'httpd/unix-directory';
 		} elseif ($this->file_exists($path)) {
@@ -226,7 +226,7 @@ 
 		}
 	}
 
-	public function hash(string $type, string $path, bool $raw = false): string|false {
+	public function hash(string $type, string $path, bool $raw = false): string | false {
 		$fh = $this->fopen($path, 'rb');
 		if (!$fh) {
 			return false;
@@ -237,7 +237,7 @@ 
 		return hash_final($ctx, $raw);
 	}
 
-	public function getLocalFile(string $path): string|false {
+	public function getLocalFile(string $path): string | false {
 		return $this->getCachedFile($path);
 	}
 
@@ -246,12 +246,12 @@ 
 		if (is_resource($dh)) {
 			while (($file = readdir($dh)) !== false) {
 				if (!Filesystem::isIgnoredDir($file)) {
-					if ($this->is_dir($path . '/' . $file)) {
-						mkdir($target . '/' . $file);
-						$this->addLocalFolder($path . '/' . $file, $target . '/' . $file);
+					if ($this->is_dir($path.'/'.$file)) {
+						mkdir($target.'/'.$file);
+						$this->addLocalFolder($path.'/'.$file, $target.'/'.$file);
 					} else {
-						$tmp = $this->toTmpFile($path . '/' . $file);
-						rename($tmp, $target . '/' . $file);
+						$tmp = $this->toTmpFile($path.'/'.$file);
+						rename($tmp, $target.'/'.$file);
 					}
 				}
 			}
@@ -267,10 +267,10 @@ 
 					continue;
 				}
 				if (strstr(strtolower($item), strtolower($query)) !== false) {
-					$files[] = $dir . '/' . $item;
+					$files[] = $dir.'/'.$item;
 				}
-				if ($this->is_dir($dir . '/' . $item)) {
-					$files = array_merge($files, $this->searchInDir($query, $dir . '/' . $item));
+				if ($this->is_dir($dir.'/'.$item)) {
+					$files = array_merge($files, $this->searchInDir($query, $dir.'/'.$item));
 				}
 			}
 		}
@@ -330,7 +330,7 @@ 
 		if (!isset($this->watcher)) {
 			$this->watcher = new Watcher($storage);
 			$globalPolicy = Server::get(IConfig::class)->getSystemValueInt('filesystem_check_changes', Watcher::CHECK_NEVER);
-			$this->watcher->setPolicy((int)$this->getMountOption('filesystem_check_changes', $globalPolicy));
+			$this->watcher->setPolicy((int) $this->getMountOption('filesystem_check_changes', $globalPolicy));
 		}
 		return $this->watcher;
 	}
@@ -345,7 +345,7 @@ 
 		/** @var self $storage */
 		if (!isset($storage->propagator)) {
 			$config = Server::get(IConfig::class);
-			$storage->propagator = new Propagator($storage, \OC::$server->getDatabaseConnection(), ['appdata_' . $config->getSystemValueString('instanceid')]);
+			$storage->propagator = new Propagator($storage, \OC::$server->getDatabaseConnection(), ['appdata_'.$config->getSystemValueString('instanceid')]);
 		}
 		return $storage->propagator;
 	}
@@ -366,11 +366,11 @@ 
 
 	public function getStorageCache(?IStorage $storage = null): \OC\Files\Cache\Storage {
 		/** @var Cache $cache */
-		$cache = $this->getCache(storage: $storage);
+		$cache = $this->getCache(storage : $storage);
 		return $cache->getStorageCache();
 	}
 
-	public function getOwner(string $path): string|false {
+	public function getOwner(string $path): string | false {
 		if ($this->owner === null) {
 			$this->owner = \OC_User::getUser();
 		}
@@ -378,7 +378,7 @@ 
 		return $this->owner;
 	}
 
-	public function getETag(string $path): string|false {
+	public function getETag(string $path): string | false {
 		return uniqid();
 	}
 
@@ -391,7 +391,7 @@ 
 	 */
 	public function cleanPath(string $path): string {
 		if (strlen($path) == 0 || $path[0] != '/') {
-			$path = '/' . $path;
+			$path = '/'.$path;
 		}
 
 		$output = [];
@@ -418,14 +418,14 @@ 
 			return false;
 		} catch (\Exception $e) {
 			Server::get(LoggerInterface::class)->warning(
-				'External storage not available: ' . $e->getMessage(),
+				'External storage not available: '.$e->getMessage(),
 				['exception' => $e]
 			);
 			return false;
 		}
 	}
 
-	public function free_space(string $path): int|float|false {
+	public function free_space(string $path): int | float | false {
 		return \OCP\Files\FileInfo::SPACE_UNKNOWN;
 	}
 
@@ -447,12 +447,12 @@ 
 	}
 
 	#[Override]
-	public function getDirectDownload(string $path): array|false {
+	public function getDirectDownload(string $path): array | false {
 		return false;
 	}
 
 	#[Override]
-	public function getDirectDownloadById(string $fileId): array|false {
+	public function getDirectDownloadById(string $fileId): array | false {
 		return false;
 	}
 
@@ -505,7 +505,7 @@ 
 				$result = true;
 				while ($result && ($file = readdir($dh)) !== false) {
 					if (!Filesystem::isIgnoredDir($file)) {
-						$result &= $this->copyFromStorage($sourceStorage, $sourceInternalPath . '/' . $file, $targetInternalPath . '/' . $file);
+						$result &= $this->copyFromStorage($sourceStorage, $sourceInternalPath.'/'.$file, $targetInternalPath.'/'.$file);
 					}
 				}
 			}
@@ -533,7 +533,7 @@ 
 				$this->getCache()->remove($targetInternalPath);
 			}
 		}
-		return (bool)$result;
+		return (bool) $result;
 	}
 
 	/**
@@ -595,7 +595,7 @@ 
 
 	public function getMetaData(string $path): ?array {
 		if (Filesystem::isFileBlacklisted($path)) {
-			throw new ForbiddenException('Invalid path: ' . $path, false);
+			throw new ForbiddenException('Invalid path: '.$path, false);
 		}
 
 		$permissions = $this->getPermissions($path);
@@ -640,7 +640,7 @@ 
 			);
 		}
 		try {
-			$provider->acquireLock('files/' . md5($this->getId() . '::' . trim($path, '/')), $type, $this->getId() . '::' . $path);
+			$provider->acquireLock('files/'.md5($this->getId().'::'.trim($path, '/')), $type, $this->getId().'::'.$path);
 		} catch (LockedException $e) {
 			$e = new LockedException($e->getPath(), $e, $e->getExistingLock(), $path);
 			if ($logger) {
@@ -667,7 +667,7 @@ 
 			);
 		}
 		try {
-			$provider->releaseLock('files/' . md5($this->getId() . '::' . trim($path, '/')), $type);
+			$provider->releaseLock('files/'.md5($this->getId().'::'.trim($path, '/')), $type);
 		} catch (LockedException $e) {
 			$e = new LockedException($e->getPath(), $e, $e->getExistingLock(), $path);
 			if ($logger) {
@@ -694,7 +694,7 @@ 
 			);
 		}
 		try {
-			$provider->changeLock('files/' . md5($this->getId() . '::' . trim($path, '/')), $type);
+			$provider->changeLock('files/'.md5($this->getId().'::'.trim($path, '/')), $type);
 		} catch (LockedException $e) {
 			$e = new LockedException($e->getPath(), $e, $e->getExistingLock(), $path);
 			if ($logger) {
@@ -759,7 +759,7 @@ 
 			$basePath = rtrim($directory, '/');
 			while (($file = readdir($dh)) !== false) {
 				if (!Filesystem::isIgnoredDir($file)) {
-					$childPath = $basePath . '/' . trim($file, '/');
+					$childPath = $basePath.'/'.trim($file, '/');
 					$metadata = $this->getMetaData($childPath);
 					if ($metadata !== null) {
 						yield $metadata;

apps/files_sharing/lib/SharedStorage.php

Spacing +15 added lines, -15 removed lines

@@ -136,7 +136,7 @@ 
 				// this is probably because some code path has caused recursion during the share setup
 				// we setup a "failed storage" so `getWrapperStorage` doesn't return null.
 				// If the share setup completes after this the "failed storage" will be overwritten by the correct one
-				$ex = new \Exception('Possible share setup recursion detected for share ' . $this->superShare->getId());
+				$ex = new \Exception('Possible share setup recursion detected for share '.$this->superShare->getId());
 				$this->logger->warning($ex->getMessage(), ['exception' => $ex, 'app' => 'files_sharing']);
 				$this->storage = new FailedStorage(['exception' => $ex]);
 				$this->cache = new FailedCache();
@@ -238,7 +238,7 @@ 
 	}
 
 	public function getId(): string {
-		return 'shared::' . $this->getMountPoint();
+		return 'shared::'.$this->getMountPoint();
 	}
 
 	public function getPermissions(string $path = ''): int {
@@ -260,7 +260,7 @@ 
 	}
 
 	public function isCreatable(string $path): bool {
-		return (bool)($this->getPermissions($path) & Constants::PERMISSION_CREATE);
+		return (bool) ($this->getPermissions($path) & Constants::PERMISSION_CREATE);
 	}
 
 	public function isReadable(string $path): bool {
@@ -277,18 +277,18 @@ 
 	}
 
 	public function isUpdatable(string $path): bool {
-		return (bool)($this->getPermissions($path) & Constants::PERMISSION_UPDATE);
+		return (bool) ($this->getPermissions($path) & Constants::PERMISSION_UPDATE);
 	}
 
 	public function isDeletable(string $path): bool {
-		return (bool)($this->getPermissions($path) & Constants::PERMISSION_DELETE);
+		return (bool) ($this->getPermissions($path) & Constants::PERMISSION_DELETE);
 	}
 
 	public function isSharable(string $path): bool {
 		if (Util::isSharingDisabledForUser() || !Share::isResharingAllowed()) {
 			return false;
 		}
-		return (bool)($this->getPermissions($path) & Constants::PERMISSION_SHARE);
+		return (bool) ($this->getPermissions($path) & Constants::PERMISSION_SHARE);
 	}
 
 	public function fopen(string $path, string $mode) {
@@ -335,7 +335,7 @@ 
 				}
 		}
 		$info = [
-			'target' => $this->getMountPoint() . '/' . $path,
+			'target' => $this->getMountPoint().'/'.$path,
 			'source' => $source,
 			'mode' => $mode,
 		];
@@ -429,7 +429,7 @@ 
 		return new Scanner($storage);
 	}
 
-	public function getOwner(string $path): string|false {
+	public function getOwner(string $path): string | false {
 		return $this->superShare->getShareOwner();
 	}
 
@@ -523,7 +523,7 @@ 
 		 * @psalm-suppress DocblockTypeContradiction
 		 */
 		if (!$this->storage) {
-			$message = 'no storage set after init for share ' . $this->getShareId();
+			$message = 'no storage set after init for share '.$this->getShareId();
 			$this->logger->error($message);
 			$this->storage = new FailedStorage(['exception' => new \Exception($message)]);
 		}
@@ -531,18 +531,18 @@ 
 		return $this->storage;
 	}
 
-	public function file_get_contents(string $path): string|false {
+	public function file_get_contents(string $path): string | false {
 		$info = [
-			'target' => $this->getMountPoint() . '/' . $path,
+			'target' => $this->getMountPoint().'/'.$path,
 			'source' => $this->getUnjailedPath($path),
 		];
 		Util::emitHook('\OC\Files\Storage\Shared', 'file_get_contents', $info);
 		return parent::file_get_contents($path);
 	}
 
-	public function file_put_contents(string $path, mixed $data): int|float|false {
+	public function file_put_contents(string $path, mixed $data): int | float | false {
 		$info = [
-			'target' => $this->getMountPoint() . '/' . $path,
+			'target' => $this->getMountPoint().'/'.$path,
 			'source' => $this->getUnjailedPath($path),
 		];
 		Util::emitHook('\OC\Files\Storage\Shared', 'file_put_contents', $info);
@@ -560,13 +560,13 @@ 
 	}
 
 	#[Override]
-	public function getDirectDownload(string $path): array|false {
+	public function getDirectDownload(string $path): array | false {
 		// disable direct download for shares
 		return false;
 	}
 
 	#[Override]
-	public function getDirectDownloadById(string $fileId): array|false {
+	public function getDirectDownloadById(string $fileId): array | false {
 		// disable direct download for shares
 		return false;
 	}