diff --git a/apps/files_sharing/lib/Controller/ShareController.php b/apps/files_sharing/lib/Controller/ShareController.php index 37a564d76540f..43cf741f40706 100644 --- a/apps/files_sharing/lib/Controller/ShareController.php +++ b/apps/files_sharing/lib/Controller/ShareController.php @@ -46,6 +46,7 @@ use OCP\AppFramework\Http\Template\LinkMenuAction; use OCP\AppFramework\Http\Template\PublicTemplateResponse; use OCP\Defaults; +use OCP\Files\Folder; use OCP\IL10N; use OCP\Template; use OCP\Share; @@ -527,10 +528,6 @@ public function downloadShare($token, $files = null, $path = '', $downloadStartS throw new NotFoundException(); } - if ($share->getHideDownload()) { - return new NotFoundResponse(); - } - $userFolder = $this->rootFolder->getUserFolder($share->getShareOwner()); $originalSharePath = $userFolder->getRelativePath($share->getNode()->getPath()); @@ -560,11 +557,17 @@ public function downloadShare($token, $files = null, $path = '', $downloadStartS if ($node instanceof \OCP\Files\File) { // Single file download $this->singleFileDownloaded($share, $share->getNode()); - } else if (!empty($files_list)) { - $this->fileListDownloaded($share, $files_list, $node); } else { - // The folder is downloaded - $this->singleFileDownloaded($share, $share->getNode()); + try { + if (!empty($files_list)) { + $this->fileListDownloaded($share, $files_list, $node); + } else { + // The folder is downloaded + $this->singleFileDownloaded($share, $share->getNode()); + } + } catch (NotFoundException $e) { + return new NotFoundResponse(); + } } } @@ -616,8 +619,13 @@ public function downloadShare($token, $files = null, $path = '', $downloadStartS * @param Share\IShare $share * @param array $files_list * @param \OCP\Files\Folder $node + * @throws NotFoundException when trying to download a folder or multiple files of a "hide download" share */ protected function fileListDownloaded(Share\IShare $share, array $files_list, \OCP\Files\Folder $node) { + if ($share->getHideDownload() && count($files_list) > 1) { + throw new NotFoundException('Downloading more than 1 file'); + } + foreach ($files_list as $file) { $subNode = $node->get($file); $this->singleFileDownloaded($share, $subNode); @@ -629,8 +637,12 @@ protected function fileListDownloaded(Share\IShare $share, array $files_list, \O * create activity if a single file was downloaded from a link share * * @param Share\IShare $share + * @throws NotFoundException when trying to download a folder of a "hide download" share */ protected function singleFileDownloaded(Share\IShare $share, \OCP\Files\Node $node) { + if ($share->getHideDownload() && $node instanceof Folder) { + throw new NotFoundException('Downloading a folder'); + } $fileId = $node->getId();