Fixed IRC federation's issues with the no-cache branch (#11906)

packages/rocketchat-irc/server/irc-bridge/localHandlers/onCreateRoom.js

@@ -1,15 +1,11 @@
 export default function handleOnCreateRoom(user, room) {
-	if (!room.usernames) {
+	const users = RocketChat.models.Users.findByRoomId(room._id);
-		return this.log(`Room ${ room.name } does not have a valid list of usernames`);
-	}
-
-	for (const username of room.usernames) {
-		const user = RocketChat.models.Users.findOne({ username });
 
+	users.forEach((user) => {
 		if (user.profile.irc.fromIRC) {
 			this.sendCommand('joinChannel', { room, user });
 		} else {
 			this.sendCommand('joinedChannel', { room, user });
 		}
-	}
+	});
 }

packages/rocketchat-irc/server/irc-bridge/localHandlers/onCreateUser.js

@@ -26,7 +26,7 @@ export default function handleOnCreateUser(newUser) {
 
 	this.sendCommand('registerUser', user);
 
-	const rooms = RocketChat.models.Rooms.findWithUsername(user.username).fetch();
+	const rooms = RocketChat.models.Rooms.findBySubscriptionUserId(user._id).fetch();
 
 	rooms.forEach((room) => this.sendCommand('joinedChannel', { room, user }));
 }

packages/rocketchat-irc/server/irc-bridge/localHandlers/onLogin.js

@@ -26,7 +26,7 @@ export default function handleOnLogin(login) {
 
 	this.sendCommand('registerUser', user);
 
-	const rooms = RocketChat.models.Rooms.findWithUsername(user.username).fetch();
+	const rooms = RocketChat.models.Rooms.findBySubscriptionUserId(user._id).fetch();
 
 	rooms.forEach((room) => this.sendCommand('joinedChannel', { room, user }));
 }

packages/rocketchat-irc/server/irc-bridge/peerHandlers/sentMessage.js

@@ -8,9 +8,6 @@ const getDirectRoom = (source, target) => {
 	const rid = [source._id, target._id].sort().join('');
 
 	RocketChat.models.Rooms.upsert({ _id: rid }, {
-		$set: {
-			usernames: [source.username, target.username],
-		},
 		$setOnInsert: {
 			t: 'd',
 			msgs: 0,

packages/rocketchat-lib/server/models/Rooms.js

@@ -61,10 +61,6 @@ class ModelRooms extends RocketChat.models._Base {
 
 	// FIND
 
-	findWithUsername(username, options) {
-		return this.find({ usernames: username }, options);
-	}
-
 	findById(roomId, options) {
 		return this.find({ _id: roomId }, options);
 	}

packages/rocketchat-lib/server/models/Users.js

@@ -80,6 +80,16 @@ class ModelUsers extends RocketChat.models._Base {
 		return this.find(query, options);
 	}
 
+	findByRoomId(rid, options) {
+		const data = RocketChat.models.Subscriptions.findByRoomId(rid).fetch().map((item) => item.u._id);
+		const query = {
+			_id: {
+				$in: data,
+			},
+		};
+
+		return this.find(query, options);
+	}
 
 	findByUsername(username, options) {
 		const query = { username };