fixup! feat(ocs): notify of new messages and provide API endpoint to …

…retrieve its contents
nextcloud · Jun 18, 2024 · 8858cfc · 8858cfc
1 parent ccc54f4
commit 8858cfc
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 4 deletions.
diff --git a/appinfo/routes.php b/appinfo/routes.php
@@ -492,5 +492,12 @@
 		'outbox' => ['url' => '/api/outbox'],
 		'preferences' => ['url' => '/api/preferences'],
 		'smimeCertificates' => ['url' => '/api/smime/certificates'],
-	]
+	],
+	'ocs' => [
+		[
+			'name' => 'messageApi#get',
+			'url' => '/message/{$id}',
+			'verb' => 'GET',
+		],
+	],
 ];
diff --git a/lib/Controller/MessageApiController.php b/lib/Controller/MessageApiController.php
@@ -18,7 +18,6 @@
 use OCA\Mail\Service\OutboxService;
 use OCP\AppFramework\Db\DoesNotExistException;
 use OCP\AppFramework\Http;
-use OCP\AppFramework\Http\Attribute\ApiRoute;
 use OCP\AppFramework\Http\Attribute\BruteForceProtection;
 use OCP\AppFramework\Http\Attribute\NoAdminRequired;
 use OCP\AppFramework\Http\Attribute\NoCSRFRequired;
@@ -52,13 +51,13 @@ public function __construct(
 	}
 
 	/**
+	 * @param int $id
 	 * @return JSONResponse
 	 */
-	#[ApiRoute(verb: 'GET', url: '/message/{id}')]
 	#[BruteForceProtection('mailGetMessage')]
 	#[NoAdminRequired]
 	#[NoCSRFRequired]
-	public function get(int $id) {
+	public function get(int $id): JSONResponse {
 		try {
 			$message = $this->mailManager->getMessage($this->userId, $id);
 			$mailbox = $this->mailManager->getMailbox($this->userId, $message->getMailboxId());