nextcloud / server

lib/public/Files/Folder.php

Indentation +203 added lines, -203 removed lines

@@ -16,207 +16,207 @@
  * @since 6.0.0
  */
 interface Folder extends Node {
-	/**
-	 * Get the full path of an item in the folder within owncloud's filesystem
-	 *
-	 * @param string $path relative path of an item in the folder
-	 * @return string
-	 * @throws \OCP\Files\NotPermittedException
-	 * @since 6.0.0
-	 */
-	public function getFullPath($path);
-
-	/**
-	 * Get the path of an item in the folder relative to the folder
-	 *
-	 * @param string $path absolute path of an item in the folder
-	 * @throws \OCP\Files\NotFoundException
-	 * @return string|null
-	 * @since 6.0.0
-	 */
-	public function getRelativePath($path);
-
-	/**
-	 * check if a node is a (grand-)child of the folder
-	 *
-	 * @param \OCP\Files\Node $node
-	 * @return bool
-	 * @since 6.0.0
-	 */
-	public function isSubNode($node);
-
-	/**
-	 * get the content of this directory
-	 *
-	 * @throws \OCP\Files\NotFoundException
-	 * @return \OCP\Files\Node[]
-	 * @since 6.0.0
-	 */
-	public function getDirectoryListing();
-
-	/**
-	 * Get the node at $path
-	 *
-	 * @param string $path relative path of the file or folder
-	 * @return \OCP\Files\Node
-	 * @throws \OCP\Files\NotFoundException
-	 * @throws \OCP\Files\NotPermittedException
-	 * @since 6.0.0
-	 */
-	public function get($path);
-
-	/**
-	 * Get or create new folder if the folder does not already exist.
-	 *
-	 * @param string $path relative path of the file or folder
-	 * @throw \OCP\Files\NotPermittedException
-	 * @since 33.0.0
-	 */
-	public function getOrCreateFolder(string $path, int $maxRetries = 5): Folder;
-
-	/**
-	 * Check if a file or folder exists in the folder
-	 *
-	 * @param string $path relative path of the file or folder
-	 * @return bool
-	 * @since 6.0.0
-	 */
-	public function nodeExists($path);
-
-	/**
-	 * Create a new folder
-	 *
-	 * @param string $path relative path of the new folder
-	 * @return \OCP\Files\Folder
-	 * @throws \OCP\Files\NotPermittedException
-	 * @since 6.0.0
-	 */
-	public function newFolder($path);
-
-	/**
-	 * Create a new file
-	 *
-	 * @param string $path relative path of the new file
-	 * @param string|resource|null $content content for the new file, since 19.0.0
-	 * @return \OCP\Files\File
-	 * @throws \OCP\Files\NotPermittedException
-	 * @since 6.0.0
-	 */
-	public function newFile($path, $content = null);
-
-	/**
-	 * search for files with the name matching $query
-	 *
-	 * @param string|ISearchQuery $query
-	 * @return \OCP\Files\Node[]
-	 * @since 6.0.0
-	 */
-	public function search($query);
-
-	/**
-	 * search for files by mimetype
-	 * $mimetype can either be a full mimetype (image/png) or a wildcard mimetype (image)
-	 *
-	 * @param string $mimetype
-	 * @return \OCP\Files\Node[]
-	 * @since 6.0.0
-	 */
-	public function searchByMime($mimetype);
-
-	/**
-	 * search for files by tag
-	 *
-	 * @param string|int $tag tag name or tag id
-	 * @param string $userId owner of the tags
-	 * @return \OCP\Files\Node[]
-	 * @since 8.0.0
-	 */
-	public function searchByTag($tag, $userId);
-
-	/**
-	 * search for files by system tag
-	 *
-	 * @param string|int $tag tag name
-	 * @param string $userId user id to ensure access on returned nodes
-	 * @return \OCP\Files\Node[]
-	 * @since 28.0.0
-	 */
-	public function searchBySystemTag(string $tagName, string $userId, int $limit = 0, int $offset = 0);
-
-	/**
-	 * get a file or folder inside the folder by its internal id
-	 *
-	 * This method could return multiple entries. For example once the file/folder
-	 * is shared or mounted (files_external) to the user multiple times.
-	 *
-	 * Note that the different entries can have different permissions.
-	 *
-	 * @param int $id
-	 * @return \OCP\Files\Node[]
-	 * @since 6.0.0
-	 */
-	public function getById($id);
-
-	/**
-	 * get a file or folder inside the folder by its internal id
-	 *
-	 * Unlike getById, this method only returns a single node even if the user has
-	 * access to the file with the requested id multiple times.
-	 *
-	 * This method provides no guarantee about which of the nodes in returned and the
-	 * returned node might, for example, have less permissions than other nodes for the same file
-	 *
-	 * Apps that require accurate information about the users access to the file should use getById
-	 * instead of pick the correct node out of the result.
-	 *
-	 * @param int $id
-	 * @return Node|null
-	 * @since 29.0.0
-	 */
-	public function getFirstNodeById(int $id): ?Node;
-
-	/**
-	 * Get the amount of free space inside the folder
-	 *
-	 * @return int
-	 * @since 6.0.0
-	 */
-	public function getFreeSpace();
-
-	/**
-	 * Check if new files or folders can be created within the folder
-	 *
-	 * @return bool
-	 * @since 6.0.0
-	 */
-	public function isCreatable();
-
-	/**
-	 * Add a suffix to the name in case the file exists
-	 *
-	 * @param string $name
-	 * @return string
-	 * @throws NotPermittedException
-	 * @since 8.1.0
-	 */
-	public function getNonExistingName($name);
-
-	/**
-	 * @param int $limit
-	 * @param int $offset
-	 * @return \OCP\Files\Node[]
-	 * @since 9.1.0
-	 */
-	public function getRecent($limit, $offset = 0);
-
-	/**
-	 * Verify if the given path is valid and allowed from this folder.
-	 *
-	 * @param string $path the path from this folder
-	 * @param string $fileName
-	 * @param bool $readonly Check only if the path is allowed for read-only access
-	 * @throws InvalidPathException
-	 * @since 32.0.0
-	 */
-	public function verifyPath($fileName, $readonly = false): void;
+    /**
+     * Get the full path of an item in the folder within owncloud's filesystem
+     *
+     * @param string $path relative path of an item in the folder
+     * @return string
+     * @throws \OCP\Files\NotPermittedException
+     * @since 6.0.0
+     */
+    public function getFullPath($path);
+
+    /**
+     * Get the path of an item in the folder relative to the folder
+     *
+     * @param string $path absolute path of an item in the folder
+     * @throws \OCP\Files\NotFoundException
+     * @return string|null
+     * @since 6.0.0
+     */
+    public function getRelativePath($path);
+
+    /**
+     * check if a node is a (grand-)child of the folder
+     *
+     * @param \OCP\Files\Node $node
+     * @return bool
+     * @since 6.0.0
+     */
+    public function isSubNode($node);
+
+    /**
+     * get the content of this directory
+     *
+     * @throws \OCP\Files\NotFoundException
+     * @return \OCP\Files\Node[]
+     * @since 6.0.0
+     */
+    public function getDirectoryListing();
+
+    /**
+     * Get the node at $path
+     *
+     * @param string $path relative path of the file or folder
+     * @return \OCP\Files\Node
+     * @throws \OCP\Files\NotFoundException
+     * @throws \OCP\Files\NotPermittedException
+     * @since 6.0.0
+     */
+    public function get($path);
+
+    /**
+     * Get or create new folder if the folder does not already exist.
+     *
+     * @param string $path relative path of the file or folder
+     * @throw \OCP\Files\NotPermittedException
+     * @since 33.0.0
+     */
+    public function getOrCreateFolder(string $path, int $maxRetries = 5): Folder;
+
+    /**
+     * Check if a file or folder exists in the folder
+     *
+     * @param string $path relative path of the file or folder
+     * @return bool
+     * @since 6.0.0
+     */
+    public function nodeExists($path);
+
+    /**
+     * Create a new folder
+     *
+     * @param string $path relative path of the new folder
+     * @return \OCP\Files\Folder
+     * @throws \OCP\Files\NotPermittedException
+     * @since 6.0.0
+     */
+    public function newFolder($path);
+
+    /**
+     * Create a new file
+     *
+     * @param string $path relative path of the new file
+     * @param string|resource|null $content content for the new file, since 19.0.0
+     * @return \OCP\Files\File
+     * @throws \OCP\Files\NotPermittedException
+     * @since 6.0.0
+     */
+    public function newFile($path, $content = null);
+
+    /**
+     * search for files with the name matching $query
+     *
+     * @param string|ISearchQuery $query
+     * @return \OCP\Files\Node[]
+     * @since 6.0.0
+     */
+    public function search($query);
+
+    /**
+     * search for files by mimetype
+     * $mimetype can either be a full mimetype (image/png) or a wildcard mimetype (image)
+     *
+     * @param string $mimetype
+     * @return \OCP\Files\Node[]
+     * @since 6.0.0
+     */
+    public function searchByMime($mimetype);
+
+    /**
+     * search for files by tag
+     *
+     * @param string|int $tag tag name or tag id
+     * @param string $userId owner of the tags
+     * @return \OCP\Files\Node[]
+     * @since 8.0.0
+     */
+    public function searchByTag($tag, $userId);
+
+    /**
+     * search for files by system tag
+     *
+     * @param string|int $tag tag name
+     * @param string $userId user id to ensure access on returned nodes
+     * @return \OCP\Files\Node[]
+     * @since 28.0.0
+     */
+    public function searchBySystemTag(string $tagName, string $userId, int $limit = 0, int $offset = 0);
+
+    /**
+     * get a file or folder inside the folder by its internal id
+     *
+     * This method could return multiple entries. For example once the file/folder
+     * is shared or mounted (files_external) to the user multiple times.
+     *
+     * Note that the different entries can have different permissions.
+     *
+     * @param int $id
+     * @return \OCP\Files\Node[]
+     * @since 6.0.0
+     */
+    public function getById($id);
+
+    /**
+     * get a file or folder inside the folder by its internal id
+     *
+     * Unlike getById, this method only returns a single node even if the user has
+     * access to the file with the requested id multiple times.
+     *
+     * This method provides no guarantee about which of the nodes in returned and the
+     * returned node might, for example, have less permissions than other nodes for the same file
+     *
+     * Apps that require accurate information about the users access to the file should use getById
+     * instead of pick the correct node out of the result.
+     *
+     * @param int $id
+     * @return Node|null
+     * @since 29.0.0
+     */
+    public function getFirstNodeById(int $id): ?Node;
+
+    /**
+     * Get the amount of free space inside the folder
+     *
+     * @return int
+     * @since 6.0.0
+     */
+    public function getFreeSpace();
+
+    /**
+     * Check if new files or folders can be created within the folder
+     *
+     * @return bool
+     * @since 6.0.0
+     */
+    public function isCreatable();
+
+    /**
+     * Add a suffix to the name in case the file exists
+     *
+     * @param string $name
+     * @return string
+     * @throws NotPermittedException
+     * @since 8.1.0
+     */
+    public function getNonExistingName($name);
+
+    /**
+     * @param int $limit
+     * @param int $offset
+     * @return \OCP\Files\Node[]
+     * @since 9.1.0
+     */
+    public function getRecent($limit, $offset = 0);
+
+    /**
+     * Verify if the given path is valid and allowed from this folder.
+     *
+     * @param string $path the path from this folder
+     * @param string $fileName
+     * @param bool $readonly Check only if the path is allowed for read-only access
+     * @throws InvalidPathException
+     * @since 32.0.0
+     */
+    public function verifyPath($fileName, $readonly = false): void;
 }

lib/public/Files/Template/ITemplateManager.php

Indentation +61 added lines, -61 removed lines

@@ -28,73 +28,73 @@
  */
 #[Consumable(since: '21.0.0')]
 interface ITemplateManager {
-	/**
-	 * Register a template type support
-	 *
-	 * @param callable(): TemplateFileCreator $callback A callback which returns the TemplateFileCreator instance to register
-	 * @since 21.0.0
-	 */
-	public function registerTemplateFileCreator(callable $callback): void;
+    /**
+     * Register a template type support
+     *
+     * @param callable(): TemplateFileCreator $callback A callback which returns the TemplateFileCreator instance to register
+     * @since 21.0.0
+     */
+    public function registerTemplateFileCreator(callable $callback): void;
 
-	/**
-	 * Get a list of available file creators
-	 *
-	 * @return array
-	 * @since 21.0.0
-	 */
-	public function listCreators(): array;
+    /**
+     * Get a list of available file creators
+     *
+     * @return array
+     * @since 21.0.0
+     */
+    public function listCreators(): array;
 
-	/**
-	 * Get a list of available file creators and their offered templates
-	 *
-	 * @return list<array{app: string, label: string, extension: string, iconClass: ?string, iconSvgInline: ?string, mimetypes: list<string>, ratio: ?float, actionLabel: string, templates: list<Template>}>
-	 * @since 21.0.0
-	 */
-	public function listTemplates(): array;
+    /**
+     * Get a list of available file creators and their offered templates
+     *
+     * @return list<array{app: string, label: string, extension: string, iconClass: ?string, iconSvgInline: ?string, mimetypes: list<string>, ratio: ?float, actionLabel: string, templates: list<Template>}>
+     * @since 21.0.0
+     */
+    public function listTemplates(): array;
 
-	/**
-	 * Get the fields for a given template
-	 *
-	 * @param int $fileId
-	 * @return array
-	 * @since 32.0.0
-	 */
-	public function listTemplateFields(int $fileId): array;
+    /**
+     * Get the fields for a given template
+     *
+     * @param int $fileId
+     * @return array
+     * @since 32.0.0
+     */
+    public function listTemplateFields(int $fileId): array;
 
-	/**
-	 * @return bool
-	 * @since 21.0.0
-	 */
-	public function hasTemplateDirectory(): bool;
+    /**
+     * @return bool
+     * @since 21.0.0
+     */
+    public function hasTemplateDirectory(): bool;
 
-	/**
-	 * @param string $path
-	 * @return void
-	 * @since 21.0.0
-	 */
-	public function setTemplatePath(string $path): void;
+    /**
+     * @param string $path
+     * @return void
+     * @since 21.0.0
+     */
+    public function setTemplatePath(string $path): void;
 
-	/**
-	 * @return string
-	 * @since 21.0.0
-	 */
-	public function getTemplatePath(): string;
+    /**
+     * @return string
+     * @since 21.0.0
+     */
+    public function getTemplatePath(): string;
 
-	/**
-	 * @param string|null $path
-	 * @param string|null $userId
-	 * @since 21.0.0
-	 */
-	public function initializeTemplateDirectory(?string $path = null, ?string $userId = null, $copyTemplates = true): string;
+    /**
+     * @param string|null $path
+     * @param string|null $userId
+     * @since 21.0.0
+     */
+    public function initializeTemplateDirectory(?string $path = null, ?string $userId = null, $copyTemplates = true): string;
 
-	/**
-	 * @param string $filePath
-	 * @param string $templateId
-	 * @param string $templateType
-	 * @param array $templateFields Since 30.0.0
-	 * @return FilesTemplateFile
-	 * @throws GenericFileException
-	 * @since 21.0.0
-	 */
-	public function createFromTemplate(string $filePath, string $templateId = '', string $templateType = 'user', array $templateFields = []): array;
+    /**
+     * @param string $filePath
+     * @param string $templateId
+     * @param string $templateType
+     * @param array $templateFields Since 30.0.0
+     * @return FilesTemplateFile
+     * @throws GenericFileException
+     * @since 21.0.0
+     */
+    public function createFromTemplate(string $filePath, string $templateId = '', string $templateType = 'user', array $templateFields = []): array;
 }

lib/composer/composer/installed.php

Spacing +2 added lines, -2 removed lines

@@ -5,7 +5,7 @@ 
         'version' => 'dev-master',
         'reference' => '8c12590cf6f93ce7aa41f17817b3791e524da39e',
         'type' => 'library',
-        'install_path' => __DIR__ . '/../../../',
+        'install_path' => __DIR__.'/../../../',
         'aliases' => array(),
         'dev' => false,
     ),
@@ -15,7 +15,7 @@ 
             'version' => 'dev-master',
             'reference' => '8c12590cf6f93ce7aa41f17817b3791e524da39e',
             'type' => 'library',
-            'install_path' => __DIR__ . '/../../../',
+            'install_path' => __DIR__.'/../../../',
             'aliases' => array(),
             'dev_requirement' => false,
         ),

lib/composer/composer/InstalledVersions.php

Spacing +9 added lines, -9 removed lines

@@ -169,7 +169,7 @@ 
             return implode(' || ', $ranges);
         }
 
-        throw new \OutOfBoundsException('Package "' . $packageName . '" is not installed');
+        throw new \OutOfBoundsException('Package "'.$packageName.'" is not installed');
     }
 
     /**
@@ -190,7 +190,7 @@ 
             return $installed['versions'][$packageName]['version'];
         }
 
-        throw new \OutOfBoundsException('Package "' . $packageName . '" is not installed');
+        throw new \OutOfBoundsException('Package "'.$packageName.'" is not installed');
     }
 
     /**
@@ -211,7 +211,7 @@ 
             return $installed['versions'][$packageName]['pretty_version'];
         }
 
-        throw new \OutOfBoundsException('Package "' . $packageName . '" is not installed');
+        throw new \OutOfBoundsException('Package "'.$packageName.'" is not installed');
     }
 
     /**
@@ -232,7 +232,7 @@ 
             return $installed['versions'][$packageName]['reference'];
         }
 
-        throw new \OutOfBoundsException('Package "' . $packageName . '" is not installed');
+        throw new \OutOfBoundsException('Package "'.$packageName.'" is not installed');
     }
 
     /**
@@ -249,7 +249,7 @@ 
             return isset($installed['versions'][$packageName]['install_path']) ? $installed['versions'][$packageName]['install_path'] : null;
         }
 
-        throw new \OutOfBoundsException('Package "' . $packageName . '" is not installed');
+        throw new \OutOfBoundsException('Package "'.$packageName.'" is not installed');
     }
 
     /**
@@ -277,8 +277,8 @@ 
         if (null === self::$installed) {
             // only require the installed.php file if this file is loaded from its dumped location,
             // and not from its source location in the composer/composer package, see https://github.com/composer/composer/issues/9937
-            if (substr(__DIR__, -8, 1) !== 'C' && is_file(__DIR__ . '/installed.php')) {
-                self::$installed = include __DIR__ . '/installed.php';
+            if (substr(__DIR__, -8, 1) !== 'C' && is_file(__DIR__.'/installed.php')) {
+                self::$installed = include __DIR__.'/installed.php';
             } else {
                 self::$installed = array();
             }
@@ -378,9 +378,9 @@ 
         if (null === self::$installed) {
             // only require the installed.php file if this file is loaded from its dumped location,
             // and not from its source location in the composer/composer package, see https://github.com/composer/composer/issues/9937
-            if (substr(__DIR__, -8, 1) !== 'C' && is_file(__DIR__ . '/installed.php')) {
+            if (substr(__DIR__, -8, 1) !== 'C' && is_file(__DIR__.'/installed.php')) {
                 /** @var array{root: array{name: string, pretty_version: string, version: string, reference: string|null, type: string, install_path: string, aliases: string[], dev: bool}, versions: array<string, array{pretty_version?: string, version?: string, reference?: string|null, type?: string, install_path?: string, aliases?: string[], dev_requirement: bool, replaced?: string[], provided?: string[]}>} $required */
