minor stuff

Changed pynedit.pro to fetch from 'standard' install location
Changed saveas code to use organizer report rather than uibase - this way it's
not dependant on the vagaries of the include path, and works the same as the
rest of the organiser directory

src/SConscript

@@ -0,0 +1,20 @@
+Import('qt_env')
+
+env = qt_env.Clone()
+
+env.AppendUnique(CPPDEFINES = [
+    'DIAGNOSEBASIC_LIBRARY',
+    'NOMINMAX',
+    # suppress a few warnings caused by boost vs vc++ paranoia
+    '_SCL_SECURE_NO_WARNINGS'
+])
+
+env.AppendUnique(CPPPATH = [
+    '${BOOSTPATH}'
+])
+
+lib = env.SharedLibrary('diagnoseBasic', env.Glob('*.cpp'))
+env.InstallModule(lib)
+
+res = env['QT_USED_MODULES']
+Return('res')

src/diagnoseBasic.pro

@@ -10,9 +10,7 @@ TEMPLATE = lib
 CONFIG += plugins
 CONFIG += dll
 
-contains(QT_VERSION, "^5.*") {
-	QT += widgets
-}
+greaterThan(QT_MAJOR_VERSION, 4): QT += widgets
 
 DEFINES += DIAGNOSEBASIC_LIBRARY
 DEFINES += NOMINMAX
@@ -26,11 +24,12 @@ HEADERS += diagnosebasic.h
 
 include(../plugin_template.pri)
 
-INCLUDEPATH += "$(BOOSTPATH)"
+INCLUDEPATH += "$${BOOSTPATH}"
 
 #CONFIG += dll
 
 #DEFINES += INIEDITOR_LIBRARY
 
 OTHER_FILES += \
-    diagnosebasic.json
+    diagnosebasic.json\
+    SConscript

src/diagnosebasic.cpp

@@ -20,9 +20,13 @@
 #include "diagnosebasic.h"
 #include <report.h>
 #include <utility.h>
+#include <imodlist.h>
+#include <ipluginlist.h>
+#include <igameinfo.h>
 #include <QtPlugin>
 #include <QFile>
 #include <QDir>
+#include <QDebug>
 #include <QCoreApplication>
 #include <QMessageBox>
 #include <QDateTime>
@@ -44,6 +48,7 @@ using namespace MOBase;
 
 
 DiagnoseBasic::DiagnoseBasic()
+  : m_MOInfo(nullptr)
 {
 }
 
@@ -54,14 +59,25 @@ bool DiagnoseBasic::init(IOrganizer *moInfo)
   m_MOInfo->modList()->onModStateChanged([&] (const QString &modName, IModList::ModStates) {
                                            if (modName == "Overwrite") invalidate();
                                          });
-  m_MOInfo->pluginList()->onRefreshed([&] () { this->invalidate(); });
+  m_MOInfo->modList()->onModMoved([&] (const QString&, int, int) {
+                                         // invalidates only the assetOrder check but there is currently no way to recheck individual
+                                         // checks
+                                         invalidate();
+                                      });
+  m_MOInfo->pluginList()->onPluginMoved([&] (const QString&, int, int) {
+                                         invalidate();
+                                      });
+  m_MOInfo->pluginList()->onRefreshed([&] () { invalidate(); });
+  m_MOInfo->pluginList()->onPluginStateChanged([&] (const QString &, IPluginList::PluginStates) {
+    invalidate();
+  });
 
   return true;
 }
 
 QString DiagnoseBasic::name() const
 {
-  return tr("Basic diagnosis plugin");
+  return "Basic diagnosis plugin";
 }
 
 QString DiagnoseBasic::author() const
@@ -76,7 +92,7 @@ QString DiagnoseBasic::description() const
 
 VersionInfo DiagnoseBasic::version() const
 {
-  return VersionInfo(1, 1, 0, VersionInfo::RELEASE_FINAL);
+  return VersionInfo(1, 1, 2, VersionInfo::RELEASE_FINAL);
 }
 
 bool DiagnoseBasic::isActive() const
