General improvements

app/models/server/models/Rooms.js

@@ -552,39 +552,40 @@ export class Rooms extends Base {
 		return this._db.find(query, options);
 	}
 
-	findByNameOrFNameAndRoomIdsIncludingTeamRooms(name, teamIds, roomIds, options) {
+	findByNameOrFNameAndRoomIdsIncludingTeamRooms(text, teamIds, roomIds, options) {
+		const searchTerm = text && new RegExp(text, 'i');
+
 		const query = {
-			teamMain: {
-				$exists: false,
-			},
 			$and: [
+				{ teamMain: { $exists: false } },
+				{ prid: { $exists: false } },
 				{
 					$or: [
-						roomIds ? { _id: { $in: roomIds }, t: 'p' } : false,
 						{
 							t: 'c',
-							teamId: {
-								$exists: false,
-							},
+							teamId: { $exists: false },
 						},
 						{
-							teamId: {
-								$in: teamIds,
-							},
+							t: 'c',
+							teamId: { $in: teamIds },
 						},
+						...roomIds?.length > 0 ? [{
+							_id: {
+								$in: roomIds,
+							},
+						}] : [],
 					],
 				},
-				{
+				...searchTerm ? [{
 					$or: [{
-						name,
+						name: searchTerm,
 					}, {
-						fname: name,
+						fname: searchTerm,
 					}],
-				},
+				}] : [],
 			],
 		};
 
-		// do not use cache
 		return this._db.find(query, options);
 	}
 
@@ -604,7 +605,7 @@ export class Rooms extends Base {
 		};
 
 		if (text) {
-			const regex = new RegExp(s.trim(escapeRegExp(text)), 'i');
+			const regex = new RegExp(text, 'i');
 
 			query.$and.push({
 				$or: [{

client/views/directory/ChannelsTable.js

@@ -101,16 +101,17 @@ function ChannelsTable() {
 	);
 
 	const channelRoute = useRoute('channel');
+	const groupsRoute = useRoute('group');
 
 	const { value: data = { result: [] } } = useEndpointData('directory', query);
 
 	const onClick = useMemo(
-		() => (name) => (e) => {
+		() => (name, type) => (e) => {
 			if (e.type === 'click' || e.key === 'Enter') {
-				channelRoute.push({ name });
+				type === 'c' ? channelRoute.push({ name }) : groupsRoute.push({ name });
 			}
 		},
-		[channelRoute],
+		[channelRoute, groupsRoute],
 	);
 
 	const formatDate = useFormatDate();
@@ -122,8 +123,8 @@ function ChannelsTable() {
 			return (
 				<Table.Row
 					key={_id}
-					onKeyDown={onClick(name)}
-					onClick={onClick(name)}
+					onKeyDown={onClick(name, t)}
+					onClick={onClick(name, t)}
 					tabIndex={0}
 					role='link'
 					action

server/methods/browseChannels.js

@@ -51,9 +51,11 @@ const getChannelsAndGroups = (user, canViewAnon, searchTerm, sort, pagination) =
 
 	const teams = Promise.await(Team.getAllPublicTeams());
 	const teamIds = teams.map(({ _id }) => _id);
+
+	const userTeams = Promise.await(Team.listTeamsBySubscriberUserId(user._id, { projection: { teamId: 1 } }))?.map(({ teamId }) => teamId) || [];
 	const userRooms = user.__rooms;
 
-	const result = Rooms.findByNameOrFNameAndRoomIdsIncludingTeamRooms(searchTerm, teamIds, userRooms, {
+	const result = Rooms.findByNameOrFNameAndRoomIdsIncludingTeamRooms(searchTerm, [...userTeams, ...teamIds], userRooms, {
 		...pagination,
 		sort: {
 			featured: -1,
@@ -203,7 +205,7 @@ const getUsers = (user, text, workspace, sort, pagination) => {
 
 Meteor.methods({
 	browseChannels({ text = '', workspace = '', type = 'channels', sortBy = 'name', sortDirection = 'asc', page, offset, limit = 10 }) {
-		const regex = new RegExp(s.trim(escapeRegExp(text)), 'i');
+		const searchTerm = s.trim(escapeRegExp(text));
 
 		if (!['channels', 'users', 'teams'].includes(type) || !['asc', 'desc'].includes(sortDirection) || ((!page && page !== 0) && (!offset && offset !== 0))) {
 			return;
@@ -231,9 +233,9 @@ Meteor.methods({
 
 		switch (type) {
 			case 'channels':
-				return getChannelsAndGroups(user, canViewAnonymous, regex, sortChannels(sortBy, sortDirection), pagination);
+				return getChannelsAndGroups(user, canViewAnonymous, searchTerm, sortChannels(sortBy, sortDirection), pagination);
 			case 'teams':
-				return getTeams(user, text, sortChannels(sortBy, sortDirection), pagination);
+				return getTeams(user, searchTerm, sortChannels(sortBy, sortDirection), pagination);
 			case 'users':
 				return getUsers(user, text, workspace, sortUsers(sortBy, sortDirection), pagination);
 			default:

server/sdk/types/ITeamService.ts

@@ -82,4 +82,5 @@ export interface ITeamService {
 	getAllPublicTeams(options: FindOneOptions<ITeam>): Promise<Array<ITeam>>;
 	getMembersByTeamIds(teamIds: Array<string>, options: FindOneOptions<ITeamMember>): Promise<Array<ITeamMember>>;
 	update(uid: string, teamId: string, updateData: ITeamUpdateData): Promise<void>;
+	listTeamsBySubscriberUserId(uid: string, options?: FindOneOptions<ITeamMember>): Promise<Array<ITeamMember> | null>;
 }