When MUCServiceImpl creates a chat room (i.e. a ChatRoomWrapper instance) it should first check to see if the chat room already exists and is already known by the protocol.
By first looking for an existing chat room, findRoom(...), before creating the chat room, we can make sure that we are not redundantly creating a chat room (and causing an OperationFailedException as a consequence).
Also, this becomes particularly relevant when the IRC protocol implementation is merged, since the IRC servers may automatically join a user to a particular channel, in which case the ChatRoom instance (related to the protocol implementation) exists, but the ChatRoomWrapper (related to the Jitsi core framework) does not yet.
You can merge this Pull Request by running:
git pull https://github.com/cobratbq/jitsi chatroom-fix
Or you can view, comment on it, or merge it online at:
https://github.com/jitsi/jitsi/pull/28
-- Commit Summary --
* Only create the chat room if provider cannot find it.
-- File Changes --
M src/net/java/sip/communicator/impl/muc/MUCServiceImpl.java (17)
-- Patch Links --
https://github.com/jitsi/jitsi/pull/28.patch
https://github.com/jitsi/jitsi/pull/28.diff
···
---
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi/pull/28