aboutsummaryrefslogtreecommitdiffstats
path: root/apps
diff options
context:
space:
mode:
authorMorris Jobke <hey@morrisjobke.de>2020-10-26 16:22:27 +0100
committerMorris Jobke <hey@morrisjobke.de>2020-10-26 16:22:27 +0100
commit0390a90d70104c2b3237feb05c3398f8ea1612c9 (patch)
treed5d8549a204094fcd1420b640e81804283000c0f /apps
parentca1dc0009dfde447c90db826465defbb156966da (diff)
downloadnextcloud-server-0390a90d70104c2b3237feb05c3398f8ea1612c9.tar.gz
nextcloud-server-0390a90d70104c2b3237feb05c3398f8ea1612c9.zip
Return correct error message on share download without read permission
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
Diffstat (limited to 'apps')
-rw-r--r--apps/files_sharing/lib/Controller/ShareController.php2
-rw-r--r--apps/files_sharing/tests/Controller/ShareControllerTest.php2
2 files changed, 2 insertions, 2 deletions
diff --git a/apps/files_sharing/lib/Controller/ShareController.php b/apps/files_sharing/lib/Controller/ShareController.php
index fdf69abd9ac..3e514b3a74d 100644
--- a/apps/files_sharing/lib/Controller/ShareController.php
+++ b/apps/files_sharing/lib/Controller/ShareController.php
@@ -556,7 +556,7 @@ class ShareController extends AuthPublicShareController {
$share = $this->shareManager->getShareByToken($token);
if (!($share->getPermissions() & \OCP\Constants::PERMISSION_READ)) {
- return new \OCP\AppFramework\Http\DataResponse('Share is read-only');
+ return new \OCP\AppFramework\Http\DataResponse('Share has no read permission');
}
$files_list = null;
diff --git a/apps/files_sharing/tests/Controller/ShareControllerTest.php b/apps/files_sharing/tests/Controller/ShareControllerTest.php
index 33dff13cafe..270f38a1148 100644
--- a/apps/files_sharing/tests/Controller/ShareControllerTest.php
+++ b/apps/files_sharing/tests/Controller/ShareControllerTest.php
@@ -840,7 +840,7 @@ class ShareControllerTest extends \Test\TestCase {
// Test with a password protected share and no authentication
$response = $this->shareController->downloadShare('validtoken');
- $expectedResponse = new DataResponse('Share is read-only');
+ $expectedResponse = new DataResponse('Share has no read permission');
$this->assertEquals($expectedResponse, $response);
}