From ebda58fd30c7a4aec8f04e3751de869579b71cac Mon Sep 17 00:00:00 2001 From: Kevin Smith Date: Wed, 6 Jul 2011 08:44:24 +0100 Subject: Don't bring autojoined MUCs to front. Resolves: #909 diff --git a/Swift/Controllers/Chat/ChatsManager.cpp b/Swift/Controllers/Chat/ChatsManager.cpp index 31764f7..9d6d9c8 100644 --- a/Swift/Controllers/Chat/ChatsManager.cpp +++ b/Swift/Controllers/Chat/ChatsManager.cpp @@ -224,6 +224,7 @@ void ChatsManager::handleUIEvent(boost::shared_ptr event) { } else if (JoinMUCUIEvent::ref joinEvent = boost::dynamic_pointer_cast(event)) { handleJoinMUCRequest(joinEvent->getJID(), joinEvent->getNick(), false); + mucControllers_[joinEvent->getJID()]->activateChatWindow(); } else if (boost::shared_ptr joinEvent = boost::dynamic_pointer_cast(event)) { if (!joinMUCWindow_) { @@ -349,8 +350,8 @@ void ChatsManager::rebindControllerJID(const JID& from, const JID& to) { chatControllers_[to]->setToJID(to); } -void ChatsManager::handleJoinMUCRequest(const JID &mucJID, const boost::optional& nickMaybe, bool autoJoin) { - if (autoJoin) { +void ChatsManager::handleJoinMUCRequest(const JID &mucJID, const boost::optional& nickMaybe, bool addAutoJoin) { + if (addAutoJoin) { MUCBookmark bookmark(mucJID, mucJID.getNode()); bookmark.setAutojoin(true); if (nickMaybe) { @@ -370,7 +371,8 @@ void ChatsManager::handleJoinMUCRequest(const JID &mucJID, const boost::optional controller->setAvailableServerFeatures(serverDiscoInfo_); controller->onUserLeft.connect(boost::bind(&ChatsManager::handleUserLeftMUC, this, controller)); } - mucControllers_[mucJID]->activateChatWindow(); + + mucControllers_[mucJID]->showChatWindow(); /* FIXME: handleChatActivity connection for recents, and changes to that method.*/ } diff --git a/Swift/Controllers/Chat/ChatsManager.h b/Swift/Controllers/Chat/ChatsManager.h index 936f5a8..24b0f97 100644 --- a/Swift/Controllers/Chat/ChatsManager.h +++ b/Swift/Controllers/Chat/ChatsManager.h @@ -54,7 +54,7 @@ namespace Swift { private: void handleChatRequest(const std::string& contact); - void handleJoinMUCRequest(const JID& muc, const boost::optional& nick, bool autoJoin); + void handleJoinMUCRequest(const JID& muc, const boost::optional& nick, bool addAutoJoin); void handleSearchMUCRequest(); void handleMUCSelectedAfterSearch(const JID&); void rebindControllerJID(const JID& from, const JID& to); -- cgit v0.10.2-6-g49f6