Bug 280705 - JJ: When re-establishing broken channels text UI does not force
Summary: JJ: When re-establishing broken channels text UI does not force
Status: RESOLVED FIXED
Alias: None
Product: telepathy
Classification: Unmaintained
Component: text-ui (show other bugs)
Version: git-latest
Platform: Unlisted Binaries Linux
: NOR normal
Target Milestone: 0.4.0
Assignee: Telepathy Bugs
URL:
Keywords: junior-jobs
Depends on:
Blocks:
 
Reported: 2011-08-24 12:48 UTC by David Edmundson
Modified: 2012-07-06 12:47 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description David Edmundson 2011-08-24 12:48:44 UTC 
Version:           git-latest
OS:                Linux

When I'm chatting in the text-ui started from our contact list, if my connection drops and reconnects, the Text-Ui re-establishes a channel to whoever I was chatting with. 
However because I have empathy installed, it loads the empathy window, and gets in a right mess.



Reproducible: Didn't try

Steps to Reproduce:
N/A

Actual Results:  
N/A

Expected Results:  
At a minimum we should set the preferred handler requested from the text-ui to be the text-ui.

maybe we should be using ensureAndHandleChannel as this is actually what we want to do. Probably a much better solution.



See void ChatTab::onConnectionStatusChanged(Tp::ConnectionStatus status) for the offending line.
Comment 1 David Edmundson 2011-08-29 17:58:18 UTC 
Git commit 27d3517046dedd53679c8c703ce6adbd9e24757e by David Edmundson.
Committed on 29/08/2011 at 19:56.
Pushed by davidedmundson into branch 'master'.

When re-establishing channels, set preferred handler so that the new chat is handled by our application.
BUG: 280705

M  +1    -1    app/chat-tab.cpp

http://commits.kde.org/telepathy-chat-handler/27d3517046dedd53679c8c703ce6adbd9e24757e