Default to adding existing now big folders to whitelist when undecided

Signed-off-by: Claudio Cambra <[email protected]>
nextcloud · Jul 5, 2023 · 062960f · 062960f
1 parent c963efa
commit 062960f
Show file tree

Hide file tree

Showing 4 changed files with 24 additions and 6 deletions.
diff --git a/src/gui/folder.cpp b/src/gui/folder.cpp
@@ -1229,6 +1229,16 @@ void Folder::slotExistingFolderNowBig(const QString &folderPath)
     const auto trailSlashFolderPath = trailingSlashPath(folderPath);
     const auto journal = journalDb();
 
+    // Add the entry to the whitelist if it is neither in the blacklist or whitelist already
+    bool ok1 = false;
+    bool ok2 = false;
+    auto blacklist = journal->getSelectiveSyncList(SyncJournalDb::SelectiveSyncBlackList, &ok1);
+    auto whitelist = journal->getSelectiveSyncList(SyncJournalDb::SelectiveSyncWhiteList, &ok2);
+    if (ok1 && ok2 && !blacklist.contains(trailSlashFolderPath) && !whitelist.contains(trailSlashFolderPath)) {
+        whitelist.append(trailSlashFolderPath);
+        journal->setSelectiveSyncList(SyncJournalDb::SelectiveSyncWhiteList, whitelist);
+    }
+
     auto undecidedListQueryOk = false;
     auto undecidedList = journal->getSelectiveSyncList(SyncJournalDb::SelectiveSyncUndecidedList, &undecidedListQueryOk);
     if (undecidedListQueryOk) {
@@ -1261,6 +1271,7 @@ void Folder::slotExistingFolderNowBig(const QString &folderPath)
         existingFolderNowBigActivity._message = tr("Would you like to stop syncing this folder?");
         existingFolderNowBigActivity._accName = _accountState->account()->displayName();
         existingFolderNowBigActivity._folder = alias();
+        existingFolderNowBigActivity._file = cleanPath() + '/' + trailSlashFolderPath;
         existingFolderNowBigActivity._links = {blacklistActivityLink, whitelistActivityLink};
         existingFolderNowBigActivity._id = qHash(existingFolderNowBigActivity._file);
 

diff --git a/src/gui/folderman.cpp b/src/gui/folderman.cpp
@@ -1266,7 +1266,9 @@ void FolderMan::whitelistFolderPath(const QString &path)
         return;
     }
 
-    folder->whitelistPath(path);
+    const QString folderPath = folder->cleanPath() + QLatin1Char('/');
+    const auto relPath = path.mid(folderPath.length());
+    folder->whitelistPath(relPath);
 }
 
 void FolderMan::blacklistFolderPath(const QString &path)
@@ -1276,7 +1278,9 @@ void FolderMan::blacklistFolderPath(const QString &path)
         return;
     }
 
-    folder->blacklistPath(path);
+    const QString folderPath = folder->cleanPath() + QLatin1Char('/');
+    const auto relPath = path.mid(folderPath.length());
+    folder->blacklistPath(relPath);
 }
 
 QStringList FolderMan::findFileInLocalFolders(const QString &relPath, const AccountPtr acc)

diff --git a/src/gui/folderman.h b/src/gui/folderman.h
@@ -104,6 +104,7 @@ class FolderMan : public QObject
     /** Returns the folder which the file or directory stored in path is in */
     Folder *folderForPath(const QString &path);
 
+    // Takes local file paths and finds the corresponding folder, adding to correct selective sync list
     void whitelistFolderPath(const QString &path);
     void blacklistFolderPath(const QString &path);
 

diff --git a/src/gui/tray/activitylistmodel.cpp b/src/gui/tray/activitylistmodel.cpp
@@ -785,12 +785,14 @@ void ActivityListModel::slotTriggerAction(const int activityIndex, const int act
                (activity._syncFileItemStatus == SyncFileItem::Conflict || activity._syncFileItemStatus == SyncFileItem::FileNameClash)) {
         slotTriggerDefaultAction(activityIndex);
         return;
-    } else if (action._verb == "WHITELIST_FOLDER" && !activity._folder.isEmpty()) {
-        FolderMan::instance()->whitelistFolderPath(activity._folder);
+    } else if (action._verb == "WHITELIST_FOLDER" && !activity._file.isEmpty()) { // _folder == folder alias/name, _file == folder/file path
+        FolderMan::instance()->whitelistFolderPath(activity._file);
         removeActivityFromActivityList(activity);
-    } else if (action._verb == "BLACKLIST_FOLDER" && !activity._folder.isEmpty()) {
-        FolderMan::instance()->blacklistFolderPath(activity._folder);
+        return;
+    } else if (action._verb == "BLACKLIST_FOLDER" && !activity._file.isEmpty()) {
+        FolderMan::instance()->blacklistFolderPath(activity._file);
         removeActivityFromActivityList(activity);
+        return;
     }
 
     emit sendNotificationRequest(activity._accName, action._link, action._verb, activityIndex);