From 63eaf453e0de2087ce7d2a3c26edc7eac4bd1882 Mon Sep 17 00:00:00 2001
From: Tannin <sherb@gmx.net>
Date: Tue, 9 Feb 2016 20:05:41 +0100
Subject: portable setup is no longer offered if the directory isn't writable

---
 src/instancemanager.cpp | 21 ++++++++++++---------
 1 file changed, 12 insertions(+), 9 deletions(-)

(limited to 'src/instancemanager.cpp')

diff --git a/src/instancemanager.cpp b/src/instancemanager.cpp
index 48c12d42..d48d02d2 100644
--- a/src/instancemanager.cpp
+++ b/src/instancemanager.cpp
@@ -172,18 +172,21 @@ QString InstanceManager::determineDataPath()
 
     QStringList instanceList = instances();
 
-    qDebug("%d - %s", instanceList.size(), qPrintable(instanceList.join(";")));
-
     if (instanceList.size() == 0) {
-      switch (queryInstallMode()) {
-        case InstallationMode::PORTABLE: {
-          instanceId = QString();
-        } break;
-        case InstallationMode::REGULAR: {
-          instanceId = queryInstanceName();
-        } break;
+      if (QFileInfo(qApp->applicationDirPath()).isWritable()) {
+        switch (queryInstallMode()) {
+          case InstallationMode::PORTABLE: {
+            instanceId = QString();
+          } break;
+          case InstallationMode::REGULAR: {
+            instanceId = queryInstanceName();
+          } break;
+        }
+      } else {
+        instanceId = queryInstanceName();
       }
     } else {
+      // don't offer portable instance if we can't set one up.
       instanceId = chooseInstance(instanceList);
     }
   }
-- 
cgit v1.3.1