From 48964bc5cca6e56cd7fc745b1a4a9ae5d34f0772 Mon Sep 17 00:00:00 2001 From: Marvin W Date: Sat, 27 Jun 2020 11:23:48 +0200 Subject: Don't strip() at XML layer --- libdino/src/service/message_processor.vala | 4 +++- xmpp-vala/src/core/stanza_reader.vala | 15 ++++++++++----- xmpp-vala/tests/stanza.vala | 2 +- 3 files changed, 14 insertions(+), 7 deletions(-) diff --git a/libdino/src/service/message_processor.vala b/libdino/src/service/message_processor.vala index 6a66f343..74192e16 100644 --- a/libdino/src/service/message_processor.vala +++ b/libdino/src/service/message_processor.vala @@ -333,7 +333,9 @@ public class MessageProcessor : StreamInteractionModule, Object { } public async Entities.Message parse_message_stanza(Account account, Xmpp.MessageStanza message) { - Entities.Message new_message = new Entities.Message(message.body); + string? body = message.body; + if (body != null) body = body.strip(); + Entities.Message new_message = new Entities.Message(body); new_message.account = account; new_message.stanza_id = Xep.UniqueStableStanzaIDs.get_origin_id(message) ?? message.id; diff --git a/xmpp-vala/src/core/stanza_reader.vala b/xmpp-vala/src/core/stanza_reader.vala index 1727847d..0c07948d 100644 --- a/xmpp-vala/src/core/stanza_reader.vala +++ b/xmpp-vala/src/core/stanza_reader.vala @@ -221,7 +221,7 @@ public class StanzaReader { var res = new StanzaNode(); res.name = "#text"; res.ns_uri = ns_state.current_ns_uri; - res.encoded_val = (yield read_until_char('<')).strip(); + res.encoded_val = (yield read_until_char('<')); return res; } @@ -244,8 +244,9 @@ public class StanzaReader { var res = yield read_node_start(); if (res.has_nodes) { bool finish_node_seen = false; + StanzaNode? text_node = null; do { - yield skip_until_non_ws(); + text_node = yield read_text_node(); if ((yield peek_single()) == '<') { skip_single(); if ((yield peek_single()) == '/') { @@ -264,11 +265,15 @@ public class StanzaReader { } else { res.sub_nodes.add(yield read_stanza_node()); } - } else { - res.sub_nodes.add(yield read_text_node()); } } while (!finish_node_seen); - if (res.sub_nodes.size == 0) res.has_nodes = false; + if (res.sub_nodes.size == 0) { + if (text_node == null || text_node.val.length == 0) { + res.has_nodes = false; + } else { + res.sub_nodes.add(text_node); + } + } } ns_state = ns_state.pop(); return res; diff --git a/xmpp-vala/tests/stanza.vala b/xmpp-vala/tests/stanza.vala index 19baef49..17cd8317 100644 --- a/xmpp-vala/tests/stanza.vala +++ b/xmpp-vala/tests/stanza.vala @@ -37,7 +37,7 @@ class StanzaTest : Gee.TestCase { - I'll send a friar with speed, to Mantua, with my letters to thy lord. + I'll send a friar with speed, to Mantua, with my letters to thy lord. """; -- cgit v1.2.3-54-g00ecf