fixup! feat: improve ZipFolderPlugin behaviour for different cases

Signed-off-by: Salvatore Martire <4652631+salmart-dev@users.noreply.github.com>

Author: salmart-dev

apps/files_sharing/lib/Listener/BeforeZipCreatedListener.php

@@ -48,15 +48,20 @@ public function handle(Event $event): void {
 
 		$user = $this->userSession->getUser();
 		$folder = $event->getFolder();
-		if ($user === null && $event->getFolder() === null) {
+		if ($user === null && $folder === null) {
 			// there is no way to know if the file is downloadable or not, allow it
 			$event->setSuccessful(true);
 			return;
 		}
 
 		// in link-shares there may be no user, in that case we check that the share folder is downloadable
 		$userFolder = $user ? $this->rootFolder->getUserFolder($user->getUID()) : null;
-		$folderToCheck = $userFolder ? $userFolder->get($dir) : $folder;
+
+		$folderToCheck = $folder;
+		if ($userFolder !== null) {
+			// if we have a user, use their user folder
+			$folderToCheck = $userFolder->getPath() === $dir ? $userFolder : $userFolder->get($dir);
+		}
 
 		$viewOnlyHandler = new ViewOnly($folderToCheck);
 		$isRootDownloadable = $viewOnlyHandler->isDownloadable($folderToCheck);