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/executableslist.cpp | 118 ++++++++++++++++++++++++++++++++----------------
 1 file changed, 80 insertions(+), 38 deletions(-)

(limited to 'src/executableslist.cpp')

diff --git a/src/executableslist.cpp b/src/executableslist.cpp
index 9c291708..d884bdf2 100644
--- a/src/executableslist.cpp
+++ b/src/executableslist.cpp
@@ -81,13 +81,13 @@ void ExecutablesList::load(const MOBase::IPluginGame* game, QSettings& settings)
     if (settings.value("ownicon", false).toBool())
       flags |= Executable::UseApplicationIcon;
 
-    setExecutable({
-      settings.value("title").toString(),
-      settings.value("binary").toString(),
-      settings.value("arguments").toString(),
-      settings.value("workingDirectory", "").toString(),
-      settings.value("steamAppID", "").toString(),
-      flags});
+    setExecutable(Executable()
+      .title(settings.value("title").toString())
+      .binaryInfo(settings.value("binary").toString())
+      .arguments(settings.value("arguments").toString())
+      .steamAppID(settings.value("steamAppID", "").toString())
+      .workingDirectory(settings.value("workingDirectory", "").toString())
+      .flags(flags));
   }
 
   settings.endArray();
@@ -125,27 +125,22 @@ void ExecutablesList::addFromPlugin(IPluginGame const *game)
 
   for (const ExecutableInfo &info : game->executables()) {
     if (info.isValid()) {
-      setExecutable({
-        info.title(),
-        info.binary().absoluteFilePath(),
-        info.arguments().join(" "),
-        info.workingDirectory().absolutePath(),
-        info.steamAppID(),
-        Executable::UseApplicationIcon});
+      setExecutable({info, Executable::UseApplicationIcon});
     }
   }
 
-  ExecutableInfo explorerpp = ExecutableInfo("Explore Virtual Folder", QFileInfo(QCoreApplication::applicationDirPath() + "/explorer++/Explorer++.exe" ))
-      .withArgument(QString("\"%1\"").arg(QDir::toNativeSeparators(game->dataDirectory().absolutePath())));
-
-  if (explorerpp.isValid()) {
-    setExecutable({
-      explorerpp.title(),
-      explorerpp.binary().absoluteFilePath(),
-      explorerpp.arguments().join(" "),
-      explorerpp.workingDirectory().absolutePath(),
-      explorerpp.steamAppID(),
-      Executable::UseApplicationIcon});
+  const QFileInfo eppBin(QCoreApplication::applicationDirPath() + "/explorer++/Explorer++.exe");
+
+  if (eppBin.exists()) {
+    const auto args = QString("\"%1\"")
+      .arg(QDir::toNativeSeparators(game->dataDirectory().absolutePath()));
+
+    setExecutable(Executable()
+      .title("Explore Virtual Folder")
+      .binaryInfo(eppBin)
+      .arguments(args)
+      .workingDirectory(eppBin.absolutePath())
+      .flags(Executable::UseApplicationIcon));
   }
 }
 
@@ -213,15 +208,13 @@ void ExecutablesList::remove(const QString &title)
 }
 
 
-Executable::Executable(
-  QString title, QFileInfo binaryInfo, QString arguments,
-  QString steamAppID, QString workingDirectory, Flags flags) :
-    m_title(std::move(title)),
-    m_binaryInfo(std::move(binaryInfo)),
-    m_arguments(std::move(arguments)),
-    m_steamAppID(std::move(steamAppID)),
-    m_workingDirectory(std::move(workingDirectory)),
-    m_flags(flags)
+Executable::Executable(const MOBase::ExecutableInfo& info, Flags flags) :
+  m_title(info.title()),
+  m_binaryInfo(info.binary()),
+  m_arguments(info.arguments().join(" ")),
+  m_steamAppID(info.steamAppID()),
+  m_workingDirectory(info.workingDirectory().absolutePath()),
+  m_flags(flags)
 {
 }
 
@@ -255,6 +248,42 @@ Executable::Flags Executable::flags() const
   return m_flags;
 }
 
+Executable& Executable::title(const QString& s)
+{
+  m_title = s;
+  return *this;
+}
+
+Executable& Executable::binaryInfo(const QFileInfo& fi)
+{
+  m_binaryInfo = fi;
+  return *this;
+}
+
+Executable& Executable::arguments(const QString& s)
+{
+  m_arguments = s;
+  return *this;
+}
+
+Executable& Executable::steamAppID(const QString& s)
+{
+  m_steamAppID = s;
+  return *this;
+}
+
+Executable& Executable::workingDirectory(const QString& s)
+{
+  m_workingDirectory = s;
+  return *this;
+}
+
+Executable& Executable::flags(Flags f)
+{
+  m_flags = f;
+  return *this;
+}
+
 bool Executable::isCustom() const
 {
   return m_flags.testFlag(CustomExecutable);
@@ -281,12 +310,25 @@ bool Executable::usesOwnIcon() const
 
 void Executable::mergeFrom(const Executable& other)
 {
-  if (!isCustom()) {
-    // this happens when the user is trying to modify a plugin executable
+  // flags on plugin executables that the user is allowed to chnage
+  const auto allow = ShowInToolbar;
+
+
+  if (!isCustom() && !other.isCustom()) {
+    // this happens when loading plugin executables in addFromPlugin(), replace
+    // everything in case the plugin has changed
 
-    // only change some of the flags
-    const auto allow = ShowInToolbar;
+    // remember the flags though
+    const auto flags = m_flags;
 
+    // overwrite everything
+    *this = other;
+
+    // set the user flags
+    m_flags |= (flags & allow);
+  }
+  else if (!isCustom()) {
+    // this happens when the user is trying to modify a plugin executable
     m_flags |= (other.flags() & allow);
   } else {
     // this happens after executables are loaded from settings and plugin
-- 
cgit v1.3.1