From 9d19cdbf4ec5109f078193daf6ab55df6f69ec93 Mon Sep 17 00:00:00 2001 From: fiaxh Date: Sat, 11 May 2019 19:10:36 +0200 Subject: [PATCH] Process read marker from other devices for MUCs to adjust read-up-to state --- .../service/counterpart_interaction_manager.vala | 14 ++++++++++++++ xmpp-vala/src/module/xep/0333_chat_markers.vala | 1 - 2 files changed, 14 insertions(+), 1 deletion(-) diff --git a/libdino/src/service/counterpart_interaction_manager.vala b/libdino/src/service/counterpart_interaction_manager.vala index b4df9b8d..68589003 100644 --- a/libdino/src/service/counterpart_interaction_manager.vala +++ b/libdino/src/service/counterpart_interaction_manager.vala @@ -64,8 +64,18 @@ public class CounterpartInteractionManager : StreamInteractionModule, Object { } private void on_chat_marker_received(Account account, Jid jid, string marker, string stanza_id) { + + // Check if the marker comes from ourselves (own jid or our jid in a MUC) bool own_marker = account.bare_jid.to_string() == jid.bare_jid.to_string(); + if (stream_interactor.get_module(MucManager.IDENTITY).is_groupchat_occupant(jid, account)) { + Jid? own_muc_jid = stream_interactor.get_module(MucManager.IDENTITY).get_own_jid(jid.bare_jid, account); + if (own_muc_jid != null && own_muc_jid.equals(jid)) { + own_marker = true; + } + } + if (own_marker) { + // If we received a display marker from ourselves (other device), set the conversation read up to that message. if (marker != Xep.ChatMarkers.MARKER_DISPLAYED && marker != Xep.ChatMarkers.MARKER_ACKNOWLEDGED) return; Conversation? conversation = stream_interactor.get_module(MessageStorage.IDENTITY).get_conversation_for_stanza_id(account, stanza_id); if (conversation == null) return; @@ -73,15 +83,18 @@ public class CounterpartInteractionManager : StreamInteractionModule, Object { if (message == null) return; conversation.read_up_to = message; } else { + // We received a marker from someone else. Search the respective message and mark it. foreach (Conversation conversation in stream_interactor.get_module(ConversationManager.IDENTITY).get_conversations(jid, account)) { Entities.Message? message = stream_interactor.get_module(MessageStorage.IDENTITY).get_message_by_stanza_id(stanza_id, conversation); if (message != null) { switch (marker) { case Xep.ChatMarkers.MARKER_RECEIVED: + // If we got a received marker, mark the respective message received. received_message_received(account, jid, message); message.marked = Entities.Message.Marked.RECEIVED; break; case Xep.ChatMarkers.MARKER_DISPLAYED: + // If we got a display marker, set all messages up to that message as read (if we know they've been received). received_message_displayed(account, jid, message); Gee.List messages = stream_interactor.get_module(MessageStorage.IDENTITY).get_messages(conversation); foreach (Entities.Message m in messages) { @@ -92,6 +105,7 @@ public class CounterpartInteractionManager : StreamInteractionModule, Object { break; } } else { + // We might get a marker before the actual message (on catchup). Save the marker. if (marker_wo_message.has_key(stanza_id) && marker_wo_message[stanza_id] == Xep.ChatMarkers.MARKER_DISPLAYED && marker == Xep.ChatMarkers.MARKER_RECEIVED) { return; diff --git a/xmpp-vala/src/module/xep/0333_chat_markers.vala b/xmpp-vala/src/module/xep/0333_chat_markers.vala index d5f46c43..257f6967 100644 --- a/xmpp-vala/src/module/xep/0333_chat_markers.vala +++ b/xmpp-vala/src/module/xep/0333_chat_markers.vala @@ -44,7 +44,6 @@ public class Module : XmppStreamModule { public override string get_id() { return IDENTITY.id; } private void on_received_message(XmppStream stream, MessageStanza message) { - if (message.type_ != MessageStanza.TYPE_CHAT) return; Gee.List nodes = message.stanza.get_all_subnodes(); foreach (StanzaNode node in nodes) { if (node.ns_uri == NS_URI && node.name in MARKERS) {