Improve path resolution in files_version hooks

Signed-off-by: Louis Chemineau <[email protected]>

apps/files_versions/lib/Listener/FileEventsListener.php

@@ -193,6 +193,10 @@ public function write_hook(Node $node): void {
 		}
 
 		$path = $this->getPathForNode($node);
+		if ($path === null) {
+			$this->logger->error('Failed to find relative path for file for ' . $node->getPath() . ' and owner ' .$node->getOwner());
+		}
+
 		$result = Storage::store($path);
 
 		// Store the result of the version creation so it can be used in post_write_hook.
@@ -350,16 +354,24 @@ public function pre_renameOrCopy_hook(Node $source, Node $target): void {
 	private function getPathForNode(Node $node): ?string {
 		$user = $this->userSession->getUser()?->getUID();
 		if ($user) {
-			return $this->rootFolder
+			$path = $this->rootFolder
 				->getUserFolder($user)
 				->getRelativePath($node->getPath());
+
+			if ($path !== null) {
+				return $path;
+			}
 		}
 
 		$owner = $node->getOwner()?->getUid();
 		if ($owner) {
-			return $this->rootFolder
+			$path = $this->rootFolder
 				->getUserFolder($owner)
 				->getRelativePath($node->getPath());
+
+			if ($path !== null) {
+				return $path;
+			}
 		}
 
 		return null;