[FIX] Active users count on @all and @here (#25957)

Co-authored-by: Matheus Barbosa Silva <[email protected]>

apps/meteor/app/lib/server/lib/sendNotificationsOnMessage.js

@@ -244,7 +244,7 @@ export async function sendMessageNotifications(message, room, usersInThread = []
 
 	// Don't fetch all users if room exceeds max members
 	const maxMembersForNotification = settings.get('Notifications_Max_Room_Members');
-	const roomMembersCount = Subscriptions.findByRoomId(room._id).count();
+	const roomMembersCount = await Users.countRoomMembers(room._id);
 	const disableAllMessageNotifications = roomMembersCount > maxMembersForNotification && maxMembersForNotification !== 0;
 
 	const query = {

apps/meteor/app/models/server/models/Users.js

@@ -1655,6 +1655,10 @@ Find users to send a message by email if:
 			},
 		});
 	}
+
+	countRoomMembers(roomId) {
+		return this.find({ __rooms: roomId, active: true }).count();
+	}
 }
 
 export default new Users(Meteor.users, true);