Fix usePlaceholders migration

diff --git a/src/gui/folder.cpp b/src/gui/folder.cpp
index 403640994af75c2d30da250fe379cb7c7e8d652c..376a58d60572ae6d132f0a262f16b76405cf2aca 100644 (file)
--- a/src/gui/folder.cpp
+++ b/src/gui/folder.cpp
@@ -122,17 +122,18 @@ Folder::Folder(const FolderDefinition &definition,
     // Potentially upgrade suffix vfs to windows vfs
     ENFORCE(_vfs);
     if (_definition.virtualFilesMode == Vfs::WithSuffix
-        && _definition.upgradeVfsMode
-        && isVfsPluginAvailable(Vfs::WindowsCfApi)) {
-        if (auto winvfs = createVfsFromPlugin(Vfs::WindowsCfApi)) {
-            // Wipe the existing suffix files from fs and journal
-            SyncEngine::wipeVirtualFiles(path(), _journal, *_vfs);
-
-            // Then switch to winvfs mode
-            _vfs.reset(winvfs.release());
-            _definition.virtualFilesMode = Vfs::WindowsCfApi;
-            saveToSettings();
+        && _definition.upgradeVfsMode) {
+        if (isVfsPluginAvailable(Vfs::WindowsCfApi)) {
+            if (auto winvfs = createVfsFromPlugin(Vfs::WindowsCfApi)) {
+                // Wipe the existing suffix files from fs and journal
+                SyncEngine::wipeVirtualFiles(path(), _journal, *_vfs);
+
+                // Then switch to winvfs mode
+                _vfs.reset(winvfs.release());
+                _definition.virtualFilesMode = Vfs::WindowsCfApi;
+            }
         }
+        saveToSettings();
     }
 
     // Initialize the vfs plugin

diff --git a/src/gui/folderman.cpp b/src/gui/folderman.cpp
index d7876d4d138c2fb8ace115cc9faad728615addab..04d6eef60a2c4264141b6d11cf261f7aa8a3fb5e 100644 (file)
--- a/src/gui/folderman.cpp
+++ b/src/gui/folderman.cpp
@@ -299,7 +299,6 @@ void FolderMan::setupFoldersHelper(QSettings &settings, AccountStatePtr account,
                     && settings.value(QLatin1String("usePlaceholders"), false).toBool()) {
                     qCInfo(lcFolderMan) << "Migrate: From usePlaceholders to PinState::OnlineOnly";
                     f->setRootPinState(PinState::OnlineOnly);
-                    settings.remove(QStringLiteral("usePlaceholders"));
                 }
 
                 // Migration: Mark folders that shall be saved in a backwards-compatible way