Fix #372 - writeSettings / writeLibraryPaths warn user if settings ca…

…n't be written to
openstudiocoalition · Jun 1, 2021 · e6b70fd · e6b70fd
1 parent 1c7338d
commit e6b70fd
Showing 1 changed file with 10 additions and 0 deletions.
diff --git a/src/openstudio_app/OpenStudioApp.cpp b/src/openstudio_app/OpenStudioApp.cpp
@@ -132,6 +132,7 @@
 #include <QTcpServer>
 #include <QtConcurrent>
 #include <QtGlobal>
+#include <QSettings>
 
 #include <openstudio/OpenStudio.hxx>
 #include <openstudio/utilities/idd/IddEnums.hxx>
@@ -1173,6 +1174,10 @@ void OpenStudioApp::writeSettings() {
   QString organizationName = QCoreApplication::organizationName();
   QString applicationName = QCoreApplication::applicationName();
   QSettings settings(organizationName, applicationName);
+  if (!settings.isWritable()) {
+    QMessageBox::warning(nullptr, tr("Settings file not writable"),
+                         tr("Your settings file '") + settings.fileName() + tr("' is not writable. Adjust the file permissions"));
+  }
   settings.setValue("lastPath", lastPath());
 }
 
@@ -1302,6 +1307,11 @@ void OpenStudioApp::writeLibraryPaths(std::vector<openstudio::path> paths) {
 
   QSettings settings(QCoreApplication::organizationName(), QCoreApplication::applicationName());
 
+  if (!settings.isWritable()) {
+    QMessageBox::warning(nullptr, tr("Settings file not writable"),
+                         tr("Your settings file '") + settings.fileName() + tr("' is not writable. Adjust the file permissions"));
+  }
+
   if (paths == defaultPaths) {
     settings.remove("library");
   } else {