Declare whitelist and blacklist activity verbs as static constexprs in activitydata...

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>

diff --git a/src/gui/folder.cpp b/src/gui/folder.cpp
index 02caa5322e8196c561bd4f953be0d6ef740c98f3..e43bcea068c7da3739add4ce165b6ac87a7352fa 100644 (file)
--- a/src/gui/folder.cpp
+++ b/src/gui/folder.cpp
@@ -1297,7 +1297,7 @@ void Folder::slotExistingFolderNowBig(const QString &folderPath)
         auto whitelistActivityLink = ActivityLink();
         whitelistActivityLink._label = tr("Keep syncing");
         whitelistActivityLink._primary = false;
-        whitelistActivityLink._verb = "WHITELIST_FOLDER";
+        whitelistActivityLink._verb = ActivityLink::WhitelistFolderVerb;
 
         QVector<ActivityLink> activityLinks = {whitelistActivityLink};
 
@@ -1305,7 +1305,7 @@ void Folder::slotExistingFolderNowBig(const QString &folderPath)
             auto blacklistActivityLink = ActivityLink();
             blacklistActivityLink._label = tr("Stop syncing");
             blacklistActivityLink._primary = true;
-            blacklistActivityLink._verb = "BLACKLIST_FOLDER";
+            blacklistActivityLink._verb = ActivityLink::BlacklistFolderVerb;
 
             activityLinks.append(blacklistActivityLink);
         }

diff --git a/src/gui/tray/activitydata.h b/src/gui/tray/activitydata.h
index 29df3c878cc38a9675a428d1fdce9093abda2774..e613ad5bf5dd36ac893fd057175a5e984a64ab14 100644 (file)
--- a/src/gui/tray/activitydata.h
+++ b/src/gui/tray/activitydata.h
@@ -45,6 +45,9 @@ class ActivityLink
 public:
     static ActivityLink createFomJsonObject(const QJsonObject &obj);
 
+    static constexpr auto WhitelistFolderVerb = "WHITELIST_FOLDER";
+    static constexpr auto BlacklistFolderVerb = "BLACKLIST_FOLDER";
+
 public:
     QString _imageSource;
     QString _imageSourceHovered;

diff --git a/src/gui/tray/activitylistmodel.cpp b/src/gui/tray/activitylistmodel.cpp
index 76fb6f8dbe6481912c3e0e6301179b2c8c7bcf8d..b43cb45c016f28fe2cd2d92fd6e4b01a19f5d9b8 100644 (file)
--- a/src/gui/tray/activitylistmodel.cpp
+++ b/src/gui/tray/activitylistmodel.cpp
@@ -836,11 +836,11 @@ 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._file.isEmpty()) { // _folder == folder alias/name, _file == folder/file path
+    } else if (action._verb == ActivityLink::WhitelistFolderVerb && !activity._file.isEmpty()) { // _folder == folder alias/name, _file == folder/file path
         FolderMan::instance()->whitelistFolderPath(activity._file);
         removeActivityFromActivityList(activity);
         return;
-    } else if (action._verb == "BLACKLIST_FOLDER" && !activity._file.isEmpty()) {
+    } else if (action._verb == ActivityLink::BlacklistFolderVerb && !activity._file.isEmpty()) {
         FolderMan::instance()->blacklistFolderPath(activity._file);
         removeActivityFromActivityList(activity);
         return;