[FIX] Active users count on @all and @here

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 Subscriptions.countRoomMembers(room._id);
 	const disableAllMessageNotifications = roomMembersCount > maxMembersForNotification && maxMembersForNotification !== 0;
 
 	const query = {

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

@@ -1381,6 +1381,26 @@ export class Subscriptions extends Base {
 	unsetOnHold(roomId) {
 		return this.update({ rid: roomId }, { $unset: { onHold: 1 } }, { multi: true });
 	}
+
+	countRoomMembers(roomId) {
+		return this.model
+			.rawCollection()
+			.aggregate([
+				{ $match: { rid: roomId } },
+				{
+					$lookup: {
+						from: 'users',
+						localField: 'u._id',
+						foreignField: '_id',
+						as: 'receiver',
+					},
+				},
+				{ $match: { 'receiver.active': true } },
+				{ $count: 'count' },
+			])
+			.toArray()
+			.then((res) => res?.[0]?.count || 0);
+	}
 }
 
 export default new Subscriptions('subscription', true);