Also take in count audio files

Signed-off-by: Carl Schwan <carl@carlschwan.eu>
nextcloud · Nov 22, 2022 · d1ff6cf · d1ff6cf
1 parent 601d15c
commit d1ff6cf
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/apps/files_sharing/lib/Controller/ShareController.php b/apps/files_sharing/lib/Controller/ShareController.php
@@ -627,14 +627,14 @@ public function downloadShare($token, $files = null, $path = '', $downloadStartS
 		$userFolder = $this->rootFolder->getUserFolder($share->getShareOwner());
 		$originalSharePath = $userFolder->getRelativePath($share->getNode()->getPath());
 
-		$isVideo = false;
+		$isVideoAudio = false;
 
 		// Single file share
 		if ($share->getNode() instanceof \OCP\Files\File) {
 			$node = $share->getNode();
 			// Single file download
 			$this->singleFileDownloaded($share, $node);
-			$isVideo = str_starts_with($node->getMimeType(), 'video/');
+			$isVideoAutio = str_starts_with($node->getMimeType(), 'video/') || str_starts_with($node->getMimeType(), 'audio/');
 		}
 		// Directory share
 		else {
@@ -658,7 +658,7 @@ public function downloadShare($token, $files = null, $path = '', $downloadStartS
 				$node = $share->getNode();
 				// Single file download
 				$this->singleFileDownloaded($share, $node);
-				$isVideo = str_starts_with($node->getMimeType(), 'video/');
+				$isVideoAutio = str_starts_with($node->getMimeType(), 'video/') || str_starts_with($node->getMimeType(), 'audio/');
 			} else {
 				try {
 					if (!empty($files_list)) {
@@ -692,8 +692,8 @@ public function downloadShare($token, $files = null, $path = '', $downloadStartS
 
 		$this->emitAccessShareHook($share);
 
-		// Ensure download limit is counted unless we are streaming a video
-		if (!isset($_SERVER['HTTP_RANGE']) || !$isVideo) {
+		// Ensure download limit is counted unless we are streaming a video or audio file
+		if (!isset($_SERVER['HTTP_RANGE']) || !$isVideoAudio) {
 			$this->emitShareAccessEvent($share, self::SHARE_DOWNLOAD);
 		}