[Bug 288327] telepathy-text-ui does not reconnect correctly to chatroom

Elias Probst mail at eliasprobst.eu
Mon Jan 16 21:23:15 UTC 2012


https://bugs.kde.org/show_bug.cgi?id=288327


Elias Probst <mail at eliasprobst.eu> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|FIXED                       |




--- Comment #4 from Elias Probst <mail eliasprobst eu>  2012-01-16 21:23:15 ---
Sorry, need to REOPEN.
The committed patch doesn't work here (still doesn't reconnect), while the
previous nopasted patch (see below) works just fine.

Just verified by locally reverting to the previous commit and testing the "old"
patch again.

diff --git a/app/chat-tab.cpp b/app/chat-tab.cpp
index a8f6626..5a16f45 100644
--- a/app/chat-tab.cpp
+++ b/app/chat-tab.cpp
@@ -80,6 +80,14 @@ void ChatTab::onConnectionStatusChanged(Tp::ConnectionStatus
status)
 {
     // request a new text channel for the chat
     if (status == Tp::ConnectionStatusConnected) {
-        account()->ensureTextChat(textChannel()->targetId(),
QDateTime::currentDateTime(),
QLatin1String("org.freedesktop.Telepathy.Client.KTp.TextUi"));
+        if (textChannel()->targetHandleType() == Tp::HandleTypeContact) {
+            account()->ensureTextChat(textChannel()->targetId(),
+                                      QDateTime::currentDateTime(),
+                                     
QLatin1String("org.freedesktop.Telepathy.Client.KTp.TextUi"));
+        } else if (textChannel()->targetHandleType() == Tp::HandleTypeRoom) {
+            account()->ensureTextChatroom(textChannel()->targetId(),
+                                                  
QDateTime::currentDateTime(),
+                                                  
QLatin1String("org.freedesktop.Telepathy.Client.KTp.TextUi"));
+        }
     }
 }

-- 
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.


More information about the KDE-Telepathy mailing list