make addUserToRoom to receive user instead of user or string

Author: ricardogarim

apps/meteor/app/lib/server/functions/addUserToRoom.ts

@@ -21,7 +21,7 @@ import { notifyOnRoomChangedById, notifyOnSubscriptionChangedById } from '../lib
 
 export const addUserToRoom = async (
 	rid: string,
-	user: Pick<IUser, '_id' | 'username'> | string,
+	user: Pick<IUser, '_id' | 'username'>,
 	inviter?: Pick<IUser, '_id' | 'username'>,
 	{
 		skipSystemMessage,
@@ -42,7 +42,7 @@ export const addUserToRoom = async (
 		});
 	}
 
-	const userToBeAdded = typeof user === 'string' ? await Users.findOneByUsername(user.replace('@', '')) : await Users.findOneById(user._id);
+	const userToBeAdded = await Users.findOneById(user._id);
 	const roomDirectives = roomCoordinator.getRoomDirectives(room.t);
 
 	if (!userToBeAdded) {

apps/meteor/app/lib/server/methods/addUsersToRoom.ts

@@ -23,7 +23,7 @@ export const sanitizeUsername = (username: string) => {
 	if (isFederatedUsername) {
 		return username;
 	}
-	
+
 	return username.replace(/(^@)|( @)/, '');
 };
 
@@ -103,7 +103,7 @@ export const addUsersToRoomMethod = async (userId: string, data: { rid: string;
 
 			const subscription = await Subscriptions.findOneByRoomIdAndUserId(data.rid, newUser._id);
 			if (!subscription) {
-				await addUserToRoom(data.rid, newUser || username, user);
+				await addUserToRoom(data.rid, newUser, user);
 			} else {
 				if (!newUser.username) {
 					return;