From 2b6ff55fa96a0dd1a549f14bf148fdacf1288025 Mon Sep 17 00:00:00 2001
From: Florian Storz <fstorz@users.noreply.github.com>
Date: Fri, 25 Jun 2021 16:09:47 +0200
Subject: [PATCH 1/3] hide download button for public preview of audio files

When the option to hide downloads was selected at public link creation, the download button can be hidden by the audio html attribute controlsList="nodownload"

Signed-off-by: Florian Storz <florian.storz@devlix.de>
---
 apps/files_sharing/templates/public.php | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/apps/files_sharing/templates/public.php b/apps/files_sharing/templates/public.php
index ffc449f9e6381..c17734e46c1e0 100644
--- a/apps/files_sharing/templates/public.php
+++ b/apps/files_sharing/templates/public.php
@@ -63,7 +63,8 @@
 			<?php else: ?>
 				<?php if ($_['previewEnabled'] && substr($_['mimetype'], 0, strpos($_['mimetype'], '/')) == 'audio'): ?>
 					<div id="imgframe">
-						<audio tabindex="0" controls="" preload="none" style="width: 100%; max-width: <?php p($_['previewMaxX']); ?>px; max-height: <?php p($_['previewMaxY']); ?>px">
+						<audio tabindex="0" controls="" preload="none" style="width: 100%; max-width: <?php p($_['previewMaxX']); ?>px; max-height: <?php p($_['previewMaxY']); ?>px"
+							   <?php if ($_['hideDownload']) { ?>controlsList="nodownload" <?php } ?>>
 							<source src="<?php p($_['downloadURL']); ?>" type="<?php p($_['mimetype']); ?>" />
 						</audio>
 					</div>

From 6aed80913ffee35dea250ab3d002bf0cd16f5f21 Mon Sep 17 00:00:00 2001
From: Florian Storz <fstorz@users.noreply.github.com>
Date: Mon, 19 Jul 2021 16:48:37 +0200
Subject: [PATCH 2/3] Add comment linking to PR

PR contains discussion about the implementation with pros and cons

Signed-off-by: Florian Storz <florian.storz@devlix.de>
---
 apps/files_sharing/templates/public.php | 1 +
 1 file changed, 1 insertion(+)

diff --git a/apps/files_sharing/templates/public.php b/apps/files_sharing/templates/public.php
index c17734e46c1e0..c663e4b811fb3 100644
--- a/apps/files_sharing/templates/public.php
+++ b/apps/files_sharing/templates/public.php
@@ -64,6 +64,7 @@
 				<?php if ($_['previewEnabled'] && substr($_['mimetype'], 0, strpos($_['mimetype'], '/')) == 'audio'): ?>
 					<div id="imgframe">
 						<audio tabindex="0" controls="" preload="none" style="width: 100%; max-width: <?php p($_['previewMaxX']); ?>px; max-height: <?php p($_['previewMaxY']); ?>px"
+							   <?php // See https://github.com/nextcloud/server/pull/27674 ?>
 							   <?php if ($_['hideDownload']) { ?>controlsList="nodownload" <?php } ?>>
 							<source src="<?php p($_['downloadURL']); ?>" type="<?php p($_['mimetype']); ?>" />
 						</audio>

From b803043b808f7cd57ebbb9c18e9c4a49b8f8f036 Mon Sep 17 00:00:00 2001
From: Florian Storz <fstorz@users.noreply.github.com>
Date: Mon, 19 Jul 2021 18:51:56 +0200
Subject: [PATCH 3/3] Remove trailing whitespace from comment

Signed-off-by: Florian Storz <florian.storz@devlix.de>
---
 apps/files_sharing/templates/public.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/apps/files_sharing/templates/public.php b/apps/files_sharing/templates/public.php
index c663e4b811fb3..b2d72a3c3371b 100644
--- a/apps/files_sharing/templates/public.php
+++ b/apps/files_sharing/templates/public.php
@@ -64,7 +64,7 @@
 				<?php if ($_['previewEnabled'] && substr($_['mimetype'], 0, strpos($_['mimetype'], '/')) == 'audio'): ?>
 					<div id="imgframe">
 						<audio tabindex="0" controls="" preload="none" style="width: 100%; max-width: <?php p($_['previewMaxX']); ?>px; max-height: <?php p($_['previewMaxY']); ?>px"
-							   <?php // See https://github.com/nextcloud/server/pull/27674 ?>
+							   <?php // See https://github.com/nextcloud/server/pull/27674?>
 							   <?php if ($_['hideDownload']) { ?>controlsList="nodownload" <?php } ?>>
 							<source src="<?php p($_['downloadURL']); ?>" type="<?php p($_['mimetype']); ?>" />
 						</audio>