Save changes on close

kraxarn · May 9, 2020 · 8fa14ee · 8fa14ee
1 parent b01cf2a
commit 8fa14ee
Show file tree

Hide file tree

Showing 4 changed files with 25 additions and 8 deletions.
diff --git a/src/mainwindow.hpp b/src/mainwindow.hpp
@@ -84,6 +84,7 @@ class MainWindow : public QMainWindow
 	QString currentLibraryItem();
 	void reloadTrayIcon();
 	spt::User getCurrentUser();
+	void refreshPlaylists();
 	// I know these should be methods, I'm just lazy
 	QString					cacheLocation;
 	QVector<spt::Playlist>	*sptPlaylists;
@@ -117,7 +118,6 @@ class MainWindow : public QMainWindow
 	// Methods
 	QWidget *createCentralWidget();
 	QToolBar *createToolBar();
-	void refreshPlaylists();
 	bool loadSongs(const QVector<spt::Track> &tracks);
 	void setAlbumImage(const QString &url);
 	static QString formatTime(int ms);

diff --git a/src/playlisteditdialog.cpp b/src/playlisteditdialog.cpp
@@ -1,6 +1,7 @@
 #include "playlisteditdialog.hpp"
 
-PlaylistEditDialog::PlaylistEditDialog(const spt::Playlist &playlist, int selectedIndex, QWidget *parent)
+PlaylistEditDialog::PlaylistEditDialog(spt::Spotify *spotify, const spt::Playlist &playlist,
+	int selectedIndex, QWidget *parent)
 	: QDialog(parent)
 {
 	setWindowTitle(playlist.name);
@@ -34,8 +35,20 @@ PlaylistEditDialog::PlaylistEditDialog(const spt::Playlist &playlist, int select
 	// Dialog buttons
 	auto buttons = new QDialogButtonBox(this);
 	buttons->setStandardButtons(QDialogButtonBox::Ok | QDialogButtonBox::Cancel);
-	QDialogButtonBox::connect(buttons, &QDialogButtonBox::accepted, [this] {
-		accept();
+	QDialogButtonBox::connect(buttons, &QDialogButtonBox::accepted, [this, playlist, spotify] {
+		auto pl = playlist;
+		pl.name				= name->text();
+		pl.description		= description->toPlainText();
+		pl.isPublic			= isPublic->isChecked();
+		pl.collaborative	= isCollaborative->isChecked();
+		auto result = spotify->editPlaylist(pl);
+		if (result.isEmpty())
+		{
+			accept();
+			return;
+		}
+		QMessageBox::warning(this, "Edit failed",
+			QString("Failed to save changes: %1").arg(result));
 	});
 	QDialogButtonBox::connect(buttons, &QDialogButtonBox::rejected, [this] {
 		reject();

diff --git a/src/playlisteditdialog.hpp b/src/playlisteditdialog.hpp
@@ -10,13 +10,15 @@
 #include <QPushButton>
 #include <QDialogButtonBox>
 #include <QCheckBox>
+#include <QMessageBox>
 
 class PlaylistEditDialog : public QDialog
 {
 	Q_OBJECT
 
 public:
-	PlaylistEditDialog(const spt::Playlist &playlist, int selectedIndex, QWidget *parent = nullptr);
+	PlaylistEditDialog(spt::Spotify *spotify, const spt::Playlist &playlist,
+		int selectedIndex, QWidget *parent = nullptr);
 
 private:
 	QLineEdit	*name;

diff --git a/src/playlistmenu.cpp b/src/playlistmenu.cpp
@@ -31,12 +31,14 @@ PlaylistMenu::PlaylistMenu(spt::Spotify &spotify, const spt::Playlist &playlist,
 			window->setStatus(status, true);
 	});
 	if (isOwner)
-		QAction::connect(addAction(Icon::get("document-edit"), "Edit"), &QAction::triggered, [this, playlist](bool checked)
+		QAction::connect(addAction(Icon::get("document-edit"), "Edit"), &QAction::triggered,
+			[this, playlist, &spotify, window](bool checked)
 		{
 			if (editDialog != nullptr)
 				delete editDialog;
-			editDialog = new PlaylistEditDialog(playlist, -1, parentWidget());
-			editDialog->show();
+			editDialog = new PlaylistEditDialog(&spotify, playlist, -1, parentWidget());
+			if (editDialog->exec() == QDialog::Accepted)
+				window->refreshPlaylists();
 		});
 	auto share = addMenu(Icon::get("document-share"), "Share");
 	auto sharePlaylist = share->addAction("Copy playlist link");