Merge pull request ModOrganizer2#1621 from LostDragonist/misc_fixes

Misc fixes

Author: LostDragonist

src/createinstancedialog.cpp

@@ -341,28 +341,25 @@ void CreateInstanceDialog::finish()
       s.game().setEdition(ci.gameVariant);
     }
 
-    if (ci.type == Global) {
-      if (ci.paths.base != ci.dataPath) {
-        s.paths().setBase(ci.paths.base);
-      }
-
-      if (ci.paths.downloads != cid::makeDefaultPath(AppConfig::downloadPath())) {
-        s.paths().setDownloads(ci.paths.downloads);
-      }
+    if (ci.paths.base != ci.dataPath) {
+      s.paths().setBase(ci.paths.base);
+    }
 
-      if (ci.paths.mods != cid::makeDefaultPath(AppConfig::modsPath())) {
-        s.paths().setMods(ci.paths.mods);
-      }
+    if (ci.paths.downloads != cid::makeDefaultPath(AppConfig::downloadPath())) {
+      s.paths().setDownloads(ci.paths.downloads);
+    }
 
-      if (ci.paths.profiles != cid::makeDefaultPath(AppConfig::profilesPath())) {
-        s.paths().setProfiles(ci.paths.profiles);
-      }
+    if (ci.paths.mods != cid::makeDefaultPath(AppConfig::modsPath())) {
+      s.paths().setMods(ci.paths.mods);
+    }
 
-      if (ci.paths.overwrite != cid::makeDefaultPath(AppConfig::overwritePath())) {
-        s.paths().setOverwrite(ci.paths.overwrite);
-      }
+    if (ci.paths.profiles != cid::makeDefaultPath(AppConfig::profilesPath())) {
+      s.paths().setProfiles(ci.paths.profiles);
     }
 
+    if (ci.paths.overwrite != cid::makeDefaultPath(AppConfig::overwritePath())) {
+      s.paths().setOverwrite(ci.paths.overwrite);
+    }
 
     logCreation(tr("Writing %1...").arg(ci.iniPath));
 

src/createinstancedialog.ui

@@ -67,7 +67,7 @@
      <property name="currentIndex">
       <number>0</number>
      </property>
-     <widget class="QWidget" name="page_10">
+     <widget class="QWidget" name="page_1">
       <layout class="QVBoxLayout" name="verticalLayout_21">
        <item>
         <widget class="QLabel" name="label_15">
@@ -115,7 +115,7 @@
        </item>
       </layout>
      </widget>
-     <widget class="QWidget" name="page">
+     <widget class="QWidget" name="page_2">
       <layout class="QVBoxLayout" name="verticalLayout_3">
        <item>
         <widget class="QWidget" name="widget_7" native="true">
@@ -226,7 +226,7 @@
        </item>
       </layout>
      </widget>
-     <widget class="QWidget" name="page_2">
+     <widget class="QWidget" name="page_3">
       <layout class="QVBoxLayout" name="verticalLayout_7">
        <item>
         <widget class="QWidget" name="widget_5" native="true">
@@ -367,7 +367,7 @@
        </item>
       </layout>
      </widget>
-     <widget class="QWidget" name="page_9">
+     <widget class="QWidget" name="page_4">
       <layout class="QVBoxLayout" name="verticalLayout_19">
        <item>
         <widget class="QWidget" name="widget_16" native="true">
@@ -594,7 +594,7 @@
             <property name="currentIndex">
              <number>0</number>
             </property>
-            <widget class="QWidget" name="page_7">
+            <widget class="QWidget" name="page_6_simple">
              <layout class="QVBoxLayout" name="verticalLayout_12">
               <property name="leftMargin">
                <number>0</number>
@@ -676,7 +676,7 @@
               </item>
              </layout>
             </widget>
-            <widget class="QWidget" name="page_8">
+            <widget class="QWidget" name="page_6_advanced">
              <layout class="QVBoxLayout" name="verticalLayout_13">
               <property name="leftMargin">
                <number>0</number>
@@ -906,7 +906,7 @@
        </item>
       </layout>
      </widget>
-     <widget class="QWidget" name="page_3">
+     <widget class="QWidget" name="page_7">
       <layout class="QVBoxLayout" name="verticalLayout_4">
        <item>
         <widget class="QWidget" name="widget_4" native="true">
@@ -1048,7 +1048,7 @@
        </item>
       </layout>
      </widget>
-     <widget class="QWidget" name="page_11">
+     <widget class="QWidget" name="page_8">
       <layout class="QVBoxLayout" name="verticalLayout_22">
        <item>
         <widget class="QWidget" name="widget_18" native="true">
@@ -1232,6 +1232,38 @@
    <header location="global">linklabel.h</header>
   </customwidget>
  </customwidgets>
+ <tabstops>
+  <tabstop>next</tabstop>
+  <tabstop>cancel</tabstop>
+  <tabstop>hideIntro</tabstop>
+  <tabstop>createGlobal</tabstop>
+  <tabstop>createPortable</tabstop>
+  <tabstop>scrollArea_3</tabstop>
+  <tabstop>showAllGames</tabstop>
+  <tabstop>gamesFilter</tabstop>
+  <tabstop>scrollArea_2</tabstop>
+  <tabstop>instanceName</tabstop>
+  <tabstop>location</tabstop>
+  <tabstop>browseLocation</tabstop>
+  <tabstop>advancedPathOptions</tabstop>
+  <tabstop>base</tabstop>
+  <tabstop>browseBase</tabstop>
+  <tabstop>downloads</tabstop>
+  <tabstop>browseDownloads</tabstop>
+  <tabstop>mods</tabstop>
+  <tabstop>browseMods</tabstop>
+  <tabstop>profiles</tabstop>
+  <tabstop>browseProfiles</tabstop>
+  <tabstop>overwrite</tabstop>
+  <tabstop>browseOverwrite</tabstop>
+  <tabstop>nexusConnect</tabstop>
+  <tabstop>nexusManual</tabstop>
+  <tabstop>nexusLog</tabstop>
+  <tabstop>review</tabstop>
+  <tabstop>creationLog</tabstop>
+  <tabstop>launch</tabstop>
+  <tabstop>back</tabstop>
+ </tabstops>
  <resources>
   <include location="resources.qrc"/>
  </resources>

src/envwindows.cpp

@@ -94,11 +94,6 @@ QString WindowsInfo::toString() const
     sl.push_back(m_release.buildLab);
   }
 
