aboutsummaryrefslogtreecommitdiffstats
path: root/apps/files/lib/Controller/TransferOwnershipController.php
diff options
context:
space:
mode:
authorprovokateurin <kate@provokateurin.de>2024-07-25 13:14:46 +0200
committerprovokateurin <kate@provokateurin.de>2024-07-27 22:44:01 +0200
commit3fde60db853454fda3b2c82258ae87b6f29f4726 (patch)
tree9e0be1b7e1c0e37e0599b8df8efc3b3a72487f16 /apps/files/lib/Controller/TransferOwnershipController.php
parent212a621697cd32b65ea78fa90015cec9d9d1dfe3 (diff)
downloadnextcloud-server-3fde60db853454fda3b2c82258ae87b6f29f4726.tar.gz
nextcloud-server-3fde60db853454fda3b2c82258ae87b6f29f4726.zip
refactor(files): Replace security annotations with respective attributes
Signed-off-by: provokateurin <kate@provokateurin.de>
Diffstat (limited to 'apps/files/lib/Controller/TransferOwnershipController.php')
-rw-r--r--apps/files/lib/Controller/TransferOwnershipController.php10
1 files changed, 4 insertions, 6 deletions
diff --git a/apps/files/lib/Controller/TransferOwnershipController.php b/apps/files/lib/Controller/TransferOwnershipController.php
index 924c200f375..a71373baae4 100644
--- a/apps/files/lib/Controller/TransferOwnershipController.php
+++ b/apps/files/lib/Controller/TransferOwnershipController.php
@@ -13,6 +13,7 @@ use OCA\Files\Db\TransferOwnership as TransferOwnershipEntity;
use OCA\Files\Db\TransferOwnershipMapper;
use OCP\AppFramework\Db\DoesNotExistException;
use OCP\AppFramework\Http;
+use OCP\AppFramework\Http\Attribute\NoAdminRequired;
use OCP\AppFramework\Http\DataResponse;
use OCP\AppFramework\OCSController;
use OCP\AppFramework\Utility\ITimeFactory;
@@ -62,8 +63,6 @@ class TransferOwnershipController extends OCSController {
/**
- * @NoAdminRequired
- *
* Transfer the ownership to another user
*
* @param string $recipient Username of the recipient
@@ -75,6 +74,7 @@ class TransferOwnershipController extends OCSController {
* 400: Transferring ownership is not possible
* 403: Transferring ownership is not allowed
*/
+ #[NoAdminRequired]
public function transfer(string $recipient, string $path): DataResponse {
$recipientUser = $this->userManager->get($recipient);
@@ -118,8 +118,6 @@ class TransferOwnershipController extends OCSController {
}
/**
- * @NoAdminRequired
- *
* Accept an ownership transfer
*
* @param int $id ID of the ownership transfer
@@ -130,6 +128,7 @@ class TransferOwnershipController extends OCSController {
* 403: Accepting ownership transfer is not allowed
* 404: Ownership transfer not found
*/
+ #[NoAdminRequired]
public function accept(int $id): DataResponse {
try {
$transferOwnership = $this->mapper->getById($id);
@@ -161,8 +160,6 @@ class TransferOwnershipController extends OCSController {
}
/**
- * @NoAdminRequired
- *
* Reject an ownership transfer
*
* @param int $id ID of the ownership transfer
@@ -173,6 +170,7 @@ class TransferOwnershipController extends OCSController {
* 403: Rejecting ownership transfer is not allowed
* 404: Ownership transfer not found
*/
+ #[NoAdminRequired]
public function reject(int $id): DataResponse {
try {
$transferOwnership = $this->mapper->getById($id);