Sink merges the sync requests now.

1 files changed, 1 insertions, 20 deletions

diff --git a/framework/domain/maillistmodel.cpp b/framework/domain/maillistmodel.cpp
index a995451e..c96f6dd1 100644
--- a/framework/domain/maillistmodel.cpp
+++ b/framework/domain/maillistmodel.cpp
@@ -27,8 +27,6 @@ MailListModel::MailListModel(QObject *parent)
 {
     setDynamicSortFilter(true);
     sort(0, Qt::DescendingOrder);
-    connect(&mFetchTimer, &QTimer::timeout, this, &MailListModel::fetch);
-    mFetchTimer.setSingleShot(true);
 }
 
 MailListModel::~MailListModel()
@@ -78,24 +76,7 @@ void MailListModel::fetchMail(Sink::ApplicationDomain::Mail::Ptr mail)
     if (mail && !mail->getFullPayloadAvailable() && !mFetchedMails.contains(mail->identifier())) {
         qDebug() << "Fetching mail: " << mail->identifier() << mail->getSubject();
         mFetchedMails.insert(mail->identifier());
-        mMailsToFetch << *mail;
-        //TODO it would be nicer if Sink could just transparently merge synchronization requeusts.
-        if (!mFetchTimer.isActive()) {
-            mFetchTimer.start(50);
-        }
-    }
-}
-
-void MailListModel::fetch()
-{
-    if (!mMailsToFetch.isEmpty()) {
-        auto first = mMailsToFetch.first();
-        auto scope = Sink::SyncScope{first};
-        for (const auto &m : mMailsToFetch) {
-            scope.filter(m.identifier());
-        }
-        Sink::Store::synchronize(scope).exec();
-        mMailsToFetch.clear();
+        Sink::Store::synchronize(Sink::SyncScope{*mail}).exec();
     }
 }