From 4f84e9ad148e917a351de06c794ed5e39becf0d5 Mon Sep 17 00:00:00 2001
From: Teo Mrnjavac <teo@kde.org>
Date: Wed, 9 Sep 2015 18:44:43 +0200
Subject: [PATCH] InstallationViewStep is now ExecutionViewStep. It also uses
 JobQueue directly.

---
 ...tionViewStep.cpp => ExecutionViewStep.cpp} | 56 +++++++++++++------
 ...allationViewStep.h => ExecutionViewStep.h} | 18 ++++--
 2 files changed, 51 insertions(+), 23 deletions(-)
 rename src/libcalamaresui/{InstallationViewStep.cpp => ExecutionViewStep.cpp} (71%)
 rename src/libcalamaresui/{InstallationViewStep.h => ExecutionViewStep.h} (80%)

diff --git a/src/libcalamaresui/InstallationViewStep.cpp b/src/libcalamaresui/ExecutionViewStep.cpp
similarity index 71%
rename from src/libcalamaresui/InstallationViewStep.cpp
rename to src/libcalamaresui/ExecutionViewStep.cpp
index 9c9570bf0..fb43122b9 100644
--- a/src/libcalamaresui/InstallationViewStep.cpp
+++ b/src/libcalamaresui/ExecutionViewStep.cpp
@@ -17,14 +17,17 @@
  *   along with Calamares. If not, see <http://www.gnu.org/licenses/>.
  */
 
-#include <InstallationViewStep.h>
+#include <ExecutionViewStep.h>
 
-#include "JobQueue.h"
 #include "Branding.h"
+#include "JobQueue.h"
+#include "modulesystem/Module.h"
+#include "modulesystem/ModuleManager.h"
+#include "Settings.h"
 #include "utils/CalamaresUtilsGui.h"
 #include "utils/Logger.h"
-#include "Settings.h"
 #include "utils/Retranslator.h"
+#include "ViewManager.h"
 
 #include <QDir>
 #include <QLabel>
@@ -37,7 +40,7 @@
 namespace Calamares
 {
 
-InstallationViewStep::InstallationViewStep( QObject* parent )
+ExecutionViewStep::ExecutionViewStep( QObject* parent )
     : ViewStep( parent )
     , m_widget( new QWidget )
 {
@@ -62,65 +65,68 @@ InstallationViewStep::InstallationViewStep( QObject* parent )
     innerLayout->addWidget( m_label );
 
     cDebug() << "QML import paths:" << m_slideShow->engine()->importPathList();
+
+    connect( JobQueue::instance(), &JobQueue::progress,
+             this, &ExecutionViewStep::updateFromJobQueue );
 }
 
 
 QString
-InstallationViewStep::prettyName() const
+ExecutionViewStep::prettyName() const
 {
     return tr( "Install" );
 }
 
 
 QWidget*
-InstallationViewStep::widget()
+ExecutionViewStep::widget()
 {
     return m_widget;
 }
 
 
 void
-InstallationViewStep::next()
+ExecutionViewStep::next()
 {
 }
 
 
 void
-InstallationViewStep::back()
+ExecutionViewStep::back()
 {
 }
 
 
 bool
-InstallationViewStep::isNextEnabled() const
+ExecutionViewStep::isNextEnabled() const
 {
     return false;
 }
 
 
 bool
-InstallationViewStep::isBackEnabled() const
+ExecutionViewStep::isBackEnabled() const
 {
     return false;
 }
 
 
 bool
-InstallationViewStep::isAtBeginning() const
+ExecutionViewStep::isAtBeginning() const
 {
     return true;
 }
 
 
 bool
-InstallationViewStep::isAtEnd() const
+ExecutionViewStep::isAtEnd() const
 {
     return true;
 }
 
 
 void
-InstallationViewStep::onActivate()
+ExecutionViewStep::onActivate()
 {
     CALAMARES_RETRANSLATE_WIDGET( m_widget,
         if ( !Calamares::Branding::instance()->slideshowPath().isEmpty() )
@@ -128,20 +134,36 @@ InstallationViewStep::onActivate()
                                                          ->slideshowPath() ) );
     )
 
-    connect( JobQueue::instance(), &JobQueue::progress,
-             this, &InstallationViewStep::updateFromJobQueue );
+
+    JobQueue* queue = JobQueue::instance();
+    foreach ( const QString& instanceKey, m_jobInstanceKeys )
+    {
+        Calamares::Module* module = Calamares::ModuleManager::instance()
+                                    ->moduleInstance( instanceKey );
+        if ( module )
+            queue->enqueue( module->jobs() );
+    }
+
+    queue->start();
 }
 
 
 QList< Calamares::job_ptr >
-InstallationViewStep::jobs() const
+ExecutionViewStep::jobs() const
 {
     return QList< Calamares::job_ptr >();
 }
 
 
 void
-InstallationViewStep::updateFromJobQueue( qreal percent, const QString& message )
+ExecutionViewStep::appendJobModuleInstanceKey( const QString& instanceKey )
+{
+    m_jobInstanceKeys.append( instanceKey );
+}
+
+
+void
+ExecutionViewStep::updateFromJobQueue( qreal percent, const QString& message )
 {
     m_progressBar->setValue( percent * m_progressBar->maximum() );
     m_label->setText( message );
diff --git a/src/libcalamaresui/InstallationViewStep.h b/src/libcalamaresui/ExecutionViewStep.h
similarity index 80%
rename from src/libcalamaresui/InstallationViewStep.h
rename to src/libcalamaresui/ExecutionViewStep.h
index f4c8daf2c..3101cf479 100644
--- a/src/libcalamaresui/InstallationViewStep.h
+++ b/src/libcalamaresui/ExecutionViewStep.h
@@ -17,11 +17,13 @@
  *   along with Calamares. If not, see <http://www.gnu.org/licenses/>.
  */
 
-#ifndef INSTALLATIONVIEWSTEP_H
-#define INSTALLATIONVIEWSTEP_H
+#ifndef EXECUTIONVIEWSTEP_H
+#define EXECUTIONVIEWSTEP_H
 
 #include <viewpages/ViewStep.h>
 
+#include <QStringList>
+
 class QLabel;
 class QProgressBar;
 class QQuickWidget;
@@ -29,11 +31,11 @@ class QQuickWidget;
 namespace Calamares
 {
 
-class InstallationViewStep : public ViewStep
+class ExecutionViewStep : public ViewStep
 {
     Q_OBJECT
 public:
-    explicit InstallationViewStep( QObject* parent = nullptr );
+    explicit ExecutionViewStep( QObject* parent = nullptr );
 
     QString prettyName() const override;
 
@@ -50,7 +52,9 @@ public:
 
     void onActivate() override;
 
-    QList< Calamares::job_ptr > jobs() const override;
+    QList< job_ptr > jobs() const override;
+
+    void appendJobModuleInstanceKey( const QString& instanceKey );
 
 private:
     QWidget* m_widget;
@@ -58,9 +62,11 @@ private:
     QLabel* m_label;
     QQuickWidget* m_slideShow;
 
+    QStringList m_jobInstanceKeys;
+
     void updateFromJobQueue( qreal percent, const QString& message );
 };
 
 }
 
-#endif /* INSTALLATIONVIEWSTEP_H */
+#endif /* EXECUTIONVIEWSTEP_H */