fix: Sends set admin only when not empty.

jitsi · Apr 24, 2024 · eed9bbc · eed9bbc
1 parent f5f1c31
commit eed9bbc
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/modules/xmpp/ChatRoom.js b/modules/xmpp/ChatRoom.js
@@ -1452,16 +1452,19 @@ export default class ChatRoom extends Listenable {
                 type: 'set' })
                 .c('query', {
                     xmlns: 'http://jabber.org/protocol/muc#admin' });
+            let sendIq = false;
 
             Object.values(this.members).forEach(m => {
                 if (m.jid && !MEMBERS_AFFILIATIONS.includes(m.affiliation)) {
                     affiliationsIq.c('item', {
                         'affiliation': 'member',
                         'jid': Strophe.getBareJidFromJid(m.jid)
                     }).up();
+                    sendIq = true;
                 }
             });
-            this.xmpp.connection.sendIQ(affiliationsIq.up());
+
+            sendIq && this.xmpp.connection.sendIQ(affiliationsIq.up());
         }
 
         const errorCallback = onError ? onError : () => {}; // eslint-disable-line no-empty-function