From fdd7f3f61bb6f35f638b4ca909bab85646baa516 Mon Sep 17 00:00:00 2001
From: bluebrother <bluebrother@a1c6a512-1295-4272-9138-f99709370657>
Date: Sun, 16 Sep 2007 18:32:49 +0000
Subject: [PATCH] Check target / mountpoint selection before performing any
 installation.

git-svn-id: svn://svn.rockbox.org/rockbox/trunk@14729 a1c6a512-1295-4272-9138-f99709370657
---
 rbutil/rbutilqt/rbutilqt.cpp | 38 ++++++++++++++++++++++++++++++++++++++
 rbutil/rbutilqt/rbutilqt.h   |  1 +
 2 files changed, 39 insertions(+)

diff --git a/rbutil/rbutilqt/rbutilqt.cpp b/rbutil/rbutilqt/rbutilqt.cpp
index 0f6a856bd..4cb52504e 100644
--- a/rbutil/rbutilqt/rbutilqt.cpp
+++ b/rbutil/rbutilqt/rbutilqt.cpp
@@ -161,6 +161,16 @@ void RbUtilQt::downloadDone(bool error)
         bleeding->setCache(userSettings->value("cachepath", QDir::tempPath()).toString());
     bleeding->setFile(&bleedingInfo);
     bleeding->getFile(QUrl(devices->value("bleeding_info").toString()));
+
+    if(chkConfig(false)) {
+        QApplication::processEvents();
+        QMessageBox::critical(this, tr("Configuration error"),
+            tr("Your configuration is invalid. This is most likely due "
+                "to a new installation of Rockbox Utility or a changed device "
+                "path. The configuation dialog will now open to allow you "
+                "correcting the problem."));
+        configDialog();
+    }
 }
 
 
@@ -311,6 +321,7 @@ void RbUtilQt::updateManual()
 
 void RbUtilQt::completeInstall()
 {
+    if(chkConfig(true)) return;
     if(QMessageBox::question(this, tr("Confirm Installation"),
            tr("Do you really want to make a complete Installation?"),
               QMessageBox::Yes | QMessageBox::No) != QMessageBox::Yes) return;
@@ -362,6 +373,7 @@ void RbUtilQt::completeInstall()
 
 void RbUtilQt::smallInstall()
 {
+    if(chkConfig(true)) return;
     if(QMessageBox::question(this, tr("Confirm Installation"),
            tr("Do you really want to make a small Installation?"),
               QMessageBox::Yes | QMessageBox::No) != QMessageBox::Yes) return;
@@ -424,6 +436,7 @@ void RbUtilQt::installdone(bool error)
 
 void RbUtilQt::installBtn()
 {
+    if(chkConfig(true)) return;
     install();
 }
 
@@ -498,6 +511,7 @@ bool RbUtilQt::installBootloaderAuto()
 
 void RbUtilQt::installBootloaderBtn()
 {
+    if(chkConfig(true)) return;
     if(QMessageBox::question(this, tr("Confirm Installation"),
            tr("Do you really want to install the Bootloader?"),
               QMessageBox::Yes | QMessageBox::No) != QMessageBox::Yes) return;
@@ -593,6 +607,7 @@ void RbUtilQt::installBootloader()
 
 void RbUtilQt::installFontsBtn()
 {
+    if(chkConfig(true)) return;
     if(QMessageBox::question(this, tr("Confirm Installation"),
            tr("Do you really want to install the fonts package?"),
               QMessageBox::Yes | QMessageBox::No) != QMessageBox::Yes) return;
@@ -627,6 +642,7 @@ void RbUtilQt::installFonts()
 
 void RbUtilQt::installVoice()
 {
+    if(chkConfig(true)) return;
     if(QMessageBox::question(this, tr("Confirm Installation"),
        tr("Do you really want to install the voice file?"),
        QMessageBox::Yes | QMessageBox::No) != QMessageBox::Yes) return;
@@ -658,6 +674,7 @@ void RbUtilQt::installVoice()
 
 void RbUtilQt::installDoomBtn()
 {
+    if(chkConfig(true)) return;
     if(!hasDoom()){
         QMessageBox::critical(this, tr("Error"), tr("Your device doesn't have a doom plugin. Aborting."));
         return;
@@ -704,6 +721,7 @@ void RbUtilQt::installDoom()
 
 void RbUtilQt::installThemes()
 {
+    if(chkConfig(true)) return;
     ThemesInstallWindow* tw = new ThemesInstallWindow(this);
     tw->setDeviceSettings(devices);
     tw->setUserSettings(userSettings);
@@ -715,6 +733,7 @@ void RbUtilQt::installThemes()
 
 void RbUtilQt::createTalkFiles(void)
 {
+    if(chkConfig(true)) return;
     InstallTalkWindow *installWindow = new InstallTalkWindow(this);
     installWindow->setUserSettings(userSettings);
     installWindow->setDeviceSettings(devices);
@@ -724,6 +743,7 @@ void RbUtilQt::createTalkFiles(void)
 
 void RbUtilQt::uninstall(void)
 {
+    if(chkConfig(true)) return;
     UninstallWindow *uninstallWindow = new UninstallWindow(this);
     uninstallWindow->setUserSettings(userSettings);
     uninstallWindow->setDeviceSettings(devices);
@@ -732,6 +752,7 @@ void RbUtilQt::uninstall(void)
 
 void RbUtilQt::uninstallBootloader(void)
 {
+    if(chkConfig(true)) return;
     if(QMessageBox::question(this, tr("Confirm Uninstallation"),
            tr("Do you really want to uninstall the Bootloader?"),
            QMessageBox::Yes | QMessageBox::No) != QMessageBox::Yes) return;
@@ -761,6 +782,7 @@ void RbUtilQt::uninstallBootloader(void)
 
 void RbUtilQt::downloadManual(void)
 {
+    if(chkConfig(true)) return;
     if(QMessageBox::question(this, tr("Confirm download"),
        tr("Do you really want to download the manual? The manual will be saved "
             "to the root folder of your player."),
@@ -921,3 +943,19 @@ QUrl RbUtilQt::proxy()
     return QUrl("");
 }
 
+
+bool RbUtilQt::chkConfig(bool warn)
+{
+    bool error = false;
+    if(userSettings->value("platform").toString().isEmpty()
+        || userSettings->value("mountpoint").toString().isEmpty()
+        || !QFileInfo(userSettings->value("mountpoint").toString()).isWritable()) {
+        error = true;
+
+        if(warn) QMessageBox::critical(this, tr("Configuration error"),
+            tr("You configuration is invalid. Please go to the configuration "
+                "dialog and make sure the selected values are correct."));
+    }
+    return error;
+}
+
diff --git a/rbutil/rbutilqt/rbutilqt.h b/rbutil/rbutilqt/rbutilqt.h
index cdabd6c57..f656caf45 100644
--- a/rbutil/rbutilqt/rbutilqt.h
+++ b/rbutil/rbutilqt/rbutilqt.h
@@ -56,6 +56,7 @@ class RbUtilQt : public QMainWindow
         BootloaderInstaller* blinstaller;
         QUrl proxy(void);
         QMap<QString, QString> versmap;
+        bool chkConfig(bool);
 
         volatile bool m_installed;
         volatile bool m_error;
-- 
2.11.4.GIT