From 40362ee985a52290883176acdde7c7def8cc844a Mon Sep 17 00:00:00 2001
From: Dave Conway-Jones <conway@uk.ibm.com>
Date: Tue, 23 Feb 2021 12:45:55 +0000
Subject: [PATCH] Let xmpp in handle a list of rooms

---
 social/xmpp/92-xmpp.html |  1 +
 social/xmpp/92-xmpp.js   | 13 ++++++++-----
 2 files changed, 9 insertions(+), 5 deletions(-)

diff --git a/social/xmpp/92-xmpp.html b/social/xmpp/92-xmpp.html
index 1b81a8ef..dbf3eb9a 100644
--- a/social/xmpp/92-xmpp.html
+++ b/social/xmpp/92-xmpp.html
@@ -19,6 +19,7 @@
     </div>
     <div class="form-tips"><b>Note:</b> By leaving Buddy empty and ticking "Is a chat room",
         the node will try to listen to all the rooms the user has access to.
+        You can specify multiple rooms by separating them by a :
     </div>
 </script>
 
diff --git a/social/xmpp/92-xmpp.js b/social/xmpp/92-xmpp.js
index d056ef51..ce6b90d9 100644
--- a/social/xmpp/92-xmpp.js
+++ b/social/xmpp/92-xmpp.js
@@ -322,7 +322,8 @@ module.exports = function(RED) {
         this.join = n.join || false;
         this.sendAll = n.sendObject;
         // Yes, it's called "from", don't ask me why; I don't know why
-        this.from = (n.to || "").trim();
+        // (because it's where you are asking to get messages from...)
+        this.from = ((n.to || "").split(':')).map(s => s.trim());
         this.quiet = false;
         // MUC == Multi-User-Chat == chatroom
         //this.muc = this.join && (this.from !== "")
@@ -357,13 +358,15 @@ module.exports = function(RED) {
             node.quiet = false;
             node.status({fill:"green",shape:"dot",text:"node-red:common.status.connected"});
             if (node.join) {
-                if (node.from === "") {
+                if (node.from[0] === "") {
                     // try to get list of all rooms and join them all.
                     getItems(this.serverConfig.server,this.serverConfig.id,xmpp);
                 }
                 else {
                     // if we want to use a chatroom, we need to tell the server we want to join it
-                    joinMUC(node, xmpp, node.from+'/'+node.nick);
+                    for (var i=0; i<node.from.length; i++) {
+                        joinMUC(node, xmpp, node.from[i]+'/'+node.nick);
+                    }
                 }
             }
         });
@@ -413,7 +416,7 @@ module.exports = function(RED) {
                         }
                         else { msg.topic = ids[0]; }
                         // if (RED.settings.verbose || LOGITALL) {node.log("Received a message from "+stanza.attrs.from); }
-                        if (!node.join && ((node.from === "") || (node.from === stanza.attrs.to))) {
+                        if (!node.join && ((node.from[0] === "") || (node.from.includes(stanza.attrs.to)))) {
                             node.send([msg,null]);
                         }
                     }
@@ -429,7 +432,7 @@ module.exports = function(RED) {
                     }
                     var msg = { topic:from, payload:payload, room:conference };
                     //if (from && stanza.attrs.from != node.nick && from != node.nick) {
-                    if (from && node.join && (node.from === "" || node.from === conference)) {
+                    if (from && node.join && (node.from[0] === "" || node.from.includes(conference))) {
                         node.send([msg,null]);
                     }
                     //}