From 4ab33fdfbb23fdfed5327d402989fd53d79f5380 Mon Sep 17 00:00:00 2001
From: Stanislaw Halik <sthalik@misaki.pl>
Date: Sat, 22 Aug 2015 19:02:25 +0200
Subject: main, ui: store config files in a predefined directory

Settings files are now stored in Documents/opentrack-version.

The transition path is to copy all .ini files into that directory,
accessible through the "Profile" menu dropdown.

Issue: #179
---
 opentrack/options.hpp | 53 ++++++++++++++++++++++++++++++++++-----------------
 1 file changed, 36 insertions(+), 17 deletions(-)

(limited to 'opentrack/options.hpp')

diff --git a/opentrack/options.hpp b/opentrack/options.hpp
index f4deb8a1..8e623092 100644
--- a/opentrack/options.hpp
+++ b/opentrack/options.hpp
@@ -30,6 +30,7 @@
 #include <QCoreApplication>
 #include <QFileInfo>
 #include <QDir>
+#include <QStandardPaths>
 
 #include <cinttypes>
 
@@ -38,8 +39,8 @@
 #include <memory>
 template<typename t> using mem = std::shared_ptr<t>;
 
-#define OPENTRACK_CONFIG_FILENAME_KEY "settings-file"
-#define OPENTRACK_DEFAULT_CONFIG_PATH "/settings/default.ini"
+#define OPENTRACK_CONFIG_FILENAME_KEY "settings-filename"
+#define OPENTRACK_DEFAULT_CONFIG "default.ini"
 #define OPENTRACK_ORG "opentrack-2.3"
 
 namespace options {
@@ -90,30 +91,30 @@ namespace options {
     public:
         group(const string& name) : name(name)
         {
-            QSettings conf(ini_pathname(), QSettings::IniFormat);
+            auto conf = ini_file();
             auto q_name = QString::fromStdString(name);
-            conf.beginGroup(q_name);
-            for (auto& k_ : conf.childKeys())
+            conf->beginGroup(q_name);
+            for (auto& k_ : conf->childKeys())
             {
                 auto tmp = k_.toUtf8();
                 string k(tmp);
-                kvs[k] = conf.value(k_);
+                kvs[k] = conf->value(k_);
             }
-            conf.endGroup();
+            conf->endGroup();
         }
 
         void save()
         {
-            QSettings s(ini_pathname(), QSettings::IniFormat);
+            auto s = ini_file();
             auto q_name = QString::fromStdString(name);
-            s.beginGroup(q_name);
+            s->beginGroup(q_name);
             for (auto& i : kvs)
             {
                 auto k = QString::fromStdString(i.first);
-                s.setValue(k, i.second);
+                s->setValue(k, i.second);
             }
-            s.endGroup();
-            s.sync();
+            s->endGroup();
+            s->sync();
         }
 
         template<typename t>
@@ -132,22 +133,40 @@ namespace options {
             return kvs.count(s) != 0;
         }
 
-        static const QString ini_pathname()
+        static QString ini_directory()
         {
+            const auto dirs = QStandardPaths::standardLocations(QStandardPaths::DocumentsLocation);
+            if (dirs.size() == 0)
+                return "";
+            if (QDir(dirs[0]).mkpath(OPENTRACK_ORG))
+                return dirs[0] + "/" OPENTRACK_ORG;
+            return "";
+        }
+
+        static QString ini_pathname()
+        {
+            const auto dir = ini_directory();
+            if (dir == "")
+                return "";
             QSettings settings(OPENTRACK_ORG);
-            return settings.value(OPENTRACK_CONFIG_FILENAME_KEY, QCoreApplication::applicationDirPath() + OPENTRACK_DEFAULT_CONFIG_PATH).toString();
+            return dir + "/" + settings.value(OPENTRACK_CONFIG_FILENAME_KEY, OPENTRACK_DEFAULT_CONFIG).toString();
         }
         
         static const QStringList ini_list()
         {
-            QFileInfo info(group::ini_pathname());
-            QDir settings_dir(info.dir());
+            const auto dirname = ini_directory();
+            if (dirname == "")
+                return QStringList();
+            QDir settings_dir(dirname);
             return settings_dir.entryList( QStringList { "*.ini" } , QDir::Files, QDir::Name );
         }
         
         static const mem<QSettings> ini_file()
         {
-            return std::make_shared<QSettings>(ini_pathname(), QSettings::IniFormat);
+            const auto pathname = ini_pathname();
+            if (pathname != "")
+                return std::make_shared<QSettings>(ini_pathname(), QSettings::IniFormat);
+            return std::make_shared<QSettings>();
         }
     };
 
-- 
cgit v1.2.3


From 8adcd7acc7280c84a959db0da2d79a0ed918a1ca Mon Sep 17 00:00:00 2001
From: Stanislaw Halik <sthalik@misaki.pl>
Date: Wed, 26 Aug 2015 19:53:40 +0200
Subject: options: return .ini basename easier

---
 facetracknoir/ui.cpp  | 4 ++--
 opentrack/options.hpp | 6 ++++++
 2 files changed, 8 insertions(+), 2 deletions(-)

(limited to 'opentrack/options.hpp')

diff --git a/facetracknoir/ui.cpp b/facetracknoir/ui.cpp
index 6aa47fb6..15c94bd7 100644
--- a/facetracknoir/ui.cpp
+++ b/facetracknoir/ui.cpp
@@ -203,7 +203,7 @@ void MainWindow::refresh_config_list()
 
      QStringList ini_list = group::ini_list();
      set_title();
-     QString current = QFileInfo(group::ini_pathname()).fileName();
+     QString current = group::ini_filename();
      ui.iconcomboProfile->clear();
      for (auto x : ini_list)
          ui.iconcomboProfile->addItem(QIcon(":/images/settings16.png"), x);
@@ -343,7 +343,7 @@ void MainWindow::set_title(const QString& game_title_)
     QString game_title;
     if (game_title_ != "")
         game_title = " :: " + game_title_;
-    QString current = QFileInfo(group::ini_pathname()).fileName();
+    QString current = group::ini_filename();
     setWindowTitle(const_cast<const char*>(opentrack_version) + QStringLiteral(" :: ") + current + game_title);
 }
 
diff --git a/opentrack/options.hpp b/opentrack/options.hpp
index 8e623092..f8475877 100644
--- a/opentrack/options.hpp
+++ b/opentrack/options.hpp
@@ -143,6 +143,12 @@ namespace options {
             return "";
         }
 
+        static QString ini_filename()
+        {
+            QSettings settings(OPENTRACK_ORG);
+            return settings.value(OPENTRACK_CONFIG_FILENAME_KEY, OPENTRACK_DEFAULT_CONFIG).toString();
+        }
+
         static QString ini_pathname()
         {
             const auto dir = ini_directory();
-- 
cgit v1.2.3