diff options
author | louiz’ <louiz@louiz.org> | 2020-03-11 15:25:38 +0100 |
---|---|---|
committer | louiz’ <louiz@louiz.org> | 2020-03-11 15:27:16 +0100 |
commit | 2c4ffbf25ae2647c586c9069c182444152c28622 (patch) | |
tree | 139776849a1e81a605496cc91374aa84bd43f9d6 /src/xmpp | |
parent | 8013bb1cfecf410466370c5d337490e30c02ced1 (diff) | |
download | biboumi-2c4ffbf25ae2647c586c9069c182444152c28622.tar.gz biboumi-2c4ffbf25ae2647c586c9069c182444152c28622.tar.bz2 biboumi-2c4ffbf25ae2647c586c9069c182444152c28622.tar.xz biboumi-2c4ffbf25ae2647c586c9069c182444152c28622.zip |
Consider the nick change only if it is NOT a join
Diffstat (limited to 'src/xmpp')
-rw-r--r-- | src/xmpp/biboumi_component.cpp | 25 |
1 files changed, 13 insertions, 12 deletions
diff --git a/src/xmpp/biboumi_component.cpp b/src/xmpp/biboumi_component.cpp index 514b270..91be091 100644 --- a/src/xmpp/biboumi_component.cpp +++ b/src/xmpp/biboumi_component.cpp @@ -159,18 +159,6 @@ void BiboumiComponent::handle_presence(const Stanza& stanza) const std::string own_nick = bridge->get_own_nick(iid); const XmlNode* x = stanza.get_child("x", MUC_NS); const IrcClient* irc = bridge->find_irc_client(iid.get_server()); - if (irc) - { - const auto chan = irc->find_channel(iid.get_local()); - if (chan->joined) - bridge->send_irc_nick_change(iid, to.resource, from.resource); - else if (!x) - { // send an error if we are not joined yet, instead of treating it as a join - this->send_stanza_error("presence", from_str, to_str, id, - "modify", "not-acceptable", - "You are not joined to this MUC."); - } - } // if there is no <x/>, this is a presence status update, we don’t care about those if (x) { @@ -202,6 +190,19 @@ void BiboumiComponent::handle_presence(const Stanza& stanza) bridge->join_irc_channel(iid, to.resource, password ? password->get_inner(): "", from.resource, history_limit); } + else + { + if (irc) + { + const auto chan = irc->find_channel(iid.get_local()); + if (chan && chan->joined) + bridge->send_irc_nick_change(iid, to.resource, from.resource); + else + { // send an error if we are not joined yet, instead of treating it as a join + this->send_stanza_error("presence", from_str, to_str, id, "modify", "not-acceptable", "You are not joined to this MUC."); + } + } + } } else if (type == "unavailable") { |