Don't thread drafts and sent

To do this we:
* Expose from the model wether or not the model is threaded
* Set the relevant properties from the model on the controller (so we
  can switch between aggregate and non-aggregate versions)
* Keep the controller in the view it belongs to.

While this works it highlights a couple of issues:
* Controllers are view specific and should be kept within the view.
* The actions we execute in the controller are closely related to the
  model. The model is essentially what the user sees, and therefore what
  he operatees on.
* Sink should perhaps expose aggregates better. We have to pass around
  the values from the model because the model dispatches between
  aggregate and non-aggregate property depending on the threaded state.
  Similary the controller operates on the thread or not depending on the
  threaded state. Perhaps it would be more useful if sink actually
  returned the aggregate somehow, with the regular properties. That way
  the controller could use the regular properties from the entity it
  gets (which would simply either be the aggregate or non-aggregate
  depending on the executed query). If the aggregate already contains
  all matched ids, then we would also not have to execute an additional
  query to get the thread again, the modification would simply be
  applied to all ids originally returned.

1 files changed, 43 insertions, 4 deletions

diff --git a/framework/src/domain/maillistmodel.cpp b/framework/src/domain/maillistmodel.cpp
index 83340f69..0f477624 100644
--- a/framework/src/domain/maillistmodel.cpp
+++ b/framework/src/domain/maillistmodel.cpp
@@ -114,9 +114,17 @@ QVariant MailListModel::data(const QModelIndex &idx, int role) const
         case Date:
             return mail->getDate();
         case Unread:
-            return mail->getProperty("unreadCollected").toList().contains(true);
+            if (mIsThreaded) {
+                return mail->getProperty("unreadCollected").toList().contains(true);
+            } else {
+                return mail->getUnread();
+            }
         case Important:
-            return mail->getProperty("importantCollected").toList().contains(true);
+            if (mIsThreaded) {
+                return mail->getProperty("importantCollected").toList().contains(true);
+            } else {
+                return mail->getImportant();
+            }
         case Draft:
             return mail->getDraft();
         case Sent:
@@ -133,7 +141,11 @@ QVariant MailListModel::data(const QModelIndex &idx, int role) const
             }
             return mail->getMimeMessage();
         case ThreadSize:
-            return mail->getProperty("count").toInt();
+            if (mIsThreaded) {
+                return mail->getProperty("count").toInt();
+            } else {
+                return 1;
+            }
         case Mail:
             return QVariant::fromValue(mail);
         case Incomplete:
@@ -176,6 +188,11 @@ void MailListModel::runQuery(const Sink::Query &query)
     setSourceModel(m_model.data());
 }
 
+bool MailListModel::isThreaded() const
+{
+    return mIsThreaded;
+}
+
 void MailListModel::setParentFolder(const QVariant &parentFolder)
 {
     using namespace Sink::ApplicationDomain;
@@ -189,11 +206,33 @@ void MailListModel::setParentFolder(const QVariant &parentFolder)
         return;
     }
     mCurrentQueryItem = folder->identifier();
-    Sink::Query query = Sink::StandardQueries::threadLeaders(*folder);
+    if (folder->getSpecialPurpose().contains(Sink::ApplicationDomain::SpecialPurpose::Mail::drafts) ||
+        folder->getSpecialPurpose().contains(Sink::ApplicationDomain::SpecialPurpose::Mail::sent)) {
+        mIsThreaded = false;
+    } else {
+        mIsThreaded = true;
+    }
+    emit isThreadedChanged();
+
+    Sink::Query query = [&] {
+        if (mIsThreaded) {
+            return Sink::StandardQueries::threadLeaders(*folder);
+        } else {
+            Sink::Query query;
+            query.setId("threadleaders-unthreaded");
+            if (!folder->resourceInstanceIdentifier().isEmpty()) {
+                query.resourceFilter(folder->resourceInstanceIdentifier());
+            }
+            query.filter<Sink::ApplicationDomain::Mail::Folder>(*folder);
+            query.sort<Sink::ApplicationDomain::Mail::Date>();
+            return query;
+        }
+    }();
     if (!folder->getSpecialPurpose().contains(Sink::ApplicationDomain::SpecialPurpose::Mail::trash)) {
         //Filter trash if this is not a trash folder
         query.filter<Sink::ApplicationDomain::Mail::Trash>(false);
     }
+
     query.setFlags(Sink::Query::LiveQuery);
     query.limit(100);
     query.request<Mail::Subject>();