From 9324025c41ca482e59253de2e90a3f3d6b1fcd22 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Alban=20F=C3=A9ron?= <alban.feron@gmail.com>
Date: Sat, 10 Apr 2021 15:47:31 +0200
Subject: [PATCH] Added disclaimer before exporting playlist

---
 supysonic/frontend/playlist.py     |  8 +++++---
 supysonic/static/js/supysonic.js   |  2 +-
 supysonic/templates/playlists.html | 24 ++++++++++++++++++++++--
 3 files changed, 28 insertions(+), 6 deletions(-)

diff --git a/supysonic/frontend/playlist.py b/supysonic/frontend/playlist.py
index a0cbeb4..4cd0418 100644
--- a/supysonic/frontend/playlist.py
+++ b/supysonic/frontend/playlist.py
@@ -40,6 +40,7 @@ def playlist_details(uid):
 
     return render_template("playlist.html", playlist=playlist)
 
+
 @frontend.route("/playlist/<uid>/export")
 def playlist_export(uid):
     try:
@@ -57,9 +58,10 @@ def playlist_export(uid):
     return Response(
         render_template("playlist_export.m3u", playlist=playlist),
         mimetype="audio/mpegurl",
-        headers={"Content-disposition": f"attachment; filename={playlist.name}.m3u"}
-        )
-
+        headers={
+            "Content-disposition": "attachment; filename={}.m3u".format(playlist.name)
+        },
+    )
 
 
 @frontend.route("/playlist/<uid>", methods=["POST"])
diff --git a/supysonic/static/js/supysonic.js b/supysonic/static/js/supysonic.js
index e207c69..44c58b0 100644
--- a/supysonic/static/js/supysonic.js
+++ b/supysonic/static/js/supysonic.js
@@ -12,6 +12,6 @@ $(function () {
   $('[data-toggle="tooltip"]').tooltip()
 });
 
-$('#confirm-delete').on('show.bs.modal', function(e) {
+$('.modal').on('show.bs.modal', function(e) {
   $(this).find('.btn-ok').attr('href', $(e.relatedTarget).data('href'));
 });
diff --git a/supysonic/templates/playlists.html b/supysonic/templates/playlists.html
index 4949f15..8e5c479 100644
--- a/supysonic/templates/playlists.html
+++ b/supysonic/templates/playlists.html
@@ -32,8 +32,8 @@
                                  aria-label="Public playlist"></span>{% else %}<span
                                  class="glyphicon glyphicon-unchecked"
                                  aria-label="Private playlist"></span>{% endif %}</td>
-      <td><a href="{{ url_for('frontend.playlist_export', uid = p.id) }}">
-        <button class="btn btn-primary btn-xs"><span class="glyphicon glyphicon-arrow-down" title="Export playlist"></span></button></a></td>
+      <td><button class="btn btn-primary btn-xs" data-href="{{ url_for('frontend.playlist_export', uid = p.id) }}" data-toggle="modal" data-target="#confirm-export" aria-label="Export playlist">
+        <span class="glyphicon glyphicon-arrow-down" aria-hidden="true" data-toggle="tooltip" data-placement="top" title="Export M3U playlist"></span></button></td>
       <td><button class="btn btn-danger btn-xs" data-href="{{ url_for('frontend.playlist_delete', uid = p.id) }}" data-toggle="modal" data-target="#confirm-delete" aria-label="Delete playlist">
         <span class="glyphicon glyphicon-remove-circle" aria-hidden="true" data-toggle="tooltip" data-placement="top" title="Delete playlist"></span></button></td>
     </tr>
@@ -77,4 +77,24 @@
     </div><!-- /.modal-content -->
   </div><!-- /.modal-dialog -->
 </div><!-- /.modal -->
+<div class="modal fade" id="confirm-export" tabindex="-1" role="dialog">
+  <div class="modal-dialog" role="document">
+    <div class="modal-content">
+      <div class="modal-header">
+        <button type="button" class="close" data-dismiss="modal" aria-label="Close"><span aria-hidden="true">&times;</span></button>
+        <h4 class="modal-title">Warning</h4>
+      </div>
+      <div class="modal-body">
+        <p>The exported playlist will use the same local paths the server uses itself. This means the playlist will only
+          work on the very same system the Supysonic server is running. If you were to export it from another machine it
+          won't be usable.
+        </p>
+      </div>
+      <div class="modal-footer">
+        <button type="button" class="btn btn-default" data-dismiss="modal">Oh ok. Forget it</button>
+        <a type="button" class="btn btn-primary btn-ok">Got it, export the playlist</a>
+      </div>
+    </div><!-- /.modal-content -->
+  </div><!-- /.modal-dialog -->
+</div><!-- /.modal -->
 {% endblock %}