fix: notification parameters need to be string

All values of a notification need to be a string Signed-off-by: Vitor Mattos <vitor@php.rio>
LibreSign · Sep 13, 2024 · d61c79d · d61c79d
1 parent 4e8fe31
commit d61c79d
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 8 deletions.
diff --git a/lib/Activity/Listener.php b/lib/Activity/Listener.php
@@ -98,7 +98,7 @@ protected function generateNewSignNotificationActivity(
 				),
 				'signRequest' => [
 					'type' => 'sign-request',
-					'id' => $signRequest->getId(),
+					'id' => (string)$signRequest->getId(),
 					'name' => $actor->getDisplayName(),
 				],
 			]);
@@ -110,12 +110,12 @@ protected function generateNewSignNotificationActivity(
 	}
 
 	/**
-	 * @return array{type: 'file', id: int, name: string, path: string, link: string}
+	 * @return array{type: 'file', id: string, name: string, path: string, link: string}
 	 */
 	protected function getFileParameter(SignRequest $signRequest, FileEntity $libreSignFile): array {
 		return [
 			'type' => 'file',
-			'id' => $libreSignFile->getNodeId(),
+			'id' => (string)$libreSignFile->getNodeId(),
 			'name' => $libreSignFile->getName(),
 			'path' => $libreSignFile->getName(),
 			'link' => $this->url->linkToRouteAbsolute('libresign.page.sign', ['uuid' => $signRequest->getUuid()]),

diff --git a/lib/Listener/NotificationListener.php b/lib/Listener/NotificationListener.php
@@ -77,7 +77,7 @@ private function sendSignNotification(
 			'file' => $this->getFileParameter($signRequest, $libreSignFile),
 			'signRequest' => [
 				'type' => 'sign-request',
-				'id' => $signRequest->getId(),
+				'id' => (string)$signRequest->getId(),
 				'name' => $actor->getDisplayName(),
 			],
 		]);
@@ -106,14 +106,12 @@ public function isNotificationDisabledAtActivity(IIdentifyMethod $identifyMethod
 	}
 
 	/**
-	 * @return (int|string)[]
-	 *
-	 * @psalm-return array{type: 'file', id: int, name: string, path: string, link: string}
+	 * @psalm-return array{type: 'file', id: string, name: string, path: string, link: string}
 	 */
 	protected function getFileParameter(SignRequest $signRequest, FileEntity $libreSignFile): array {
 		return [
 			'type' => 'file',
-			'id' => $libreSignFile->getNodeId(),
+			'id' => (string)$libreSignFile->getNodeId(),
 			'name' => $libreSignFile->getName(),
 			'path' => $libreSignFile->getName(),
 			'link' => $this->url->linkToRouteAbsolute('libresign.page.sign', ['uuid' => $signRequest->getUuid()]),