feat: preserve the last selected account between runs

src/gui/accountmanager.cpp

@@ -108,18 +108,21 @@ AccountManager::AccountsRestoreResult AccountManager::restore(const bool alsoRes
     if (skipSettingsKeys.contains(settings->group())) {
         // Should not happen: bad container keys should have been deleted
         qCWarning(lcAccountManager) << "Accounts structure is too new, ignoring";
+        emit(accountListInitialized());
         return AccountsRestoreSuccessWithSkipped;
     }
 
     // If there are no accounts, check the old format.
 #if !DISABLE_ACCOUNT_MIGRATION
     if (settings->childGroups().isEmpty() && !settings->contains(QLatin1String(versionC)) && alsoRestoreLegacySettings) {
         restoreFromLegacySettings();
+        emit(accountListInitialized());
         return AccountsRestoreSuccessFromLegacyVersion;
     }
 #endif
 
     if (settings->childGroups().isEmpty()) {
+        emit(accountListInitialized());
         return AccountsNotFound;
     }
 
@@ -149,6 +152,8 @@ AccountManager::AccountsRestoreResult AccountManager::restore(const bool alsoRes
         }
     }
 
+    emit(accountListInitialized());
+
     ConfigFile().cleanupGlobalNetworkConfiguration();
     ClientProxy().cleanupGlobalNetworkConfiguration();   
 

src/gui/accountmanager.h

@@ -6,7 +6,7 @@
 
 #pragma once
 
 #include "account.h"
 #include "accountstate.h"
 
 namespace OCC {
@@ -108,6 +108,7 @@
     void removeAccountFolders(OCC::AccountState *account);
     void forceLegacyImportChanged();
     void capabilitiesChanged();
+    void accountListInitialized();
 
 private:
     // saving and loading Account to settings

src/gui/tray/usermodel.cpp

@@ -1,3 +1,3 @@
 /*
  * SPDX-FileCopyrightText: 2021 Nextcloud GmbH and Nextcloud contributors
  * SPDX-License-Identifier: GPL-2.0-or-later
@@ -1334,13 +1334,14 @@
 UserModel::UserModel(QObject *parent)
     : QAbstractListModel(parent)
 {
-    // TODO: Remember selected user from last quit via settings file
     if (AccountManager::instance()->accounts().size() > 0) {
-        buildUserList();
+        setInitialUser();
     }
 
     connect(AccountManager::instance(), &AccountManager::accountAdded,
-        this, &UserModel::buildUserList);
+        this, &UserModel::addAccsToUserList);
+    connect(AccountManager::instance(), &AccountManager::accountListInitialized,
+        this, &UserModel::setInitialUser);
 }
 
 void UserModel::buildUserList()
@@ -1349,10 +1350,41 @@
         auto user = AccountManager::instance()->accounts().at(i);
         addUser(user);
     }
+}
+
+void UserModel::addAccsToUserList()
+{
     if (_init) {
-        _users.first()->setCurrentUser(true);
-        _init = false;
+        return;
     }
+
+    buildUserList();
+}
+
+void UserModel::setInitialUser()
+{
+    if (!_init) {
+        return;
+    }
+
+    buildUserList();
+
+    if(!_users.isEmpty()) {
+        ConfigFile cfg;
+        const uint lastSelectedAccountId = cfg.lastSelectedAccount();
+
+        for (int i = 0; i <  _users.size(); i++) {
+            if (_users.at(i)->account()->id().toUInt() == lastSelectedAccountId) {
+                setCurrentUserId(i);
+            }
+        }
+
+        if (_currentUserId < 0) {
+            setCurrentUserId(0);
+        }
+    }
+
+    _init = false;
 }
 
 int UserModel::numUsers()
@@ -1430,7 +1462,7 @@
         });
 
         _users << u;
-        if (isCurrent || _currentUserId < 0) {
+        if (isCurrent || (_currentUserId < 0 && !_init)) {
             setCurrentUserId(_users.size() - 1);
         }
 
@@ -1524,6 +1556,8 @@
         // order has changed, index remained the same
         emit currentUserChanged();
     } else if (_currentUserId != id) {
+        ConfigFile cfg;
+        cfg.setLastSelectedAccount(_users[id]->account()->id().toUInt());
         _currentUserId = id;
         emit currentUserChanged();
     }

src/gui/tray/usermodel.h

@@ -6,7 +6,7 @@
 #ifndef USERMODEL_H
 #define USERMODEL_H
 
 #include <QAbstractListModel>
 #include <QImage>
 #include <QDateTime>
 #include <QStringList>
@@ -292,6 +292,8 @@
     bool _init = true;
 
     void buildUserList();
+    void addAccsToUserList();
+    void setInitialUser();
 };
 
 class ImageProvider : public QQuickAsyncImageProvider

src/libsync/configfile.cpp

@@ -4,7 +4,7 @@
  * SPDX-License-Identifier: GPL-2.0-or-later
  */
 
 #include "config.h"
 
 #include "configfile.h"
 #include "theme.h"
@@ -83,7 +83,10 @@
 static const QString defaultEnterpriseChannel = "enterprise";
 
 static constexpr char languageC[] = "language";
+
+static constexpr char lastSelectedAccountC[] = "lastSelectedAccount";
+
 static constexpr int deleteFilesThresholdDefaultValue = 100;
 }
 
 namespace OCC {
@@ -1227,7 +1230,19 @@
     settings.setValue(QLatin1String(languageC), language);
 }
 
+uint ConfigFile::lastSelectedAccount() const
+{
+    QSettings settings(configFile(), QSettings::IniFormat);
+    return settings.value(QLatin1String(lastSelectedAccountC), QLatin1String("")).toUInt();
+}
+
+void ConfigFile::setLastSelectedAccount(const uint accountId)
+{
+    QSettings settings(configFile(), QSettings::IniFormat);
+    settings.setValue(QLatin1String(lastSelectedAccountC), accountId);
+}
+
 Q_GLOBAL_STATIC(QString, g_configFileName)
 
 std::unique_ptr<QSettings> ConfigFile::settingsWithGroup(const QString &group, QObject *parent)
 {

src/libsync/configfile.h

@@ -7,7 +7,7 @@
 #ifndef CONFIGFILE_H
 #define CONFIGFILE_H
 
 #include "owncloudlib.h"
 #include <memory>
 #include <QSharedPointer>
 #include <QSettings>
@@ -242,6 +242,10 @@
     [[nodiscard]] QString language() const;
     void setLanguage(const QString &language);
 
+    /// Store and retrieve the last selected account identifier
+    [[nodiscard]] uint lastSelectedAccount() const;
+    void setLastSelectedAccount(const uint accountId);
+
     /**  Returns a new settings pre-set in a specific group.  The Settings will be created
          with the given parent. If no parent is specified, the caller must destroy the settings */
     static std::unique_ptr<QSettings> settingsWithGroup(const QString &group, QObject *parent = nullptr);