-  // product name
-  if (!m_release.productName.isEmpty()) {
-    sl.push_back(m_release.productName);
-  }
-
   // elevated
   QString elevated = "?";
   if (m_elevated.has_value()) {
@@ -187,11 +182,11 @@ WindowsInfo::Release WindowsInfo::getRelease() const
     }
   }
 
-  // localized name of windows, such as "Windows 10 Pro"
-  r.productName = settings.value("ProductName", "").toString();
-
   // release ID, such as 1803
-  r.ID = settings.value("ReleaseId", "").toString();
+  r.ID = settings.value("DisplayVersion", "").toString();
+  if (r.ID.isEmpty()) {
+    r.ID = settings.value("ReleaseId", "").toString();
+  }
 
   // some other build number, shown in winver.exe
   r.UBR = settings.value("UBR", 0).toUInt();

src/envwindows.h

@@ -40,11 +40,7 @@ class WindowsInfo
     // the BuildLab entry from the registry, may be empty
     QString buildLab;
 
-    // product name such as "Windows 10 Pro", may not be in English, may be
-    // empty
-    QString productName;
-
-    // release ID such as 1809, may be mepty
+    // release ID such as 1809, may be empty
     QString ID;
 
     // some sub-build number, undocumented, may be empty

src/loglist.cpp

@@ -359,6 +359,8 @@ void initLogging()
   log::getDefault().setCallback(
     [](log::Entry e){ LogModel::instance().add(e); });
 
+  log::getDefault().addToBlacklist(getenv("USERNAME"), "USERNAME");
+
   qInstallMessageHandler(qtLogCallback);
 }
 

src/mainwindow.cpp

@@ -1447,15 +1447,20 @@ bool MainWindow::registerNexusPage(const QString& gameName)
   if (plugin == nullptr)
     return false;
 
+  // Get the gameURL
+  QString gameURL = NexusInterface::instance().getGameURL(gameName);
+  if (gameURL.isEmpty())
+    return false;
+
   // Create an action
   QAction* action = new QAction(
       plugin->gameIcon(),
-      QObject::tr("Visit %1 on Nexus").arg(plugin->gameName()),
+      QObject::tr("Visit %1 on Nexus").arg(gameName),
       this);
 
   // Bind the action
-  connect(action, &QAction::triggered, this, [this, gameName]() {
-    shell::Open(QUrl(NexusInterface::instance().getGameURL(gameName)));
+  connect(action, &QAction::triggered, this, [this, gameURL]() {
+    shell::Open(QUrl(gameURL));
   }, Qt::QueuedConnection);
 
   // Add the action
@@ -1479,7 +1484,7 @@ void MainWindow::updateModPageMenu()
     }
   );
 
