From e2615cc7efb84c5e673b977cf00b26f5e03fe330 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 fd20c27501d68dc092c87eb9c110017995a5a1f1 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 34973fad26edb3f8117f5a05678c9b6cd16858cc 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>