refs #443 Add simulator enable/disable option

* Added CPluginSelector widget
* IContextSimulator starts/stops only specified listener(s)
* No more "auto" CSimulatorPluginInfo

src/blackgui/components/settingssimulatorcomponent.cpp

@@ -39,13 +39,14 @@ namespace BlackGui
             Q_ASSERT_X(this->getIContextSimulator(), Q_FUNC_INFO, "missing simulator");
 
             // set values
-            for (const auto &p : getAvailablePlugins(true))
+            for (const auto &p : getAvailablePlugins())
             {
-                ui->cb_Plugins->addItem(p.toQString(), QVariant::fromValue(p));
+                ui->ps_EnabledSimulators->addPlugin(p.getIdentifier(), p.getName(), false);
             }
 
             // connects
-            connect(this->ui->cb_Plugins, static_cast<void (QComboBox::*)(int)> (&QComboBox::currentIndexChanged), this, &CSettingsSimulatorComponent::ps_pluginHasBeenSelectedInComboBox);
+            connect(this->getIContextSimulator(), &IContextSimulator::simulatorPluginChanged, this, &CSettingsSimulatorComponent::ps_simulatorPluginChanged);
+            connect(this->ui->ps_EnabledSimulators, &CPluginSelector::pluginStateChanged, this, &CSettingsSimulatorComponent::ps_pluginStateChanged);
             connect(this->ui->pb_ApplyMaxAircraft, &QCheckBox::pressed, this, &CSettingsSimulatorComponent::ps_onApplyMaxRenderedAircraft);
             connect(this->ui->pb_ApplyTimeSync, &QCheckBox::pressed, this, &CSettingsSimulatorComponent::ps_onApplyTimeSync);
             connect(this->ui->pb_ApplyMaxDistance, &QCheckBox::pressed, this, &CSettingsSimulatorComponent::ps_onApplyMaxRenderedDistance);
@@ -58,29 +59,6 @@ namespace BlackGui
             this->ps_simulatorPluginChanged(getIContextSimulator()->getSimulatorPluginInfo());
         }
 
-        void CSettingsSimulatorComponent::setCurrentPluginInComboBox(const CSimulatorPluginInfo &plugin)
-        {
-            if (plugin.isUnspecified())
-            {
-                ui->cb_Plugins->setCurrentIndex(0);
-                return;
-            }
-
-            for (int i = 0; i < this->ui->cb_Plugins->count(); ++i)
-            {
-                QVariant data = this->ui->cb_Plugins->itemData(i);
-                Q_ASSERT(data.canConvert<CSimulatorPluginInfo>());
-                CSimulatorPluginInfo p = data.value<CSimulatorPluginInfo>();
-
-                if (p.getIdentifier() == plugin.getIdentifier())
-                {
-                    if (i == this->ui->cb_Plugins->currentIndex()) { return; }
-                    this->ui->cb_Plugins->setCurrentIndex(i);
-                    break;
-                }
-            }
-        }
-
         void CSettingsSimulatorComponent::setGuiValues()
         {
             Q_ASSERT(getIContextSimulator());
@@ -125,33 +103,37 @@ namespace BlackGui
             }
         }
 
-        CSimulatorPluginInfoList CSettingsSimulatorComponent::getAvailablePlugins(bool plusAuto) const
+        CSimulatorPluginInfoList CSettingsSimulatorComponent::getAvailablePlugins() const
         {
-            CSimulatorPluginInfoList l(getIContextSimulator()->getAvailableSimulatorPlugins());
-            if (plusAuto) { l.push_front(CSimulatorPluginInfo::autoPlugin()); }
-            return l;
+            return getIContextSimulator()->getAvailableSimulatorPlugins();
         }
 