@@ -86,13 +102,19 @@ bool DiagnoseBasic::isActive() const
 
 QList<PluginSetting> DiagnoseBasic::settings() const
 {
-  return QList<PluginSetting>();
+  return QList<PluginSetting>()
+      << PluginSetting("check_errorlog", tr("Warn when an error occured last time an application was run"), true)
+      << PluginSetting("check_overwrite", tr("Warn when there are files in the overwrite directory"), true)
+      << PluginSetting("check_font", tr("Warn when the font configuration refers to files that aren't installed"), true)
+      << PluginSetting("check_conflict", tr("Warn when mods are installed that conflict with MO functionality"), true)
+      << PluginSetting("check_modorder", tr("Warn when MO determins the mod order may cause problems"), true)
+      << PluginSetting("check_missingmasters", tr("Warn when there are esps with missing masters"), true);
 }
 
 
 bool DiagnoseBasic::errorReported() const
 {
-  QDir dir(QCoreApplication::applicationDirPath() + "/logs");
+  QDir dir(qApp->property("dataPath").toString() + "/logs");
   QFileInfoList files = dir.entryInfoList(QStringList("ModOrganizer_??_??_??_??_??.log"),
                                           QDir::Files, QDir::Name | QDir::Reversed);
 
@@ -141,7 +163,7 @@ bool DiagnoseBasic::errorReported() const
 
 bool DiagnoseBasic::overwriteFiles() const
 {
-  QDir dir(QCoreApplication::applicationDirPath() + "/overwrite");
+  QDir dir(qApp->property("dataPath").toString() + "/overwrite");
 
   return dir.count() != 2; // account for . and ..
 }
@@ -202,16 +224,18 @@ void DiagnoseBasic::topoSort(std::vector<DiagnoseBasic::ListElement> &list) cons
     typedef graph_traits<Graph>::vertex_descriptor Vertex;
     typedef std::list<Vertex> Order;
 
-    // figure out unconnected components of the graph.
+    // figure out disconnected components of the graph
     std::vector<int> component(num_vertices(graph));
+    if (component.size() == 0) {
+      throw MyException(tr("failed to sort"));
+    }
     connected_components(graph, &component[0]);
-
     for (int i = 0; i != component.size(); ++i) {
       list[i].sortGroup = component[i];
     }
 
     Order order;
-    // do the actual sorting. This sorts the graph in full though the order between unconnected components doesn't
+    // do the actual sorting. This sorts the graph in full although the order between unconnected components doesn't
     // really matter to us
     boost::topological_sort(graph, std::front_inserter(order));
   }
