fix: only set signer as me when identifier matches current user

Remove owner-based shortcut that marked all signers as me for the file owner,
and rely solely on identifier matching (identifyMethodId, UID, or email).

Signed-off-by: Vitor Mattos <[email protected]>

Author: vitormattos

lib/Service/File/SignersLoader.php

@@ -64,7 +64,6 @@ public function loadLibreSignSigners(
 				if (!empty($found)) {
 					$index = key($found);
 				} else {
-					// Find next available index
 					$index = count($fileData->signers);
 				}
 			} else {
@@ -144,21 +143,15 @@ public function loadLibreSignSigners(
 			if ($options->getMe() || $options->getIdentifyMethodId()) {
 				$currentUserData = new stdClass();
 				$currentUserData->me = false;
-				if ($options->getMe()?->getUID() === $file->getUserId()) {
-					$currentUserData->me = true;
-				}
-				if (!$currentUserData->me) {
-					foreach ($identifyMethods as $methods) {
-						foreach ($methods as $identifyMethod) {
-							$entity = $identifyMethod->getEntity();
-							if (!$currentUserData->me) {
-								if ($options->getIdentifyMethodId() === $entity->getId()
-									|| $options->getMe()?->getUID() === $entity->getIdentifierValue()
-									|| $options->getMe()?->getEMailAddress() === $entity->getIdentifierValue()
-								) {
-									$currentUserData->me = true;
-								}
-							}
+				foreach ($identifyMethods as $methods) {
+					foreach ($methods as $identifyMethod) {
+						$entity = $identifyMethod->getEntity();
+						if ($options->getIdentifyMethodId() === $entity->getId()
+							|| $options->getMe()?->getUID() === $entity->getIdentifierValue()
+							|| $options->getMe()?->getEMailAddress() === $entity->getIdentifierValue()
+						) {
+							$currentUserData->me = true;
+							break 2;
 						}
 					}
 				}