-        void CSettingsSimulatorComponent::ps_pluginHasBeenSelectedInComboBox(int index)
+        void CSettingsSimulatorComponent::ps_pluginStateChanged(const QString &identifier, bool enabled)
         {
-            Q_ASSERT(this->getIContextSimulator());
-            if (!this->getIContextSimulator()) { return; }
+            Q_ASSERT(getIContextSimulator());
 
-            CSimulatorPluginInfoList simDrivers(getAvailablePlugins(true));
-            if (simDrivers.isEmpty())
+            CSimulatorPluginInfoList simDrivers(getAvailablePlugins());
+            auto selected = std::find_if(simDrivers.begin(), simDrivers.end(),
+                                         [&identifier](const CSimulatorPluginInfo &info)
+                {
+                    return info.getIdentifier() == identifier;
+                });
+
+            if (selected->isUnspecified())
             {
-                CLogMessage(this).error("No drivers available");
-                return;
-            }
-            if (simDrivers.size() <= index)
-            {
-                CLogMessage(this).error("Wrong driver index");
-                return;
+                CLogMessage(this).error("Simulator plugin does not exist: %1") << identifier;
             }
 
-            // update
-            CSimulatorPluginInfo selectedPlugin = simDrivers[index];
-            this->getIContextSimulator()->startSimulatorPlugin(selectedPlugin);
+            if (enabled)
+            {
+                getIContextSimulator()->startSimulatorPlugin(*selected);
+                CLogMessage(this).info("Started listening for %1") << selected->getSimulator();
+            }
+            else
+            {
+                getIContextSimulator()->stopSimulatorPlugin(*selected);
+                CLogMessage(this).info("Stopped listening for %1") << selected->getSimulator();
+            }
 
             // changing of GUI state will be done via received signal
         }

src/blackgui/components/settingssimulatorcomponent.h

@@ -39,8 +39,9 @@ namespace BlackGui
             virtual void runtimeHasBeenSet() override;
 
         private slots:
-            //! Driver changed
-            void ps_pluginHasBeenSelectedInComboBox(int index);
+            //! Driver plugin enabled/disabled
+            //! \todo Unload plugin if user disables it while running
+            void ps_pluginStateChanged(const QString &identifier, bool enabled);
 
             //! Apply max.aircraft
             void ps_onApplyMaxRenderedAircraft();
@@ -62,14 +63,11 @@ namespace BlackGui
 
         private:
 
-            //! Smarter way to set current driver, avoids unnecessary signals and less formatting dependend
-            void setCurrentPluginInComboBox(const BlackMisc::Simulation::CSimulatorPluginInfo &plugin);
-
             //! Set the GUI values
             void setGuiValues();
 
             //! Available plugins, auto pseudo plugin added
-            BlackMisc::Simulation::CSimulatorPluginInfoList getAvailablePlugins(bool plusAuto) const;
+            BlackMisc::Simulation::CSimulatorPluginInfoList getAvailablePlugins() const;
 
             QScopedPointer<Ui::CSettingsSimulatorComponent> ui; //!< UI
             bool m_pluginLoaded = false; //!< plugin loaded

src/blackgui/components/settingssimulatorcomponent.ui

@@ -53,26 +53,6 @@
       <property name="verticalSpacing">
        <number>6</number>
       </property>
-      <item row="0" column="0">
-       <widget class="QLabel" name="lbl_Plugin">
-        <property name="text">
-         <string>Driver</string>
-        </property>
-       </widget>
-      </item>
-      <item row="0" column="1">
-       <widget class="QComboBox" name="cb_Plugins">
-        <property name="maximumSize">
-         <size>
-          <width>16777215</width>
-          <height>16777215</height>
-         </size>
-        </property>
-        <property name="sizeAdjustPolicy">
-         <enum>QComboBox::AdjustToMinimumContentsLength</enum>
-        </property>
-       </widget>
-      </item>
       <item row="1" column="0">
        <widget class="QLabel" name="lbl_LoadedPlugin">
         <property name="text">
@@ -380,6 +360,9 @@
         </item>
        </layout>
       </item>
+      <item row="0" column="0" colspan="2">
+       <widget class="BlackGui::CPluginSelector" name="ps_EnabledSimulators" native="true"/>
+      </item>
      </layout>
     </widget>
    </item>
@@ -424,6 +407,12 @@
    <header>blackgui/led.h</header>
    <container>1</container>
   </customwidget>
