fix(crl): correct type hints for Psalm

vitormattos · vitormattos · commit 54cd86f6be78 · 2025-11-27T16:05:36.000-03:00
- Use CRLReason enum instead of int in Revoke command
- Fix docblock to use full namespace for Crl entity

Signed-off-by: Vitor Mattos &lt;1079143+vitormattos@users.noreply.github.com&gt;
diff --git a/lib/Command/Crl/Revoke.php b/lib/Command/Crl/Revoke.php
@@ -145,7 +145,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
 		try {
 			$success = $this->crlService->revokeCertificate(
 				$serialNumber,
-				$reasonCode,
+				$reason,
 				$reasonText,
 				'cli-admin'
 			);
diff --git a/lib/Service/CrlService.php b/lib/Service/CrlService.php
@@ -92,7 +92,7 @@ public function revokeUserCertificates(
 	/**
 	 * Revoke a list of certificates
 	 *
-	 * @param array<Crl> $certificates Array of Crl entities
+	 * @param array<\OCA\Libresign\Db\Crl> $certificates Array of Crl entities
 	 * @param CRLReason $reason Revocation reason
 	 * @param string|null $reasonText Optional text describing the reason
 	 * @param string|null $revokedBy Who is revoking the certificates

Original file line number	Diff line number	Diff line change
`@@ -92,7 +92,7 @@ public function revokeUserCertificates(`
`92`	`92`	`/**`
`93`	`93`	`* Revoke a list of certificates`
`94`	`94`	`*`
`95`		`- * @param array<Crl> $certificates Array of Crl entities`
	`95`	`+ * @param array<\OCA\Libresign\Db\Crl> $certificates Array of Crl entities`
`96`	`96`	`* @param CRLReason $reason Revocation reason`
`97`	`97`	`* @param string\|null $reasonText Optional text describing the reason`
`98`	`98`	`* @param string\|null $revokedBy Who is revoking the certificates`