diff options
author | Christian Kamm <mail@ckamm.de> | 2018-02-21 15:55:33 +0300 |
---|---|---|
committer | Camila San <hello@camila.codes> | 2018-05-16 17:47:27 +0300 |
commit | 800f1ace0c5bc89119a8712ec977a82c741e89ac (patch) | |
tree | ad09ce4115ac65c052304fa50adfb08008392844 /src/gui/folder.h | |
parent | 75194d1821641892b6ec8bf72f8cfd3787e46a2b (diff) |
SyncResult: Make sure the number of conflicts is correct #6226
If the SyncResult incorrectly believes that there are no conflicts, the
tray icon won't be correct and there will be no warning about unresolved
conflicts on the account.
Nevertheless, it's pretty awkward that the IssuesWidget is better
informed about pending conflicts than the Folder itself. This kind of
backwards data flow is very confusing.
Unfortunately the only alternative I see is to either keep track of
this information in two places (also in Folder), or create a common
data-holding class that can serve as a model instance for the issues
view as well as provide data directly to the Folder - which would
have been a much larger change.
Diffstat (limited to 'src/gui/folder.h')
-rw-r--r-- | src/gui/folder.h | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/src/gui/folder.h b/src/gui/folder.h index ef4d70541..fedaa46a5 100644 --- a/src/gui/folder.h +++ b/src/gui/folder.h @@ -315,6 +315,13 @@ private slots: /** Ensures that the next sync performs a full local discovery. */ void slotNextSyncFullLocalDiscovery(); + /** Adjust sync result based on conflict data from IssuesWidget. + * + * This is pretty awkward, but IssuesWidget just keeps better track + * of conflicts across partial local discovery. + */ + void slotFolderConflicts(const QString &folder, const QStringList &conflictPaths); + private: bool reloadExcludes(); |