Asynchronous updating for remote info in SettingsDlg

[irwi] / src / settingsdlg.cpp

diff --git a/src/settingsdlg.cpp b/src/settingsdlg.cpp
index 14c0a6c..a5a0d16 100644 (file)
--- a/src/settingsdlg.cpp
+++ b/src/settingsdlg.cpp
@@ -10,14 +10,15 @@
 #include <QPushButton>
 #include <QLabel>
 #include <QDebug>
+#include <QNetworkConfiguration>
 
 SettingsDlg::SettingsDlg(QWidget *parent)
     : QDialog(parent)
 {
     QSettings settings(this);
     m_layout = new QVBoxLayout(this);
-    m_btnLayout = new QHBoxLayout(this);
-    m_remoteNameLayout = new QHBoxLayout(this);
+    m_btnLayout = new QHBoxLayout();
+    m_remoteNameLayout = new QHBoxLayout();
     
     m_advSettingsBtn = new QPushButton(tr("Advanced"), this);
     m_selectRemoteBtn = new QPushButton(tr("Select remote"), this);
@@ -61,7 +62,21 @@ SettingsDlg::SettingsDlg(QWidget *parent)
     m_layout->addLayout(m_btnLayout);
     this->setLayout(m_layout);
 
-    initRemote();
+    QString selectedRemote = settings.value("remoteName", "").toString();
+    if (selectedRemote == "") {
+        m_remoteNameLabel->setText("No remote selected");
+        enableRateBtns(false);
+    } else {
+        // Create remote by name and update it's info if online
+        m_remote = Remote(selectedRemote);
+        m_remoteNameLabel->setText(selectedRemote);
+        m_netConfMan = new QTM_PREPEND_NAMESPACE(
+                QNetworkConfigurationManager)(this);
+        connect(m_netConfMan, SIGNAL(updateCompleted()),
+                this, SLOT(onNetworkStatusUpdate()));
+        m_netConfMan->updateConfigurations();
+        enableRateBtns(false);
+    }
 }
 
 SettingsDlg::~SettingsDlg()
@@ -76,17 +91,24 @@ SettingsDlg::~SettingsDlg()
     delete m_btnLayout;
     delete m_remoteNameLayout;
     delete m_layout;
+    delete m_netConfMan;
+}
+
+void SettingsDlg::setBusy(bool busy)
+{
+    setAttribute(Qt::WA_Maemo5ShowProgressIndicator, busy);
+    setEnabled(!busy);
 }
 
 void SettingsDlg::showAdvSettingsDlg()
 {
-    AdvSettingsDlg dlg(this);
+    AdvSettingsDlg dlg;
     dlg.exec();
 }
 
 void SettingsDlg::showSelectRemoteDlg()
 {
-    SelectRemoteDlg dlg(this);
+    SelectRemoteDlg dlg;
     connect(&dlg, SIGNAL(remoteChanged(Remote)),
             this, SLOT(setRemote(Remote)));
     dlg.exec();
@@ -94,18 +116,21 @@ void SettingsDlg::showSelectRemoteDlg()
 
 void SettingsDlg::showAboutDlg()
 {
-    AboutDlg dlg(this);
+    AboutDlg dlg;
     dlg.exec();
 }
 
-void SettingsDlg::initRemote()
+void SettingsDlg::setRemote(Remote r)
 {
-    QString selectedRemote = QSettings(this).value("remoteName", "").toString();
-    if (selectedRemote == "") {
-        m_remoteNameLabel->setText("No remote selected");
-        enableRateBtns(false);
-    } else {
-        m_remote = Remote(selectedRemote);
+    m_remote = r;
+    updateRemoteInfo();
+    enableRateBtns();
+}
+
+void SettingsDlg::onNetworkStatusUpdate()
+{
+    if (m_netConfMan->isOnline()) {
+        setBusy();
         connect(&m_remote, SIGNAL(infoUpdated()),
                 this, SLOT(updateRemoteInfo()));
         m_remote.updateInfo();
@@ -113,15 +138,9 @@ void SettingsDlg::initRemote()
     }
 }
 
-void SettingsDlg::setRemote(Remote r)
-{
-    m_remote = r;
-    enableRateBtns();
-    updateRemoteInfo();
-}
-
 void SettingsDlg::updateRemoteInfo()
 {
+    setBusy(false);
     m_remoteNameLabel->setText(m_remote.mfg() + " " + m_remote.name());
     m_ratingLabel->setText(tr("Rating") + ": "
             + QString::number(m_remote.rating()));
@@ -130,13 +149,19 @@ void SettingsDlg::updateRemoteInfo()
 void SettingsDlg::rateUpClicked()
 {
     m_remote.sendRating(Rating::Up);
-    enableRateBtns(false);
+    processRatingSent();
 }
 
 void SettingsDlg::rateDownClicked()
 {
     m_remote.sendRating(Rating::Down);
+    processRatingSent();
+}
+
+void SettingsDlg::processRatingSent()
+{
     enableRateBtns(false);
+    m_remote.updateInfo();
 }
 
 void SettingsDlg::enableRateBtns(bool enable)