fixed visibility and saving of profiles

2 files changed, 23 insertions, 20 deletions

diff --git a/README.md b/README.md
index 4d37b387..5a063299 100644
--- a/README.md
+++ b/README.md
@@ -55,7 +55,7 @@ Planned features
 * Re-encryption after users-change (optional ofcourse)
 * Showing path in Add and Edit screen (currently sometimes confusing where I'm adding this password)
 * Right click handlers for file/folder and content
-* ~~first use wizards to set up password-store (and decryption key, currently always the gpg default key)~~
+* ~~First use wizards to set up password-store (and decryption key, currently always the gpg default key)~~
 * ~~Profiles (to allow use of multiple password stores and decryption keys) with dropdown in main screen~~
 * Password generation with options for what kind you'd like
 * Templates (username, url etc) in Add / Edit screen (configurable templates)
diff --git a/mainwindow.cpp b/mainwindow.cpp
index aaf76e30..a88be65f 100644
--- a/mainwindow.cpp
+++ b/mainwindow.cpp
@@ -282,24 +282,27 @@ void MainWindow::config() {
             settings.setValue("hidePassword", hidePassword ? "true" : "false");
             settings.setValue("hideContent", hideContent ? "true" : "false");
             settings.setValue("addGPGId", addGPGId ? "true" : "false");
-            settings.beginGroup("profiles");
-            settings.remove("");
-            bool profileExists = false;
-            QHashIterator<QString, QString> i(profiles);
-            while (i.hasNext()) {
-                i.next();
-                //qDebug() << i.key() + "|" + i.value();
-                if (i.key() == profile) {
-                    profileExists = true;
+            if (!profiles.isEmpty()) {
+                settings.beginGroup("profiles");
+                settings.remove("");
+                bool profileExists = false;
+                QHashIterator<QString, QString> i(profiles);
+                while (i.hasNext()) {
+                    i.next();
+                    //qDebug() << i.key() + "|" + i.value();
+                    if (i.key() == profile) {
+                        profileExists = true;
+                    }
+                    settings.setValue(i.key(), i.value());
                 }
-                settings.setValue(i.key(), i.value());
-            }
-            if (!profileExists) {
-                // just take the last one
-                profile = i.key();
+                if (!profileExists) {
+                    // just take the last one
+                    profile = i.key();
+                }
+                settings.endGroup();
+            } else {
+                settings.remove("profiles");
             }
-            settings.endGroup();
-
             updateProfileBox();
             ui->treeView->setRootIndex(proxyModel.mapFromSource(model.setRootPath(passStore)));
 
@@ -1045,11 +1048,11 @@ void MainWindow::genKey(QString batch, QDialog *keygenWindow)
  */
 void MainWindow::updateProfileBox()
 {
-    qDebug() << profiles.size();
+    //qDebug() << profiles.size();
     if (profiles.isEmpty()) {
-        ui->profileBox->setVisible(false);
+        ui->profileBox->hide();
     } else {
-        ui->profileBox->setVisible(true);
+        ui->profileBox->show();
         if (profiles.size() < 2) {
             ui->profileBox->setEnabled(false);
         } else {