feat(lobby): Admin set function for multiple items.

jitsi · Feb 13, 2024 · 95f7e7b · 95f7e7b
1 parent cc9b0e6
commit 95f7e7b
Showing 1 changed file with 11 additions and 12 deletions.
diff --git a/modules/xmpp/ChatRoom.js b/modules/xmpp/ChatRoom.js
@@ -1442,22 +1442,21 @@ export default class ChatRoom extends Listenable {
     setMembersOnly(enabled, onSuccess, onError) {
         if (enabled && Object.values(this.members).filter(m => !m.isFocus).length) {
             // first grant membership to all that are in the room
-            // currently there is a bug in prosody where it handles only the first item
-            // that's why we will send iq per member
+            const affiliationsIq = $iq({
+                to: this.roomjid,
+                type: 'set' })
+                .c('query', {
+                    xmlns: 'http://jabber.org/protocol/muc#admin' });
+
             Object.values(this.members).forEach(m => {
                 if (m.jid && !MEMBERS_AFFILIATIONS.includes(m.affiliation)) {
-                    this.xmpp.connection.sendIQ(
-                        $iq({
-                            to: this.roomjid,
-                            type: 'set' })
-                        .c('query', {
-                            xmlns: 'http://jabber.org/protocol/muc#admin' })
-                        .c('item', {
-                            'affiliation': 'member',
-                            'jid': Strophe.getBareJidFromJid(m.jid)
-                        }).up().up());
+                    affiliationsIq.c('item', {
+                        'affiliation': 'member',
+                        'jid': Strophe.getBareJidFromJid(m.jid)
+                    }).up();
                 }
             });
+            this.xmpp.connection.sendIQ(affiliationsIq.up());
         }
 
         const errorCallback = onError ? onError : () => {}; // eslint-disable-line no-empty-function