Handle no selected themes properly.

When installing themes without any themes selected show a notice in the log and
continue instead of stopping.

Change-Id: Ieeb03e4656b041ce1dda25b2c44b6b6f0aa0ca80

2 files changed, 3 insertions, 2 deletions

diff --git a/rbutil/rbutilqt/gui/selectiveinstallwidget.cpp b/rbutil/rbutilqt/gui/selectiveinstallwidget.cpp
index 555c99c546..dbbe4afb4d 100644
--- a/rbutil/rbutilqt/gui/selectiveinstallwidget.cpp
+++ b/rbutil/rbutilqt/gui/selectiveinstallwidget.cpp
@@ -465,10 +465,10 @@ void SelectiveInstallWidget::installThemes(void)
         if(m_themesinstaller == NULL)
             m_themesinstaller = new ThemesInstallWindow(this);
 
+        connect(m_themesinstaller, SIGNAL(done(bool)), this, SLOT(continueInstall(bool)));
         m_themesinstaller->setLogger(m_logger);
         m_themesinstaller->setModal(true);
         m_themesinstaller->install();
-        connect(m_themesinstaller, SIGNAL(done(bool)), this, SLOT(continueInstall(bool)));
     }
     else {
         qDebug() << "[SelectiveInstallWidget] Themes install disabled.";
diff --git a/rbutil/rbutilqt/themesinstallwindow.cpp b/rbutil/rbutilqt/themesinstallwindow.cpp
index 9218a085d0..1b10ce9e6c 100644
--- a/rbutil/rbutilqt/themesinstallwindow.cpp
+++ b/rbutil/rbutilqt/themesinstallwindow.cpp
@@ -321,7 +321,8 @@ void ThemesInstallWindow::accept(void)
 void ThemesInstallWindow::install()
 {
     if(ui.listThemes->selectedItems().size() == 0) {
-        this->close();
+        logger->addItem(tr("No themes selected, skipping"), LOGINFO);
+        emit done(false);
         return;
     }
     QStringList themes;