Merge pull request #3703 from LibreSign/backport/3699/stable28

[stable28] chore: adjust filter condition
LibreSign · Sep 14, 2024 · 0cb95a8 · 0cb95a8
2 parents 5802dde + aa5805d
commit 0cb95a8
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/lib/Db/SignRequestMapper.php b/lib/Db/SignRequestMapper.php
@@ -139,7 +139,7 @@ public function getByUuid(string $uuid): SignRequest {
 			);
 		/** @var SignRequest */
 		$signRequest = $this->findEntity($qb);
-		if (!array_filter($this->signers, fn ($s) => $s->getId() !== $signRequest->getId())) {
+		if (!array_filter($this->signers, fn ($s) => $s->getId() === $signRequest->getId())) {
 			$this->signers[] = $signRequest;
 		}
 		return $signRequest;
@@ -188,7 +188,7 @@ public function getByFileId(int $fileId): array {
 		/** @var SignRequest[] */
 		$signers = $this->findEntities($qb);
 		foreach ($signers as $signRequest) {
-			if (!array_filter($signers, fn ($s) => $s->getId() !== $signRequest->getId())) {
+			if (!array_filter($this->signers, fn ($s) => $s->getId() === $signRequest->getId())) {
 				$this->signers[] = $signRequest;
 			}
 		}
@@ -214,7 +214,7 @@ public function getById(int $signRequestId): SignRequest {
 
 		/** @var SignRequest */
 		$signRequest = $this->findEntity($qb);
-		if (!array_filter($this->signers, fn ($s) => $s->getId() !== $signRequest->getId())) {
+		if (!array_filter($this->signers, fn ($s) => $s->getId() === $signRequest->getId())) {
 			$this->signers[] = $signRequest;
 		}
 		return $signRequest;
@@ -277,7 +277,7 @@ public function getByFileUuid(string $uuid) {
 		/** @var SignRequest[] */
 		$signers = $this->findEntities($qb);
 		foreach ($signers as $signRequest) {
-			if (!array_filter($signers, fn ($s) => $s->getId() !== $signRequest->getId())) {
+			if (!array_filter($this->signers, fn ($s) => $s->getId() === $signRequest->getId())) {
 				$this->signers[] = $signRequest;
 			}
 		}
@@ -295,7 +295,7 @@ public function getBySignerUuidAndUserId(string $uuid): SignRequest {
 
 		/** @var SignRequest */
 		$signRequest = $this->findEntity($qb);
-		if (!array_filter($this->signers, fn ($s) => $s->getId() !== $signRequest->getId())) {
+		if (!array_filter($this->signers, fn ($s) => $s->getId() === $signRequest->getId())) {
 			$this->signers[] = $signRequest;
 		}
 		return $signRequest;
@@ -350,7 +350,7 @@ public function getByFileIdAndSignRequestId(int $fileId, int $signRequestId): Si
 			);
 
 		$signRequest = $this->findEntity($qb);
-		if (!array_filter($this->signers, fn ($s) => $s->getId() !== $signRequest->getId())) {
+		if (!array_filter($this->signers, fn ($s) => $s->getId() === $signRequest->getId())) {
 			$this->signers[] = $signRequest;
 		}
 		/** @var SignRequest */