fix: Migrate getById to getFirstNodeById

Signed-off-by: Louis Chmn <louis@chmn.me>

Author: artonge

apps/workflowengine/lib/Entity/File.php

@@ -136,12 +136,9 @@ protected function getNode(): Node {
 				if (!$this->event instanceof MapperEvent || $this->event->getObjectType() !== 'files') {
 					throw new NotFoundException();
 				}
-				$nodes = $this->root->getById((int)$this->event->getObjectId());
-				if (is_array($nodes) && isset($nodes[0])) {
-					$this->node = $nodes[0];
-					return $this->node;
-				}
-				break;
+				$node = $this->root->getFirstNodeById((int)$this->event->getObjectId());
+				$this->node = $node;
+				return $this->node;
 		}
 		throw new NotFoundException();
 	}
@@ -228,11 +225,11 @@ public function importContextIDs(array $contextIDs): void {
 		$this->eventName = $contextIDs['eventName'];
 		if ($contextIDs['nodeOwnerId'] !== null) {
 			$userFolder = $this->root->getUserFolder($contextIDs['nodeOwnerId']);
-			$nodes = $userFolder->getById($contextIDs['nodeId']);
+			$node = $userFolder->getFirstNodeById($contextIDs['nodeId']);
 		} else {
-			$nodes = $this->root->getById($contextIDs['nodeId']);
+			$node = $this->root->getFirstNodeById($contextIDs['nodeId']);
 		}
-		$this->node = $nodes[0] ?? null;
+		$this->node = $node;
 		if ($contextIDs['actingUserId']) {
 			$this->actingUser = $this->userManager->get($contextIDs['actingUserId']);
 		}

lib/private/Files/Node/Root.php

@@ -388,7 +388,7 @@ public function getFirstNodeByIdInPath(int $id, string $path): ?INode {
 				}
 			}
 		}
-		$node = current($this->getByIdInPath($id, $path));
+		$node = $this->getFirstNodeByIdInPath($id, $path);
 		if (!$node) {
 			return null;
 		}