From e86224ee7ffdb18b7c7d17bc2c5cef2457665103 Mon Sep 17 00:00:00 2001
From: isanae <14251494+isanae@users.noreply.github.com>
Date: Sun, 9 Jun 2019 11:38:44 -0400
Subject: moved commiting changes to EditExecutablesDialog itself instead of
 doing it from the main window use a separate enabled state for custom
 overwrites and forced libraries, this remembers the values even when
 unchecking the checkbox, as long as the dialog stays opened pass the whole
 OrganizerCore to EditExecutablesDialog, simplifies a bunch of things

---
 src/editexecutablesdialog.h | 39 +++++++++++++++++++++++++--------------
 1 file changed, 25 insertions(+), 14 deletions(-)

(limited to 'src/editexecutablesdialog.h')

diff --git a/src/editexecutablesdialog.h b/src/editexecutablesdialog.h
index 6e006ae2..4957ebff 100644
--- a/src/editexecutablesdialog.h
+++ b/src/editexecutablesdialog.h
@@ -34,7 +34,7 @@ namespace Ui {
 }
 
 class ModList;
-
+class OrganizerCore;
 
 /** helper class to manage custom overwrites within the edit executables
  *  dialog
@@ -42,15 +42,22 @@ class ModList;
 class CustomOverwrites
 {
 public:
+  struct Info
+  {
+    bool enabled;
+    QString modName;
+  };
+
   void load(Profile* p, const ExecutablesList& exes);
-  std::optional<QString> find(const QString& title) const;
+  std::optional<Info> find(const QString& title) const;
 
-  void set(const QString& title, const QString& mod);
+  void setEnabled(const QString& title, bool b);
+  void setMod(const QString& title, const QString& mod);
   void rename(const QString& oldTitle, const QString& newTitle);
   void remove(const QString& title);
 
 private:
-  std::map<QString, QString> m_map;
+  std::map<QString, Info> m_map;
 };
 
 
@@ -61,15 +68,22 @@ class ForcedLibraries
 public:
   using list_type = QList<MOBase::ExecutableForcedLoadSetting>;
 
+  struct Info
+  {
+    bool enabled;
+    list_type list;
+  };
+
   void load(Profile* p, const ExecutablesList& exes);
-  std::optional<list_type> find(const QString& title) const;
+  std::optional<Info> find(const QString& title) const;
 
-  void set(const QString& title, const list_type& list);
+  void setEnabled(const QString& title, bool b);
+  void setList(const QString& title, const list_type& list);
   void rename(const QString& oldTitle, const QString& newTitle);
   void remove(const QString& title);
 
 private:
-  std::map<QString, list_type> m_map;
+  std::map<QString, Info> m_map;
 };
 
 
@@ -85,9 +99,7 @@ public:
    * @param executablesList current list of executables
    * @param parent parent widget
    **/
-  explicit EditExecutablesDialog(
-    const ExecutablesList &executablesList, const ModList &modList,
-    Profile *profile, const MOBase::IPluginGame *game, QWidget *parent = 0);
+  explicit EditExecutablesDialog(OrganizerCore& oc, QWidget* parent=nullptr);
 
   ~EditExecutablesDialog();
 
@@ -115,15 +127,13 @@ private slots:
 
 private:
   std::unique_ptr<Ui::EditExecutablesDialog> ui;
+  OrganizerCore& m_organizerCore;
+  const ExecutablesList m_originalExecutables;
   ExecutablesList m_executablesList;
   CustomOverwrites m_customOverwrites;
   ForcedLibraries m_forcedLibraries;
-  Profile *m_profile;
-  const MOBase::IPluginGame *m_gamePlugin;
   bool m_settingUI;
 
-  QListWidgetItem *m_currentItem;
-
 
   QListWidgetItem* selectedItem();
   Executable* selectedExe();
@@ -137,6 +147,7 @@ private:
   void setJarBinary(const QString& binaryName);
   std::optional<QString> makeNonConflictingTitle(const QString& prefix);
   bool isTitleConflicting(const QString& s);
+  void commitChanges();
 };
 
 #endif // EDITEXECUTABLESDIALOG_H
-- 
cgit v1.3.1