From c5705de4cc4e2c3df69cd5c44141ff7a4d3b3bff Mon Sep 17 00:00:00 2001
From: Christian Kamm <mail@ckamm.de>
Date: Mon, 14 Jan 2019 15:44:50 +0100
Subject: [PATCH] SyncEngine: Rename phase finishing slots

slotDiscoveryJobFinished -> slotDiscoveryFinished
slotFinished -> slotPropagationFinished

This should be clearer. Particular the
slotFinished -> finalize -> emit finished()
chain was confusing before.
---
 src/libsync/progressdispatcher.h | 2 +-
 src/libsync/syncengine.cpp       | 8 ++++----
 src/libsync/syncengine.h         | 4 ++--
 3 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/libsync/progressdispatcher.h b/src/libsync/progressdispatcher.h
index f153ef785c..df50116cc4 100644
--- a/src/libsync/progressdispatcher.h
+++ b/src/libsync/progressdispatcher.h
@@ -63,7 +63,7 @@ public:
          * Emitted once when done
          *
          * Except when SyncEngine jumps directly to finalize() without going
-         * through slotFinished().
+         * through slotPropagationFinished().
          */
         Done
     };
diff --git a/src/libsync/syncengine.cpp b/src/libsync/syncengine.cpp
index f800f417ca..e46ebe54fe 100644
--- a/src/libsync/syncengine.cpp
+++ b/src/libsync/syncengine.cpp
@@ -548,7 +548,7 @@ void SyncEngine::startSync()
         syncError(errorString);
         finalize(false);
     });
-    connect(_discoveryPhase.data(), &DiscoveryPhase::finished, this, &SyncEngine::slotDiscoveryJobFinished);
+    connect(_discoveryPhase.data(), &DiscoveryPhase::finished, this, &SyncEngine::slotDiscoveryFinished);
 
     auto discoveryJob = new ProcessDirectoryJob(SyncFileItemPtr(), ProcessDirectoryJob::NormalQuery, ProcessDirectoryJob::NormalQuery,
         _discoveryPhase.data(), _discoveryPhase.data());
@@ -591,7 +591,7 @@ void SyncEngine::slotNewItem(const SyncFileItemPtr &item)
     _progressInfo->adjustTotalsForFile(*item);
 }
 
-void SyncEngine::slotDiscoveryJobFinished()
+void SyncEngine::slotDiscoveryFinished()
 {
     if (!_discoveryPhase) {
         // There was an error that was already taken care of
@@ -681,7 +681,7 @@ void SyncEngine::slotDiscoveryJobFinished()
         this, &SyncEngine::slotProgress);
     connect(_propagator.data(), &OwncloudPropagator::updateFileTotal,
         this, &SyncEngine::updateFileTotal);
-    connect(_propagator.data(), &OwncloudPropagator::finished, this, &SyncEngine::slotFinished, Qt::QueuedConnection);
+    connect(_propagator.data(), &OwncloudPropagator::finished, this, &SyncEngine::slotPropagationFinished, Qt::QueuedConnection);
     connect(_propagator.data(), &OwncloudPropagator::seenLockedFile, this, &SyncEngine::seenLockedFile);
     connect(_propagator.data(), &OwncloudPropagator::touchedFile, this, &SyncEngine::slotAddTouchedFile);
     connect(_propagator.data(), &OwncloudPropagator::insufficientLocalStorage, this, &SyncEngine::slotInsufficientLocalStorage);
@@ -743,7 +743,7 @@ void SyncEngine::slotItemCompleted(const SyncFileItemPtr &item)
     emit itemCompleted(item);
 }
 
-void SyncEngine::slotFinished(bool success)
+void SyncEngine::slotPropagationFinished(bool success)
 {
     if (_propagator->_anotherSyncNeeded && _anotherSyncNeeded == NoFollowUpSync) {
         _anotherSyncNeeded = ImmediateFollowUp;
diff --git a/src/libsync/syncengine.h b/src/libsync/syncengine.h
index 06d29c2275..5ce6b1ba19 100644
--- a/src/libsync/syncengine.h
+++ b/src/libsync/syncengine.h
@@ -176,10 +176,10 @@ private slots:
     void slotNewItem(const SyncFileItemPtr &item);
 
     void slotItemCompleted(const SyncFileItemPtr &item);
-    void slotFinished(bool success);
+    void slotDiscoveryFinished();
+    void slotPropagationFinished(bool success);
     void slotProgress(const SyncFileItem &item, quint64 curent);
     void updateFileTotal(const SyncFileItem &item, quint64 newSize);
-    void slotDiscoveryJobFinished();
     void slotCleanPollsJobAborted(const QString &error);
 
     /** Records that a file was touched by a job. */