1 files changed, 8 insertions, 13 deletions
diff --git a/apps/workflowengine/lib/Controller/UserWorkflowsController.php b/apps/workflowengine/lib/Controller/UserWorkflowsController.php
index 9be95a2ede9..953ce149233 100644
--- a/apps/workflowengine/lib/Controller/UserWorkflowsController.php
+++ b/apps/workflowengine/lib/Controller/UserWorkflowsController.php
@@ -10,6 +10,7 @@ namespace OCA\WorkflowEngine\Controller;
 
 use OCA\WorkflowEngine\Helper\ScopeContext;
 use OCA\WorkflowEngine\Manager;
+use OCP\AppFramework\Http\Attribute\NoAdminRequired;
 use OCP\AppFramework\Http\Attribute\PasswordConfirmationRequired;
 use OCP\AppFramework\Http\DataResponse;
 use OCP\AppFramework\OCS\OCSBadRequestException;
@@ -21,9 +22,6 @@ use Psr\Log\LoggerInterface;
 
 class UserWorkflowsController extends AWorkflowController {
 
-	/** @var IUserSession */
-	private $session;
-
 	/** @var ScopeContext */
 	private $scopeContext;
 
@@ -31,12 +29,10 @@ class UserWorkflowsController extends AWorkflowController {
 		$appName,
 		IRequest $request,
 		Manager $manager,
-		IUserSession $session,
-		LoggerInterface $logger
+		private IUserSession $session,
+		LoggerInterface $logger,
 	) {
 		parent::__construct($appName, $request, $manager, $logger);
-
-		$this->session = $session;
 	}
 
 	/**
@@ -44,47 +40,46 @@ class UserWorkflowsController extends AWorkflowController {
 	 *
 	 * Example: curl -u joann -H "OCS-APIREQUEST: true" "http://my.nc.srvr/ocs/v2.php/apps/workflowengine/api/v1/workflows/user?format=json"
 	 *
-	 * @NoAdminRequired
 	 * @throws OCSForbiddenException
 	 */
+	#[NoAdminRequired]
 	public function index(): DataResponse {
 		return parent::index();
 	}
 
 	/**
-	 * @NoAdminRequired
-	 *
 	 * Example: curl -u joann -H "OCS-APIREQUEST: true" "http://my.nc.srvr/ocs/v2.php/apps/workflowengine/api/v1/workflows/user/OCA\\Workflow_DocToPdf\\Operation?format=json"
 	 * @throws OCSForbiddenException
 	 */
+	#[NoAdminRequired]
 	public function show(string $id): DataResponse {
 		return parent::show($id);
 	}
 
 	/**
-	 * @NoAdminRequired
 	 * @throws OCSBadRequestException
 	 * @throws OCSForbiddenException
 	 */
+	#[NoAdminRequired]
 	#[PasswordConfirmationRequired]
 	public function create(string $class, string $name, array $checks, string $operation, string $entity, array $events): DataResponse {
 		return parent::create($class, $name, $checks, $operation, $entity, $events);
 	}
 
 	/**
-	 * @NoAdminRequired
 	 * @throws OCSBadRequestException
 	 * @throws OCSForbiddenException
 	 */
+	#[NoAdminRequired]
 	#[PasswordConfirmationRequired]
 	public function update(int $id, string $name, array $checks, string $operation, string $entity, array $events): DataResponse {
 		return parent::update($id, $name, $checks, $operation, $entity, $events);
 	}
 
 	/**
-	 * @NoAdminRequired
 	 * @throws OCSForbiddenException
 	 */
+	#[NoAdminRequired]
 	#[PasswordConfirmationRequired]
 	public function destroy(int $id): DataResponse {
 		return parent::destroy($id);