+  <customwidget>
+   <class>BlackGui::CPluginSelector</class>
+   <extends>QWidget</extends>
+   <header>blackgui/pluginselector.h</header>
+   <container>1</container>
+  </customwidget>
  </customwidgets>
  <resources/>
  <connections/>

src/blackgui/pluginselector.cpp

@@ -0,0 +1,67 @@
+#include "pluginselector.h"
+
+#include <QCheckBox>
+#include <QHBoxLayout>
+#include <QVBoxLayout>
+#include <QPushButton>
+#include <QVariant>
+
+namespace BlackGui
+{
+
+    CPluginSelector::CPluginSelector(QWidget *parent) : QWidget(parent)
+    {
+        setObjectName("CPluginSelector");
+
+        QVBoxLayout *layout = new QVBoxLayout;
+        setLayout(layout);
+    }
+
+    void CPluginSelector::addPlugin(const QString& identifier, const QString &name, bool enabled)
+    {
+        QWidget *pw = new QWidget;
+        QHBoxLayout *layout = new QHBoxLayout;
+        layout->setContentsMargins(0, 0, 0, 0);
+        pw->setLayout(layout);
+
+        QCheckBox *cb = new QCheckBox(name);
+        cb->setProperty("pluginIdentifier", identifier);
+        connect(cb, &QCheckBox::stateChanged, this, &CPluginSelector::ps_handlePluginStateChange);
+        if (enabled)
+        {
+            cb->setCheckState(Qt::Checked);
+        }
+        else
+        {
+            cb->setCheckState(Qt::Unchecked);
+        }
+
+        pw->layout()->addWidget(cb);
+
+        /* Might be useful for #392 */
+#if 0
+        QPushButton *pb = new QPushButton("...");
+        pw->layout()->addWidget(pb);
+
+        layout->setStretch(0, 1);
+        layout->setStretch(1, 0);
+#endif
+
+        this->layout()->addWidget(pw);
+    }
+
+    void CPluginSelector::ps_handlePluginStateChange()
+    {
+        QCheckBox *cb = qobject_cast<QCheckBox*>(sender());
+        Q_ASSERT(cb);
+
+        bool enabled = cb->checkState() != Qt::Unchecked;
+        Q_ASSERT(cb->property("pluginIdentifier").isValid());
+        QString identifier = cb->property("pluginIdentifier").toString();
+        Q_ASSERT(!identifier.isEmpty());
+
+        emit pluginStateChanged(identifier, enabled);
+    }
+
+
+} // namespace

src/blackgui/pluginselector.h

@@ -0,0 +1,50 @@
+/* Copyright (C) 2014
+ * swift project Community / Contributors
+ *
+ * This file is part of swift Project. It is subject to the license terms in the LICENSE file found in the top-level
+ * directory of this distribution and at http://www.swift-project.org/license.html. No part of swift project,
+ * including this file, may be copied, modified, propagated, or distributed except according to the terms
+ * contained in the LICENSE file.
+ */
+
+//! \file
+
+#ifndef PLUGINSELECTOR_H
+#define PLUGINSELECTOR_H
+
+#include "blackgui/blackguiexport.h"
+
+#include <QWidget>
+
+namespace BlackGui {
+
+    /*!
+     * \brief The CPluginSelector class is used to select which plugins are to be loaded
+     * and (optionally) configure them.
+     */
+    class BLACKGUI_EXPORT CPluginSelector : public QWidget
+    {
+        Q_OBJECT
+
+    signals:
+        //! Emitted when user enables/disables the particular plugin
+        void pluginStateChanged(QString identifier, bool enabled);
+
+    public:
+        //! Constructor
+        explicit CPluginSelector(QWidget *parent = 0);
+
+        //! Adds the new plugin to the list.
+        //! \param identifier Identifier of the plugin.
+        //! \param name Name of the plugin
+        //! \param enabled Defines whether the plugin is initially enabled or not
+        void addPlugin(const QString &identifier, const QString &name, bool enabled = true);
+
+    private slots:
+        void ps_handlePluginStateChange();
+
+    };
+
+} // namespace BlackGui
+
+#endif // PLUGINSELECTOR_H