Ensure all sync result status types are handled for file provider in …

…owncloudgui Signed-off-by: Claudio Cambra <[email protected]>
nextcloud · Jul 19, 2024 · 58d953f · 58d953f
1 parent c852506
commit 58d953f
Showing 1 changed file with 21 additions and 4 deletions.
diff --git a/src/gui/owncloudgui.cpp b/src/gui/owncloudgui.cpp
@@ -311,11 +311,28 @@ void ownCloudGui::slotComputeOverallSyncStatus()
                 continue;
             }
             const auto fileProvider = Mac::FileProvider::instance();
-            const auto latestStatus = fileProvider->socketServer()->latestReceivedSyncStatusForAccount(accountState->account());
-            if (latestStatus == SyncResult::Problem || latestStatus == SyncResult::Error || !fileProvider->xpc()->fileProviderExtReachable(accountFpId)) {
+
+            if (!fileProvider->xpc()->fileProviderExtReachable(accountFpId)) {
                 problemFileProviderAccounts.append(accountFpId);
-            } else if (latestStatus == SyncResult::SyncRunning) {
-                syncingFileProviderAccounts.append(accountFpId);
+            } else {
+                switch (const auto latestStatus = fileProvider->socketServer()->latestReceivedSyncStatusForAccount(accountState->account())) {
+                case SyncResult::Undefined:
+                case SyncResult::NotYetStarted:
+                    break;
+                case SyncResult::SyncPrepare:
+                case SyncResult::SyncRunning:
+                case SyncResult::SyncAbortRequested:
+                case SyncResult::Success:
+                    syncingFileProviderAccounts.append(accountFpId);
+                    break;
+                case SyncResult::Problem:
+                case SyncResult::Error:
+                case SyncResult::SetupError:
+                    problemFileProviderAccounts.append(accountFpId);
+                    break;
+                case SyncResult::Paused:
+                    break;
+                }
             }
         }
     }