-                $required = require __DIR__ . '/installed.php';
+                $required = require __DIR__.'/installed.php';
                 self::$installed = $required;
             } else {
                 self::$installed = array();

lib/private/Files/Node/Folder.php

Indentation +475 added lines, -475 removed lines

@@ -31,479 +31,479 @@
 
 class Folder extends Node implements IFolder {
 
-	private ?IUserManager $userManager = null;
-
-	private bool $wasDeleted = false;
-
-	/**
-	 * Creates a Folder that represents a non-existing path
-	 *
-	 * @param string $path path
-	 * @return NonExistingFolder non-existing node
-	 */
-	protected function createNonExistingNode($path) {
-		return new NonExistingFolder($this->root, $this->view, $path);
-	}
-
-	/**
-	 * @param string $path path relative to the folder
-	 * @return string
-	 * @throws \OCP\Files\NotPermittedException
-	 */
-	public function getFullPath($path) {
-		$path = $this->normalizePath($path);
-		if (!$this->isValidPath($path)) {
-			throw new NotPermittedException('Invalid path "' . $path . '"');
-		}
-		return $this->path . $path;
-	}
-
-	/**
-	 * @param string $path
-	 * @return string|null
-	 */
-	public function getRelativePath($path) {
-		return PathHelper::getRelativePath($this->getPath(), $path);
-	}
-
-	/**
-	 * check if a node is a (grand-)child of the folder
-	 *
-	 * @param \OC\Files\Node\Node $node
-	 * @return bool
-	 */
-	public function isSubNode($node) {
-		return str_starts_with($node->getPath(), $this->path . '/');
-	}
-
-	/**
-	 * get the content of this directory
-	 *
-	 * @return Node[]
-	 * @throws \OCP\Files\NotFoundException
-	 */
-	public function getDirectoryListing() {
-		$folderContent = $this->view->getDirectoryContent($this->path, '', $this->getFileInfo(false));
-
-		return array_map(function (FileInfo $info) {
-			if ($info->getMimetype() === FileInfo::MIMETYPE_FOLDER) {
-				return new Folder($this->root, $this->view, $info->getPath(), $info, $this);
-			} else {
-				return new File($this->root, $this->view, $info->getPath(), $info, $this);
-			}
-		}, $folderContent);
-	}
-
-	protected function createNode(string $path, ?FileInfo $info = null, bool $infoHasSubMountsIncluded = true): INode {
-		if (is_null($info)) {
-			$isDir = $this->view->is_dir($path);
-		} else {
-			$isDir = $info->getType() === FileInfo::TYPE_FOLDER;
-		}
-		$parent = dirname($path) === $this->getPath() ? $this : null;
-		if ($isDir) {
-			return new Folder($this->root, $this->view, $path, $info, $parent, $infoHasSubMountsIncluded);
-		} else {
-			return new File($this->root, $this->view, $path, $info, $parent);
-		}
-	}
-
-	public function get($path) {
-		return $this->root->get($this->getFullPath($path));
-	}
-
-	public function nodeExists($path) {
-		try {
-			$this->get($path);
-			return true;
-		} catch (NotFoundException|NotPermittedException) {
-			return false;
-		}
-	}
-
-	/**
-	 * @param string $path
-	 * @return \OC\Files\Node\Folder
-	 * @throws \OCP\Files\NotPermittedException
-	 */
-	public function newFolder($path) {
-		if ($this->checkPermissions(\OCP\Constants::PERMISSION_CREATE)) {
-			$fullPath = $this->getFullPath($path);
-			$nonExisting = new NonExistingFolder($this->root, $this->view, $fullPath);
-			$this->sendHooks(['preWrite', 'preCreate'], [$nonExisting]);
-			if (!$this->view->mkdir($fullPath)) {
-				// maybe another concurrent process created the folder already
-				if (!$this->view->is_dir($fullPath)) {
-					throw new NotPermittedException('Could not create folder "' . $fullPath . '"');
-				} else {
-					// we need to ensure we don't return before the concurrent request has finished updating the cache
-					$tries = 5;
-					while (!$this->view->getFileInfo($fullPath)) {
-						if ($tries < 1) {
-							throw new NotPermittedException('Could not create folder "' . $fullPath . '", folder exists but unable to get cache entry');
-						}
-						usleep(5 * 1000);
-						$tries--;
-					}
-				}
-			}
-			$parent = dirname($fullPath) === $this->getPath() ? $this : null;
-			$node = new Folder($this->root, $this->view, $fullPath, null, $parent);
-			$this->sendHooks(['postWrite', 'postCreate'], [$node]);
-			return $node;
-		} else {
-			throw new NotPermittedException('No create permission for folder "' . $path . '"');
-		}
-	}
-
-	/**
-	 * @param string $path
-	 * @param string | resource | null $content
-	 * @return \OC\Files\Node\File
-	 * @throws \OCP\Files\NotPermittedException
-	 */
-	public function newFile($path, $content = null) {
-		if ($path === '') {
-			throw new NotPermittedException('Could not create as provided path is empty');
-		}
-		$this->recreateIfNeeded();
-		if ($this->checkPermissions(\OCP\Constants::PERMISSION_CREATE)) {
-			$fullPath = $this->getFullPath($path);
-			$nonExisting = new NonExistingFile($this->root, $this->view, $fullPath);
-			$this->sendHooks(['preWrite', 'preCreate'], [$nonExisting]);
-			if ($content !== null) {
-				$result = $this->view->file_put_contents($fullPath, $content);
-			} else {
-				$result = $this->view->touch($fullPath);
-			}
-			if ($result === false) {
-				throw new NotPermittedException('Could not create path "' . $fullPath . '"');
-			}
-			$node = new File($this->root, $this->view, $fullPath, null, $this);
-			$this->sendHooks(['postWrite', 'postCreate'], [$node]);
-			return $node;
-		}
-		throw new NotPermittedException('No create permission for path "' . $path . '"');
-	}
-
-	private function queryFromOperator(ISearchOperator $operator, ?string $uid = null, int $limit = 0, int $offset = 0): ISearchQuery {
-		if ($uid === null) {
-			$user = null;
-		} else {
-			/** @var IUserManager $userManager */
-			$userManager = \OCP\Server::get(IUserManager::class);
-			$user = $userManager->get($uid);
-		}
-		return new SearchQuery($operator, $limit, $offset, [], $user);
-	}
-
-	/**
-	 * search for files with the name matching $query
-	 *
-	 * @param string|ISearchQuery $query
-	 * @return \OC\Files\Node\Node[]
-	 */
-	public function search($query) {
-		if (is_string($query)) {
-			$query = $this->queryFromOperator(new SearchComparison(ISearchComparison::COMPARE_LIKE, 'name', '%' . $query . '%'));
-		}
-
-		// search is handled by a single query covering all caches that this folder contains
-		// this is done by collect
-
-		$limitToHome = $query->limitToHome();
-		if ($limitToHome && count(explode('/', $this->path)) !== 3) {
-			throw new \InvalidArgumentException('searching by owner is only allowed in the users home folder');
-		}
-
-		/** @var QuerySearchHelper $searchHelper */
-		$searchHelper = \OC::$server->get(QuerySearchHelper::class);
-		[$caches, $mountByMountPoint] = $searchHelper->getCachesAndMountPointsForSearch($this->root, $this->path, $limitToHome);
-		$resultsPerCache = $searchHelper->searchInCaches($query, $caches);
-
-		// loop through all results per-cache, constructing the FileInfo object from the CacheEntry and merge them all
-		$files = array_merge(...array_map(function (array $results, string $relativeMountPoint) use ($mountByMountPoint) {
-			$mount = $mountByMountPoint[$relativeMountPoint];
-			return array_map(function (ICacheEntry $result) use ($relativeMountPoint, $mount) {
-				return $this->cacheEntryToFileInfo($mount, $relativeMountPoint, $result);
-			}, $results);
-		}, array_values($resultsPerCache), array_keys($resultsPerCache)));
-
-		// don't include this folder in the results
-		$files = array_values(array_filter($files, function (FileInfo $file) {
-			return $file->getPath() !== $this->getPath();
-		}));
-
-		// since results were returned per-cache, they are no longer fully sorted
-		$order = $query->getOrder();
-		if ($order) {
-			usort($files, function (FileInfo $a, FileInfo $b) use ($order) {
-				foreach ($order as $orderField) {
-					$cmp = $orderField->sortFileInfo($a, $b);
-					if ($cmp !== 0) {
-						return $cmp;
-					}
-				}
-				return 0;
-			});
-		}
-
-		return array_map(function (FileInfo $file) {
-			return $this->createNode($file->getPath(), $file);
-		}, $files);
-	}
-
-	private function cacheEntryToFileInfo(IMountPoint $mount, string $appendRoot, ICacheEntry $cacheEntry): FileInfo {
-		$cacheEntry['internalPath'] = $cacheEntry['path'];
-		$cacheEntry['path'] = rtrim($appendRoot . $cacheEntry->getPath(), '/');
-		$subPath = $cacheEntry['path'] !== '' ? '/' . $cacheEntry['path'] : '';
-		$storage = $mount->getStorage();
-
-		$owner = null;
-		$ownerId = $storage->getOwner($cacheEntry['internalPath']);
-		if ($ownerId !== false) {
-			// Cache the user manager (for performance)
-			if ($this->userManager === null) {
-				$this->userManager = \OCP\Server::get(IUserManager::class);
-			}
-			$owner = new LazyUser($ownerId, $this->userManager);
-		}
-
-		return new \OC\Files\FileInfo(
-			$this->path . $subPath,
-			$storage,
-			$cacheEntry['internalPath'],
-			$cacheEntry,
-			$mount,
-			$owner,
-		);
-	}
-
-	/**
-	 * search for files by mimetype
-	 *
-	 * @param string $mimetype
-	 * @return Node[]
-	 */
-	public function searchByMime($mimetype) {
-		if (!str_contains($mimetype, '/')) {
-			$query = $this->queryFromOperator(new SearchComparison(ISearchComparison::COMPARE_LIKE, 'mimetype', $mimetype . '/%'));
-		} else {
-			$query = $this->queryFromOperator(new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'mimetype', $mimetype));
-		}
-		return $this->search($query);
-	}
-
-	/**
-	 * search for files by tag
-	 *
-	 * @param string|int $tag name or tag id
-	 * @param string $userId owner of the tags
-	 * @return Node[]
-	 */
-	public function searchByTag($tag, $userId) {
-		$query = $this->queryFromOperator(new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'tagname', $tag), $userId);
-		return $this->search($query);
-	}
-
-	public function searchBySystemTag(string $tagName, string $userId, int $limit = 0, int $offset = 0): array {
-		$query = $this->queryFromOperator(new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'systemtag', $tagName), $userId, $limit, $offset);
-		return $this->search($query);
-	}
-
-	/**
-	 * @param int $id
-	 * @return \OCP\Files\Node[]
-	 */
-	public function getById($id) {
-		return $this->root->getByIdInPath((int)$id, $this->getPath());
-	}
-
-	public function getFirstNodeById(int $id): ?\OCP\Files\Node {
-		return $this->root->getFirstNodeByIdInPath($id, $this->getPath());
-	}
-
-	public function getAppDataDirectoryName(): string {
-		$instanceId = \OC::$server->getConfig()->getSystemValueString('instanceid');
-		return 'appdata_' . $instanceId;
-	}
-
-	/**
-	 * In case the path we are currently in is inside the appdata_* folder,
-	 * the original getById method does not work, because it can only look inside
-	 * the user's mount points. But the user has no mount point for the root storage.
-	 *
-	 * So in that case we directly check the mount of the root if it contains
-	 * the id. If it does we check if the path is inside the path we are working
-	 * in.
-	 *
-	 * @param int $id
-	 * @return array
-	 */
-	protected function getByIdInRootMount(int $id): array {
-		if (!method_exists($this->root, 'createNode')) {
-			// Always expected to be false. Being a method of Folder, this is
-			// always implemented. For it is an internal method and should not
-			// be exposed and made public, it is not part of an interface.
-			return [];
-		}
-		$mount = $this->root->getMount('');
-		$storage = $mount->getStorage();
-		$cacheEntry = $storage?->getCache($this->path)->get($id);
-		if (!$cacheEntry) {
-			return [];
-		}
-
-		$absolutePath = '/' . ltrim($cacheEntry->getPath(), '/');
-		$currentPath = rtrim($this->path, '/') . '/';
-
-		if (!str_starts_with($absolutePath, $currentPath)) {
-			return [];
-		}
-
-		return [$this->root->createNode(
-			$absolutePath, new \OC\Files\FileInfo(
-				$absolutePath,
-				$storage,
-				$cacheEntry->getPath(),
-				$cacheEntry,
-				$mount
-			))];
-	}
-
-	public function getFreeSpace() {
-		return $this->view->free_space($this->path);
-	}
-
-	public function delete() {
-		if ($this->checkPermissions(\OCP\Constants::PERMISSION_DELETE)) {
-			$this->sendHooks(['preDelete']);
-			$fileInfo = $this->getFileInfo();
-			$this->view->rmdir($this->path);
-			$nonExisting = new NonExistingFolder($this->root, $this->view, $this->path, $fileInfo);
-			$this->sendHooks(['postDelete'], [$nonExisting]);
-			$this->wasDeleted = true;
-		} else {
-			throw new NotPermittedException('No delete permission for path "' . $this->path . '"');
-		}
-	}
-
-	/**
-	 * Add a suffix to the name in case the file exists
-	 *
-	 * @param string $name
-	 * @return string
-	 * @throws NotPermittedException
-	 */
-	public function getNonExistingName($name) {
-		$uniqueName = \OC_Helper::buildNotExistingFileNameForView($this->getPath(), $name, $this->view);
-		return trim($this->getRelativePath($uniqueName), '/');
-	}
-
-	/**
-	 * @param int $limit
-	 * @param int $offset
-	 * @return INode[]
-	 */
-	public function getRecent($limit, $offset = 0) {
-		$filterOutNonEmptyFolder = new SearchBinaryOperator(
-			// filter out non empty folders
-			ISearchBinaryOperator::OPERATOR_OR,
-			[
-				new SearchBinaryOperator(
-					ISearchBinaryOperator::OPERATOR_NOT,
-					[
-						new SearchComparison(
-							ISearchComparison::COMPARE_EQUAL,
-							'mimetype',
-							FileInfo::MIMETYPE_FOLDER
-						),
-					]
-				),
-				new SearchComparison(
-					ISearchComparison::COMPARE_EQUAL,
-					'size',
-					0
-				),
-			]
-		);
-
-		$filterNonRecentFiles = new SearchComparison(
-			ISearchComparison::COMPARE_GREATER_THAN,
-			'mtime',
-			strtotime('-2 week')
-		);
-		if ($offset === 0 && $limit <= 100) {
-			$query = new SearchQuery(
-				new SearchBinaryOperator(
-					ISearchBinaryOperator::OPERATOR_AND,
-					[
-						$filterOutNonEmptyFolder,
-						$filterNonRecentFiles,
-					],
-				),
-				$limit,
-				$offset,
-				[
-					new SearchOrder(
-						ISearchOrder::DIRECTION_DESCENDING,
-						'mtime'
-					),
-				]
-			);
-		} else {
-			$query = new SearchQuery(
-				$filterOutNonEmptyFolder,
-				$limit,
-				$offset,
-				[
-					new SearchOrder(
-						ISearchOrder::DIRECTION_DESCENDING,
-						'mtime'
-					),
-				]
-			);
-		}
-
-		return $this->search($query);
-	}
-
-	public function verifyPath($fileName, $readonly = false): void {
-		$this->view->verifyPath(
-			$this->getPath(),
-			$fileName,
-			$readonly,
-		);
-	}
-
-	private function recreateIfNeeded(): void {
-		if ($this->wasDeleted) {
-			$this->newFolder('');
-			$this->wasDeleted = false;
-		}
-	}
-
-	#[Override]
-	public function getOrCreateFolder(string $path, int $maxRetries = 5): IFolder {
-		$i = 0;
-		while (true) {
-			$path = $i === 0 ? $path : $path . ' (' . $i . ')';
-			try {
-				$folder = $this->get($path);
-				if ($folder instanceof IFolder) {
-					return $folder;
-				}
-			} catch (NotFoundException) {
-				$folder = dirname($path) === '.' ? $this : $this->get(dirname($path));
-				if (!($folder instanceof Folder)) {
-					throw new NotPermittedException("Unable to create folder $path. Parent is not a directory.");
-				}
-				return $folder->newFolder(basename($path));
-			}
-			$i++;
-			if ($i === $maxRetries) {
-				throw new NotPermittedException('Unable to load or create folder.');
-			}
-		}
-	}
+    private ?IUserManager $userManager = null;
+
+    private bool $wasDeleted = false;
+
+    /**
+     * Creates a Folder that represents a non-existing path
+     *
+     * @param string $path path
+     * @return NonExistingFolder non-existing node
+     */
+    protected function createNonExistingNode($path) {
+        return new NonExistingFolder($this->root, $this->view, $path);
+    }
+
+    /**
+     * @param string $path path relative to the folder
+     * @return string
+     * @throws \OCP\Files\NotPermittedException
+     */
+    public function getFullPath($path) {
+        $path = $this->normalizePath($path);
+        if (!$this->isValidPath($path)) {
+            throw new NotPermittedException('Invalid path "' . $path . '"');
+        }
+        return $this->path . $path;
+    }
+
+    /**
+     * @param string $path
+     * @return string|null
+     */
+    public function getRelativePath($path) {
+        return PathHelper::getRelativePath($this->getPath(), $path);
+    }
+
+    /**
+     * check if a node is a (grand-)child of the folder
+     *
+     * @param \OC\Files\Node\Node $node
+     * @return bool
+     */
+    public function isSubNode($node) {
+        return str_starts_with($node->getPath(), $this->path . '/');
+    }
+
+    /**
+     * get the content of this directory
+     *
+     * @return Node[]
+     * @throws \OCP\Files\NotFoundException
+     */
+    public function getDirectoryListing() {
+        $folderContent = $this->view->getDirectoryContent($this->path, '', $this->getFileInfo(false));
+
+        return array_map(function (FileInfo $info) {
+            if ($info->getMimetype() === FileInfo::MIMETYPE_FOLDER) {
+                return new Folder($this->root, $this->view, $info->getPath(), $info, $this);
+            } else {
+                return new File($this->root, $this->view, $info->getPath(), $info, $this);
+            }
+        }, $folderContent);
+    }
+
+    protected function createNode(string $path, ?FileInfo $info = null, bool $infoHasSubMountsIncluded = true): INode {
+        if (is_null($info)) {
+            $isDir = $this->view->is_dir($path);
+        } else {
+            $isDir = $info->getType() === FileInfo::TYPE_FOLDER;
+        }
+        $parent = dirname($path) === $this->getPath() ? $this : null;
+        if ($isDir) {
+            return new Folder($this->root, $this->view, $path, $info, $parent, $infoHasSubMountsIncluded);
+        } else {
+            return new File($this->root, $this->view, $path, $info, $parent);
+        }
+    }
+
+    public function get($path) {
+        return $this->root->get($this->getFullPath($path));
+    }
+
+    public function nodeExists($path) {
+        try {
+            $this->get($path);
+            return true;
+        } catch (NotFoundException|NotPermittedException) {
+            return false;
+        }
+    }
+
+    /**
+     * @param string $path
+     * @return \OC\Files\Node\Folder
+     * @throws \OCP\Files\NotPermittedException
+     */
+    public function newFolder($path) {
+        if ($this->checkPermissions(\OCP\Constants::PERMISSION_CREATE)) {
+            $fullPath = $this->getFullPath($path);
+            $nonExisting = new NonExistingFolder($this->root, $this->view, $fullPath);
+            $this->sendHooks(['preWrite', 'preCreate'], [$nonExisting]);
+            if (!$this->view->mkdir($fullPath)) {
+                // maybe another concurrent process created the folder already
+                if (!$this->view->is_dir($fullPath)) {
+                    throw new NotPermittedException('Could not create folder "' . $fullPath . '"');
+                } else {
+                    // we need to ensure we don't return before the concurrent request has finished updating the cache
+                    $tries = 5;
+                    while (!$this->view->getFileInfo($fullPath)) {
+                        if ($tries < 1) {
+                            throw new NotPermittedException('Could not create folder "' . $fullPath . '", folder exists but unable to get cache entry');
+                        }
+                        usleep(5 * 1000);
+                        $tries--;
+                    }
+                }
+            }
+            $parent = dirname($fullPath) === $this->getPath() ? $this : null;
+            $node = new Folder($this->root, $this->view, $fullPath, null, $parent);
+            $this->sendHooks(['postWrite', 'postCreate'], [$node]);
+            return $node;
+        } else {
+            throw new NotPermittedException('No create permission for folder "' . $path . '"');
+        }
+    }
+
+    /**
+     * @param string $path
+     * @param string | resource | null $content
+     * @return \OC\Files\Node\File
+     * @throws \OCP\Files\NotPermittedException
+     */
+    public function newFile($path, $content = null) {
+        if ($path === '') {
+            throw new NotPermittedException('Could not create as provided path is empty');
+        }
+        $this->recreateIfNeeded();
+        if ($this->checkPermissions(\OCP\Constants::PERMISSION_CREATE)) {
+            $fullPath = $this->getFullPath($path);
+            $nonExisting = new NonExistingFile($this->root, $this->view, $fullPath);
+            $this->sendHooks(['preWrite', 'preCreate'], [$nonExisting]);
+            if ($content !== null) {
+                $result = $this->view->file_put_contents($fullPath, $content);
+            } else {
+                $result = $this->view->touch($fullPath);
+            }
+            if ($result === false) {
+                throw new NotPermittedException('Could not create path "' . $fullPath . '"');
+            }
+            $node = new File($this->root, $this->view, $fullPath, null, $this);
+            $this->sendHooks(['postWrite', 'postCreate'], [$node]);
+            return $node;
+        }
+        throw new NotPermittedException('No create permission for path "' . $path . '"');
+    }
+
+    private function queryFromOperator(ISearchOperator $operator, ?string $uid = null, int $limit = 0, int $offset = 0): ISearchQuery {
+        if ($uid === null) {
+            $user = null;
+        } else {
+            /** @var IUserManager $userManager */
+            $userManager = \OCP\Server::get(IUserManager::class);
+            $user = $userManager->get($uid);
+        }
+        return new SearchQuery($operator, $limit, $offset, [], $user);
+    }
+
+    /**
+     * search for files with the name matching $query
+     *
+     * @param string|ISearchQuery $query
+     * @return \OC\Files\Node\Node[]
+     */
+    public function search($query) {
+        if (is_string($query)) {
+            $query = $this->queryFromOperator(new SearchComparison(ISearchComparison::COMPARE_LIKE, 'name', '%' . $query . '%'));
+        }
+
+        // search is handled by a single query covering all caches that this folder contains
+        // this is done by collect
+
+        $limitToHome = $query->limitToHome();
+        if ($limitToHome && count(explode('/', $this->path)) !== 3) {
+            throw new \InvalidArgumentException('searching by owner is only allowed in the users home folder');
+        }
+
+        /** @var QuerySearchHelper $searchHelper */
+        $searchHelper = \OC::$server->get(QuerySearchHelper::class);
+        [$caches, $mountByMountPoint] = $searchHelper->getCachesAndMountPointsForSearch($this->root, $this->path, $limitToHome);
+        $resultsPerCache = $searchHelper->searchInCaches($query, $caches);
+
+        // loop through all results per-cache, constructing the FileInfo object from the CacheEntry and merge them all
+        $files = array_merge(...array_map(function (array $results, string $relativeMountPoint) use ($mountByMountPoint) {
+            $mount = $mountByMountPoint[$relativeMountPoint];
+            return array_map(function (ICacheEntry $result) use ($relativeMountPoint, $mount) {
+                return $this->cacheEntryToFileInfo($mount, $relativeMountPoint, $result);
+            }, $results);
+        }, array_values($resultsPerCache), array_keys($resultsPerCache)));
+
+        // don't include this folder in the results
+        $files = array_values(array_filter($files, function (FileInfo $file) {
+            return $file->getPath() !== $this->getPath();
+        }));
+
+        // since results were returned per-cache, they are no longer fully sorted
+        $order = $query->getOrder();
+        if ($order) {
+            usort($files, function (FileInfo $a, FileInfo $b) use ($order) {
+                foreach ($order as $orderField) {
+                    $cmp = $orderField->sortFileInfo($a, $b);
+                    if ($cmp !== 0) {
+                        return $cmp;
+                    }
+                }
+                return 0;
+            });
+        }
+
+        return array_map(function (FileInfo $file) {
+            return $this->createNode($file->getPath(), $file);
+        }, $files);
+    }
+
+    private function cacheEntryToFileInfo(IMountPoint $mount, string $appendRoot, ICacheEntry $cacheEntry): FileInfo {
+        $cacheEntry['internalPath'] = $cacheEntry['path'];
+        $cacheEntry['path'] = rtrim($appendRoot . $cacheEntry->getPath(), '/');
+        $subPath = $cacheEntry['path'] !== '' ? '/' . $cacheEntry['path'] : '';
+        $storage = $mount->getStorage();
+
+        $owner = null;
+        $ownerId = $storage->getOwner($cacheEntry['internalPath']);
+        if ($ownerId !== false) {
+            // Cache the user manager (for performance)
+            if ($this->userManager === null) {
+                $this->userManager = \OCP\Server::get(IUserManager::class);
+            }
+            $owner = new LazyUser($ownerId, $this->userManager);
+        }
+
+        return new \OC\Files\FileInfo(
+            $this->path . $subPath,
+            $storage,
+            $cacheEntry['internalPath'],
+            $cacheEntry,
+            $mount,
+            $owner,
+        );
+    }
+
+    /**
+     * search for files by mimetype
+     *
+     * @param string $mimetype
+     * @return Node[]
+     */
+    public function searchByMime($mimetype) {
+        if (!str_contains($mimetype, '/')) {
+            $query = $this->queryFromOperator(new SearchComparison(ISearchComparison::COMPARE_LIKE, 'mimetype', $mimetype . '/%'));
+        } else {
+            $query = $this->queryFromOperator(new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'mimetype', $mimetype));
+        }
+        return $this->search($query);
+    }
+
+    /**
+     * search for files by tag
+     *
+     * @param string|int $tag name or tag id
+     * @param string $userId owner of the tags
+     * @return Node[]
+     */
+    public function searchByTag($tag, $userId) {
+        $query = $this->queryFromOperator(new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'tagname', $tag), $userId);
+        return $this->search($query);
+    }
+
+    public function searchBySystemTag(string $tagName, string $userId, int $limit = 0, int $offset = 0): array {
+        $query = $this->queryFromOperator(new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'systemtag', $tagName), $userId, $limit, $offset);
+        return $this->search($query);
+    }
+
+    /**
+     * @param int $id
+     * @return \OCP\Files\Node[]
+     */
+    public function getById($id) {
+        return $this->root->getByIdInPath((int)$id, $this->getPath());
+    }
+
+    public function getFirstNodeById(int $id): ?\OCP\Files\Node {
+        return $this->root->getFirstNodeByIdInPath($id, $this->getPath());
+    }
+
+    public function getAppDataDirectoryName(): string {
+        $instanceId = \OC::$server->getConfig()->getSystemValueString('instanceid');
+        return 'appdata_' . $instanceId;
+    }
+
+    /**
+     * In case the path we are currently in is inside the appdata_* folder,
+     * the original getById method does not work, because it can only look inside
+     * the user's mount points. But the user has no mount point for the root storage.
+     *
+     * So in that case we directly check the mount of the root if it contains
+     * the id. If it does we check if the path is inside the path we are working
+     * in.
+     *
+     * @param int $id
+     * @return array
+     */
+    protected function getByIdInRootMount(int $id): array {
+        if (!method_exists($this->root, 'createNode')) {
+            // Always expected to be false. Being a method of Folder, this is
+            // always implemented. For it is an internal method and should not
+            // be exposed and made public, it is not part of an interface.
+            return [];
+        }
+        $mount = $this->root->getMount('');
+        $storage = $mount->getStorage();
+        $cacheEntry = $storage?->getCache($this->path)->get($id);
+        if (!$cacheEntry) {
+            return [];
+        }
+
+        $absolutePath = '/' . ltrim($cacheEntry->getPath(), '/');
+        $currentPath = rtrim($this->path, '/') . '/';
+
+        if (!str_starts_with($absolutePath, $currentPath)) {
+            return [];
+        }
+
+        return [$this->root->createNode(
+            $absolutePath, new \OC\Files\FileInfo(
+                $absolutePath,
+                $storage,
+                $cacheEntry->getPath(),
+                $cacheEntry,
+                $mount
+            ))];
+    }
+
+    public function getFreeSpace() {
+        return $this->view->free_space($this->path);
+    }
+
+    public function delete() {
+        if ($this->checkPermissions(\OCP\Constants::PERMISSION_DELETE)) {
+            $this->sendHooks(['preDelete']);
+            $fileInfo = $this->getFileInfo();
+            $this->view->rmdir($this->path);
+            $nonExisting = new NonExistingFolder($this->root, $this->view, $this->path, $fileInfo);
+            $this->sendHooks(['postDelete'], [$nonExisting]);
+            $this->wasDeleted = true;
+        } else {
+            throw new NotPermittedException('No delete permission for path "' . $this->path . '"');
+        }
+    }
+
+    /**
+     * Add a suffix to the name in case the file exists
+     *
+     * @param string $name
+     * @return string
+     * @throws NotPermittedException
+     */
+    public function getNonExistingName($name) {
+        $uniqueName = \OC_Helper::buildNotExistingFileNameForView($this->getPath(), $name, $this->view);
+        return trim($this->getRelativePath($uniqueName), '/');
+    }
+
+    /**
+     * @param int $limit
+     * @param int $offset
+     * @return INode[]
+     */
+    public function getRecent($limit, $offset = 0) {
+        $filterOutNonEmptyFolder = new SearchBinaryOperator(
+            // filter out non empty folders
+            ISearchBinaryOperator::OPERATOR_OR,
+            [
+                new SearchBinaryOperator(
+                    ISearchBinaryOperator::OPERATOR_NOT,
+                    [
+                        new SearchComparison(
+                            ISearchComparison::COMPARE_EQUAL,
+                            'mimetype',
+                            FileInfo::MIMETYPE_FOLDER
+                        ),
+                    ]
+                ),
+                new SearchComparison(
+                    ISearchComparison::COMPARE_EQUAL,
+                    'size',
+                    0
+                ),
+            ]
+        );
+
+        $filterNonRecentFiles = new SearchComparison(
+            ISearchComparison::COMPARE_GREATER_THAN,
+            'mtime',
+            strtotime('-2 week')
+        );
+        if ($offset === 0 && $limit <= 100) {
+            $query = new SearchQuery(
+                new SearchBinaryOperator(
+                    ISearchBinaryOperator::OPERATOR_AND,
+                    [
+                        $filterOutNonEmptyFolder,
+                        $filterNonRecentFiles,
+                    ],
+                ),
+                $limit,
+                $offset,
+                [
+                    new SearchOrder(
+                        ISearchOrder::DIRECTION_DESCENDING,
+                        'mtime'
+                    ),
+                ]
+            );
+        } else {
+            $query = new SearchQuery(
+                $filterOutNonEmptyFolder,
+                $limit,
+                $offset,
+                [
+                    new SearchOrder(
+                        ISearchOrder::DIRECTION_DESCENDING,
+                        'mtime'
+                    ),
+                ]
+            );
+        }
+
+        return $this->search($query);
+    }
+
+    public function verifyPath($fileName, $readonly = false): void {
+        $this->view->verifyPath(
+            $this->getPath(),
+            $fileName,
+            $readonly,
+        );
+    }
+
+    private function recreateIfNeeded(): void {
+        if ($this->wasDeleted) {
+            $this->newFolder('');
+            $this->wasDeleted = false;
+        }
+    }
+
+    #[Override]
+    public function getOrCreateFolder(string $path, int $maxRetries = 5): IFolder {
+        $i = 0;
+        while (true) {
+            $path = $i === 0 ? $path : $path . ' (' . $i . ')';
+            try {
+                $folder = $this->get($path);
+                if ($folder instanceof IFolder) {
+                    return $folder;
+                }
+            } catch (NotFoundException) {
+                $folder = dirname($path) === '.' ? $this : $this->get(dirname($path));
+                if (!($folder instanceof Folder)) {
+                    throw new NotPermittedException("Unable to create folder $path. Parent is not a directory.");
+                }
+                return $folder->newFolder(basename($path));
+            }
+            $i++;
+            if ($i === $maxRetries) {
+                throw new NotPermittedException('Unable to load or create folder.');
+            }
+        }
+    }
 }

