From: Holger Weiss <holger@zedat.fu-berlin.de>
Date: Sun, 7 Feb 2016 23:46:33 +0000 (+0100)
Subject: Omit redundant check for overcrowded MUC room
X-Git-Tag: 16.02~49
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=bf49c292f8374c6b61523deceec655bde705f817;p=ejabberd

Omit redundant check for overcrowded MUC room

The send_update_presence/4 function already checked whether the room is
overcrowded before calling send_update_presence1/4, so there's no need
to have send_new_presence/4 perform the same check.
---

diff --git a/src/mod_muc_room.erl b/src/mod_muc_room.erl
index 3c5776de1..1b3b9c2bb 100644
--- a/src/mod_muc_room.erl
+++ b/src/mod_muc_room.erl
@@ -2117,7 +2117,7 @@ send_update_presence1(JID, Reason, StateData, OldStateData) ->
 		  end
 	    end,
     lists:foreach(fun (J) ->
-			  send_new_presence(J, Reason, StateData, OldStateData)
+			  send_new_presence1(J, Reason, StateData, OldStateData)
 		  end,
 		  LJIDs).