-  // Remove disabled plugins:
+  // Remove disabled plugins
   modPagePlugins.erase(
     std::remove_if(std::begin(modPagePlugins), std::end(modPagePlugins), [&](auto* tool) {
       return !m_PluginContainer.isEnabled(tool);
@@ -1490,23 +1495,35 @@ void MainWindow::updateModPageMenu()
     registerModPage(modPagePlugin);
   }
 
-  // Add the primary game (with a separator)
-  registerNexusPage(m_OrganizerCore.managedGame()->gameShortName());
-  ui->actionModPage->menu()->addSeparator();
+  QStringList registeredSources;
+
+  // Add the primary game
+  QString gameShortName = m_OrganizerCore.managedGame()->gameShortName();
+  if (registerNexusPage(gameShortName))
+    registeredSources << gameShortName;
+
+  // Add the primary sources
+  for (auto gameName : m_OrganizerCore.managedGame()->primarySources())
+  {
+    if (!registeredSources.contains(gameName) && registerNexusPage(gameName))
+      registeredSources << gameName;
+  }
+
+  // Add a separator if needed
+  if (registeredSources.length() > 0)
+    ui->actionModPage->menu()->addSeparator();
 
   // Add the secondary games (sorted)
-  bool secondaryGameAdded = false;
   QStringList secondaryGames = m_OrganizerCore.managedGame()->validShortNames();
   secondaryGames.sort(Qt::CaseInsensitive);
   for (auto gameName : secondaryGames)
   {
-    if (registerNexusPage(gameName)) {
-      secondaryGameAdded = true;
-    }
+    if (!registeredSources.contains(gameName) && registerNexusPage(gameName))
+      registeredSources << gameName;
   }
 
-  // No mod page plugin and the menu was visible:
-  bool keepOriginalAction = modPagePlugins.size() == 0 && !secondaryGameAdded;
+  // No mod page plugin and the menu was visible
+  bool keepOriginalAction = modPagePlugins.size() == 0 && registeredSources.length() <= 1;
   if (keepOriginalAction) {
     ui->toolBar->insertAction(ui->actionAdd_Profile, ui->actionNexus);
   }
@@ -1646,7 +1663,7 @@ void MainWindow::on_profileBox_currentIndexChanged(int index)
   }
 }
 
-bool MainWindow::refreshProfiles(bool selectProfile)
+bool MainWindow::refreshProfiles(bool selectProfile, QString newProfile)
 {
   QComboBox* profileBox = findChild<QComboBox*>("profileBox");
 
@@ -1675,7 +1692,12 @@ bool MainWindow::refreshProfiles(bool selectProfile)
 
   if (selectProfile) {
     if (profileBox->count() > 1) {
-      profileBox->setCurrentText(currentProfileName);
+      if (newProfile.isEmpty()) {
+        profileBox->setCurrentText(currentProfileName);
+      }
+      else {
+        profileBox->setCurrentText(newProfile);
+      }
       if (profileBox->currentIndex() == 0) {
         profileBox->setCurrentIndex(1);
       }
@@ -2188,16 +2210,7 @@ void MainWindow::on_actionAdd_Profile_triggered()
     profilesDialog.exec();
     m_SavesTab->refreshSaveList(); // since the save list may now be outdated we have to refresh it completely
 
-    if (profilesDialog.selectedProfile())
-    {
-      // Change profile while blocking signals to prevent extra signals being sent
-      // Doesn't matter much as refreshProfiles() is being called after this
-      ui->profileBox->blockSignals(true);
-      ui->profileBox->setCurrentText(profilesDialog.selectedProfile().value());
-      ui->profileBox->blockSignals(false);
-    }
-
-    if (refreshProfiles() && !profilesDialog.failed()) {
+    if (refreshProfiles(true, profilesDialog.selectedProfile().value()) && !profilesDialog.failed()) {
       break;
     }
   }

src/mainwindow.h

@@ -198,7 +198,7 @@ private slots:
   QMenu* createPopupMenu() override;
   void activateSelectedProfile();
 
-  bool refreshProfiles(bool selectProfile = true);
+  bool refreshProfiles(bool selectProfile = true, QString newProfile = QString());
   void refreshExecutablesList();
 
   bool modifyExecutablesDialog(int selection);

src/nexusinterface.cpp

@@ -371,7 +371,13 @@ QString NexusInterface::getGameURL(QString gameName) const
 {
   IPluginGame *game = getGame(gameName);
   if (game != nullptr) {
-    return "https://www.nexusmods.com/" + game->gameNexusName().toLower();
+    QString gameNexusName = game->gameNexusName().toLower();
+    if (gameNexusName.isEmpty()) {
+      return "";
+    }
+    else {
+      return "https://www.nexusmods.com/" + gameNexusName;
+    }
   } else {
     log::error("getGameURL can't find plugin for {}", gameName);
     return "";

src/pluginlist.cpp

@@ -42,6 +42,7 @@ along with Mod Organizer.  If not, see <http://www.gnu.org/licenses/>.
 #include <QDateTime>
 #include <QDir>
 #include <QFile>
+#include <QFileInfo>
 #include <QTextCodec>
 #include <QFileInfo>
 #include <QListWidgetItem>
@@ -201,9 +202,9 @@ void PluginList::refresh(const QString &profileName
     }
     QString filename = ToQString(current->getName());
 
-    QString extension = filename.right(3).toLower();
-
-    if ((extension == "esp") || (extension == "esm") || (extension == "esl")) {
+    if (filename.endsWith(".esp", Qt::CaseInsensitive) ||
+        filename.endsWith(".esm", Qt::CaseInsensitive) ||
+        filename.endsWith(".esl", Qt::CaseInsensitive)) {
 
       availablePlugins.append(filename);
 

src/settings.cpp

@@ -271,6 +271,7 @@ QString Settings::executablesBlacklist() const
     << "Discord.exe"
     << "GalaxyClient.exe"
     << "Spotify.exe"
+    << "Brave.exe"
     ).join(";");
 
   return get<QString>(m_Settings, "Settings", "executable_blacklist", def);