@@ -224,22 +248,36 @@ void DiagnoseBasic::Sorter::sortGroup(std::vector<ListElement> modList)
   {
     auto maxSeqBegin = modList.end();
     auto maxSeqEnd = modList.end();
+    int maxSeqAvoidMove = 0;
 
     // first, determine the longest sequence of correctly sorted mods
     auto curSeqBegin = modList.begin();
     auto curSeqEnd = modList.begin();
 
+    int curSeqAvoidMove = 0;
+
     auto iter = modList.begin() + 1;
     for (; iter != modList.end(); ++iter) {
-      if (iter->modPriority < curSeqEnd->modPriority) {
+      if ((iter->pluginPriority != curSeqEnd->pluginPriority)
+          && (iter->modPriority < curSeqEnd->modPriority)) {
         // sequence ends
-        if ((maxSeqBegin == modList.end()) || ((curSeqEnd - curSeqBegin) > (maxSeqEnd - maxSeqBegin))) {
+        // use this sequence of correctly sorted mods if it is longer than the previously longest
+        // sequence and doesn't have fewer mods that don't want to move. Thus the need for mods to
+        // stay in place beats our gole to have the minimal number of moves
+        if ((maxSeqBegin == modList.end())
+            || (((curSeqEnd - curSeqBegin) > (maxSeqEnd - maxSeqBegin))
+                && (curSeqAvoidMove >= maxSeqAvoidMove))) {
           maxSeqBegin = curSeqBegin;
           maxSeqEnd = iter;
+          maxSeqAvoidMove = curSeqAvoidMove;
         }
         curSeqBegin = curSeqEnd = iter;
+        curSeqAvoidMove = 0;
       } else {
         curSeqEnd = iter;
+        if (iter->avoidMove) {
+          ++curSeqAvoidMove;
+        }
       }
     }
 
@@ -263,7 +301,8 @@ void DiagnoseBasic::Sorter::sortGroup(std::vector<ListElement> modList)
         break;
       }
     }
-    if (!found) {
+    if (!found
+        && (iter->pluginPriority != (*sorted.rbegin()).pluginPriority)) {
       // add to end!
       moves.push_back(Move(*iter, *sorted.rbegin(), Move::AFTER));
     }
@@ -302,60 +341,112 @@ bool DiagnoseBasic::assetOrder() const
 
   // list of mods containing conflicted scripts. We care only for those
   std::map<QString, QSet<QString>> scriptMods;
-  foreach (const IOrganizer::FileInfo & pex, m_MOInfo->findFileInfos("scripts",
-        [] (const IOrganizer::FileInfo &file) -> bool { return file.filePath.endsWith(".pex", Qt::CaseInsensitive); })) {
+  auto filter = [] (const IOrganizer::FileInfo &file) -> bool {
+    return file.filePath.endsWith(".pex", Qt::CaseInsensitive); };
+
+  for (const IOrganizer::FileInfo &pex : m_MOInfo->findFileInfos("scripts", filter)) {
     QStringList origins = pex.origins;
-    origins.removeAll("data"); // ignore files in base directory
+    // ignore files in base directories
+    origins.removeAll("data");
+    origins.removeAll("Overwrite");
     if (origins.size() > 1) {
-      foreach(const QString &origin, origins) {
+      for(const QString &origin : origins) {
         scriptMods[origin].insert(pex.filePath);
       }
     }
   }
 
   // produce a list with the information we need: plugin, mod and the priority for each
-  QStringList esps = m_MOInfo->findFiles("", [] (const QString &fileName) -> bool { return fileName.endsWith(".esp", Qt::CaseInsensitive); });
-  foreach (const QString &esp, esps) {
-    ListElement ele;
+  QStringList esps = m_MOInfo->findFiles("",
+                                         [] (const QString &fileName) -> bool {
+    return fileName.endsWith(".esp", Qt::CaseInsensitive)
+        || fileName.endsWith(".esm", Qt::CaseInsensitive);
+  });
+  for (const QString &esp : esps) {
+    for (const QString origin : m_MOInfo->getFileOrigins(esp)) {
+      ListElement ele;
 
-    ele.espName = QFileInfo(esp).fileName();
-    ele.modName = m_MOInfo->pluginList()->origin(ele.espName);
-    ele.pluginPriority = m_MOInfo->pluginList()->priority(ele.espName);
-    ele.modPriority = m_MOInfo->modList()->priority(ele.modName);
-    IModList::ModStates state = m_MOInfo->modList()->state(ele.modName);
-    auto iter = scriptMods.find(ele.modName);
-    if (state.testFlag(IModList::STATE_EXISTS) && !state.testFlag(IModList::STATE_ESSENTIAL) &&
-        (iter != scriptMods.end())) {
-      ele.relevantScripts = iter->second;
-      modList.push_back(ele);
+      ele.espName = QFileInfo(esp).fileName();
+      ele.modName = origin;
+      ele.pluginPriority = m_MOInfo->pluginList()->priority(ele.espName);
+      ele.modPriority = m_MOInfo->modList()->priority(ele.modName);
+      IModList::ModStates state = m_MOInfo->modList()->state(ele.modName);
+      ele.avoidMove = state.testFlag(IModList::STATE_ESSENTIAL);
+      auto iter = scriptMods.find(ele.modName);
+      if (state.testFlag(IModList::STATE_EXISTS)
+          && (iter != scriptMods.end())) {
+        ele.relevantScripts = iter->second;
+        modList.push_back(ele);
+      }
     }
   }
-
   // generate a copy of list that contains each mod only once, otherwise
   // strange things happen if a mod contains multiple esps that are mixed
   // with esps from other mods
   std::vector<ListElement> distinctModList;
   {
+    // sort the input list so we get predictable results
+    std::sort(modList.begin(), modList.end(),
+              [] (const ListElement &lhs, const ListElement &rhs) -> bool {
+      return lhs.pluginPriority < rhs.pluginPriority;
+    });
+
     std::set<QString> includedMods;
-    foreach(const ListElement &ele, modList) {
+    for (const ListElement &ele : modList) {
       if (includedMods.find(ele.modName) == includedMods.end()) {
         distinctModList.push_back(ele);
         includedMods.insert(ele.modName);
       }
     }
   }
+  if (distinctModList.size() == 0) {
+    return false;
+  }
 
-  // sort the list by plugin priority
+  // sort the list by plugin priority. This step is probably unnecessary as the list was already sorted when we removed duplicates
   std::sort(distinctModList.begin(), distinctModList.end(),
             [] (const ListElement &lhs, const ListElement &rhs) -> bool { return lhs.pluginPriority < rhs.pluginPriority; });
 
-  topoSort(distinctModList);
+  if (distinctModList.size() > 0) {
+    topoSort(distinctModList);
 
-  // now determine the moves necessary to bring the mod list into this order
-  minSorter(distinctModList);
-  m_SuggestedMoves = minSorter.moves;
+    // now determine the moves necessary to bring the mod list into this order
+    minSorter(distinctModList);
+    m_SuggestedMoves = minSorter.moves;
+    return m_SuggestedMoves.size() > 0;
+  } else {
+    return false;
+  }
+}
 
-  return m_SuggestedMoves.size() > 0;
+bool DiagnoseBasic::missingMasters() const
+{
+  std::set<QString> enabledPlugins;
+
+  QStringList esps = m_MOInfo->findFiles("",
+      [] (const QString &fileName) -> bool { return fileName.endsWith(".esp", Qt::CaseInsensitive)
+                                                  || fileName.endsWith(".esm", Qt::CaseInsensitive); });
+  // gather enabled masters first
+  foreach (const QString &esp, esps) {
+    QString baseName = QFileInfo(esp).fileName();
+    if (m_MOInfo->pluginList()->state(baseName) == IPluginList::STATE_ACTIVE) {
+      enabledPlugins.insert(baseName.toLower());
+    }
+  }
+
+  m_MissingMasters.clear();
+  // for each required master in each esp, test if it's in the list of enabled masters.
+  foreach (const QString &esp, esps) {
+    QString baseName = QFileInfo(esp).fileName();
+    if (m_MOInfo->pluginList()->state(baseName) == IPluginList::STATE_ACTIVE) {
+      foreach (const QString master, m_MOInfo->pluginList()->masters(baseName)) {
+        if (enabledPlugins.find(master.toLower()) == enabledPlugins.end()) {
+          m_MissingMasters.insert(master);
+        }
+      }
+    }
+  }
+  return !m_MissingMasters.empty();
 }
 
 bool DiagnoseBasic::invalidFontConfig() const
@@ -402,32 +493,28 @@ bool DiagnoseBasic::invalidFontConfig() const
   return false;
 }
 
-
 std::vector<unsigned int> DiagnoseBasic::activeProblems() const
 {
   std::vector<unsigned int> result;
 
-  if (errorReported()) {
+  if (m_MOInfo->pluginSetting(name(), "check_errorlog").toBool() && errorReported()) {
     result.push_back(PROBLEM_ERRORLOG);
   }
-  if (overwriteFiles()) {
+  if (m_MOInfo->pluginSetting(name(), "check_overwrite").toBool() && overwriteFiles()) {
     result.push_back(PROBLEM_OVERWRITE);
   }
-  if (invalidFontConfig()) {
+  if (m_MOInfo->pluginSetting(name(), "check_font").toBool() && invalidFontConfig()) {
     result.push_back(PROBLEM_INVALIDFONT);
   }
-  if (nitpickInstalled()) {
+  if (m_MOInfo->pluginSetting(name(), "check_conflict").toBool() && nitpickInstalled()) {
     result.push_back(PROBLEM_NITPICKINSTALLED);
   }
-  if (assetOrder()) {
+  if (m_MOInfo->pluginSetting(name(), "check_modorder").toBool() && assetOrder()) {
     result.push_back(PROBLEM_ASSETORDER);
   }
-  QStringList backups = QDir(m_MOInfo->profilePath()).entryList(QStringList() << "modlist.txt_backup_*");
-  if (backups.size() > 0) {
-    m_NewestModlistBackup = backups.last();
-    result.push_back(PROBLEM_MODLISTBACKUP);
+  if (m_MOInfo->pluginSetting(name(), "check_missingmasters").toBool() && missingMasters()) {
+    result.push_back(PROBLEM_MISSINGMASTERS);
   }
-
   if (QFile::exists(m_MOInfo->profilePath() + "/profile_tweaks.ini")) {
     result.push_back(PROBLEM_PROFILETWEAKS);
   }
@@ -448,10 +535,10 @@ QString DiagnoseBasic::shortDescription(unsigned int key) const
       return tr("Nitpick installed");
     case PROBLEM_ASSETORDER:
       return tr("Potential Mod order problem");
-    case PROBLEM_MODLISTBACKUP:
-      return tr("Modlist backup exists");
     case PROBLEM_PROFILETWEAKS:
       return tr("Ini Tweaks overwritten");
+    case PROBLEM_MISSINGMASTERS:
+      return tr("Missing Masters");
     default:
       throw MyException(tr("invalid problem key %1").arg(key));
   }
@@ -464,7 +551,7 @@ QString DiagnoseBasic::fullDescription(unsigned int key) const
       return "<code>" + m_ErrorMessage.replace("\n", "<br>") + "</code>";
     case PROBLEM_OVERWRITE:
       return tr("Files in the <font color=\"red\"><i>Overwrite</i></font> mod are are usually files created by an external tool (i.e. Wrye Bash, Automatic Variants, ...).<br>"
-                "It is advisable you empty Overwrite directory by moving those files to an existing mod. You can do this by double-clicking the <font color=\"red\"><i>Overwrite</i></font> mod and use drag&drop to move the files to a mod.<br>"
+                "It is advisable you empty the Overwrite directory by moving those files to an existing mod. You can do this by double-clicking the <font color=\"red\"><i>Overwrite</i></font> mod and use drag&drop to move the files to a mod.<br>"
                 "Alternatively, right-click on <font color=\"red\"><i>Overwrite</i></font> and create a new regular mod from the files there.<br>"
                 "<br>"
                 "Why is this necessary? Generated files may depend on the other mods active in a profile and may thus be incompatible with a different profile (i.e. bashed patches from Wrye Bash). "
@@ -478,28 +565,22 @@ QString DiagnoseBasic::fullDescription(unsigned int key) const
                 "Worse: The two solutions may conflict so it's strongly suggested you remove this plugin.");
     case PROBLEM_ASSETORDER: {
       QString res = tr("The conflict resolution order for some mods containing scripts differs from that of the corresponding esp.<br>"
-                       "This may lead to subtle, hard to locate bugs. You should re-order the affected mods (left list!).<br>"
-                       "There is no way to reliably know if these changes are necessary but its definitively safer.<br>"
+                       "This may lead to subtle, hard to locate bugs. <b>You should re-order the affected mods (<font color=\"red\">left list!</font>).</b><br>"
+                       "There is no way to reliably know if each of these changes is absolutely necessary but its definitively safer.<br>"
                        "If someone suggested you ignore this message, please give them a proper slapping from me. <b>Do not ignore this warning</b><br>"
-                       "The following changes should fix the issue:") + "<ul>";
+                       "The following changes should prevent these kinds of errors:") + "<ul>";
       foreach(const Move &op, m_SuggestedMoves) {
+        QString itemName = m_MOInfo->modList()->displayName(op.item.modName);
+        QString referenceName = m_MOInfo->modList()->displayName(op.reference.modName);
         if (op.type == Move::BEFORE) {
-          res += "<li>" + tr("Move %1 before %2").arg(op.item.modName).arg(op.reference.modName) + "</li>";
+          res += "<li>" + tr("Move %1 before %2").arg(itemName).arg(referenceName) + "</li>";
         } else {
-          res += "<li>" + tr("Move %1 after %2").arg(op.item.modName).arg(op.reference.modName) + "</li>";
+          res += "<li>" + tr("Move %1 after %2").arg(itemName).arg(referenceName) + "</li>";
         }
       }
       res += "</ul>";
       return res;
     } break;
-    case PROBLEM_MODLISTBACKUP: {
-      uint timestamp = m_NewestModlistBackup.right(10).toULong();
-      QDateTime time;
-      time.setTime_t(timestamp);
-      return tr("A previous operation created a backup of your mod list on %1.<br>"
-                "This backup contains both the info which mods are enabled and the ordering.<br>"
-                "You can restore that backup here.").arg(time.toString());
-    } break;
     case PROBLEM_PROFILETWEAKS: {
       QString fileContent = readFileText(m_MOInfo->profilePath() + "/profile_tweaks.ini");
       return tr("Settings provided in ini tweaks have been overwritten in-game or in an applications.<br>"
@@ -510,6 +591,11 @@ QString DiagnoseBasic::fullDescription(unsigned int key) const
                 "Hitting the <i>Fix</i> button will delete that file")
              + "<hr><i>profile_tweaks.ini:</i><pre>" + fileContent + "</pre>";
     } break;
+    case PROBLEM_MISSINGMASTERS: {
+      return tr("The masters for some plugins (esp/esm) are not enabled.<br>"
+                "The game will crash unless you install and enable the following plugins: ")
+             + "<ul><li>" + SetJoin(m_MissingMasters, "</li><li>") + "</li></ul>";
+    } break;
     default:
       throw MyException(tr("invalid problem key %1").arg(key));
   }
@@ -517,51 +603,35 @@ QString DiagnoseBasic::fullDescription(unsigned int key) const
 
 bool DiagnoseBasic::hasGuidedFix(unsigned int key) const
 {
-  return /*(key == PROBLEM_ASSETORDER) || */ (key == PROBLEM_MODLISTBACKUP) || (key == PROBLEM_PROFILETWEAKS);
+  return (key == PROBLEM_ASSETORDER) || (key == PROBLEM_PROFILETWEAKS);
 }
 
 void DiagnoseBasic::startGuidedFix(unsigned int key) const
 {
   switch (key) {
-/*    case PROBLEM_ASSETORDER: {
- *    if (QMessageBox::warning(NULL, tr("Continue?"), tr("This <b>BETA</b> feature will rearrange your mods to eliminate all "
- *            "possible ordering conflicts. A backup of your mod list will be created. Proceed?"), QMessageBox::Yes | QMessageBox::No) == QMessageBox::Yes) {
- *      shellCopy(QStringList(m_MOInfo->profilePath() + "/modlist.txt"),
- *                QStringList(m_MOInfo->profilePath() + "/modlist.txt_backup_" + QString("%1").arg(QDateTime::currentDateTime().toTime_t())));
- *      foreach (const Move &op, m_SuggestedMoves) {
- *        int oldPriority = m_MOInfo->modList()->priority(op.item.modName);
- *        int targetPriority = -1;
- *        if (op.type == Move::BEFORE) {
- *          targetPriority = m_MOInfo->modList()->priority(op.reference.modName);
- *        } else {
- *          targetPriority = m_MOInfo->modList()->priority(op.reference.modName) + 1;
- *        }
- *        if (oldPriority < targetPriority) {
- *          --targetPriority;
- *        }
- *        m_MOInfo->modList()->setPriority(op.item.modName, targetPriority);
- *      }
- *    }
- *  } break;*/
-    case PROBLEM_MODLISTBACKUP: {
-      QMessageBox question(QMessageBox::Question, tr("Restore backup?"),
-                           tr("Do you want to restore this backup or delete it?"),
-                           QMessageBox::Yes | QMessageBox::No | QMessageBox::Cancel);
-      question.setButtonText(QMessageBox::Yes, tr("Restore"));
-      question.setButtonText(QMessageBox::No, tr("Delete"));
-
-      question.exec();
-      if (question.result() == QMessageBox::Yes) {
-        shellMove(QStringList(m_MOInfo->profilePath() + "/" + m_NewestModlistBackup), QStringList(m_MOInfo->profilePath() + "/modlist.txt"));
-        m_MOInfo->refreshModList(false);
-      } else if (question.result() == QMessageBox::No) {
-        shellDelete(QStringList(m_MOInfo->profilePath() + "/" + m_NewestModlistBackup));
+    case PROBLEM_ASSETORDER: {
+      if (QMessageBox::warning(nullptr, tr("Continue?"), tr("This <b>BETA</b> feature will rearrange your mods to eliminate all "
+              "possible ordering conflicts. Proceed?"), QMessageBox::Yes | QMessageBox::No) == QMessageBox::Yes) {
+        foreach (const Move &op, m_SuggestedMoves) {
+          int oldPriority = m_MOInfo->modList()->priority(op.item.modName);
+          int targetPriority = -1;
+          if (op.type == Move::BEFORE) {
+            targetPriority = m_MOInfo->modList()->priority(op.reference.modName);
+          } else {
+            targetPriority = m_MOInfo->modList()->priority(op.reference.modName) + 1;
+          }
+          if (oldPriority < targetPriority) {
+            --targetPriority;
+          }
+          m_MOInfo->modList()->setPriority(op.item.modName, targetPriority);
+        }
       }
     } break;
     case PROBLEM_PROFILETWEAKS: {
       shellDeleteQuiet(m_MOInfo->profilePath() + "/profile_tweaks.ini");
     } break;
-    default: throw MyException(tr("invalid problem key %1").arg(key));
+    default:
+      throw MyException(tr("invalid problem key %1").arg(key));
   }
 }
 

src/diagnosebasic.h

@@ -29,7 +29,7 @@ along with this plugin.  If not, see <http://www.gnu.org/licenses/>.
 #include <QSet>
 
 
-class DiagnoseBasic : public QObject, MOBase::IPlugin, MOBase::IPluginDiagnose
+class DiagnoseBasic : public QObject, public MOBase::IPlugin, public MOBase::IPluginDiagnose
 {
   Q_OBJECT
   Q_INTERFACES(MOBase::IPlugin MOBase::IPluginDiagnose)
@@ -66,6 +66,7 @@ private:
   bool invalidFontConfig() const;
   bool nitpickInstalled() const;
   bool assetOrder() const;
+  bool missingMasters() const;
 
 private:
 
@@ -74,8 +75,8 @@ private:
   static const unsigned int PROBLEM_INVALIDFONT = 3;
   static const unsigned int PROBLEM_NITPICKINSTALLED = 4;
   static const unsigned int PROBLEM_ASSETORDER = 5;
-  static const unsigned int PROBLEM_MODLISTBACKUP = 6;
   static const unsigned int PROBLEM_PROFILETWEAKS = 7;
+  static const unsigned int PROBLEM_MISSINGMASTERS = 8;
 
   static const unsigned int NUM_CONTEXT_ROWS = 5;
 
@@ -86,8 +87,8 @@ private:
     QString modName;
     int pluginPriority;
     int modPriority;
-    int sortPriority;
     int sortGroup;
+    bool avoidMove;
     QSet<QString> relevantScripts;
   };
 
@@ -130,6 +131,7 @@ private:
   mutable QString m_ErrorMessage;
   mutable std::vector <Move> m_SuggestedMoves;
   mutable QString m_NewestModlistBackup;
+  mutable std::set<QString> m_MissingMasters;
 
 };