Build: #1730 was successful Changes by Florian Schmaus <flo@geekplace.eu>
Code commits
Smack (master)
-
Florian Schmaus <flo@geekplace.eu> 0f7b7df1f0cd6a3d2f75a983143d02fda8990111
muc: fix roomDestroyed() callback
The previous site where the callback was invoked was only reached if
there was also a user status on the unvailable presence. But those are
not part of unavilable presences upon room destruction.
Fixes SMACK-888. -
Florian Schmaus <flo@geekplace.eu> 20aaef26289fcaa95fd8ecd23ad907f5a371a021
muc: remove 'joined' boolean from MultiUserChat, use myRoomJid instead
If myRoomJid is set, we are joined. -
Florian Schmaus <flo@geekplace.eu> 26ab8324520f10aa8009f5054ee862c77ba1925c
muc: only call userHasLeft() at one site
There is no need to duplicate that code. Also ensure that
userHasLeft() is invoked *before* the listeners are invoked, so that
e.g. isJoined() returns false in the listeners. -
Florian Schmaus <flo@geekplace.eu> ab2822be3ef31e2598d2607c3ecc95b3dc849679
muc: also set myRoomJid to null if we have left the room
-
Florian Schmaus <flo@geekplace.eu> c519dd1213bd96d9ff5a92d8f64e3957c7b8d981
muc: do check for equality twice
We already performed the presence.getFrom().equals(myRoomJID) check
and saved its result. No need to do it again. -
Florian Schmaus <flo@geekplace.eu> e2e228fc93ef0cbdd31cde119413d1561a251bd0
muc: synchronize Stats.create(Integer)
Since this method is used by the MUCUserProvider, it is potentially
invoked concurrently and the access to the statusMap must be
synchronized then. -
Florian Schmaus <flo@geekplace.eu> fa643f12d5deef6a1e782d660159e4b298196e3f
muc: add MUC status code 33
SMACK-882