Spacing +26 added lines, -26 removed lines

@@ -53,9 +53,9 @@ 
 	public function getFullPath($path) {
 		$path = $this->normalizePath($path);
 		if (!$this->isValidPath($path)) {
-			throw new NotPermittedException('Invalid path "' . $path . '"');
+			throw new NotPermittedException('Invalid path "'.$path.'"');
 		}
-		return $this->path . $path;
+		return $this->path.$path;
 	}
 
 	/**
@@ -73,7 +73,7 @@ 
 	 * @return bool
 	 */
 	public function isSubNode($node) {
-		return str_starts_with($node->getPath(), $this->path . '/');
+		return str_starts_with($node->getPath(), $this->path.'/');
 	}
 
 	/**
@@ -85,7 +85,7 @@ 
 	public function getDirectoryListing() {
 		$folderContent = $this->view->getDirectoryContent($this->path, '', $this->getFileInfo(false));
 
-		return array_map(function (FileInfo $info) {
+		return array_map(function(FileInfo $info) {
 			if ($info->getMimetype() === FileInfo::MIMETYPE_FOLDER) {
 				return new Folder($this->root, $this->view, $info->getPath(), $info, $this);
 			} else {
@@ -116,7 +116,7 @@ 
 		try {
 			$this->get($path);
 			return true;
-		} catch (NotFoundException|NotPermittedException) {
+		} catch (NotFoundException | NotPermittedException) {
 			return false;
 		}
 	}
@@ -134,13 +134,13 @@ 
 			if (!$this->view->mkdir($fullPath)) {
 				// maybe another concurrent process created the folder already
 				if (!$this->view->is_dir($fullPath)) {
-					throw new NotPermittedException('Could not create folder "' . $fullPath . '"');
+					throw new NotPermittedException('Could not create folder "'.$fullPath.'"');
 				} else {
 					// we need to ensure we don't return before the concurrent request has finished updating the cache
 					$tries = 5;
 					while (!$this->view->getFileInfo($fullPath)) {
 						if ($tries < 1) {
-							throw new NotPermittedException('Could not create folder "' . $fullPath . '", folder exists but unable to get cache entry');
+							throw new NotPermittedException('Could not create folder "'.$fullPath.'", folder exists but unable to get cache entry');
 						}
 						usleep(5 * 1000);
 						$tries--;
@@ -152,7 +152,7 @@ 
 			$this->sendHooks(['postWrite', 'postCreate'], [$node]);
 			return $node;
 		} else {
-			throw new NotPermittedException('No create permission for folder "' . $path . '"');
+			throw new NotPermittedException('No create permission for folder "'.$path.'"');
 		}
 	}
 
@@ -177,13 +177,13 @@ 
 				$result = $this->view->touch($fullPath);
 			}
 			if ($result === false) {
-				throw new NotPermittedException('Could not create path "' . $fullPath . '"');
+				throw new NotPermittedException('Could not create path "'.$fullPath.'"');
 			}
 			$node = new File($this->root, $this->view, $fullPath, null, $this);
 			$this->sendHooks(['postWrite', 'postCreate'], [$node]);
 			return $node;
 		}
-		throw new NotPermittedException('No create permission for path "' . $path . '"');
+		throw new NotPermittedException('No create permission for path "'.$path.'"');
 	}
 
 	private function queryFromOperator(ISearchOperator $operator, ?string $uid = null, int $limit = 0, int $offset = 0): ISearchQuery {
@@ -205,7 +205,7 @@ 
 	 */
 	public function search($query) {
 		if (is_string($query)) {
-			$query = $this->queryFromOperator(new SearchComparison(ISearchComparison::COMPARE_LIKE, 'name', '%' . $query . '%'));
+			$query = $this->queryFromOperator(new SearchComparison(ISearchComparison::COMPARE_LIKE, 'name', '%'.$query.'%'));
 		}
 
 		// search is handled by a single query covering all caches that this folder contains
@@ -222,22 +222,22 @@ 
 		$resultsPerCache = $searchHelper->searchInCaches($query, $caches);
 
 		// loop through all results per-cache, constructing the FileInfo object from the CacheEntry and merge them all
-		$files = array_merge(...array_map(function (array $results, string $relativeMountPoint) use ($mountByMountPoint) {
+		$files = array_merge(...array_map(function(array $results, string $relativeMountPoint) use ($mountByMountPoint) {
 			$mount = $mountByMountPoint[$relativeMountPoint];
-			return array_map(function (ICacheEntry $result) use ($relativeMountPoint, $mount) {
+			return array_map(function(ICacheEntry $result) use ($relativeMountPoint, $mount) {
 				return $this->cacheEntryToFileInfo($mount, $relativeMountPoint, $result);
 			}, $results);
 		}, array_values($resultsPerCache), array_keys($resultsPerCache)));
 
 		// don't include this folder in the results
-		$files = array_values(array_filter($files, function (FileInfo $file) {
+		$files = array_values(array_filter($files, function(FileInfo $file) {
 			return $file->getPath() !== $this->getPath();
 		}));
 
 		// since results were returned per-cache, they are no longer fully sorted
 		$order = $query->getOrder();
 		if ($order) {
-			usort($files, function (FileInfo $a, FileInfo $b) use ($order) {
+			usort($files, function(FileInfo $a, FileInfo $b) use ($order) {
 				foreach ($order as $orderField) {
 					$cmp = $orderField->sortFileInfo($a, $b);
 					if ($cmp !== 0) {
@@ -248,15 +248,15 @@ 
 			});
 		}
 
-		return array_map(function (FileInfo $file) {
+		return array_map(function(FileInfo $file) {
 			return $this->createNode($file->getPath(), $file);
 		}, $files);
 	}
 
 	private function cacheEntryToFileInfo(IMountPoint $mount, string $appendRoot, ICacheEntry $cacheEntry): FileInfo {
 		$cacheEntry['internalPath'] = $cacheEntry['path'];
-		$cacheEntry['path'] = rtrim($appendRoot . $cacheEntry->getPath(), '/');
-		$subPath = $cacheEntry['path'] !== '' ? '/' . $cacheEntry['path'] : '';
+		$cacheEntry['path'] = rtrim($appendRoot.$cacheEntry->getPath(), '/');
+		$subPath = $cacheEntry['path'] !== '' ? '/'.$cacheEntry['path'] : '';
 		$storage = $mount->getStorage();
 
 		$owner = null;
@@ -270,7 +270,7 @@ 
 		}
 
 		return new \OC\Files\FileInfo(
-			$this->path . $subPath,
+			$this->path.$subPath,
 			$storage,
 			$cacheEntry['internalPath'],
 			$cacheEntry,
@@ -287,7 +287,7 @@ 
 	 */
 	public function searchByMime($mimetype) {
 		if (!str_contains($mimetype, '/')) {
-			$query = $this->queryFromOperator(new SearchComparison(ISearchComparison::COMPARE_LIKE, 'mimetype', $mimetype . '/%'));
+			$query = $this->queryFromOperator(new SearchComparison(ISearchComparison::COMPARE_LIKE, 'mimetype', $mimetype.'/%'));
 		} else {
 			$query = $this->queryFromOperator(new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'mimetype', $mimetype));
 		}
@@ -316,7 +316,7 @@ 
 	 * @return \OCP\Files\Node[]
 	 */
 	public function getById($id) {
-		return $this->root->getByIdInPath((int)$id, $this->getPath());
+		return $this->root->getByIdInPath((int) $id, $this->getPath());
 	}
 
 	public function getFirstNodeById(int $id): ?\OCP\Files\Node {
@@ -325,7 +325,7 @@ 
 
 	public function getAppDataDirectoryName(): string {
 		$instanceId = \OC::$server->getConfig()->getSystemValueString('instanceid');
-		return 'appdata_' . $instanceId;
+		return 'appdata_'.$instanceId;
 	}
 
 	/**
@@ -354,8 +354,8 @@ 
 			return [];
 		}
 
-		$absolutePath = '/' . ltrim($cacheEntry->getPath(), '/');
-		$currentPath = rtrim($this->path, '/') . '/';
+		$absolutePath = '/'.ltrim($cacheEntry->getPath(), '/');
+		$currentPath = rtrim($this->path, '/').'/';
 
 		if (!str_starts_with($absolutePath, $currentPath)) {
 			return [];
@@ -384,7 +384,7 @@ 
 			$this->sendHooks(['postDelete'], [$nonExisting]);
 			$this->wasDeleted = true;
 		} else {
-			throw new NotPermittedException('No delete permission for path "' . $this->path . '"');
+			throw new NotPermittedException('No delete permission for path "'.$this->path.'"');
 		}
 	}
 
@@ -487,7 +487,7 @@ 
 	public function getOrCreateFolder(string $path, int $maxRetries = 5): IFolder {
 		$i = 0;
 		while (true) {
-			$path = $i === 0 ? $path : $path . ' (' . $i . ')';
+			$path = $i === 0 ? $path : $path.' ('.$i.')';
 			try {
 				$folder = $this->get($path);
 				if ($folder instanceof IFolder) {

lib/private/Files/Node/LazyFolder.php

Indentation +543 added lines, -543 removed lines

@@ -25,547 +25,547 @@
  * @package OC\Files\Node
  */
 class LazyFolder implements Folder {
-	/** @var \Closure(): Folder */
-	private \Closure $folderClosure;
-	protected ?Folder $folder = null;
-	protected IRootFolder $rootFolder;
-	protected array $data;
-
-	/**
-	 * @param IRootFolder $rootFolder
-	 * @param \Closure(): Folder $folderClosure
-	 * @param array $data
-	 */
-	public function __construct(IRootFolder $rootFolder, \Closure $folderClosure, array $data = []) {
-		$this->rootFolder = $rootFolder;
-		$this->folderClosure = $folderClosure;
-		$this->data = $data;
-	}
-
-	protected function getRootFolder(): IRootFolder {
-		return $this->rootFolder;
-	}
-
-	protected function getRealFolder(): Folder {
-		if ($this->folder === null) {
-			$this->folder = call_user_func($this->folderClosure);
-		}
-		return $this->folder;
-	}
-
-	/**
-	 * Magic method to first get the real rootFolder and then
-	 * call $method with $args on it
-	 *
-	 * @param $method
-	 * @param $args
-	 * @return mixed
-	 */
-	public function __call($method, $args) {
-		return call_user_func_array([$this->getRealFolder(), $method], $args);
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function getUser() {
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function listen($scope, $method, callable $callback) {
-		$this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function removeListener($scope = null, $method = null, ?callable $callback = null) {
-		$this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function emit($scope, $method, $arguments = []) {
-		$this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function mount($storage, $mountPoint, $arguments = []) {
-		$this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function getMount(string $mountPoint): IMountPoint {
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @return IMountPoint[]
-	 */
-	public function getMountsIn(string $mountPoint): array {
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function getMountByStorageId($storageId) {
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function getMountByNumericStorageId($numericId) {
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function unMount($mount) {
-		$this->__call(__FUNCTION__, func_get_args());
-	}
-
-	public function get($path) {
-		return $this->getRootFolder()->get($this->getFullPath($path));
-	}
-
-	#[Override]
-	public function getOrCreateFolder(string $path, int $maxRetries = 5): Folder {
-		return $this->getRootFolder()->getOrCreateFolder($this->getFullPath($path), $maxRetries);
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function rename($targetPath) {
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function delete() {
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function copy($targetPath) {
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function touch($mtime = null) {
-		$this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function getStorage() {
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function getPath() {
-		if (isset($this->data['path'])) {
-			return $this->data['path'];
-		}
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function getInternalPath() {
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function getId() {
-		if (isset($this->data['fileid'])) {
-			return $this->data['fileid'];
-		}
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function stat() {
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function getMTime() {
-		if (isset($this->data['mtime'])) {
-			return $this->data['mtime'];
-		}
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function getSize($includeMounts = true): int|float {
-		if (isset($this->data['size'])) {
-			return $this->data['size'];
-		}
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function getEtag() {
-		if (isset($this->data['etag'])) {
-			return $this->data['etag'];
-		}
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function getPermissions() {
-		if (isset($this->data['permissions'])) {
-			return $this->data['permissions'];
-		}
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function isReadable() {
-		if (isset($this->data['permissions'])) {
-			return ($this->data['permissions'] & Constants::PERMISSION_READ) == Constants::PERMISSION_READ;
-		}
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function isUpdateable() {
-		if (isset($this->data['permissions'])) {
-			return ($this->data['permissions'] & Constants::PERMISSION_UPDATE) == Constants::PERMISSION_UPDATE;
-		}
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function isDeletable() {
-		if (isset($this->data['permissions'])) {
-			return ($this->data['permissions'] & Constants::PERMISSION_DELETE) == Constants::PERMISSION_DELETE;
-		}
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function isShareable() {
-		if (isset($this->data['permissions'])) {
-			return ($this->data['permissions'] & Constants::PERMISSION_SHARE) == Constants::PERMISSION_SHARE;
-		}
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function getParent() {
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function getName() {
-		if (isset($this->data['path'])) {
-			return basename($this->data['path']);
-		}
-		if (isset($this->data['name'])) {
-			return $this->data['name'];
-		}
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function getUserFolder($userId) {
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	public function getMimetype(): string {
-		if (isset($this->data['mimetype'])) {
-			return $this->data['mimetype'];
-		}
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function getMimePart() {
-		if (isset($this->data['mimetype'])) {
-			[$part,] = explode('/', $this->data['mimetype']);
-			return $part;
-		}
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function isEncrypted() {
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function getType() {
-		if (isset($this->data['type'])) {
-			return $this->data['type'];
-		}
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function isShared() {
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function isMounted() {
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function getMountPoint() {
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function getOwner() {
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function getChecksum() {
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	public function getExtension(): string {
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function getFullPath($path) {
-		if (isset($this->data['path'])) {
-			$path = PathHelper::normalizePath($path);
-			if (!Filesystem::isValidPath($path)) {
-				throw new NotPermittedException('Invalid path "' . $path . '"');
-			}
-			return $this->data['path'] . $path;
-		}
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function isSubNode($node) {
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function getDirectoryListing() {
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	public function nodeExists($path) {
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function newFolder($path) {
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function newFile($path, $content = null) {
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function search($query) {
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function searchByMime($mimetype) {
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function searchByTag($tag, $userId) {
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	public function searchBySystemTag(string $tagName, string $userId, int $limit = 0, int $offset = 0) {
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function getById($id) {
-		return $this->getRootFolder()->getByIdInPath((int)$id, $this->getPath());
-	}
-
-	public function getFirstNodeById(int $id): ?\OCP\Files\Node {
-		return $this->getRootFolder()->getFirstNodeByIdInPath($id, $this->getPath());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function getFreeSpace() {
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function isCreatable() {
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function getNonExistingName($name) {
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function move($targetPath) {
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function lock($type) {
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function changeLock($targetType) {
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function unlock($type) {
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function getRecent($limit, $offset = 0) {
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function getCreationTime(): int {
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function getUploadTime(): int {
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	public function getRelativePath($path) {
-		return PathHelper::getRelativePath($this->getPath(), $path);
-	}
-
-	public function getParentId(): int {
-		if (isset($this->data['parent'])) {
-			return $this->data['parent'];
-		}
-		return $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	/**
-	 * @inheritDoc
-	 * @return array<string, int|string|bool|float|string[]|int[]>
-	 */
-	public function getMetadata(): array {
-		return $this->data['metadata'] ?? $this->__call(__FUNCTION__, func_get_args());
-	}
-
-	public function verifyPath($fileName, $readonly = false): void {
-		$this->__call(__FUNCTION__, func_get_args());
-	}
+    /** @var \Closure(): Folder */
+    private \Closure $folderClosure;
+    protected ?Folder $folder = null;
+    protected IRootFolder $rootFolder;
+    protected array $data;
+
+    /**
+     * @param IRootFolder $rootFolder
+     * @param \Closure(): Folder $folderClosure
+     * @param array $data
+     */
+    public function __construct(IRootFolder $rootFolder, \Closure $folderClosure, array $data = []) {
+        $this->rootFolder = $rootFolder;
+        $this->folderClosure = $folderClosure;
+        $this->data = $data;
+    }
+
+    protected function getRootFolder(): IRootFolder {
+        return $this->rootFolder;
+    }
+
+    protected function getRealFolder(): Folder {
+        if ($this->folder === null) {
+            $this->folder = call_user_func($this->folderClosure);
+        }
+        return $this->folder;
+    }
+
+    /**
+     * Magic method to first get the real rootFolder and then
+     * call $method with $args on it
+     *
+     * @param $method
+     * @param $args
+     * @return mixed
+     */
+    public function __call($method, $args) {
+        return call_user_func_array([$this->getRealFolder(), $method], $args);
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getUser() {
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function listen($scope, $method, callable $callback) {
+        $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function removeListener($scope = null, $method = null, ?callable $callback = null) {
+        $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function emit($scope, $method, $arguments = []) {
+        $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function mount($storage, $mountPoint, $arguments = []) {
+        $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getMount(string $mountPoint): IMountPoint {
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @return IMountPoint[]
+     */
+    public function getMountsIn(string $mountPoint): array {
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getMountByStorageId($storageId) {
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getMountByNumericStorageId($numericId) {
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function unMount($mount) {
+        $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    public function get($path) {
+        return $this->getRootFolder()->get($this->getFullPath($path));
+    }
+
+    #[Override]
+    public function getOrCreateFolder(string $path, int $maxRetries = 5): Folder {
+        return $this->getRootFolder()->getOrCreateFolder($this->getFullPath($path), $maxRetries);
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function rename($targetPath) {
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function delete() {
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function copy($targetPath) {
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function touch($mtime = null) {
+        $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getStorage() {
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getPath() {
+        if (isset($this->data['path'])) {
+            return $this->data['path'];
+        }
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getInternalPath() {
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getId() {
+        if (isset($this->data['fileid'])) {
+            return $this->data['fileid'];
+        }
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function stat() {
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getMTime() {
+        if (isset($this->data['mtime'])) {
+            return $this->data['mtime'];
+        }
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getSize($includeMounts = true): int|float {
+        if (isset($this->data['size'])) {
+            return $this->data['size'];
+        }
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getEtag() {
+        if (isset($this->data['etag'])) {
+            return $this->data['etag'];
+        }
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getPermissions() {
+        if (isset($this->data['permissions'])) {
+            return $this->data['permissions'];
+        }
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function isReadable() {
+        if (isset($this->data['permissions'])) {
+            return ($this->data['permissions'] & Constants::PERMISSION_READ) == Constants::PERMISSION_READ;
+        }
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function isUpdateable() {
+        if (isset($this->data['permissions'])) {
+            return ($this->data['permissions'] & Constants::PERMISSION_UPDATE) == Constants::PERMISSION_UPDATE;
+        }
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function isDeletable() {
+        if (isset($this->data['permissions'])) {
+            return ($this->data['permissions'] & Constants::PERMISSION_DELETE) == Constants::PERMISSION_DELETE;
+        }
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function isShareable() {
+        if (isset($this->data['permissions'])) {
+            return ($this->data['permissions'] & Constants::PERMISSION_SHARE) == Constants::PERMISSION_SHARE;
+        }
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getParent() {
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getName() {
+        if (isset($this->data['path'])) {
+            return basename($this->data['path']);
+        }
+        if (isset($this->data['name'])) {
+            return $this->data['name'];
+        }
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getUserFolder($userId) {
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    public function getMimetype(): string {
+        if (isset($this->data['mimetype'])) {
+            return $this->data['mimetype'];
+        }
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getMimePart() {
+        if (isset($this->data['mimetype'])) {
+            [$part,] = explode('/', $this->data['mimetype']);
+            return $part;
+        }
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function isEncrypted() {
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getType() {
+        if (isset($this->data['type'])) {
+            return $this->data['type'];
+        }
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function isShared() {
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function isMounted() {
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getMountPoint() {
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getOwner() {
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getChecksum() {
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    public function getExtension(): string {
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getFullPath($path) {
+        if (isset($this->data['path'])) {
+            $path = PathHelper::normalizePath($path);
+            if (!Filesystem::isValidPath($path)) {
+                throw new NotPermittedException('Invalid path "' . $path . '"');
+            }
+            return $this->data['path'] . $path;
+        }
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function isSubNode($node) {
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getDirectoryListing() {
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    public function nodeExists($path) {
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function newFolder($path) {
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function newFile($path, $content = null) {
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function search($query) {
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function searchByMime($mimetype) {
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function searchByTag($tag, $userId) {
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    public function searchBySystemTag(string $tagName, string $userId, int $limit = 0, int $offset = 0) {
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getById($id) {
+        return $this->getRootFolder()->getByIdInPath((int)$id, $this->getPath());
+    }
+
+    public function getFirstNodeById(int $id): ?\OCP\Files\Node {
+        return $this->getRootFolder()->getFirstNodeByIdInPath($id, $this->getPath());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getFreeSpace() {
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function isCreatable() {
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getNonExistingName($name) {
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function move($targetPath) {
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function lock($type) {
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function changeLock($targetType) {
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function unlock($type) {
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getRecent($limit, $offset = 0) {
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getCreationTime(): int {
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getUploadTime(): int {
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    public function getRelativePath($path) {
+        return PathHelper::getRelativePath($this->getPath(), $path);
+    }
+
+    public function getParentId(): int {
+        if (isset($this->data['parent'])) {
+            return $this->data['parent'];
+        }
+        return $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    /**
+     * @inheritDoc
+     * @return array<string, int|string|bool|float|string[]|int[]>
+     */
+    public function getMetadata(): array {
+        return $this->data['metadata'] ?? $this->__call(__FUNCTION__, func_get_args());
+    }
+
+    public function verifyPath($fileName, $readonly = false): void {
+        $this->__call(__FUNCTION__, func_get_args());
+    }
 }

lib/private/Files/Template/TemplateManager.php

Indentation +366 added lines, -366 removed lines

@@ -40,372 +40,372 @@
  * @psalm-import-type FilesTemplateFile from ResponseDefinitions
  */
 class TemplateManager implements ITemplateManager {
-	/** @var list<callable(): TemplateFileCreator> */
-	private array $registeredTypes = [];
-	/** @var list<TemplateFileCreator> */
-	private array $types = [];
-	/** @var array<class-string<ICustomTemplateProvider>, ICustomTemplateProvider>|null */
-	private ?array $providers = null;
-	private IL10n $l10n;
-	private ?string $userId;
-
-	public function __construct(
-		private readonly ContainerInterface $serverContainer,
-		private readonly IEventDispatcher $eventDispatcher,
-		private readonly Coordinator $bootstrapCoordinator,
-		private readonly IRootFolder $rootFolder,
-		IUserSession $userSession,
-		private readonly IUserManager $userManager,
-		private readonly IPreview $previewManager,
-		private readonly IConfig $config,
-		private readonly IFactory $l10nFactory,
-		private readonly LoggerInterface $logger,
-		private readonly IFilenameValidator $filenameValidator,
-	) {
-		$this->l10n = $l10nFactory->get('lib');
-		$this->userId = $userSession->getUser()?->getUID();
-	}
-
-	#[Override]
-	public function registerTemplateFileCreator(callable $callback): void {
-		$this->registeredTypes[] = $callback;
-	}
-
-	/**
-	 * @return array<class-string<ICustomTemplateProvider>, ICustomTemplateProvider>
-	 */
-	private function getRegisteredProviders(): array {
-		if ($this->providers !== null) {
-			return $this->providers;
-		}
-
-		$context = $this->bootstrapCoordinator->getRegistrationContext();
-
-		$this->providers = [];
-		foreach ($context->getTemplateProviders() as $provider) {
-			$class = $provider->getService();
-			$this->providers[$class] = $this->serverContainer->get($class);
-		}
-		return $this->providers;
-	}
-
-	/**
-	 * @return list<TemplateFileCreator>
-	 */
-	private function getTypes(): array {
-		if (!empty($this->types)) {
-			return $this->types;
-		}
-		$this->eventDispatcher->dispatchTyped(new RegisterTemplateCreatorEvent($this));
-		foreach ($this->registeredTypes as $registeredType) {
-			$this->types[] = $registeredType();
-		}
-		return $this->types;
-	}
-
-	#[Override]
-	public function listCreators(): array {
-		$types = $this->getTypes();
-		usort($types, function (TemplateFileCreator $a, TemplateFileCreator $b) {
-			return $a->getOrder() - $b->getOrder();
-		});
-		return $types;
-	}
-
-	#[Override]
-	public function listTemplates(): array {
-		return array_values(array_map(function (TemplateFileCreator $entry) {
-			return array_merge($entry->jsonSerialize(), [
-				'templates' => $this->getTemplateFiles($entry)
-			]);
-		}, $this->listCreators()));
-	}
-
-	#[Override]
-	public function listTemplateFields(int $fileId): array {
-		foreach ($this->listCreators() as $creator) {
-			$fields = $this->getTemplateFields($creator, $fileId);
-			if (empty($fields)) {
-				continue;
-			}
-
-			return $fields;
-		}
-
-		return [];
-	}
-
-	#[Override]
-	public function createFromTemplate(string $filePath, string $templateId = '', string $templateType = 'user', array $templateFields = []): array {
-		$userFolder = $this->rootFolder->getUserFolder($this->userId);
-		try {
-			$userFolder->get($filePath);
-			throw new GenericFileException($this->l10n->t('File already exists'));
-		} catch (NotFoundException $e) {
-		}
-		try {
-			if (!$userFolder->nodeExists(dirname($filePath))) {
-				throw new GenericFileException($this->l10n->t('Invalid path'));
-			}
-			/** @var Folder $folder */
-			$folder = $userFolder->get(dirname($filePath));
-			$template = null;
-			if ($templateType === 'user' && $templateId !== '') {
-				$template = $userFolder->get($templateId);
-			} else {
-				$matchingProvider = array_filter($this->getRegisteredProviders(), function (ICustomTemplateProvider $provider) use ($templateType) {
-					return $templateType === get_class($provider);
-				});
-				$provider = array_shift($matchingProvider);
-				if ($provider) {
-					$template = $provider->getCustomTemplate($templateId);
-				}
-			}
-
-			$filename = basename($filePath);
-			$this->filenameValidator->validateFilename($filename);
-			$targetFile = $folder->newFile($filename, ($template instanceof File ? $template->fopen('rb') : null));
-
-			$this->eventDispatcher->dispatchTyped(new FileCreatedFromTemplateEvent($template, $targetFile, $templateFields));
-			/** @var File $file */
-			$file = $userFolder->get($filePath);
-			return $this->formatFile($file);
-		} catch (\Exception $e) {
-			$this->logger->error($e->getMessage(), ['exception' => $e]);
-			throw new GenericFileException($this->l10n->t('Failed to create file from template'));
-		}
-	}
-
-	/**
-	 * @throws \OCP\Files\NotFoundException
-	 * @throws \OCP\Files\NotPermittedException
-	 * @throws \OC\User\NoUserException
-	 */
-	private function getTemplateFolder(): Folder {
-		if ($this->getTemplatePath() !== '') {
-			$path = $this->rootFolder->getUserFolder($this->userId)->get($this->getTemplatePath());
-			if ($path instanceof Folder) {
-				return $path;
-			}
-		}
-		throw new NotFoundException();
-	}
-
-	/**
-	 * @return list<Template>
-	 */
-	private function getTemplateFiles(TemplateFileCreator $type): array {
-		$templates = array_merge(
-			$this->getProviderTemplates($type),
-			$this->getUserTemplates($type)
-		);
-
-		$this->eventDispatcher->dispatchTyped(new BeforeGetTemplatesEvent($templates, false));
-
-		return $templates;
-	}
-
-	/**
-	 * @return list<Template>
-	 */
-	private function getProviderTemplates(TemplateFileCreator $type): array {
-		$templates = [];
-		foreach ($this->getRegisteredProviders() as $provider) {
-			foreach ($type->getMimetypes() as $mimetype) {
-				foreach ($provider->getCustomTemplates($mimetype) as $template) {
-					$templates[] = $template;
-				}
-			}
-		}
-
-		return $templates;
-	}
-
-	/**
-	 * @return list<Template>
-	 */
-	private function getUserTemplates(TemplateFileCreator $type): array {
-		$templates = [];
-
-		try {
-			$userTemplateFolder = $this->getTemplateFolder();
-		} catch (\Exception $e) {
-			return $templates;
-		}
-
-		foreach ($type->getMimetypes() as $mimetype) {
-			foreach ($userTemplateFolder->searchByMime($mimetype) as $templateFile) {
-				if (!($templateFile instanceof File)) {
-					continue;
-				}
-				$template = new Template(
-					'user',
-					$this->rootFolder->getUserFolder($this->userId)->getRelativePath($templateFile->getPath()),
-					$templateFile
-				);
-				$template->setHasPreview($this->previewManager->isAvailable($templateFile));
-				$templates[] = $template;
-			}
-		}
-
-		return $templates;
-	}
-
-	/*
+    /** @var list<callable(): TemplateFileCreator> */
+    private array $registeredTypes = [];
+    /** @var list<TemplateFileCreator> */
+    private array $types = [];
+    /** @var array<class-string<ICustomTemplateProvider>, ICustomTemplateProvider>|null */
+    private ?array $providers = null;
+    private IL10n $l10n;
+    private ?string $userId;
+
+    public function __construct(
+        private readonly ContainerInterface $serverContainer,
+        private readonly IEventDispatcher $eventDispatcher,
+        private readonly Coordinator $bootstrapCoordinator,
+        private readonly IRootFolder $rootFolder,
+        IUserSession $userSession,
+        private readonly IUserManager $userManager,
+        private readonly IPreview $previewManager,
+        private readonly IConfig $config,
+        private readonly IFactory $l10nFactory,
+        private readonly LoggerInterface $logger,
+        private readonly IFilenameValidator $filenameValidator,
+    ) {
+        $this->l10n = $l10nFactory->get('lib');
+        $this->userId = $userSession->getUser()?->getUID();
+    }
+
+    #[Override]
+    public function registerTemplateFileCreator(callable $callback): void {
+        $this->registeredTypes[] = $callback;
+    }
+
+    /**
+     * @return array<class-string<ICustomTemplateProvider>, ICustomTemplateProvider>
+     */
+    private function getRegisteredProviders(): array {
+        if ($this->providers !== null) {
+            return $this->providers;
+        }
+
+        $context = $this->bootstrapCoordinator->getRegistrationContext();
+
+        $this->providers = [];
+        foreach ($context->getTemplateProviders() as $provider) {
+            $class = $provider->getService();
+            $this->providers[$class] = $this->serverContainer->get($class);
+        }
+        return $this->providers;
+    }
+
+    /**
+     * @return list<TemplateFileCreator>
+     */
+    private function getTypes(): array {
+        if (!empty($this->types)) {
+            return $this->types;
+        }
+        $this->eventDispatcher->dispatchTyped(new RegisterTemplateCreatorEvent($this));
+        foreach ($this->registeredTypes as $registeredType) {
+            $this->types[] = $registeredType();
+        }
+        return $this->types;
+    }
+
+    #[Override]
+    public function listCreators(): array {
+        $types = $this->getTypes();
+        usort($types, function (TemplateFileCreator $a, TemplateFileCreator $b) {
+            return $a->getOrder() - $b->getOrder();
+        });
+        return $types;
+    }
+
+    #[Override]
+    public function listTemplates(): array {
+        return array_values(array_map(function (TemplateFileCreator $entry) {
+            return array_merge($entry->jsonSerialize(), [
+                'templates' => $this->getTemplateFiles($entry)
+            ]);
+        }, $this->listCreators()));
+    }
+
+    #[Override]
+    public function listTemplateFields(int $fileId): array {
+        foreach ($this->listCreators() as $creator) {
+            $fields = $this->getTemplateFields($creator, $fileId);
+            if (empty($fields)) {
+                continue;
+            }
+
+            return $fields;
+        }
+
+        return [];
+    }
+
+    #[Override]
+    public function createFromTemplate(string $filePath, string $templateId = '', string $templateType = 'user', array $templateFields = []): array {
+        $userFolder = $this->rootFolder->getUserFolder($this->userId);
+        try {
+            $userFolder->get($filePath);
+            throw new GenericFileException($this->l10n->t('File already exists'));
+        } catch (NotFoundException $e) {
+        }
+        try {
+            if (!$userFolder->nodeExists(dirname($filePath))) {
+                throw new GenericFileException($this->l10n->t('Invalid path'));
+            }
+            /** @var Folder $folder */
+            $folder = $userFolder->get(dirname($filePath));
+            $template = null;
+            if ($templateType === 'user' && $templateId !== '') {
+                $template = $userFolder->get($templateId);
+            } else {
+                $matchingProvider = array_filter($this->getRegisteredProviders(), function (ICustomTemplateProvider $provider) use ($templateType) {
+                    return $templateType === get_class($provider);
+                });
+                $provider = array_shift($matchingProvider);
+                if ($provider) {
+                    $template = $provider->getCustomTemplate($templateId);
+                }
+            }
+
+            $filename = basename($filePath);
+            $this->filenameValidator->validateFilename($filename);
+            $targetFile = $folder->newFile($filename, ($template instanceof File ? $template->fopen('rb') : null));
+
+            $this->eventDispatcher->dispatchTyped(new FileCreatedFromTemplateEvent($template, $targetFile, $templateFields));
+            /** @var File $file */
+            $file = $userFolder->get($filePath);
+            return $this->formatFile($file);
+        } catch (\Exception $e) {
+            $this->logger->error($e->getMessage(), ['exception' => $e]);
+            throw new GenericFileException($this->l10n->t('Failed to create file from template'));
+        }
+    }
+
+    /**
+     * @throws \OCP\Files\NotFoundException
+     * @throws \OCP\Files\NotPermittedException
+     * @throws \OC\User\NoUserException
+     */
+    private function getTemplateFolder(): Folder {
+        if ($this->getTemplatePath() !== '') {
+            $path = $this->rootFolder->getUserFolder($this->userId)->get($this->getTemplatePath());
+            if ($path instanceof Folder) {
+                return $path;
+            }
+        }
+        throw new NotFoundException();
+    }
+
+    /**
+     * @return list<Template>
+     */
+    private function getTemplateFiles(TemplateFileCreator $type): array {
+        $templates = array_merge(
+            $this->getProviderTemplates($type),
+            $this->getUserTemplates($type)
+        );
+
+        $this->eventDispatcher->dispatchTyped(new BeforeGetTemplatesEvent($templates, false));
+
+        return $templates;
+    }
+
+    /**
+     * @return list<Template>
+     */
+    private function getProviderTemplates(TemplateFileCreator $type): array {
+        $templates = [];
+        foreach ($this->getRegisteredProviders() as $provider) {
+            foreach ($type->getMimetypes() as $mimetype) {
+                foreach ($provider->getCustomTemplates($mimetype) as $template) {
+                    $templates[] = $template;
+                }
+            }
+        }
+
+        return $templates;
+    }
+
+    /**
+     * @return list<Template>
+     */
+    private function getUserTemplates(TemplateFileCreator $type): array {
+        $templates = [];
+
+        try {
+            $userTemplateFolder = $this->getTemplateFolder();
+        } catch (\Exception $e) {
+            return $templates;
+        }
+
+        foreach ($type->getMimetypes() as $mimetype) {
+            foreach ($userTemplateFolder->searchByMime($mimetype) as $templateFile) {
+                if (!($templateFile instanceof File)) {
+                    continue;
+                }
+                $template = new Template(
+                    'user',
+                    $this->rootFolder->getUserFolder($this->userId)->getRelativePath($templateFile->getPath()),
+                    $templateFile
+                );
+                $template->setHasPreview($this->previewManager->isAvailable($templateFile));
+                $templates[] = $template;
+            }
+        }
+
+        return $templates;
+    }
+
+    /*
 	 * @return list<Field>
 	 */
-	private function getTemplateFields(TemplateFileCreator $type, int $fileId): array {
-		$providerTemplates = $this->getProviderTemplates($type);
-		$userTemplates = $this->getUserTemplates($type);
-
-		$matchedTemplates = array_filter(
-			array_merge($providerTemplates, $userTemplates),
-			fn (Template $template): bool => $template->jsonSerialize()['fileid'] === $fileId);
-
-		if (empty($matchedTemplates)) {
-			return [];
-		}
-
-		$this->eventDispatcher->dispatchTyped(new BeforeGetTemplatesEvent($matchedTemplates, true));
-
-		return array_values(array_map(static fn (Template $template): array => $template->jsonSerialize()['fields'] ?? [], $matchedTemplates));
-	}
-
-	/**
-	 * @return FilesTemplateFile
-	 * @throws NotFoundException
-	 * @throws \OCP\Files\InvalidPathException
-	 */
-	private function formatFile(File $file): array {
-		return [
-			'basename' => $file->getName(),
-			'etag' => $file->getEtag(),
-			'fileid' => $file->getId() ?? -1,
-			'filename' => $this->rootFolder->getUserFolder($this->userId)->getRelativePath($file->getPath()),
-			'lastmod' => $file->getMTime(),
-			'mime' => $file->getMimetype(),
-			'size' => $file->getSize(),
-			'type' => $file->getType(),
-			'hasPreview' => $this->previewManager->isAvailable($file),
-			'permissions' => $file->getPermissions(),
-		];
-	}
-
-	public function hasTemplateDirectory(): bool {
-		try {
-			$this->getTemplateFolder();
-			return true;
-		} catch (\Exception $e) {
-		}
-		return false;
-	}
-
-	#[Override]
-	public function setTemplatePath(string $path): void {
-		$this->config->setUserValue($this->userId, 'core', 'templateDirectory', $path);
-	}
-
-	#[Override]
-	public function getTemplatePath(): string {
-		return $this->config->getUserValue($this->userId, 'core', 'templateDirectory', '');
-	}
-
-	#[Override]
-	public function initializeTemplateDirectory(?string $path = null, ?string $userId = null, $copyTemplates = true): string {
-		if ($userId !== null) {
-			$this->userId = $userId;
-		}
-
-		$defaultSkeletonDirectory = \OC::$SERVERROOT . '/core/skeleton';
-		$defaultTemplateDirectory = \OC::$SERVERROOT . '/core/skeleton/Templates';
-		$skeletonPath = $this->config->getSystemValueString('skeletondirectory', $defaultSkeletonDirectory);
-		$skeletonTemplatePath = $this->config->getSystemValueString('templatedirectory', $defaultTemplateDirectory);
-		$isDefaultSkeleton = $skeletonPath === $defaultSkeletonDirectory;
-		$isDefaultTemplates = $skeletonTemplatePath === $defaultTemplateDirectory;
-		$userLang = $this->l10nFactory->getUserLanguage($this->userManager->get($this->userId));
-
-		if ($skeletonTemplatePath === '') {
-			$this->setTemplatePath('');
-			return '';
-		}
-
-		try {
-			$l10n = $this->l10nFactory->get('lib', $userLang);
-			$userFolder = $this->rootFolder->getUserFolder($this->userId);
-			$userTemplatePath = $path ?? $this->config->getAppValue('core', 'defaultTemplateDirectory', $l10n->t('Templates')) . '/';
-
-			// Initial user setup without a provided path
-			if ($path === null) {
-				// All locations are default so we just need to rename the directory to the users language
-				if ($isDefaultSkeleton && $isDefaultTemplates) {
-					if (!$userFolder->nodeExists('Templates')) {
-						return '';
-					}
-					$newPath = Filesystem::normalizePath($userFolder->getPath() . '/' . $userTemplatePath);
-					if ($newPath !== $userFolder->get('Templates')->getPath()) {
-						$userFolder->get('Templates')->move($newPath);
-					}
-					$this->setTemplatePath($userTemplatePath);
-					return $userTemplatePath;
-				}
-
-				if ($isDefaultSkeleton && !empty($skeletonTemplatePath) && !$isDefaultTemplates && $userFolder->nodeExists('Templates')) {
-					$shippedSkeletonTemplates = $userFolder->get('Templates');
-					$shippedSkeletonTemplates->delete();
-				}
-			}
-
-			$folder = $userFolder->getOrCreateFolder($userTemplatePath);
-
-			$folderIsEmpty = count($folder->getDirectoryListing()) === 0;
-
-			if (!$copyTemplates) {
-				$this->setTemplatePath($userTemplatePath);
-				return $userTemplatePath;
-			}
-
-			if (!$isDefaultTemplates && $folderIsEmpty) {
-				$localizedSkeletonTemplatePath = $this->getLocalizedTemplatePath($skeletonTemplatePath, $userLang);
-				if (!empty($localizedSkeletonTemplatePath) && file_exists($localizedSkeletonTemplatePath)) {
-					\OC_Util::copyr($localizedSkeletonTemplatePath, $folder);
-					$userFolder->getStorage()->getScanner()->scan($folder->getInternalPath(), Scanner::SCAN_RECURSIVE);
-					$this->setTemplatePath($userTemplatePath);
-					return $userTemplatePath;
-				}
-			}
-
-			if ($path !== null && $isDefaultSkeleton && $isDefaultTemplates && $folderIsEmpty) {
-				$localizedSkeletonPath = $this->getLocalizedTemplatePath($skeletonPath . '/Templates', $userLang);
-				if (!empty($localizedSkeletonPath) && file_exists($localizedSkeletonPath)) {
-					\OC_Util::copyr($localizedSkeletonPath, $folder);
-					$userFolder->getStorage()->getScanner()->scan($folder->getInternalPath(), Scanner::SCAN_RECURSIVE);
-					$this->setTemplatePath($userTemplatePath);
-					return $userTemplatePath;
-				}
-			}
-
-			$this->setTemplatePath($path ?? '');
-			return $this->getTemplatePath();
-		} catch (\Throwable $e) {
-			$this->logger->error('Failed to initialize templates directory to user language ' . $userLang . ' for ' . $userId, ['app' => 'files_templates', 'exception' => $e]);
-		}
-		$this->setTemplatePath('');
-		return $this->getTemplatePath();
-	}
-
-	private function getLocalizedTemplatePath(string $skeletonTemplatePath, string $userLang): string {
-		$localizedSkeletonTemplatePath = str_replace('{lang}', $userLang, $skeletonTemplatePath);
-
-		if (!file_exists($localizedSkeletonTemplatePath)) {
-			$dialectStart = strpos($userLang, '_');
-			if ($dialectStart !== false) {
-				$localizedSkeletonTemplatePath = str_replace('{lang}', substr($userLang, 0, $dialectStart), $skeletonTemplatePath);
-			}
-			if ($dialectStart === false || !file_exists($localizedSkeletonTemplatePath)) {
-				$localizedSkeletonTemplatePath = str_replace('{lang}', 'default', $skeletonTemplatePath);
-			}
-		}
-
-		return $localizedSkeletonTemplatePath;
-	}
+    private function getTemplateFields(TemplateFileCreator $type, int $fileId): array {
+        $providerTemplates = $this->getProviderTemplates($type);
+        $userTemplates = $this->getUserTemplates($type);
+
+        $matchedTemplates = array_filter(
+            array_merge($providerTemplates, $userTemplates),
+            fn (Template $template): bool => $template->jsonSerialize()['fileid'] === $fileId);
+
+        if (empty($matchedTemplates)) {
+            return [];
+        }
+
+        $this->eventDispatcher->dispatchTyped(new BeforeGetTemplatesEvent($matchedTemplates, true));
+
+        return array_values(array_map(static fn (Template $template): array => $template->jsonSerialize()['fields'] ?? [], $matchedTemplates));
+    }
+
+    /**
+     * @return FilesTemplateFile
+     * @throws NotFoundException
+     * @throws \OCP\Files\InvalidPathException
+     */
+    private function formatFile(File $file): array {
+        return [
+            'basename' => $file->getName(),
+            'etag' => $file->getEtag(),
+            'fileid' => $file->getId() ?? -1,
+            'filename' => $this->rootFolder->getUserFolder($this->userId)->getRelativePath($file->getPath()),
+            'lastmod' => $file->getMTime(),
+            'mime' => $file->getMimetype(),
+            'size' => $file->getSize(),
+            'type' => $file->getType(),
+            'hasPreview' => $this->previewManager->isAvailable($file),
+            'permissions' => $file->getPermissions(),
+        ];
+    }
+
+    public function hasTemplateDirectory(): bool {
+        try {
+            $this->getTemplateFolder();
+            return true;
+        } catch (\Exception $e) {
+        }
+        return false;
+    }
+
+    #[Override]
+    public function setTemplatePath(string $path): void {
+        $this->config->setUserValue($this->userId, 'core', 'templateDirectory', $path);
+    }
+
+    #[Override]
+    public function getTemplatePath(): string {
+        return $this->config->getUserValue($this->userId, 'core', 'templateDirectory', '');
+    }
+
+    #[Override]
+    public function initializeTemplateDirectory(?string $path = null, ?string $userId = null, $copyTemplates = true): string {
+        if ($userId !== null) {
+            $this->userId = $userId;
+        }
+
+        $defaultSkeletonDirectory = \OC::$SERVERROOT . '/core/skeleton';
+        $defaultTemplateDirectory = \OC::$SERVERROOT . '/core/skeleton/Templates';
+        $skeletonPath = $this->config->getSystemValueString('skeletondirectory', $defaultSkeletonDirectory);
+        $skeletonTemplatePath = $this->config->getSystemValueString('templatedirectory', $defaultTemplateDirectory);
+        $isDefaultSkeleton = $skeletonPath === $defaultSkeletonDirectory;
+        $isDefaultTemplates = $skeletonTemplatePath === $defaultTemplateDirectory;
+        $userLang = $this->l10nFactory->getUserLanguage($this->userManager->get($this->userId));
+
+        if ($skeletonTemplatePath === '') {
+            $this->setTemplatePath('');
+            return '';
+        }
+
+        try {
+            $l10n = $this->l10nFactory->get('lib', $userLang);
+            $userFolder = $this->rootFolder->getUserFolder($this->userId);
+            $userTemplatePath = $path ?? $this->config->getAppValue('core', 'defaultTemplateDirectory', $l10n->t('Templates')) . '/';
+
+            // Initial user setup without a provided path
+            if ($path === null) {
+                // All locations are default so we just need to rename the directory to the users language
+                if ($isDefaultSkeleton && $isDefaultTemplates) {
+                    if (!$userFolder->nodeExists('Templates')) {
+                        return '';
+                    }
+                    $newPath = Filesystem::normalizePath($userFolder->getPath() . '/' . $userTemplatePath);
+                    if ($newPath !== $userFolder->get('Templates')->getPath()) {
+                        $userFolder->get('Templates')->move($newPath);
+                    }
+                    $this->setTemplatePath($userTemplatePath);
+                    return $userTemplatePath;
+                }
+
+                if ($isDefaultSkeleton && !empty($skeletonTemplatePath) && !$isDefaultTemplates && $userFolder->nodeExists('Templates')) {
+                    $shippedSkeletonTemplates = $userFolder->get('Templates');
+                    $shippedSkeletonTemplates->delete();
+                }
+            }
+
+            $folder = $userFolder->getOrCreateFolder($userTemplatePath);
+
+            $folderIsEmpty = count($folder->getDirectoryListing()) === 0;
+
+            if (!$copyTemplates) {
+                $this->setTemplatePath($userTemplatePath);
+                return $userTemplatePath;
+            }
+
+            if (!$isDefaultTemplates && $folderIsEmpty) {
+                $localizedSkeletonTemplatePath = $this->getLocalizedTemplatePath($skeletonTemplatePath, $userLang);
+                if (!empty($localizedSkeletonTemplatePath) && file_exists($localizedSkeletonTemplatePath)) {
+                    \OC_Util::copyr($localizedSkeletonTemplatePath, $folder);
+                    $userFolder->getStorage()->getScanner()->scan($folder->getInternalPath(), Scanner::SCAN_RECURSIVE);
+                    $this->setTemplatePath($userTemplatePath);
+                    return $userTemplatePath;
+                }
+            }
+
+            if ($path !== null && $isDefaultSkeleton && $isDefaultTemplates && $folderIsEmpty) {
+                $localizedSkeletonPath = $this->getLocalizedTemplatePath($skeletonPath . '/Templates', $userLang);
+                if (!empty($localizedSkeletonPath) && file_exists($localizedSkeletonPath)) {
+                    \OC_Util::copyr($localizedSkeletonPath, $folder);
+                    $userFolder->getStorage()->getScanner()->scan($folder->getInternalPath(), Scanner::SCAN_RECURSIVE);
+                    $this->setTemplatePath($userTemplatePath);
+                    return $userTemplatePath;
+                }
+            }
+
+            $this->setTemplatePath($path ?? '');
+            return $this->getTemplatePath();
+        } catch (\Throwable $e) {
+            $this->logger->error('Failed to initialize templates directory to user language ' . $userLang . ' for ' . $userId, ['app' => 'files_templates', 'exception' => $e]);
+        }
+        $this->setTemplatePath('');
+        return $this->getTemplatePath();
+    }
+
+    private function getLocalizedTemplatePath(string $skeletonTemplatePath, string $userLang): string {
+        $localizedSkeletonTemplatePath = str_replace('{lang}', $userLang, $skeletonTemplatePath);
+
+        if (!file_exists($localizedSkeletonTemplatePath)) {
+            $dialectStart = strpos($userLang, '_');
+            if ($dialectStart !== false) {
+                $localizedSkeletonTemplatePath = str_replace('{lang}', substr($userLang, 0, $dialectStart), $skeletonTemplatePath);
+            }
+            if ($dialectStart === false || !file_exists($localizedSkeletonTemplatePath)) {
+                $localizedSkeletonTemplatePath = str_replace('{lang}', 'default', $skeletonTemplatePath);
+            }
+        }
+
+        return $localizedSkeletonTemplatePath;
+    }
 }

Spacing +9 added lines, -9 removed lines

@@ -106,7 +106,7 @@ 
 	#[Override]
 	public function listCreators(): array {
 		$types = $this->getTypes();
-		usort($types, function (TemplateFileCreator $a, TemplateFileCreator $b) {
+		usort($types, function(TemplateFileCreator $a, TemplateFileCreator $b) {
 			return $a->getOrder() - $b->getOrder();
 		});
 		return $types;
@@ -114,7 +114,7 @@ 
 
 	#[Override]
 	public function listTemplates(): array {
-		return array_values(array_map(function (TemplateFileCreator $entry) {
+		return array_values(array_map(function(TemplateFileCreator $entry) {
 			return array_merge($entry->jsonSerialize(), [
 				'templates' => $this->getTemplateFiles($entry)
 			]);
@@ -153,7 +153,7 @@ 
 			if ($templateType === 'user' && $templateId !== '') {
 				$template = $userFolder->get($templateId);
 			} else {
-				$matchingProvider = array_filter($this->getRegisteredProviders(), function (ICustomTemplateProvider $provider) use ($templateType) {
+				$matchingProvider = array_filter($this->getRegisteredProviders(), function(ICustomTemplateProvider $provider) use ($templateType) {
 					return $templateType === get_class($provider);
 				});
 				$provider = array_shift($matchingProvider);
@@ -316,8 +316,8 @@ 
 			$this->userId = $userId;
 		}
 
-		$defaultSkeletonDirectory = \OC::$SERVERROOT . '/core/skeleton';
-		$defaultTemplateDirectory = \OC::$SERVERROOT . '/core/skeleton/Templates';
+		$defaultSkeletonDirectory = \OC::$SERVERROOT.'/core/skeleton';
+		$defaultTemplateDirectory = \OC::$SERVERROOT.'/core/skeleton/Templates';
 		$skeletonPath = $this->config->getSystemValueString('skeletondirectory', $defaultSkeletonDirectory);
 		$skeletonTemplatePath = $this->config->getSystemValueString('templatedirectory', $defaultTemplateDirectory);
 		$isDefaultSkeleton = $skeletonPath === $defaultSkeletonDirectory;
@@ -332,7 +332,7 @@ 
 		try {
 			$l10n = $this->l10nFactory->get('lib', $userLang);
 			$userFolder = $this->rootFolder->getUserFolder($this->userId);
-			$userTemplatePath = $path ?? $this->config->getAppValue('core', 'defaultTemplateDirectory', $l10n->t('Templates')) . '/';
+			$userTemplatePath = $path ?? $this->config->getAppValue('core', 'defaultTemplateDirectory', $l10n->t('Templates')).'/';
 
 			// Initial user setup without a provided path
 			if ($path === null) {
@@ -341,7 +341,7 @@ 
 					if (!$userFolder->nodeExists('Templates')) {
 						return '';
 					}
-					$newPath = Filesystem::normalizePath($userFolder->getPath() . '/' . $userTemplatePath);
+					$newPath = Filesystem::normalizePath($userFolder->getPath().'/'.$userTemplatePath);
 					if ($newPath !== $userFolder->get('Templates')->getPath()) {
 						$userFolder->get('Templates')->move($newPath);
 					}
@@ -375,7 +375,7 @@ 
 			}
 
 			if ($path !== null && $isDefaultSkeleton && $isDefaultTemplates && $folderIsEmpty) {
-				$localizedSkeletonPath = $this->getLocalizedTemplatePath($skeletonPath . '/Templates', $userLang);
+				$localizedSkeletonPath = $this->getLocalizedTemplatePath($skeletonPath.'/Templates', $userLang);
 				if (!empty($localizedSkeletonPath) && file_exists($localizedSkeletonPath)) {
 					\OC_Util::copyr($localizedSkeletonPath, $folder);
 					$userFolder->getStorage()->getScanner()->scan($folder->getInternalPath(), Scanner::SCAN_RECURSIVE);
@@ -387,7 +387,7 @@ 
 			$this->setTemplatePath($path ?? '');
 			return $this->getTemplatePath();
 		} catch (\Throwable $e) {
-			$this->logger->error('Failed to initialize templates directory to user language ' . $userLang . ' for ' . $userId, ['app' => 'files_templates', 'exception' => $e]);
+			$this->logger->error('Failed to initialize templates directory to user language '.$userLang.' for '.$userId, ['app' => 'files_templates', 'exception' => $e]);
 		}
 		$this->setTemplatePath('');
 		return $this->getTemplatePath();

tests/lib/Files/Node/FileTest.php

Indentation +290 added lines, -290 removed lines

@@ -26,294 +26,294 @@
  */
 #[\PHPUnit\Framework\Attributes\Group('DB')]
 class FileTest extends NodeTestCase {
-	protected function createTestNode(IRootFolder $root, View&MockObject $view, string $path, array $data = [], string $internalPath = '', ?IStorage $storage = null): File {
-		if ($data || $internalPath || $storage) {
-			return new File($root, $view, $path, $this->getFileInfo($data, $internalPath, $storage));
-		} else {
-			return new File($root, $view, $path);
-		}
-	}
-
-	protected function getNodeClass(): string {
-		return File::class;
-	}
-
-	protected function getNonExistingNodeClass(): string {
-		return NonExistingFile::class;
-	}
-
-	protected function getViewDeleteMethod(): string {
-		return 'unlink';
-	}
-
-	public function testGetContent(): void {
-		/** @var \OC\Files\Node\Root|\PHPUnit\Framework\MockObject\MockObject $root */
-		$root = $this->getMockBuilder(Root::class)
-			->setConstructorArgs([$this->manager, $this->view, $this->user, $this->userMountCache, $this->logger, $this->userManager, $this->eventDispatcher, $this->cacheFactory, $this->appConfig])
-			->getMock();
-
-		$hook = function ($file): void {
-			throw new \Exception('Hooks are not supposed to be called');
-		};
-
-		$root->listen('\OC\Files', 'preWrite', $hook);
-		$root->listen('\OC\Files', 'postWrite', $hook);
-
-		$this->view->expects($this->once())
-			->method('file_get_contents')
-			->with('/bar/foo')
-			->willReturn('bar');
-
-		$this->view->expects($this->once())
-			->method('getFileInfo')
-			->with('/bar/foo')
-			->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_READ]));
-
-		$node = new File($root, $this->view, '/bar/foo');
-		$this->assertEquals('bar', $node->getContent());
-	}
-
-
-	public function testGetContentNotPermitted(): void {
-		$this->expectException(NotPermittedException::class);
-
-		/** @var \OC\Files\Node\Root|\PHPUnit\Framework\MockObject\MockObject $root */
-		$root = $this->getMockBuilder(Root::class)
-			->setConstructorArgs([$this->manager, $this->view, $this->user, $this->userMountCache, $this->logger, $this->userManager, $this->eventDispatcher, $this->cacheFactory, $this->appConfig])
-			->getMock();
-
-		$root->expects($this->any())
-			->method('getUser')
-			->willReturn($this->user);
-
-		$this->view->expects($this->once())
-			->method('getFileInfo')
-			->with('/bar/foo')
-			->willReturn($this->getFileInfo(['permissions' => 0]));
-
-		$node = new File($root, $this->view, '/bar/foo');
-		$node->getContent();
-	}
-
-	public function testPutContent(): void {
-		/** @var \OC\Files\Node\Root|\PHPUnit\Framework\MockObject\MockObject $root */
-		$root = $this->getMockBuilder(Root::class)
-			->setConstructorArgs([$this->manager, $this->view, $this->user, $this->userMountCache, $this->logger, $this->userManager, $this->eventDispatcher, $this->cacheFactory, $this->appConfig])
-			->getMock();
-
-		$root->expects($this->any())
-			->method('getUser')
-			->willReturn($this->user);
-
-		$this->view->expects($this->once())
-			->method('getFileInfo')
-			->with('/bar/foo')
-			->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_ALL]));
-
-		$this->view->expects($this->once())
-			->method('file_put_contents')
-			->with('/bar/foo', 'bar')
-			->willReturn(true);
-
-		$node = new File($root, $this->view, '/bar/foo');
-		$node->putContent('bar');
-	}
-
-
-	public function testPutContentNotPermitted(): void {
-		$this->expectException(NotPermittedException::class);
-
-		/** @var \OC\Files\Node\Root|\PHPUnit\Framework\MockObject\MockObject $root */
-		$root = $this->getMockBuilder(Root::class)
-			->setConstructorArgs([$this->manager, $this->view, $this->user, $this->userMountCache, $this->logger, $this->userManager, $this->eventDispatcher, $this->cacheFactory, $this->appConfig])
-			->getMock();
-
-		$this->view->expects($this->once())
-			->method('getFileInfo')
-			->with('/bar/foo')
-			->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_READ]));
-
-		$node = new File($root, $this->view, '/bar/foo');
-		$node->putContent('bar');
-	}
-
-	public function testGetMimeType(): void {
-		/** @var \OC\Files\Node\Root|\PHPUnit\Framework\MockObject\MockObject $root */
-		$root = $this->getMockBuilder(Root::class)
-			->setConstructorArgs([$this->manager, $this->view, $this->user, $this->userMountCache, $this->logger, $this->userManager, $this->eventDispatcher, $this->cacheFactory, $this->appConfig])
-			->getMock();
-
-		$this->view->expects($this->once())
-			->method('getFileInfo')
-			->with('/bar/foo')
-			->willReturn($this->getFileInfo(['mimetype' => 'text/plain']));
-
-		$node = new File($root, $this->view, '/bar/foo');
-		$this->assertEquals('text/plain', $node->getMimeType());
-	}
-
-	public function testFOpenRead(): void {
-		$stream = fopen('php://memory', 'w+');
-		fwrite($stream, 'bar');
-		rewind($stream);
-
-		$root = new Root(
-			$this->manager,
-			$this->view,
-			$this->user,
-			$this->userMountCache,
-			$this->logger,
-			$this->userManager,
-			$this->eventDispatcher,
-			$this->cacheFactory,
-			$this->appConfig,
-		);
-
-		$hook = function ($file): void {
-			throw new \Exception('Hooks are not supposed to be called');
-		};
-
-		$root->listen('\OC\Files', 'preWrite', $hook);
-		$root->listen('\OC\Files', 'postWrite', $hook);
-
-		$this->view->expects($this->once())
-			->method('fopen')
-			->with('/bar/foo', 'r')
-			->willReturn($stream);
-
-		$this->view->expects($this->once())
-			->method('getFileInfo')
-			->with('/bar/foo')
-			->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_ALL]));
-
-		$node = new File($root, $this->view, '/bar/foo');
-		$fh = $node->fopen('r');
-		$this->assertEquals($stream, $fh);
-		$this->assertEquals('bar', fread($fh, 3));
-	}
-
-	public function testFOpenWrite(): void {
-		$stream = fopen('php://memory', 'w+');
-
-		$root = new Root(
-			$this->manager,
-			$this->view,
-			$this->user,
-			$this->userMountCache,
-			$this->logger,
-			$this->userManager,
-			$this->eventDispatcher,
-			$this->cacheFactory,
-			$this->appConfig,
-		);
-		$hooksCalled = 0;
-		$hook = function ($file) use (&$hooksCalled): void {
-			$hooksCalled++;
-		};
-
-		$root->listen('\OC\Files', 'preWrite', $hook);
-		$root->listen('\OC\Files', 'postWrite', $hook);
-
-		$this->view->expects($this->once())
-			->method('fopen')
-			->with('/bar/foo', 'w')
-			->willReturn($stream);
-
-		$this->view->expects($this->once())
-			->method('getFileInfo')
-			->with('/bar/foo')
-			->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_ALL]));
-
-		$node = new File($root, $this->view, '/bar/foo');
-		$fh = $node->fopen('w');
-		$this->assertEquals($stream, $fh);
-		fwrite($fh, 'bar');
-		rewind($fh);
-		$this->assertEquals('bar', fread($stream, 3));
-		$this->assertEquals(2, $hooksCalled);
-	}
-
-
-	public function testFOpenReadNotPermitted(): void {
-		$this->expectException(NotPermittedException::class);
-
-		$root = new Root(
-			$this->manager,
-			$this->view,
-			$this->user,
-			$this->userMountCache,
-			$this->logger,
-			$this->userManager,
-			$this->eventDispatcher,
-			$this->cacheFactory,
-			$this->appConfig,
-		);
-		$hook = function ($file): void {
-			throw new \Exception('Hooks are not supposed to be called');
-		};
-
-		$this->view->expects($this->once())
-			->method('getFileInfo')
-			->with('/bar/foo')
-			->willReturn($this->getFileInfo(['permissions' => 0]));
-
-		$node = new File($root, $this->view, '/bar/foo');
-		$node->fopen('r');
-	}
-
-
-	public function testFOpenReadWriteNoReadPermissions(): void {
-		$this->expectException(NotPermittedException::class);
-
-		$root = new Root(
-			$this->manager,
-			$this->view,
-			$this->user,
-			$this->userMountCache,
-			$this->logger,
-			$this->userManager,
-			$this->eventDispatcher,
-			$this->cacheFactory,
-			$this->appConfig,
-		);
-		$hook = function (): void {
-			throw new \Exception('Hooks are not supposed to be called');
-		};
-
-		$this->view->expects($this->once())
-			->method('getFileInfo')
-			->with('/bar/foo')
-			->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_UPDATE]));
-
-		$node = new File($root, $this->view, '/bar/foo');
-		$node->fopen('w');
-	}
-
-
-	public function testFOpenReadWriteNoWritePermissions(): void {
-		$this->expectException(NotPermittedException::class);
-
-		$root = new Root(
-			$this->manager,
-			$this->view,
-			$this->user,
-			$this->userMountCache,
-			$this->logger,
-			$this->userManager,
-			$this->eventDispatcher,
-			$this->cacheFactory,
-			$this->appConfig,
-		);
-		$hook = function (): void {
-			throw new \Exception('Hooks are not supposed to be called');
-		};
-
-		$this->view->expects($this->once())
-			->method('getFileInfo')
-			->with('/bar/foo')
-			->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_READ]));
-
-		$node = new File($root, $this->view, '/bar/foo');
-		$node->fopen('w');
-	}
+    protected function createTestNode(IRootFolder $root, View&MockObject $view, string $path, array $data = [], string $internalPath = '', ?IStorage $storage = null): File {
+        if ($data || $internalPath || $storage) {
+            return new File($root, $view, $path, $this->getFileInfo($data, $internalPath, $storage));
+        } else {
+            return new File($root, $view, $path);
+        }
+    }
+
+    protected function getNodeClass(): string {
+        return File::class;
+    }
+
+    protected function getNonExistingNodeClass(): string {
+        return NonExistingFile::class;
+    }
+
+    protected function getViewDeleteMethod(): string {
+        return 'unlink';
+    }
+
+    public function testGetContent(): void {
+        /** @var \OC\Files\Node\Root|\PHPUnit\Framework\MockObject\MockObject $root */
+        $root = $this->getMockBuilder(Root::class)
+            ->setConstructorArgs([$this->manager, $this->view, $this->user, $this->userMountCache, $this->logger, $this->userManager, $this->eventDispatcher, $this->cacheFactory, $this->appConfig])
+            ->getMock();
+
+        $hook = function ($file): void {
+            throw new \Exception('Hooks are not supposed to be called');
+        };
+
+        $root->listen('\OC\Files', 'preWrite', $hook);
+        $root->listen('\OC\Files', 'postWrite', $hook);
+
+        $this->view->expects($this->once())
+            ->method('file_get_contents')
+            ->with('/bar/foo')
+            ->willReturn('bar');
+
+        $this->view->expects($this->once())
+            ->method('getFileInfo')
+            ->with('/bar/foo')
+            ->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_READ]));
+
+        $node = new File($root, $this->view, '/bar/foo');
+        $this->assertEquals('bar', $node->getContent());
+    }
+
+
+    public function testGetContentNotPermitted(): void {
+        $this->expectException(NotPermittedException::class);
+
+        /** @var \OC\Files\Node\Root|\PHPUnit\Framework\MockObject\MockObject $root */
+        $root = $this->getMockBuilder(Root::class)
+            ->setConstructorArgs([$this->manager, $this->view, $this->user, $this->userMountCache, $this->logger, $this->userManager, $this->eventDispatcher, $this->cacheFactory, $this->appConfig])
+            ->getMock();
+
+        $root->expects($this->any())
+            ->method('getUser')
+            ->willReturn($this->user);
+
+        $this->view->expects($this->once())
+            ->method('getFileInfo')
+            ->with('/bar/foo')
+            ->willReturn($this->getFileInfo(['permissions' => 0]));
+
+        $node = new File($root, $this->view, '/bar/foo');
+        $node->getContent();
+    }
+
+    public function testPutContent(): void {
+        /** @var \OC\Files\Node\Root|\PHPUnit\Framework\MockObject\MockObject $root */
+        $root = $this->getMockBuilder(Root::class)
+            ->setConstructorArgs([$this->manager, $this->view, $this->user, $this->userMountCache, $this->logger, $this->userManager, $this->eventDispatcher, $this->cacheFactory, $this->appConfig])
+            ->getMock();
+
+        $root->expects($this->any())
+            ->method('getUser')
+            ->willReturn($this->user);
+
+        $this->view->expects($this->once())
+            ->method('getFileInfo')
+            ->with('/bar/foo')
+            ->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_ALL]));
+
+        $this->view->expects($this->once())
+            ->method('file_put_contents')
+            ->with('/bar/foo', 'bar')
+            ->willReturn(true);
+
+        $node = new File($root, $this->view, '/bar/foo');
+        $node->putContent('bar');
+    }
+
+
+    public function testPutContentNotPermitted(): void {
+        $this->expectException(NotPermittedException::class);
+
+        /** @var \OC\Files\Node\Root|\PHPUnit\Framework\MockObject\MockObject $root */
+        $root = $this->getMockBuilder(Root::class)
+            ->setConstructorArgs([$this->manager, $this->view, $this->user, $this->userMountCache, $this->logger, $this->userManager, $this->eventDispatcher, $this->cacheFactory, $this->appConfig])
+            ->getMock();
+
+        $this->view->expects($this->once())
+            ->method('getFileInfo')
+            ->with('/bar/foo')
+            ->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_READ]));
+
+        $node = new File($root, $this->view, '/bar/foo');
+        $node->putContent('bar');
+    }
+
+    public function testGetMimeType(): void {
+        /** @var \OC\Files\Node\Root|\PHPUnit\Framework\MockObject\MockObject $root */
+        $root = $this->getMockBuilder(Root::class)
+            ->setConstructorArgs([$this->manager, $this->view, $this->user, $this->userMountCache, $this->logger, $this->userManager, $this->eventDispatcher, $this->cacheFactory, $this->appConfig])
+            ->getMock();
+
+        $this->view->expects($this->once())
+            ->method('getFileInfo')
+            ->with('/bar/foo')
+            ->willReturn($this->getFileInfo(['mimetype' => 'text/plain']));
+
+        $node = new File($root, $this->view, '/bar/foo');
+        $this->assertEquals('text/plain', $node->getMimeType());
+    }
+
+    public function testFOpenRead(): void {
+        $stream = fopen('php://memory', 'w+');
+        fwrite($stream, 'bar');
+        rewind($stream);
+
+        $root = new Root(
+            $this->manager,
+            $this->view,
+            $this->user,
+            $this->userMountCache,
+            $this->logger,
+            $this->userManager,
+            $this->eventDispatcher,
+            $this->cacheFactory,
+            $this->appConfig,
+        );
+
+        $hook = function ($file): void {
+            throw new \Exception('Hooks are not supposed to be called');
+        };
+
+        $root->listen('\OC\Files', 'preWrite', $hook);
+        $root->listen('\OC\Files', 'postWrite', $hook);
+
+        $this->view->expects($this->once())
+            ->method('fopen')
+            ->with('/bar/foo', 'r')
+            ->willReturn($stream);
+
+        $this->view->expects($this->once())
+            ->method('getFileInfo')
+            ->with('/bar/foo')
+            ->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_ALL]));
+
+        $node = new File($root, $this->view, '/bar/foo');
+        $fh = $node->fopen('r');
+        $this->assertEquals($stream, $fh);
+        $this->assertEquals('bar', fread($fh, 3));
+    }
+
+    public function testFOpenWrite(): void {
+        $stream = fopen('php://memory', 'w+');
+
+        $root = new Root(
+            $this->manager,
+            $this->view,
+            $this->user,
+            $this->userMountCache,
+            $this->logger,
+            $this->userManager,
+            $this->eventDispatcher,
+            $this->cacheFactory,
+            $this->appConfig,
+        );
+        $hooksCalled = 0;
+        $hook = function ($file) use (&$hooksCalled): void {
+            $hooksCalled++;
+        };
+
+        $root->listen('\OC\Files', 'preWrite', $hook);
+        $root->listen('\OC\Files', 'postWrite', $hook);
+
+        $this->view->expects($this->once())
+            ->method('fopen')
+            ->with('/bar/foo', 'w')
+            ->willReturn($stream);
+
+        $this->view->expects($this->once())
+            ->method('getFileInfo')
+            ->with('/bar/foo')
+            ->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_ALL]));
+
+        $node = new File($root, $this->view, '/bar/foo');
+        $fh = $node->fopen('w');
+        $this->assertEquals($stream, $fh);
+        fwrite($fh, 'bar');
+        rewind($fh);
+        $this->assertEquals('bar', fread($stream, 3));
+        $this->assertEquals(2, $hooksCalled);
+    }
+
+
+    public function testFOpenReadNotPermitted(): void {
+        $this->expectException(NotPermittedException::class);
+
+        $root = new Root(
+            $this->manager,
+            $this->view,
+            $this->user,
+            $this->userMountCache,
+            $this->logger,
+            $this->userManager,
+            $this->eventDispatcher,
+            $this->cacheFactory,
+            $this->appConfig,
+        );
+        $hook = function ($file): void {
+            throw new \Exception('Hooks are not supposed to be called');
+        };
+
+        $this->view->expects($this->once())
+            ->method('getFileInfo')
+            ->with('/bar/foo')
+            ->willReturn($this->getFileInfo(['permissions' => 0]));
+
+        $node = new File($root, $this->view, '/bar/foo');
+        $node->fopen('r');
+    }
+
+
+    public function testFOpenReadWriteNoReadPermissions(): void {
+        $this->expectException(NotPermittedException::class);
+
+        $root = new Root(
+            $this->manager,
+            $this->view,
+            $this->user,
+            $this->userMountCache,
+            $this->logger,
+            $this->userManager,
+            $this->eventDispatcher,
+            $this->cacheFactory,
+            $this->appConfig,
+        );
+        $hook = function (): void {
+            throw new \Exception('Hooks are not supposed to be called');
+        };
+
+        $this->view->expects($this->once())
+            ->method('getFileInfo')
+            ->with('/bar/foo')
+            ->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_UPDATE]));
+
+        $node = new File($root, $this->view, '/bar/foo');
+        $node->fopen('w');
+    }
+
+
+    public function testFOpenReadWriteNoWritePermissions(): void {
+        $this->expectException(NotPermittedException::class);
+
+        $root = new Root(
+            $this->manager,
+            $this->view,
+            $this->user,
+            $this->userMountCache,
+            $this->logger,
+            $this->userManager,
+            $this->eventDispatcher,
+            $this->cacheFactory,
+            $this->appConfig,
+        );
+        $hook = function (): void {
+            throw new \Exception('Hooks are not supposed to be called');
+        };
+
+        $this->view->expects($this->once())
+            ->method('getFileInfo')
+            ->with('/bar/foo')
+            ->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_READ]));
+
+        $node = new File($root, $this->view, '/bar/foo');
+        $node->fopen('w');
+    }
 }

tests/lib/Files/Node/NodeTestCase.php

Indentation +699 added lines, -699 removed lines

@@ -39,746 +39,746 @@
  * @package Test\Files\Node
  */
 abstract class NodeTestCase extends \Test\TestCase {
-	protected IUser&MockObject $user;
-	protected Manager&MockObject $manager;
-	protected View&MockObject $view;
-	protected Root&MockObject $root;
-	protected IUserMountCache&MockObject $userMountCache;
-	protected LoggerInterface&MockObject $logger;
-	protected IUserManager&MockObject $userManager;
-	protected IEventDispatcher&MockObject $eventDispatcher;
-	protected ICacheFactory&MockObject $cacheFactory;
-	protected IAppConfig&MockObject $appConfig;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->user = $this->createMock(IUser::class);
-		$this->manager = $this->getMockBuilder(Manager::class)
-			->disableOriginalConstructor()
-			->getMock();
-		$this->view = $this->getMockBuilder(View::class)
-			->disableOriginalConstructor()
-			->getMock();
-		$this->view->expects($this->any())
-			->method('getRoot')
-			->willReturn('');
-		$this->userMountCache = $this->getMockBuilder('\OCP\Files\Config\IUserMountCache')
-			->disableOriginalConstructor()
-			->getMock();
-		$this->logger = $this->createMock(LoggerInterface::class);
-		$this->userManager = $this->createMock(IUserManager::class);
-		$this->eventDispatcher = $this->createMock(IEventDispatcher::class);
-		$this->cacheFactory = $this->createMock(ICacheFactory::class);
-		$this->cacheFactory->method('createLocal')
-			->willReturnCallback(function () {
-				return new ArrayCache();
-			});
-		$this->appConfig = $this->createMock(IAppConfig::class);
-
-		$this->root = $this->getMockBuilder(Root::class)
-			->setConstructorArgs([$this->manager, $this->view, $this->user, $this->userMountCache, $this->logger, $this->userManager, $this->eventDispatcher, $this->cacheFactory, $this->appConfig])
-			->getMock();
-	}
-
-	/**
-	 * @return View|\PHPUnit\Framework\MockObject\MockObject $view
-	 */
-	protected function getRootViewMock() {
-		$view = $this->createMock(View::class);
-		$view->expects($this->any())
-			->method('getRoot')
-			->willReturn('');
-		return $view;
-	}
-
-	abstract protected function createTestNode(IRootFolder $root, View&MockObject $view, string $path, array $data = [], string $internalPath = '', ?IStorage $storage = null): Node;
-
-	/**
-	 * @return class-string<Node>
-	 */
-	abstract protected function getNodeClass(): string;
-
-	/**
-	 * @return class-string<Node>
-	 */
-	abstract protected function getNonExistingNodeClass(): string;
-
-	/**
-	 * @return string
-	 */
-	abstract protected function getViewDeleteMethod(): string;
-
-	protected function getMockStorage(): IStorage&MockObject {
-		$storage = $this->getMockBuilder(IStorage::class)
-			->disableOriginalConstructor()
-			->getMock();
-		$storage->expects($this->any())
-			->method('getId')
-			->willReturn('home::someuser');
-		return $storage;
-	}
-
-	protected function getFileInfo($data, $internalPath = '', ?IStorage $storage = null) {
-		$mount = $this->createMock(IMountPoint::class);
-		$mount->method('getStorage')
-			->willReturn($storage);
-		return new FileInfo('', $this->getMockStorage(), $internalPath, $data, $mount);
-	}
-
-	public function testDelete(): void {
-		$this->root->expects($this->exactly(2))
-			->method('emit')
-			->willReturn(true);
-		$this->root->expects($this->any())
-			->method('getUser')
-			->willReturn($this->user);
-
-		$this->view->expects($this->once())
-			->method('getFileInfo')
-			->with('/bar/foo')
-			->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_ALL]));
-
-		$this->view->expects($this->once())
-			->method($this->getViewDeleteMethod())
-			->with('/bar/foo')
-			->willReturn(true);
-
-		$node = $this->createTestNode($this->root, $this->view, '/bar/foo');
-		$node->delete();
-	}
-
-	public function testDeleteHooks(): void {
-		$test = $this;
-		$hooksRun = 0;
-		/**
-		 * @param \OC\Files\Node\File $node
-		 */
-		$preListener = function ($node) use (&$test, &$hooksRun): void {
-			$test->assertInstanceOf($this->getNodeClass(), $node);
-			$test->assertEquals('foo', $node->getInternalPath());
-			$test->assertEquals('/bar/foo', $node->getPath());
-			$test->assertEquals(1, $node->getId());
-			$hooksRun++;
-		};
-
-		/**
-		 * @param \OC\Files\Node\File $node
-		 */
-		$postListener = function ($node) use (&$test, &$hooksRun): void {
-			$test->assertInstanceOf($this->getNonExistingNodeClass(), $node);
-			$test->assertEquals('foo', $node->getInternalPath());
-			$test->assertEquals('/bar/foo', $node->getPath());
-			$test->assertEquals(1, $node->getId());
-			$test->assertEquals('text/plain', $node->getMimeType());
-			$hooksRun++;
-		};
-
-		$root = new Root(
-			$this->manager,
-			$this->view,
-			$this->user,
-			$this->userMountCache,
-			$this->logger,
-			$this->userManager,
-			$this->eventDispatcher,
-			$this->cacheFactory,
-			$this->createMock(IAppConfig::class),
-		);
-
-		$root->listen('\OC\Files', 'preDelete', $preListener);
-		$root->listen('\OC\Files', 'postDelete', $postListener);
-
-		$this->view->expects($this->any())
-			->method('getFileInfo')
-			->with('/bar/foo')
-			->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_ALL, 'fileid' => 1, 'mimetype' => 'text/plain'], 'foo'));
-
-		$this->view->expects($this->once())
-			->method($this->getViewDeleteMethod())
-			->with('/bar/foo')
-			->willReturn(true);
-
-		$node = $this->createTestNode($root, $this->view, '/bar/foo');
-		$node->delete();
-		$this->assertEquals(2, $hooksRun);
-	}
-
-
-	public function testDeleteNotPermitted(): void {
-		$this->expectException(NotPermittedException::class);
-
-		$this->root->expects($this->any())
-			->method('getUser')
-			->willReturn($this->user);
-
-		$this->view->expects($this->once())
-			->method('getFileInfo')
-			->with('/bar/foo')
-			->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_READ]));
-
-		$node = $this->createTestNode($this->root, $this->view, '/bar/foo');
-		$node->delete();
-	}
-
-
-	public function testStat(): void {
-		$this->root->expects($this->any())
-			->method('getUser')
-			->willReturn($this->user);
-
-		$stat = [
-			'fileid' => 1,
-			'size' => 100,
-			'etag' => 'qwerty',
-			'mtime' => 50,
-			'permissions' => 0
-		];
-
-		$this->view->expects($this->once())
-			->method('stat')
-			->with('/bar/foo')
-			->willReturn($stat);
-
-		$node = $this->createTestNode($this->root, $this->view, '/bar/foo');
-		$this->assertEquals($stat, $node->stat());
-	}
-
-	public function testGetId(): void {
-		$this->root->expects($this->any())
-			->method('getUser')
-			->willReturn($this->user);
-
-		$stat = $this->getFileInfo([
-			'fileid' => 1,
-			'size' => 100,
-			'etag' => 'qwerty',
-			'mtime' => 50
-		]);
-
-		$this->view->expects($this->once())
-			->method('getFileInfo')
-			->with('/bar/foo')
-			->willReturn($stat);
-
-		$node = $this->createTestNode($this->root, $this->view, '/bar/foo');
-		$this->assertEquals(1, $node->getId());
-	}
-
-	public function testGetSize(): void {
-		$this->root->expects($this->any())
-			->method('getUser')
-			->willReturn($this->user);
-
-
-		$stat = $this->getFileInfo([
-			'fileid' => 1,
-			'size' => 100,
-			'etag' => 'qwerty',
-			'mtime' => 50
-		]);
-
-		$this->view->expects($this->once())
-			->method('getFileInfo')
-			->with('/bar/foo')
-			->willReturn($stat);
-
-		$node = $this->createTestNode($this->root, $this->view, '/bar/foo');
-		$this->assertEquals(100, $node->getSize());
-	}
-
-	public function testGetEtag(): void {
-		$this->root->expects($this->any())
-			->method('getUser')
-			->willReturn($this->user);
-
-		$stat = $this->getFileInfo([
-			'fileid' => 1,
-			'size' => 100,
-			'etag' => 'qwerty',
-			'mtime' => 50
-		]);
-
-		$this->view->expects($this->once())
-			->method('getFileInfo')
-			->with('/bar/foo')
-			->willReturn($stat);
-
-		$node = $this->createTestNode($this->root, $this->view, '/bar/foo');
-		$this->assertEquals('qwerty', $node->getEtag());
-	}
-
-	public function testGetMTime(): void {
-		$this->root->expects($this->any())
-			->method('getUser')
-			->willReturn($this->user);
-
-		$stat = $this->getFileInfo([
-			'fileid' => 1,
-			'size' => 100,
-			'etag' => 'qwerty',
-			'mtime' => 50
-		]);
-
-		$this->view->expects($this->once())
-			->method('getFileInfo')
-			->with('/bar/foo')
-			->willReturn($stat);
-
-		$node = $this->createTestNode($this->root, $this->view, '/bar/foo');
-		$this->assertEquals(50, $node->getMTime());
-	}
-
-	public function testGetStorage(): void {
-		$this->root->expects($this->any())
-			->method('getUser')
-			->willReturn($this->user);
-		/**
-		 * @var Storage|\PHPUnit\Framework\MockObject\MockObject $storage
-		 */
-		$storage = $this->getMockBuilder('\OC\Files\Storage\Storage')
-			->disableOriginalConstructor()
-			->getMock();
-
-		$node = $this->createTestNode($this->root, $this->view, '/bar/foo', [], 'foo', $storage);
-		$this->assertEquals($storage, $node->getStorage());
-	}
-
-	public function testGetPath(): void {
-		$this->root->expects($this->any())
-			->method('getUser')
-			->willReturn($this->user);
-
-		$node = $this->createTestNode($this->root, $this->view, '/bar/foo');
-		$this->assertEquals('/bar/foo', $node->getPath());
-	}
-
-	public function testGetInternalPath(): void {
-		$this->root->expects($this->any())
-			->method('getUser')
-			->willReturn($this->user);
-		/**
-		 * @var Storage|\PHPUnit\Framework\MockObject\MockObject $storage
-		 */
-		$storage = $this->getMockBuilder('\OC\Files\Storage\Storage')
-			->disableOriginalConstructor()
-			->getMock();
-
-		$this->view->expects($this->once())
-			->method('getFileInfo')
-			->with('/bar/foo')
-			->willReturn($this->getFileInfo([], 'foo'));
-
-
-		$node = $this->createTestNode($this->root, $this->view, '/bar/foo');
-		$this->assertEquals('foo', $node->getInternalPath());
-	}
-
-	public function testGetName(): void {
-		$this->root->expects($this->any())
-			->method('getUser')
-			->willReturn($this->user);
-
-		$node = $this->createTestNode($this->root, $this->view, '/bar/foo');
-		$this->assertEquals('foo', $node->getName());
-	}
-
-	public function testTouchSetMTime(): void {
-		$this->root->expects($this->any())
-			->method('getUser')
-			->willReturn($this->user);
-
-		$this->view->expects($this->once())
-			->method('touch')
-			->with('/bar/foo', 100)
-			->willReturn(true);
-
-		$this->view->expects($this->once())
-			->method('getFileInfo')
-			->with('/bar/foo')
-			->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_ALL]));
-
-		$node = $this->createTestNode($this->root, $this->view, '/bar/foo');
-		$node->touch(100);
-		$this->assertEquals(100, $node->getMTime());
-	}
-
-	public function testTouchHooks(): void {
-		$test = $this;
-		$hooksRun = 0;
-		/**
-		 * @param \OC\Files\Node\File $node
-		 */
-		$preListener = function ($node) use (&$test, &$hooksRun): void {
-			$test->assertEquals('foo', $node->getInternalPath());
-			$test->assertEquals('/bar/foo', $node->getPath());
-			$hooksRun++;
-		};
-
-		/**
-		 * @param \OC\Files\Node\File $node
-		 */
-		$postListener = function ($node) use (&$test, &$hooksRun): void {
-			$test->assertEquals('foo', $node->getInternalPath());
-			$test->assertEquals('/bar/foo', $node->getPath());
-			$hooksRun++;
-		};
-
-		$root = new Root(
-			$this->manager,
-			$this->view,
-			$this->user,
-			$this->userMountCache,
-			$this->logger,
-			$this->userManager,
-			$this->eventDispatcher,
-			$this->cacheFactory,
-			$this->createMock(IAppConfig::class),
-		);
-		$root->listen('\OC\Files', 'preTouch', $preListener);
-		$root->listen('\OC\Files', 'postTouch', $postListener);
-
-		$this->view->expects($this->once())
-			->method('touch')
-			->with('/bar/foo', 100)
-			->willReturn(true);
-
-		$this->view->expects($this->any())
-			->method('getFileInfo')
-			->with('/bar/foo')
-			->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_ALL], 'foo'));
-
-		$node = $this->createTestNode($root, $this->view, '/bar/foo');
-		$node->touch(100);
-		$this->assertEquals(2, $hooksRun);
-	}
-
-
-	public function testTouchNotPermitted(): void {
-		$this->expectException(NotPermittedException::class);
-
-		$this->root->expects($this->any())
-			->method('getUser')
-			->willReturn($this->user);
-
-		$this->view->expects($this->any())
-			->method('getFileInfo')
-			->with('/bar/foo')
-			->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_READ]));
-
-		$node = $this->createTestNode($this->root, $this->view, '/bar/foo');
-		$node->touch(100);
-	}
-
-
-	public function testInvalidPath(): void {
-		$this->expectException(InvalidPathException::class);
-
-		$node = $this->createTestNode($this->root, $this->view, '/../foo');
-		$node->getFileInfo();
-	}
-
-	public function testCopySameStorage(): void {
-		$this->view->expects($this->any())
-			->method('copy')
-			->with('/bar/foo', '/bar/asd')
-			->willReturn(true);
-
-		$this->view->expects($this->any())
-			->method('getFileInfo')
-			->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_ALL, 'fileid' => 3]));
-
-		$node = $this->createTestNode($this->root, $this->view, '/bar/foo');
-		$parentNode = new Folder($this->root, $this->view, '/bar');
-		$newNode = $this->createTestNode($this->root, $this->view, '/bar/asd');
-
-		$this->root->method('get')
-			->willReturnMap([
-				['/bar/asd', $newNode],
-				['/bar', $parentNode]
-			]);
-
-		$target = $node->copy('/bar/asd');
-		$this->assertInstanceOf($this->getNodeClass(), $target);
-		$this->assertEquals(3, $target->getId());
-	}
-
-
-	public function testCopyNotPermitted(): void {
-		$this->expectException(NotPermittedException::class);
-
-		/**
-		 * @var Storage|\PHPUnit\Framework\MockObject\MockObject $storage
-		 */
-		$storage = $this->createMock('\OC\Files\Storage\Storage');
-
-		$this->root->expects($this->never())
-			->method('getMount');
-
-		$storage->expects($this->never())
-			->method('copy');
-
-		$this->view->expects($this->any())
-			->method('getFileInfo')
-			->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_READ, 'fileid' => 3]));
-
-		$node = $this->createTestNode($this->root, $this->view, '/bar/foo');
-		$parentNode = new Folder($this->root, $this->view, '/bar');
-
-		$this->root->expects($this->once())
-			->method('get')
-			->willReturnMap([
-				['/bar', $parentNode]
-			]);
-
-		$node->copy('/bar/asd');
-	}
-
-
-	public function testCopyNoParent(): void {
-		$this->expectException(NotFoundException::class);
+    protected IUser&MockObject $user;
+    protected Manager&MockObject $manager;
+    protected View&MockObject $view;
+    protected Root&MockObject $root;
+    protected IUserMountCache&MockObject $userMountCache;
+    protected LoggerInterface&MockObject $logger;
+    protected IUserManager&MockObject $userManager;
+    protected IEventDispatcher&MockObject $eventDispatcher;
+    protected ICacheFactory&MockObject $cacheFactory;
+    protected IAppConfig&MockObject $appConfig;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->user = $this->createMock(IUser::class);
+        $this->manager = $this->getMockBuilder(Manager::class)
+            ->disableOriginalConstructor()
+            ->getMock();
+        $this->view = $this->getMockBuilder(View::class)
+            ->disableOriginalConstructor()
+            ->getMock();
+        $this->view->expects($this->any())
+            ->method('getRoot')
+            ->willReturn('');
+        $this->userMountCache = $this->getMockBuilder('\OCP\Files\Config\IUserMountCache')
+            ->disableOriginalConstructor()
+            ->getMock();
+        $this->logger = $this->createMock(LoggerInterface::class);
+        $this->userManager = $this->createMock(IUserManager::class);
+        $this->eventDispatcher = $this->createMock(IEventDispatcher::class);
+        $this->cacheFactory = $this->createMock(ICacheFactory::class);
+        $this->cacheFactory->method('createLocal')
+            ->willReturnCallback(function () {
+                return new ArrayCache();
+            });
+        $this->appConfig = $this->createMock(IAppConfig::class);
+
+        $this->root = $this->getMockBuilder(Root::class)
+            ->setConstructorArgs([$this->manager, $this->view, $this->user, $this->userMountCache, $this->logger, $this->userManager, $this->eventDispatcher, $this->cacheFactory, $this->appConfig])
+            ->getMock();
+    }
+
+    /**
+     * @return View|\PHPUnit\Framework\MockObject\MockObject $view
+     */
+    protected function getRootViewMock() {
+        $view = $this->createMock(View::class);
+        $view->expects($this->any())
+            ->method('getRoot')
+            ->willReturn('');
+        return $view;
+    }
+
+    abstract protected function createTestNode(IRootFolder $root, View&MockObject $view, string $path, array $data = [], string $internalPath = '', ?IStorage $storage = null): Node;
+
+    /**
+     * @return class-string<Node>
+     */
+    abstract protected function getNodeClass(): string;
+
+    /**
+     * @return class-string<Node>
+     */
+    abstract protected function getNonExistingNodeClass(): string;
+
+    /**
+     * @return string
+     */
+    abstract protected function getViewDeleteMethod(): string;
+
+    protected function getMockStorage(): IStorage&MockObject {
+        $storage = $this->getMockBuilder(IStorage::class)
+            ->disableOriginalConstructor()
+            ->getMock();
+        $storage->expects($this->any())
+            ->method('getId')
+            ->willReturn('home::someuser');
+        return $storage;
+    }
+
+    protected function getFileInfo($data, $internalPath = '', ?IStorage $storage = null) {
+        $mount = $this->createMock(IMountPoint::class);
+        $mount->method('getStorage')
+            ->willReturn($storage);
+        return new FileInfo('', $this->getMockStorage(), $internalPath, $data, $mount);
+    }
+
+    public function testDelete(): void {
+        $this->root->expects($this->exactly(2))
+            ->method('emit')
+            ->willReturn(true);
+        $this->root->expects($this->any())
+            ->method('getUser')
+            ->willReturn($this->user);
+
+        $this->view->expects($this->once())
+            ->method('getFileInfo')
+            ->with('/bar/foo')
+            ->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_ALL]));
+
+        $this->view->expects($this->once())
+            ->method($this->getViewDeleteMethod())
+            ->with('/bar/foo')
+            ->willReturn(true);
+
+        $node = $this->createTestNode($this->root, $this->view, '/bar/foo');
+        $node->delete();
+    }
+
+    public function testDeleteHooks(): void {
+        $test = $this;
+        $hooksRun = 0;
+        /**
+         * @param \OC\Files\Node\File $node
+         */
+        $preListener = function ($node) use (&$test, &$hooksRun): void {
+            $test->assertInstanceOf($this->getNodeClass(), $node);
+            $test->assertEquals('foo', $node->getInternalPath());
+            $test->assertEquals('/bar/foo', $node->getPath());
+            $test->assertEquals(1, $node->getId());
+            $hooksRun++;
+        };
+
+        /**
+         * @param \OC\Files\Node\File $node
+         */
+        $postListener = function ($node) use (&$test, &$hooksRun): void {
+            $test->assertInstanceOf($this->getNonExistingNodeClass(), $node);
+            $test->assertEquals('foo', $node->getInternalPath());
+            $test->assertEquals('/bar/foo', $node->getPath());
+            $test->assertEquals(1, $node->getId());
+            $test->assertEquals('text/plain', $node->getMimeType());
+            $hooksRun++;
+        };
+
+        $root = new Root(
+            $this->manager,
+            $this->view,
+            $this->user,
+            $this->userMountCache,
+            $this->logger,
+            $this->userManager,
+            $this->eventDispatcher,
+            $this->cacheFactory,
+            $this->createMock(IAppConfig::class),
+        );
+
+        $root->listen('\OC\Files', 'preDelete', $preListener);
+        $root->listen('\OC\Files', 'postDelete', $postListener);
+
+        $this->view->expects($this->any())
+            ->method('getFileInfo')
+            ->with('/bar/foo')
+            ->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_ALL, 'fileid' => 1, 'mimetype' => 'text/plain'], 'foo'));
+
+        $this->view->expects($this->once())
+            ->method($this->getViewDeleteMethod())
+            ->with('/bar/foo')
+            ->willReturn(true);
+
+        $node = $this->createTestNode($root, $this->view, '/bar/foo');
+        $node->delete();
+        $this->assertEquals(2, $hooksRun);
+    }
+
+
+    public function testDeleteNotPermitted(): void {
+        $this->expectException(NotPermittedException::class);
+
+        $this->root->expects($this->any())
+            ->method('getUser')
+            ->willReturn($this->user);
+
+        $this->view->expects($this->once())
+            ->method('getFileInfo')
+            ->with('/bar/foo')
+            ->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_READ]));
+
+        $node = $this->createTestNode($this->root, $this->view, '/bar/foo');
+        $node->delete();
+    }
+
+
+    public function testStat(): void {
+        $this->root->expects($this->any())
+            ->method('getUser')
+            ->willReturn($this->user);
+
+        $stat = [
+            'fileid' => 1,
+            'size' => 100,
+            'etag' => 'qwerty',
+            'mtime' => 50,
+            'permissions' => 0
+        ];
+
+        $this->view->expects($this->once())
+            ->method('stat')
+            ->with('/bar/foo')
+            ->willReturn($stat);
+
+        $node = $this->createTestNode($this->root, $this->view, '/bar/foo');
+        $this->assertEquals($stat, $node->stat());
+    }
+
+    public function testGetId(): void {
+        $this->root->expects($this->any())
+            ->method('getUser')
+            ->willReturn($this->user);
+
+        $stat = $this->getFileInfo([
+            'fileid' => 1,
+            'size' => 100,
+            'etag' => 'qwerty',
+            'mtime' => 50
+        ]);
+
+        $this->view->expects($this->once())
+            ->method('getFileInfo')
+            ->with('/bar/foo')
+            ->willReturn($stat);
+
+        $node = $this->createTestNode($this->root, $this->view, '/bar/foo');
+        $this->assertEquals(1, $node->getId());
+    }
+
+    public function testGetSize(): void {
+        $this->root->expects($this->any())
+            ->method('getUser')
+            ->willReturn($this->user);
+
+
+        $stat = $this->getFileInfo([
+            'fileid' => 1,
+            'size' => 100,
+            'etag' => 'qwerty',
+            'mtime' => 50
+        ]);
+
+        $this->view->expects($this->once())
+            ->method('getFileInfo')
+            ->with('/bar/foo')
+            ->willReturn($stat);
+
+        $node = $this->createTestNode($this->root, $this->view, '/bar/foo');
+        $this->assertEquals(100, $node->getSize());
+    }
+
+    public function testGetEtag(): void {
+        $this->root->expects($this->any())
+            ->method('getUser')
+            ->willReturn($this->user);
+
+        $stat = $this->getFileInfo([
+            'fileid' => 1,
+            'size' => 100,
+            'etag' => 'qwerty',
+            'mtime' => 50
+        ]);
+
+        $this->view->expects($this->once())
+            ->method('getFileInfo')
+            ->with('/bar/foo')
+            ->willReturn($stat);
+
+        $node = $this->createTestNode($this->root, $this->view, '/bar/foo');
+        $this->assertEquals('qwerty', $node->getEtag());
+    }
+
+    public function testGetMTime(): void {
+        $this->root->expects($this->any())
+            ->method('getUser')
+            ->willReturn($this->user);
+
+        $stat = $this->getFileInfo([
+            'fileid' => 1,
+            'size' => 100,
+            'etag' => 'qwerty',
+            'mtime' => 50
+        ]);
+
+        $this->view->expects($this->once())
+            ->method('getFileInfo')
+            ->with('/bar/foo')
+            ->willReturn($stat);
+
+        $node = $this->createTestNode($this->root, $this->view, '/bar/foo');
+        $this->assertEquals(50, $node->getMTime());
+    }
+
+    public function testGetStorage(): void {
+        $this->root->expects($this->any())
+            ->method('getUser')
+            ->willReturn($this->user);
+        /**
+         * @var Storage|\PHPUnit\Framework\MockObject\MockObject $storage
+         */
+        $storage = $this->getMockBuilder('\OC\Files\Storage\Storage')
+            ->disableOriginalConstructor()
+            ->getMock();
+
+        $node = $this->createTestNode($this->root, $this->view, '/bar/foo', [], 'foo', $storage);
+        $this->assertEquals($storage, $node->getStorage());
+    }
+
+    public function testGetPath(): void {
+        $this->root->expects($this->any())
+            ->method('getUser')
+            ->willReturn($this->user);
+
+        $node = $this->createTestNode($this->root, $this->view, '/bar/foo');
+        $this->assertEquals('/bar/foo', $node->getPath());
+    }
+
+    public function testGetInternalPath(): void {
+        $this->root->expects($this->any())
+            ->method('getUser')
+            ->willReturn($this->user);
+        /**
+         * @var Storage|\PHPUnit\Framework\MockObject\MockObject $storage
+         */
+        $storage = $this->getMockBuilder('\OC\Files\Storage\Storage')
+            ->disableOriginalConstructor()
+            ->getMock();
+
+        $this->view->expects($this->once())
+            ->method('getFileInfo')
+            ->with('/bar/foo')
+            ->willReturn($this->getFileInfo([], 'foo'));
+
+
+        $node = $this->createTestNode($this->root, $this->view, '/bar/foo');
+        $this->assertEquals('foo', $node->getInternalPath());
+    }
+
+    public function testGetName(): void {
+        $this->root->expects($this->any())
+            ->method('getUser')
+            ->willReturn($this->user);
+
+        $node = $this->createTestNode($this->root, $this->view, '/bar/foo');
+        $this->assertEquals('foo', $node->getName());
+    }
+
+    public function testTouchSetMTime(): void {
+        $this->root->expects($this->any())
+            ->method('getUser')
+            ->willReturn($this->user);
+
+        $this->view->expects($this->once())
+            ->method('touch')
+            ->with('/bar/foo', 100)
+            ->willReturn(true);
+
+        $this->view->expects($this->once())
+            ->method('getFileInfo')
+            ->with('/bar/foo')
+            ->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_ALL]));
+
+        $node = $this->createTestNode($this->root, $this->view, '/bar/foo');
+        $node->touch(100);
+        $this->assertEquals(100, $node->getMTime());
+    }
+
+    public function testTouchHooks(): void {
+        $test = $this;
+        $hooksRun = 0;
+        /**
+         * @param \OC\Files\Node\File $node
+         */
+        $preListener = function ($node) use (&$test, &$hooksRun): void {
+            $test->assertEquals('foo', $node->getInternalPath());
+            $test->assertEquals('/bar/foo', $node->getPath());
+            $hooksRun++;
+        };
+
+        /**
+         * @param \OC\Files\Node\File $node
+         */
+        $postListener = function ($node) use (&$test, &$hooksRun): void {
+            $test->assertEquals('foo', $node->getInternalPath());
+            $test->assertEquals('/bar/foo', $node->getPath());
+            $hooksRun++;
+        };
+
+        $root = new Root(
+            $this->manager,
+            $this->view,
+            $this->user,
+            $this->userMountCache,
+            $this->logger,
+            $this->userManager,
+            $this->eventDispatcher,
+            $this->cacheFactory,
+            $this->createMock(IAppConfig::class),
+        );
+        $root->listen('\OC\Files', 'preTouch', $preListener);
+        $root->listen('\OC\Files', 'postTouch', $postListener);
+
+        $this->view->expects($this->once())
+            ->method('touch')
+            ->with('/bar/foo', 100)
+            ->willReturn(true);
+
+        $this->view->expects($this->any())
+            ->method('getFileInfo')
+            ->with('/bar/foo')
+            ->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_ALL], 'foo'));
+
+        $node = $this->createTestNode($root, $this->view, '/bar/foo');
+        $node->touch(100);
+        $this->assertEquals(2, $hooksRun);
+    }
+
+
+    public function testTouchNotPermitted(): void {
+        $this->expectException(NotPermittedException::class);
+
+        $this->root->expects($this->any())
+            ->method('getUser')
+            ->willReturn($this->user);
+
+        $this->view->expects($this->any())
+            ->method('getFileInfo')
+            ->with('/bar/foo')
+            ->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_READ]));
+
+        $node = $this->createTestNode($this->root, $this->view, '/bar/foo');
+        $node->touch(100);
+    }
+
+
+    public function testInvalidPath(): void {
+        $this->expectException(InvalidPathException::class);
+
+        $node = $this->createTestNode($this->root, $this->view, '/../foo');
+        $node->getFileInfo();
+    }
+
+    public function testCopySameStorage(): void {
+        $this->view->expects($this->any())
+            ->method('copy')
+            ->with('/bar/foo', '/bar/asd')
+            ->willReturn(true);
+
+        $this->view->expects($this->any())
+            ->method('getFileInfo')
+            ->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_ALL, 'fileid' => 3]));
+
+        $node = $this->createTestNode($this->root, $this->view, '/bar/foo');
+        $parentNode = new Folder($this->root, $this->view, '/bar');
+        $newNode = $this->createTestNode($this->root, $this->view, '/bar/asd');
+
+        $this->root->method('get')
+            ->willReturnMap([
+                ['/bar/asd', $newNode],
+                ['/bar', $parentNode]
+            ]);
+
+        $target = $node->copy('/bar/asd');
+        $this->assertInstanceOf($this->getNodeClass(), $target);
+        $this->assertEquals(3, $target->getId());
+    }
+
+
+    public function testCopyNotPermitted(): void {
+        $this->expectException(NotPermittedException::class);
+
+        /**
+         * @var Storage|\PHPUnit\Framework\MockObject\MockObject $storage
+         */
+        $storage = $this->createMock('\OC\Files\Storage\Storage');
+
+        $this->root->expects($this->never())
+            ->method('getMount');
+
+        $storage->expects($this->never())
+            ->method('copy');
+
+        $this->view->expects($this->any())
+            ->method('getFileInfo')
+            ->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_READ, 'fileid' => 3]));
+
+        $node = $this->createTestNode($this->root, $this->view, '/bar/foo');
+        $parentNode = new Folder($this->root, $this->view, '/bar');
+
+        $this->root->expects($this->once())
+            ->method('get')
+            ->willReturnMap([
+                ['/bar', $parentNode]
+            ]);
+
+        $node->copy('/bar/asd');
+    }
+
+
+    public function testCopyNoParent(): void {
+        $this->expectException(NotFoundException::class);
 
-		$this->view->expects($this->never())
-			->method('copy');
-
-		$node = $this->createTestNode($this->root, $this->view, '/bar/foo');
-
-		$this->root->expects($this->once())
-			->method('get')
-			->with('/bar/asd')
-			->willThrowException(new NotFoundException());
-
-		$node->copy('/bar/asd/foo');
-	}
-
-
-	public function testCopyParentIsFile(): void {
-		$this->expectException(NotPermittedException::class);
-
-		$this->view->expects($this->never())
-			->method('copy');
-
-		$node = $this->createTestNode($this->root, $this->view, '/bar/foo');
-		$parentNode = new File($this->root, $this->view, '/bar');
-
-		$this->root->expects($this->once())
-			->method('get')
-			->willReturnMap([
-				['/bar', $parentNode]
-			]);
-
-		$node->copy('/bar/asd');
-	}
-
-	public function testMoveSameStorage(): void {
-		$this->view->expects($this->any())
-			->method('rename')
-			->with('/bar/foo', '/bar/asd')
-			->willReturn(true);
-
-		$this->view->expects($this->any())
-			->method('getFileInfo')
-			->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_ALL, 'fileid' => 1]));
-
-		$node = $this->createTestNode($this->root, $this->view, '/bar/foo');
-		$parentNode = new Folder($this->root, $this->view, '/bar');
-
-		$this->root->expects($this->any())
-			->method('get')
-			->willReturnMap([['/bar', $parentNode], ['/bar/asd', $node]]);
-
-		$target = $node->move('/bar/asd');
-		$this->assertInstanceOf($this->getNodeClass(), $target);
-		$this->assertEquals(1, $target->getId());
-		$this->assertEquals('/bar/asd', $node->getPath());
-	}
-
-	public static function moveOrCopyProvider(): array {
-		return [
-			['move', 'rename', 'preRename', 'postRename'],
-			['copy', 'copy', 'preCopy', 'postCopy'],
-		];
-	}
+        $this->view->expects($this->never())
+            ->method('copy');
+
+        $node = $this->createTestNode($this->root, $this->view, '/bar/foo');
+
+        $this->root->expects($this->once())
+            ->method('get')
+            ->with('/bar/asd')
+            ->willThrowException(new NotFoundException());
+
+        $node->copy('/bar/asd/foo');
+    }
+
+
+    public function testCopyParentIsFile(): void {
+        $this->expectException(NotPermittedException::class);
+
+        $this->view->expects($this->never())
+            ->method('copy');
+
+        $node = $this->createTestNode($this->root, $this->view, '/bar/foo');
+        $parentNode = new File($this->root, $this->view, '/bar');
+
+        $this->root->expects($this->once())
+            ->method('get')
+            ->willReturnMap([
+                ['/bar', $parentNode]
+            ]);
+
+        $node->copy('/bar/asd');
+    }
+
+    public function testMoveSameStorage(): void {
+        $this->view->expects($this->any())
+            ->method('rename')
+            ->with('/bar/foo', '/bar/asd')
+            ->willReturn(true);
+
+        $this->view->expects($this->any())
+            ->method('getFileInfo')
+            ->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_ALL, 'fileid' => 1]));
+
+        $node = $this->createTestNode($this->root, $this->view, '/bar/foo');
+        $parentNode = new Folder($this->root, $this->view, '/bar');
+
+        $this->root->expects($this->any())
+            ->method('get')
+            ->willReturnMap([['/bar', $parentNode], ['/bar/asd', $node]]);
+
+        $target = $node->move('/bar/asd');
+        $this->assertInstanceOf($this->getNodeClass(), $target);
+        $this->assertEquals(1, $target->getId());
+        $this->assertEquals('/bar/asd', $node->getPath());
+    }
+
+    public static function moveOrCopyProvider(): array {
+        return [
+            ['move', 'rename', 'preRename', 'postRename'],
+            ['copy', 'copy', 'preCopy', 'postCopy'],
+        ];
+    }
 
-	/**
-	 * @param string $operationMethod
-	 * @param string $viewMethod
-	 * @param string $preHookName
-	 * @param string $postHookName
-	 */
-	#[\PHPUnit\Framework\Attributes\DataProvider('moveOrCopyProvider')]
-	public function testMoveCopyHooks($operationMethod, $viewMethod, $preHookName, $postHookName): void {
-		/** @var IRootFolder|\PHPUnit\Framework\MockObject\MockObject $root */
-		$root = $this->getMockBuilder(Root::class)
-			->setConstructorArgs([$this->manager, $this->view, $this->user, $this->userMountCache, $this->logger, $this->userManager, $this->eventDispatcher, $this->cacheFactory, $this->appConfig])
-			->onlyMethods(['get'])
-			->getMock();
-
-		$this->view->expects($this->any())
-			->method($viewMethod)
-			->with('/bar/foo', '/bar/asd')
-			->willReturn(true);
+    /**
+     * @param string $operationMethod
+     * @param string $viewMethod
+     * @param string $preHookName
+     * @param string $postHookName
+     */
+    #[\PHPUnit\Framework\Attributes\DataProvider('moveOrCopyProvider')]
+    public function testMoveCopyHooks($operationMethod, $viewMethod, $preHookName, $postHookName): void {
+        /** @var IRootFolder|\PHPUnit\Framework\MockObject\MockObject $root */
+        $root = $this->getMockBuilder(Root::class)
+            ->setConstructorArgs([$this->manager, $this->view, $this->user, $this->userMountCache, $this->logger, $this->userManager, $this->eventDispatcher, $this->cacheFactory, $this->appConfig])
+            ->onlyMethods(['get'])
+            ->getMock();
+
+        $this->view->expects($this->any())
+            ->method($viewMethod)
+            ->with('/bar/foo', '/bar/asd')
+            ->willReturn(true);
 
-		$this->view->expects($this->any())
-			->method('getFileInfo')
-			->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_ALL, 'fileid' => 1]));
+        $this->view->expects($this->any())
+            ->method('getFileInfo')
+            ->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_ALL, 'fileid' => 1]));
 
-		/**
-		 * @var \OC\Files\Node\File|\PHPUnit\Framework\MockObject\MockObject $node
-		 */
-		$node = $this->createTestNode($root, $this->view, '/bar/foo');
-		$parentNode = new Folder($root, $this->view, '/bar');
-		$targetTestNode = $this->createTestNode($root, $this->view, '/bar/asd');
+        /**
+         * @var \OC\Files\Node\File|\PHPUnit\Framework\MockObject\MockObject $node
+         */
+        $node = $this->createTestNode($root, $this->view, '/bar/foo');
+        $parentNode = new Folder($root, $this->view, '/bar');
+        $targetTestNode = $this->createTestNode($root, $this->view, '/bar/asd');
 
-		$root->expects($this->any())
-			->method('get')
-			->willReturnMap([['/bar', $parentNode], ['/bar/asd', $targetTestNode]]);
+        $root->expects($this->any())
+            ->method('get')
+            ->willReturnMap([['/bar', $parentNode], ['/bar/asd', $targetTestNode]]);
 
-		$hooksRun = 0;
+        $hooksRun = 0;
 
-		$preListener = function (Node $sourceNode, Node $targetNode) use (&$hooksRun, $node): void {
-			$this->assertSame($node, $sourceNode);
-			$this->assertInstanceOf($this->getNodeClass(), $sourceNode);
-			$this->assertInstanceOf($this->getNonExistingNodeClass(), $targetNode);
-			$this->assertEquals('/bar/asd', $targetNode->getPath());
-			$hooksRun++;
-		};
-
-		$postListener = function (Node $sourceNode, Node $targetNode) use (&$hooksRun, $node, $targetTestNode): void {
-			$this->assertSame($node, $sourceNode);
-			$this->assertNotSame($node, $targetNode);
-			$this->assertSame($targetTestNode, $targetNode);
-			$this->assertInstanceOf($this->getNodeClass(), $sourceNode);
-			$this->assertInstanceOf($this->getNodeClass(), $targetNode);
-			$hooksRun++;
-		};
-
-		$preWriteListener = function (Node $targetNode) use (&$hooksRun): void {
-			$this->assertInstanceOf($this->getNonExistingNodeClass(), $targetNode);
-			$this->assertEquals('/bar/asd', $targetNode->getPath());
-			$hooksRun++;
-		};
-
-		$postWriteListener = function (Node $targetNode) use (&$hooksRun, $targetTestNode): void {
-			$this->assertSame($targetTestNode, $targetNode);
-			$hooksRun++;
-		};
+        $preListener = function (Node $sourceNode, Node $targetNode) use (&$hooksRun, $node): void {
+            $this->assertSame($node, $sourceNode);
+            $this->assertInstanceOf($this->getNodeClass(), $sourceNode);
+            $this->assertInstanceOf($this->getNonExistingNodeClass(), $targetNode);
+            $this->assertEquals('/bar/asd', $targetNode->getPath());
+            $hooksRun++;
+        };
+
+        $postListener = function (Node $sourceNode, Node $targetNode) use (&$hooksRun, $node, $targetTestNode): void {
+            $this->assertSame($node, $sourceNode);
+            $this->assertNotSame($node, $targetNode);
+            $this->assertSame($targetTestNode, $targetNode);
+            $this->assertInstanceOf($this->getNodeClass(), $sourceNode);
+            $this->assertInstanceOf($this->getNodeClass(), $targetNode);
+            $hooksRun++;
+        };
+
+        $preWriteListener = function (Node $targetNode) use (&$hooksRun): void {
+            $this->assertInstanceOf($this->getNonExistingNodeClass(), $targetNode);
+            $this->assertEquals('/bar/asd', $targetNode->getPath());
+            $hooksRun++;
+        };
+
+        $postWriteListener = function (Node $targetNode) use (&$hooksRun, $targetTestNode): void {
+            $this->assertSame($targetTestNode, $targetNode);
+            $hooksRun++;
+        };
 
-		$root->listen('\OC\Files', $preHookName, $preListener);
-		$root->listen('\OC\Files', 'preWrite', $preWriteListener);
-		$root->listen('\OC\Files', $postHookName, $postListener);
-		$root->listen('\OC\Files', 'postWrite', $postWriteListener);
+        $root->listen('\OC\Files', $preHookName, $preListener);
+        $root->listen('\OC\Files', 'preWrite', $preWriteListener);
+        $root->listen('\OC\Files', $postHookName, $postListener);
+        $root->listen('\OC\Files', 'postWrite', $postWriteListener);
 
-		$node->$operationMethod('/bar/asd');
-
-		$this->assertEquals(4, $hooksRun);
-	}
+        $node->$operationMethod('/bar/asd');
+
+        $this->assertEquals(4, $hooksRun);
+    }
 
 
-	public function testMoveNotPermitted(): void {
-		$this->expectException(NotPermittedException::class);
+    public function testMoveNotPermitted(): void {
+        $this->expectException(NotPermittedException::class);
 
-		$this->view->expects($this->any())
-			->method('getFileInfo')
-			->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_READ]));
+        $this->view->expects($this->any())
+            ->method('getFileInfo')
+            ->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_READ]));
 
-		$this->view->expects($this->never())
-			->method('rename');
+        $this->view->expects($this->never())
+            ->method('rename');
 
-		$node = $this->createTestNode($this->root, $this->view, '/bar/foo');
-		$parentNode = new Folder($this->root, $this->view, '/bar');
+        $node = $this->createTestNode($this->root, $this->view, '/bar/foo');
+        $parentNode = new Folder($this->root, $this->view, '/bar');
 
-		$this->root->expects($this->once())
-			->method('get')
-			->with('/bar')
-			->willReturn($parentNode);
+        $this->root->expects($this->once())
+            ->method('get')
+            ->with('/bar')
+            ->willReturn($parentNode);
 
-		$node->move('/bar/asd');
-	}
+        $node->move('/bar/asd');
+    }
 
 
-	public function testMoveNoParent(): void {
-		$this->expectException(NotFoundException::class);
+    public function testMoveNoParent(): void {
+        $this->expectException(NotFoundException::class);
 
-		/**
-		 * @var Storage|\PHPUnit\Framework\MockObject\MockObject $storage
-		 */
-		$storage = $this->createMock('\OC\Files\Storage\Storage');
+        /**
+         * @var Storage|\PHPUnit\Framework\MockObject\MockObject $storage
+         */
+        $storage = $this->createMock('\OC\Files\Storage\Storage');
 
-		$storage->expects($this->never())
-			->method('rename');
+        $storage->expects($this->never())
+            ->method('rename');
 
-		$node = $this->createTestNode($this->root, $this->view, '/bar/foo');
+        $node = $this->createTestNode($this->root, $this->view, '/bar/foo');
 
-		$this->root->expects($this->once())
-			->method('get')
-			->with('/bar')
-			->willThrowException(new NotFoundException());
+        $this->root->expects($this->once())
+            ->method('get')
+            ->with('/bar')
+            ->willThrowException(new NotFoundException());
 
-		$node->move('/bar/asd');
-	}
+        $node->move('/bar/asd');
+    }
 
 
-	public function testMoveParentIsFile(): void {
-		$this->expectException(NotPermittedException::class);
+    public function testMoveParentIsFile(): void {
+        $this->expectException(NotPermittedException::class);
 
-		$this->view->expects($this->never())
-			->method('rename');
+        $this->view->expects($this->never())
+            ->method('rename');
 
-		$node = $this->createTestNode($this->root, $this->view, '/bar/foo');
-		$parentNode = new File($this->root, $this->view, '/bar');
+        $node = $this->createTestNode($this->root, $this->view, '/bar/foo');
+        $parentNode = new File($this->root, $this->view, '/bar');
 
-		$this->root->expects($this->once())
-			->method('get')
-			->with('/bar')
-			->willReturn($parentNode);
+        $this->root->expects($this->once())
+            ->method('get')
+            ->with('/bar')
+            ->willReturn($parentNode);
 
-		$node->move('/bar/asd');
-	}
+        $node->move('/bar/asd');
+    }
 
 
-	public function testMoveFailed(): void {
-		$this->expectException(NotPermittedException::class);
+    public function testMoveFailed(): void {
+        $this->expectException(NotPermittedException::class);
 
-		$this->view->expects($this->any())
-			->method('rename')
-			->with('/bar/foo', '/bar/asd')
-			->willReturn(false);
+        $this->view->expects($this->any())
+            ->method('rename')
+            ->with('/bar/foo', '/bar/asd')
+            ->willReturn(false);
 
-		$this->view->expects($this->any())
-			->method('getFileInfo')
-			->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_ALL, 'fileid' => 1]));
+        $this->view->expects($this->any())
+            ->method('getFileInfo')
+            ->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_ALL, 'fileid' => 1]));
 
-		$node = $this->createTestNode($this->root, $this->view, '/bar/foo');
-		$parentNode = new Folder($this->root, $this->view, '/bar');
+        $node = $this->createTestNode($this->root, $this->view, '/bar/foo');
+        $parentNode = new Folder($this->root, $this->view, '/bar');
 
-		$this->root->expects($this->any())
-			->method('get')
-			->willReturnMap([['/bar', $parentNode], ['/bar/asd', $node]]);
+        $this->root->expects($this->any())
+            ->method('get')
+            ->willReturnMap([['/bar', $parentNode], ['/bar/asd', $node]]);
 
-		$node->move('/bar/asd');
-	}
+        $node->move('/bar/asd');
+    }
 
 
-	public function testCopyFailed(): void {
-		$this->expectException(NotPermittedException::class);
+    public function testCopyFailed(): void {
+        $this->expectException(NotPermittedException::class);
 
-		$this->view->expects($this->any())
-			->method('copy')
-			->with('/bar/foo', '/bar/asd')
-			->willReturn(false);
+        $this->view->expects($this->any())
+            ->method('copy')
+            ->with('/bar/foo', '/bar/asd')
+            ->willReturn(false);
 
-		$this->view->expects($this->any())
-			->method('getFileInfo')
-			->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_ALL, 'fileid' => 1]));
+        $this->view->expects($this->any())
+            ->method('getFileInfo')
+            ->willReturn($this->getFileInfo(['permissions' => Constants::PERMISSION_ALL, 'fileid' => 1]));
 
-		$node = $this->createTestNode($this->root, $this->view, '/bar/foo');
-		$parentNode = new Folder($this->root, $this->view, '/bar');
+        $node = $this->createTestNode($this->root, $this->view, '/bar/foo');
+        $parentNode = new Folder($this->root, $this->view, '/bar');
 
-		$this->root->expects($this->any())
-			->method('get')
-			->willReturnMap([['/bar', $parentNode], ['/bar/asd', $node]]);
+        $this->root->expects($this->any())
+            ->method('get')
+            ->willReturnMap([['/bar', $parentNode], ['/bar/asd', $node]]);
 
-		$node->copy('/bar/asd');
-	}
+        $node->copy('/bar/asd');
+    }
 }