SocketAPI: Ensure messages are consistent #3944

Before we blindly broadcasted the result of a sync action. That was
often different from what a subsequent FILE_STATUS query would report.

diff --git a/src/gui/socketapi.cpp b/src/gui/socketapi.cpp
index 1f75ad5d5f8703d811378c8cbc72ddda7c0c7f39..4d0bf5f692b41256294daaf0bb6157dfe88ffdd7 100644 (file)
--- a/src/gui/socketapi.cpp
+++ b/src/gui/socketapi.cpp
@@ -251,13 +251,9 @@ void SocketApi::slotItemCompleted(const QString &folder, const SyncFileItem &ite
         return;
     }
 
+    auto status = this->fileStatus(f, item.destination());
     const QString path = f->path() + item.destination();
-
-    QString command = QLatin1String("OK");
-    if (Progress::isWarningKind(item._status)) {
-        command = QLatin1String("ERROR");
-    }
-    broadcastMessage(QLatin1String("STATUS"), path, command);
+    broadcastMessage(QLatin1String("STATUS"), path, status.toSocketAPIString());
 }
 
 void SocketApi::slotSyncItemDiscovered(const QString &folder, const SyncFileItem &item)