Track the master plugin in a plugin group.

Author: Holt59

src/pluginmanager.cpp

@@ -248,6 +248,12 @@ bool PluginManager::isEnabled(MOBase::IPlugin* plugin) const
     return plugin == m_core->managedGame();
   }
 
+  // check the master of the group
+  const auto& d = details(plugin);
+  if (d.master() && d.master() != plugin) {
+    return isEnabled(d.master());
+  }
+
   return m_extensions.isEnabled(details(plugin).extension());
 }
 
@@ -485,8 +491,26 @@ bool PluginManager::loadPlugins(const MOBase::PluginExtension& extension)
   }
 
   for (auto& objectGroup : objects) {
+
+    // safety for min_element
+    if (objectGroup.isEmpty()) {
+      continue;
+    }
+
+    // find the best interface
+    auto it         = std::min_element(std::begin(objectGroup), std::end(objectGroup),
+                                       [&](auto const& lhs, auto const& rhs) {
+                                 return isBetterInterface(lhs, rhs);
+                               });
+    IPlugin* master = qobject_cast<IPlugin*>(*it);
+
+    // register plugins in the group
     for (auto* object : objectGroup) {
-      registerPlugin(extension, object, objectGroup);
+      IPlugin* plugin = registerPlugin(extension, object, objectGroup);
+
+      if (plugin) {
+        m_details.at(plugin).m_master = master;
+      }
     }
   }
 
@@ -693,14 +717,21 @@ std::vector<PluginManager::PluginLoaderPtr> PluginManager::makeLoaders()
 
   // load the python proxy
   {
-    auto pluginLoader = std::make_unique<QPluginLoader>(
-        QCoreApplication::applicationDirPath() + "/proxies/python/python_proxy.dll",
-        this);
+    const QString proxyPath =
+        QCoreApplication::applicationDirPath() + "/proxies/python";
+    auto pluginLoader =
+        std::make_unique<QPluginLoader>(proxyPath + "/python_proxy.dll", this);
 
     if (auto* object = pluginLoader->instance(); object) {
       auto loader = qobject_cast<MOBase::IPluginLoader*>(object);
-      loaders.push_back(
-          PluginLoaderPtr(loader, PluginLoaderDeleter{pluginLoader.release()}));
+      QString errorMessage;
+
+      if (loader->initialize(errorMessage)) {
+        loaders.push_back(
+            PluginLoaderPtr(loader, PluginLoaderDeleter{pluginLoader.release()}));
+      } else {
+        log::error("failed to initialize proxy from '{}': {}", proxyPath, errorMessage);
+      }
     }
   }
 

src/pluginmanager.h

@@ -47,6 +47,10 @@ class PluginDetails
   //
   const auto* proxy() const { return m_organizer; }
 
+  // the "master" of the group this plugin belongs to
+  //
+  MOBase::IPlugin* master() const { return m_master; }
+
   // the extension containing this plugin
   //
   const MOBase::PluginExtension& extension() const { return *m_extension; }
@@ -74,6 +78,7 @@ class PluginDetails
   PluginManager* m_manager;
   MOBase::IPlugin* m_plugin;
   const MOBase::PluginExtension* m_extension;
+  MOBase::IPlugin* m_master;
   std::vector<std::shared_ptr<const MOBase::IPluginRequirement>> m_requirements;
   OrganizerProxy* m_organizer;
 

src/settingsdialogplugins.cpp

@@ -37,18 +37,15 @@ PluginsSettingsTab::PluginsSettingsTab(Settings& s, PluginManager& pluginManager
     extensionItem->setData(0, Qt::DisplayRole, extension->metadata().name());
     ui->pluginsList->addTopLevelItem(extensionItem);
 
-    QSet<QString> handledNames;
-
-    // Handle child item:
     for (auto* plugin : plugins) {
-      if (handledNames.contains(plugin->name())) {
+
+      // only show master
+      if (pluginManager.details(plugin).master() != plugin) {
         continue;
       }
 
       QTreeWidgetItem* pluginItem = new QTreeWidgetItem(extensionItem);
       pluginItem->setData(0, Qt::DisplayRole, plugin->localizedName());
-
-      handledNames.insert(plugin->name());
     }
   }