From 9493f86de34e76e37c13f87aab3123a3efbfdd84 Mon Sep 17 00:00:00 2001
From: Vincent Petry <vincent@nextcloud.com>
Date: Tue, 24 May 2022 12:08:07 +0200
Subject: Fix share controller to accept share attributes alone

Signed-off-by: Vincent Petry <vincent@nextcloud.com>
---
 apps/files_sharing/lib/Controller/ShareAPIController.php | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

(limited to 'apps/files_sharing/lib/Controller')

diff --git a/apps/files_sharing/lib/Controller/ShareAPIController.php b/apps/files_sharing/lib/Controller/ShareAPIController.php
index e40aed0da70..f5c64c5d0d0 100644
--- a/apps/files_sharing/lib/Controller/ShareAPIController.php
+++ b/apps/files_sharing/lib/Controller/ShareAPIController.php
@@ -1077,6 +1077,8 @@ class ShareAPIController extends OCSController {
 			throw new OCSForbiddenException('You are not allowed to edit incoming shares');
 		}
 
+		$shareAttributes = $this->request->getParam('attributes', null);
+
 		if (
 			$permissions === null &&
 			$password === null &&
@@ -1085,7 +1087,8 @@ class ShareAPIController extends OCSController {
 			$expireDate === null &&
 			$note === null &&
 			$label === null &&
-			$hideDownload === null
+			$hideDownload === null &&
+			$shareAttributes === null
 		) {
 			throw new OCSBadRequestException($this->l->t('Wrong or no update parameter given'));
 		}
@@ -1224,7 +1227,7 @@ class ShareAPIController extends OCSController {
 			}
 		}
 
-		$share = $this->setShareAttributes($share, $this->request->getParam('attributes', null));
+		$share = $this->setShareAttributes($share, $shareAttributes);
 
 		try {
 			$share = $this->shareManager->updateShare($share);
-- 
cgit v1.2.3