diff options
author | provokateurin <kate@provokateurin.de> | 2024-07-25 13:14:46 +0200 |
---|---|---|
committer | provokateurin <kate@provokateurin.de> | 2024-07-27 22:44:01 +0200 |
commit | 3fde60db853454fda3b2c82258ae87b6f29f4726 (patch) | |
tree | 9e0be1b7e1c0e37e0599b8df8efc3b3a72487f16 /apps/files/lib/Controller/TransferOwnershipController.php | |
parent | 212a621697cd32b65ea78fa90015cec9d9d1dfe3 (diff) | |
download | nextcloud-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.php | 10 |
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); |