conversationmodel: avoid useless iteration

Also fix setMessageDisplayed for clients

Change-Id: Ia2f8f586d88736a597e2271d2fd59402664ea3e6
GitLab: https://git.jami.net/savoirfairelinux/ring-project/-/issues/1282

src/conversationmodel.cpp

@@ -1476,16 +1476,17 @@ ConversationModel::clearUnreadInteractions(const QString& convId)
     {
         std::lock_guard<std::mutex> lk(pimpl_->interactionsLocks[convId]);
         auto& interactions = conversation.interactions;
+        if (conversation.isSwarm()) {
+            emitUpdated = true;
+            if (!interactions.empty())
+                lastDisplayed = interactions.rbegin()->first;
+        } else {
             std::for_each(interactions.begin(),
                           interactions.end(),
                           [&](decltype(*interactions.begin())& it) {
                               if (!it.second.isRead) {
                                   emitUpdated = true;
                                   it.second.isRead = true;
-                              if (conversation.isSwarm()) {
-                                  lastDisplayed = it.first;
-                                  return;
-                              }
                                   if (owner.profileInfo.type != profile::Type::SIP)
                                       lastDisplayed = storage::getDaemonIdByInteractionId(pimpl_->db,
                                                                                           it.first);
@@ -1493,6 +1494,7 @@ ConversationModel::clearUnreadInteractions(const QString& convId)
                               }
                           });
         }
+    }
     if (!lastDisplayed.isEmpty()) {
         auto to = conversation.isSwarm()
                       ? "swarm:" + convId