From fafdb146004401355db5245cc1f7c241c00cf991 Mon Sep 17 00:00:00 2001
From: isanae <14251494+isanae@users.noreply.github.com>
Date: Fri, 7 Jun 2019 19:26:54 -0400
Subject: fixed EditExecutablesDialog being opened without a parent removed
 Executable's constructor with values, replaced with default ctor + setters,
 all these strings were much too error-prone added Executable ctor overload to
 convert from ExecutableInfo plugin executables now override most of the
 custom changes renamed browseButton to browseBinaryButton to avoid confusion
 with the other browseDirButton fixed both browse dialogs not handling cancel
 EditExecutablesDialog's list used to change the text color for custom
 executables, replaced with italics

---
 src/mainwindow.cpp | 13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)

(limited to 'src/mainwindow.cpp')

diff --git a/src/mainwindow.cpp b/src/mainwindow.cpp
index 926ba43c..267a2971 100644
--- a/src/mainwindow.cpp
+++ b/src/mainwindow.cpp
@@ -2483,7 +2483,9 @@ bool MainWindow::modifyExecutablesDialog()
     EditExecutablesDialog dialog(*m_OrganizerCore.executablesList(),
                                  *m_OrganizerCore.modList(),
                                  m_OrganizerCore.currentProfile(),
-                                 m_OrganizerCore.managedGame());
+                                 m_OrganizerCore.managedGame(),
+                                 this);
+
     QSettings &settings = m_OrganizerCore.settings().directInterface();
     QString key = QString("geometry/%1").arg(dialog.objectName());
     if (settings.contains(key)) {
@@ -5507,9 +5509,12 @@ void MainWindow::addAsExecutable()
 
         if (!name.isEmpty()) {
           //Note: If this already exists, you'll lose custom settings
-          m_OrganizerCore.executablesList()->setExecutable({
-            name, binaryInfo, arguments, targetInfo.absolutePath(), QString(),
-            Executable::CustomExecutable});
+          m_OrganizerCore.executablesList()->setExecutable(Executable()
+            .title(name)
+            .binaryInfo(binaryInfo)
+            .arguments(arguments)
+            .workingDirectory(targetInfo.absolutePath())
+            .flags(Executable::CustomExecutable));
 
           refreshExecutablesList();
         }
-- 
cgit v1.3.1