From: provokateurin <kate@provokateurin.de>
Date: Thu, 25 Jul 2024 11:14:45 +0000 (+0200)
Subject: refactor(comments): Replace security annotations with respective attributes
X-Git-Tag: v30.0.0beta2~51^2
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=refs%2Fpull%2F46745%2Fhead;p=nextcloud-server.git

refactor(comments): Replace security annotations with respective attributes

Signed-off-by: provokateurin <kate@provokateurin.de>
---

diff --git a/apps/comments/lib/Controller/NotificationsController.php b/apps/comments/lib/Controller/NotificationsController.php
index 408efc46973..a3228247884 100644
--- a/apps/comments/lib/Controller/NotificationsController.php
+++ b/apps/comments/lib/Controller/NotificationsController.php
@@ -7,7 +7,9 @@ namespace OCA\Comments\Controller;
 
 use OCP\AppFramework\Controller;
 use OCP\AppFramework\Http;
+use OCP\AppFramework\Http\Attribute\NoCSRFRequired;
 use OCP\AppFramework\Http\Attribute\OpenAPI;
+use OCP\AppFramework\Http\Attribute\PublicPage;
 use OCP\AppFramework\Http\NotFoundResponse;
 use OCP\AppFramework\Http\RedirectResponse;
 use OCP\Comments\IComment;
@@ -37,9 +39,6 @@ class NotificationsController extends Controller {
 	}
 
 	/**
-	 * @PublicPage
-	 * @NoCSRFRequired
-	 *
 	 * View a notification
 	 *
 	 * @param string $id ID of the notification
@@ -49,6 +48,8 @@ class NotificationsController extends Controller {
 	 * 303: Redirected to notification
 	 * 404: Notification not found
 	 */
+	#[PublicPage]
+	#[NoCSRFRequired]
 	public function view(string $id): RedirectResponse|NotFoundResponse {
 		$currentUser = $this->userSession->getUser();
 		if (!$currentUser instanceof IUser) {