Browse Source

Fix share controller to accept share attributes alone

Signed-off-by: Vincent Petry <vincent@nextcloud.com>
pull/32482/head
Vincent Petry 4 years ago
committed by Carl Schwan
parent
commit
9493f86de3
No known key found for this signature in database GPG Key ID: C3AA6B3A5EFA7AC5
  1. 7
      apps/files_sharing/lib/Controller/ShareAPIController.php

7
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);

Loading…
Cancel
Save