Return correct error message on share download without read permission
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
This commit is contained in:
parent
ca1dc0009d
commit
0390a90d70
|
@ -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;
|
||||
|
|
|
@ -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);
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in New Issue