Make variables in folderman::setupfoldermigration actually match what they are holding
authorClaudio Cambra <claudio.cambra@nextcloud.com>
Sun, 30 Apr 2023 10:30:55 +0000 (18:30 +0800)
committerClaudio Cambra <claudio.cambra@nextcloud.com>
Mon, 8 May 2023 14:56:33 +0000 (22:56 +0800)
Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>
src/gui/folderman.cpp

index 5eb1bf11fa358dc1823dbd5e3a8431f465b151e2..66bf57027c11fb2b44402eca3e6ebb59e39409f3 100644 (file)
@@ -352,12 +352,12 @@ int FolderMan::setupFoldersMigration()
     QDir dir(_folderConfigPath);
     //We need to include hidden files just in case the alias starts with '.'
     dir.setFilter(QDir::Files | QDir::Hidden);
-    const auto list = dir.entryList();
+    const auto dirFiles = dir.entryList();
 
-    // Normally there should be only one account when migrating.
+    // Normally there should be only one account when migrating. TODO: Change
     const auto accountState = AccountManager::instance()->accounts().value(0).data();
-    for (const auto &alias : list) {
-        const auto folder = setupFolderFromOldConfigFile(alias, accountState);
+    for (const auto &fileName : dirFiles) {
+        const auto folder = setupFolderFromOldConfigFile(fileName, accountState);
         if (folder) {
             scheduleFolder(folder);
             emit folderSyncStateChange(folder);