fix: renameParams named as renamedNodes and code changed accordingly as suggested in review comments

Signed-off-by: yemkareems <yemkareems@gmail.com>

Author: yemkareems

apps/admin_audit/lib/Actions/Files.php

@@ -27,7 +27,7 @@
  */
 class Files extends Action {
 
-	private array|null $renameParams = null;
+	private array $renamedNodes = [];
 	/**
 	 * Logs file read actions
 	 *
@@ -60,17 +60,13 @@ public function read(BeforeNodeReadEvent $event): void {
 	public function beforeRename(BeforeNodeRenamedEvent $event): void {
 		try {
 			$source = $event->getSource();
-			$params = [
-				'oldid' => $source->getId(),
-				'oldpath' => mb_substr($source->getInternalPath(), 5),
-			];
+			$this->renamedNodes[$source->getId()] = $source;
 		} catch (InvalidPathException|NotFoundException $e) {
 			\OCP\Server::get(LoggerInterface::class)->error(
 				"Exception thrown in file rename: ".$e->getMessage(), ['app' => 'admin_audit', 'exception' => $e]
 			);
 			return;
 		}
-		$this->renameParams = $params;
 	}
 
 	/**
@@ -81,10 +77,10 @@ public function beforeRename(BeforeNodeRenamedEvent $event): void {
 	public function afterRename(NodeRenamedEvent $event): void {
 		try {
 			$target = $event->getTarget();
-			$renameParams = $this->renameParams;
+			$originalSource = $this->renamedNodes[$target->getId()];
 			$params = [
 				'newid' => $target->getId(),
-				'oldpath' => $renameParams['oldpath'],
+				'oldpath' => mb_substr($originalSource->getInternalPath(), 5),
 				'newpath' => mb_substr($target->getInternalPath(), 5),
 			];
 		} catch (InvalidPathException|NotFoundException $e) {