Fix Windows sidebar entry non vfs

owncloud · Feb 7, 2023 · df96efe · df96efe
1 parent ca66ea8
commit df96efe
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 8 deletions.
diff --git a/src/gui/folder.cpp b/src/gui/folder.cpp
@@ -1480,6 +1480,12 @@ bool Folder::groupInSidebar() const
     return false;
 }
 
+void Folder::setNavigationPaneClsid(const QUuid &clsid)
+{
+    _definition.navigationPaneClsid = clsid;
+    saveToSettings();
+}
+
 bool FolderDefinition::isDeployed() const
 {
     return _deployed;

diff --git a/src/gui/folder.h b/src/gui/folder.h
@@ -212,7 +212,7 @@ class Folder : public QObject
      */
     QString remotePathTrailingSlash() const;
 
-    void setNavigationPaneClsid(const QUuid &clsid) { _definition.navigationPaneClsid = clsid; }
+    void setNavigationPaneClsid(const QUuid &clsid);
     QUuid navigationPaneClsid() const { return _definition.navigationPaneClsid; }
 
     /**

diff --git a/src/gui/folderman.cpp b/src/gui/folderman.cpp
@@ -914,9 +914,6 @@ Folder *FolderMan::addFolder(const AccountStatePtr &accountState, const FolderDe
         emit folderListChanged();
     }
 
-#ifdef Q_OS_WIN
-    _navigationPaneHelper.scheduleUpdateCloudStorageRegistry();
-#endif
     return folder;
 }
 
@@ -1399,10 +1396,11 @@ Folder *FolderMan::addFolderFromWizard(const AccountStatePtr &accountStatePtr, c
         if (!newFolder->groupInSidebar()) {
             Utility::setupFavLink(localFolder);
 #ifdef Q_OS_WIN
-            // TODO: move to setupFavLink
-            // TODO: don't call setupFavLinkt if showInExplorerNavigationPane is false?
-            if (_navigationPaneHelper.showInExplorerNavigationPane())
-                folderDefinition.navigationPaneClsid = QUuid::createUuid();
+            if (_navigationPaneHelper.showInExplorerNavigationPane()) {
+                newFolder->setNavigationPaneClsid(QUuid::createUuid());
+                // we might need to add or remove the panel entry as cfapi brings this feature out of the box
+                FolderMan::instance()->navigationPaneHelper().scheduleUpdateCloudStorageRegistry();
+            }
 #endif
         }
         if (!ConfigFile().newBigFolderSizeLimit().first) {