conflicts resolved

Author: naman9271

modules/xmpp/BreakoutRooms.js renamed to modules/xmpp/BreakoutRooms.ts

@@ -3,6 +3,8 @@ import { $msg, Strophe } from 'strophe.js';
 
 import { XMPPEvents } from '../../service/xmpp/XMPPEvents';
 
+import ChatRoom from './ChatRoom';
+
 const FEATURE_KEY = 'features/breakout-rooms';
 const BREAKOUT_ROOM_ACTIONS = {
     ADD: `${FEATURE_KEY}/add`,
@@ -17,17 +19,46 @@ const BREAKOUT_ROOM_EVENTS = {
 
 const logger = getLogger('modules/xmpp/BreakoutRooms');
 
+/**
+ * Types for breakout room participants and rooms.
+ */
+export interface IBreakoutRoomParticipant {
+    [key: string]: unknown;
+    jid: string;
+}
+
+export interface IBreakoutRoom {
+    [key: string]: unknown;
+    participants?: { [key: string]: IBreakoutRoomParticipant; };
+}
+
+export interface IBreakoutRoomsUpdatePayload {
+    [key: string]: unknown;
+    event: string;
+    rooms: { [key: string]: IBreakoutRoom; };
+}
+
+export interface IBreakoutRoomsMoveToRoomPayload {
+    [key: string]: unknown;
+    event: string;
+    roomJid: string;
+}
+
 /**
  * Helper class for handling breakout rooms.
  */
 export default class BreakoutRooms {
+    private room: ChatRoom;
+    private _rooms: { [key: string]: IBreakoutRoom; };
+    private _isBreakoutRoom?: boolean;
+    private _mainRoomJid?: string;
 
     /**
      * Constructs breakout room.
      *
      * @param {ChatRoom} room the room we are in.
      */
-    constructor(room) {
+    constructor(room: ChatRoom) {
         this.room = room;
 
         this._handleMessages = this._handleMessages.bind(this);
@@ -39,7 +70,7 @@ export default class BreakoutRooms {
     /**
      * Stops listening for events.
      */
-    dispose() {
+    dispose(): void {
         this.room.xmpp.removeListener(XMPPEvents.BREAKOUT_ROOMS_EVENT, this._handleMessages);
     }
 
@@ -48,7 +79,7 @@ export default class BreakoutRooms {
      *
      * @param {string} subject - A subject for the breakout room.
      */
-    createBreakoutRoom(subject) {
+    createBreakoutRoom(subject: string): void {
         if (!this.isSupported() || !this.room.isModerator()) {
             logger.error(`Cannot create breakout room - supported:${this.isSupported()},
                 moderator:${this.room.isModerator()}`);
@@ -69,7 +100,7 @@ export default class BreakoutRooms {
      *
      * @param {string} breakoutRoomJid - JID of the room to be removed.
      */
-    removeBreakoutRoom(breakoutRoomJid) {
+    removeBreakoutRoom(breakoutRoomJid: string): void {
         if (!this.isSupported() || !this.room.isModerator()) {
             logger.error(`Cannot remove breakout room - supported:${this.isSupported()},
                 moderator:${this.room.isModerator()}`);
@@ -91,7 +122,7 @@ export default class BreakoutRooms {
      * @param {string} breakoutRoomJid - JID of the room to be removed.
      * @param {string} subject - A new subject for the breakout room.
      */
-    renameBreakoutRoom(breakoutRoomJid, subject) {
+    renameBreakoutRoom(breakoutRoomJid: string, subject: string): void {
         if (!this.isSupported() || !this.room.isModerator()) {
             logger.error(`Cannot rename breakout room - supported:${this.isSupported()},
                 moderator:${this.room.isModerator()}`);
@@ -115,7 +146,7 @@ export default class BreakoutRooms {
      * @param {string} participantJid - JID of the participant to be sent to a room.
      * @param {string} roomJid - JID of the target room.
      */
-    sendParticipantToRoom(participantJid, roomJid) {
+    sendParticipantToRoom(participantJid: string, roomJid: string): void {
         if (!this.isSupported() || !this.room.isModerator()) {
             logger.error(`Cannot send participant to room - supported:${this.isSupported()},
                 moderator:${this.room.isModerator()}`);
@@ -138,14 +169,14 @@ export default class BreakoutRooms {
      * @param {string} feature - Feature to check.
      * @returns Wether the feature is supported.
      */
-    isFeatureSupported(feature) {
-        return Boolean((this.room.xmpp.breakoutRoomsFeatures || {})[feature]);
+    isFeatureSupported(feature: string): boolean {
+        return Boolean(this.room.xmpp.breakoutRoomsFeatures?.[feature]);
     }
 
     /**
      * Whether Breakout Rooms support is enabled in the backend or not.
      */
-    isSupported() {
+    isSupported(): boolean {
         return Boolean(this.getComponentAddress());
     }
 
@@ -154,7 +185,7 @@ export default class BreakoutRooms {
      *
      * @returns The address of the component.
      */
-    getComponentAddress() {
+    getComponentAddress(): string | undefined {
         return this.room.xmpp.breakoutRoomsComponentAddress;
     }
 
@@ -163,7 +194,7 @@ export default class BreakoutRooms {
      *
      * @param {boolean} isBreakoutRoom - Whether this room is a breakout room.
      */
-    _setIsBreakoutRoom(isBreakoutRoom) {
+    _setIsBreakoutRoom(isBreakoutRoom: boolean): void {
         this._isBreakoutRoom = isBreakoutRoom;
     }
 
@@ -172,7 +203,7 @@ export default class BreakoutRooms {
      *
      * @returns True if the room is a breakout room, false otherwise.
      */
-    isBreakoutRoom() {
+    isBreakoutRoom(): boolean {
         if (typeof this._isBreakoutRoom !== 'undefined') {
             return this._isBreakoutRoom;
         }
@@ -187,7 +218,7 @@ export default class BreakoutRooms {
      *
      * @param {string} jid - The main room JID.
      */
-    _setMainRoomJid(jid) {
+    _setMainRoomJid(jid: string): void {
         this._mainRoomJid = jid;
     }
 
@@ -196,7 +227,7 @@ export default class BreakoutRooms {
      *
      * @returns The main room JID.
      */
-    getMainRoomJid() {
+    getMainRoomJid(): string | undefined {
         return this._mainRoomJid;
     }
 
@@ -205,13 +236,13 @@ export default class BreakoutRooms {
      *
      * @param {object} payload - Arbitrary data.
      */
-    _handleMessages(payload) {
+    _handleMessages(payload: IBreakoutRoomsUpdatePayload | IBreakoutRoomsMoveToRoomPayload): void {
         switch (payload.event) {
         case BREAKOUT_ROOM_EVENTS.MOVE_TO_ROOM:
-            this.room.eventEmitter.emit(XMPPEvents.BREAKOUT_ROOMS_MOVE_TO_ROOM, payload.roomJid);
+            this.room.eventEmitter.emit(XMPPEvents.BREAKOUT_ROOMS_MOVE_TO_ROOM, (payload as IBreakoutRoomsMoveToRoomPayload).roomJid);
             break;
         case BREAKOUT_ROOM_EVENTS.UPDATE: {
-            const filteredPayload = this._filterUpdatePayload(payload);
+            const filteredPayload = this._filterUpdatePayload(payload as IBreakoutRoomsUpdatePayload);
 
             this._rooms = filteredPayload.rooms;
             this.room.eventEmitter.emit(XMPPEvents.BREAKOUT_ROOMS_UPDATED, filteredPayload);
@@ -226,14 +257,14 @@ export default class BreakoutRooms {
      * @param {Object} payload - The payload of the update message.
      * @return {Object} - The filtered payload.
      */
-    _filterUpdatePayload(payload) {
+    _filterUpdatePayload(payload: IBreakoutRoomsUpdatePayload): IBreakoutRoomsUpdatePayload {
         const hiddenDomain = this.room.options.hiddenDomain;
         const { rooms } = payload;
-        const filteredRooms = {};
+        const filteredRooms: { [key: string]: IBreakoutRoom; } = {};
 
         Object.entries(rooms).forEach(([ key, room ]) => {
             const { participants = {} } = room;
-            const filteredParticipants = {};
+            const filteredParticipants: { [key: string]: IBreakoutRoomParticipant; } = {};
 
             Object.entries(participants).forEach(([ k, participant ]) => {
                 if (Strophe.getDomainFromJid(participant.jid) !== hiddenDomain) {
@@ -258,7 +289,7 @@ export default class BreakoutRooms {
      *
      * @param {Object} message - Command that needs to be sent.
      */
-    _sendMessage(message) {
+    _sendMessage(message: { [key: string]: unknown; }): void {
         const msg = $msg({ to: this.getComponentAddress() });
 
         msg.c('breakout_rooms', message).up();