From 81036b0149eebc9033fd0cc089c1d9237c39cf14 Mon Sep 17 00:00:00 2001
From: Jeremy Rimpo <jeremy.rimpo@servermonkey.com>
Date: Wed, 9 Sep 2020 11:25:19 -0500
Subject: Fixes for bulk update check * Check game source for validity *
 Display name of source when no updates found

---
 src/mainwindow.cpp | 15 ++++++++-------
 src/modinfo.cpp    | 16 ++++++++++++++++
 2 files changed, 24 insertions(+), 7 deletions(-)

(limited to 'src')

diff --git a/src/mainwindow.cpp b/src/mainwindow.cpp
index dc8e6141..a4b87690 100644
--- a/src/mainwindow.cpp
+++ b/src/mainwindow.cpp
@@ -5524,10 +5524,10 @@ void MainWindow::nxmUpdateInfoAvailable(QString gameName, QVariant userData, QVa
   }
   QVariantList resultList = resultData.toList();
 
-  QFutureWatcher<std::set<QSharedPointer<ModInfo>>> *watcher = new QFutureWatcher<std::set<QSharedPointer<ModInfo>>>();
+  QFutureWatcher<std::pair<QString, std::set<QSharedPointer<ModInfo>>>> *watcher = new QFutureWatcher<std::pair<QString, std::set<QSharedPointer<ModInfo>>>>();
   QObject::connect(watcher, &QFutureWatcher<std::set<QSharedPointer<ModInfo>>>::finished, this, &MainWindow::finishUpdateInfo);
-  QFuture<std::set<QSharedPointer<ModInfo>>> future = QtConcurrent::run([=]() -> std::set<QSharedPointer<ModInfo>> {
-    return ModInfo::filteredMods(gameNameReal, resultList, userData.toBool(), true);
+  QFuture<std::pair<QString, std::set<QSharedPointer<ModInfo>>>> future = QtConcurrent::run([=]() -> std::pair<QString, std::set<QSharedPointer<ModInfo>>> {
+    return std::make_pair(gameNameReal, ModInfo::filteredMods(gameNameReal, resultList, userData.toBool(), true));
   });
   watcher->setFuture(future);
   if (m_ModListSortProxy != nullptr)
@@ -5536,12 +5536,13 @@ void MainWindow::nxmUpdateInfoAvailable(QString gameName, QVariant userData, QVa
 
 void MainWindow::finishUpdateInfo()
 {
-  QFutureWatcher<std::set<QSharedPointer<ModInfo>>> *watcher = static_cast<QFutureWatcher<std::set<QSharedPointer<ModInfo>>> *>(sender());
-
-  auto finalMods = watcher->result();
+  QFutureWatcher<std::pair<QString, std::set<QSharedPointer<ModInfo>>>> *watcher = static_cast<QFutureWatcher<std::pair<QString, std::set<QSharedPointer<ModInfo>>>> *>(sender());
+  
+  QString game = watcher->result().first;
+  auto finalMods = watcher->result().second;
 
   if (finalMods.empty()) {
-    log::info("None of your mods appear to have had recent file updates.");
+    log::info("None of your {} mods appear to have had recent file updates.", game);
   }
 
   std::set<std::pair<QString, int>> organizedGames;
diff --git a/src/modinfo.cpp b/src/modinfo.cpp
index 826aa5c9..19621345 100644
--- a/src/modinfo.cpp
+++ b/src/modinfo.cpp
@@ -312,6 +312,22 @@ bool ModInfo::checkAllForUpdate(PluginContainer *pluginContainer, QObject *recei
     }
   }
 
+  // Detect invalid source games
+  for (auto itr = games.begin(); itr != games.end(); ++itr) {
+    auto gamePlugins = pluginContainer->plugins<IPluginGame>();
+    IPluginGame* gamePlugin = qApp->property("managed_game").value<IPluginGame*>();
+    for (auto plugin : gamePlugins) {
+      if (plugin != nullptr && plugin->gameShortName().compare(*itr, Qt::CaseInsensitive) == 0) {
+        gamePlugin = plugin;
+        break;
+      }
+    }
+    if (gamePlugin != nullptr && gamePlugin->gameNexusName().isEmpty()) {
+      log::warn("The update check has found a mod with a Nexus ID and source game of {}, but this game is not a valid Nexus source.", gamePlugin->gameName());
+      itr = games.erase(itr);
+    }
+  }
+
   if (latest < QDateTime::currentDateTimeUtc().addMonths(-1)) {
     std::set<std::pair<QString, int>> organizedGames;
     for (auto mod : s_Collection) {
-- 
cgit v1.3.1