Bug 308217 - Mobile icon is not hidden when user goes offline
Summary: Mobile icon is not hidden when user goes offline
Status: RESOLVED FIXED
Alias: None
Product: telepathy
Classification: Frameworks and Libraries
Component: contactlist (show other bugs)
Version: git-latest
Platform: unspecified Linux
: NOR normal
Target Milestone: 0.6-next
Assignee: Telepathy Bugs
URL:
Keywords: junior-jobs
Depends on:
Blocks:
 
Reported: 2012-10-11 07:50 UTC by Daniele E. Domenichelli
Modified: 2012-10-12 11:42 UTC (History)
3 users (show)

See Also:
Latest Commit: http://commits.kde.org/telepathy-common-internals/7157d9e94744ad6f4aabffd28958cd7c0b408df5
Version Fixed In: 0.5.2

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Daniele E. Domenichelli 2012-10-11 07:50:52 UTC 
When a user goes offline from a mobile phone, the phone icon is still displayed next to the offline icon (and this makes no sense).

Steps to reproduce:
* Show offline contacts in the contact list or apply a filter on display name
* Go online from a mobile phone (check that the contact is shown online and the mobile phone icon is displayed
* Go offline
* The mobile phone icon is still displayed next to the offline icon.
* If you close and re-open the contact list, the icon is no longer there.
Comment 1 Daniel Vrátil 2012-10-11 18:41:47 UTC 
This is an upstream issue - clientTypes list is not updated when contact goes offline, so Tp::Contact still reports cached values.

I opened a report in upstream tracker: https://bugs.freedesktop.org/show_bug.cgi?id=55883

We can workaround this in ContactModelItem by returning empty list for ClientDataRole when contact is offline, or we can just wait for upstream to fix it :)
Comment 2 Daniele E. Domenichelli 2012-10-11 19:12:55 UTC 
I think we cannot require a newer version of tp-qt4 for 0.5.2, so we can apply the workaround now, and remove it in master when the upstream bug is fixed
Comment 3 Daniel Vrátil 2012-10-12 11:09:40 UTC 
Short update: the upstream bug report has been triaged as an issue in connection manager (telepathy-gabble). 

We don't have a explicit dependency on gabble, so I suggest reassigning this to 0.6-next to remember to check whether upstream has already fixed the issue and eventually removing the workaround.
Comment 4 Martin Klapetek 2012-10-12 11:17:30 UTC 
I was thinking exactly this, reassigning.
Comment 5 David Edmundson 2012-10-12 11:29:35 UTC 
No, we can't have a bug open on something that we've worked around. 

1) It'll mess up my release reports
2) Any triager will just close this

If you want I'll open a new bug "remove temporary workaround on..." and assign that to 0.6?
Comment 6 Daniel Vrátil 2012-10-12 11:36:41 UTC 
Git commit 7157d9e94744ad6f4aabffd28958cd7c0b408df5 by Dan Vrátil.
Committed on 12/10/2012 at 13:34.
Pushed by dvratil into branch 'kde-telepathy-0.5'.

Return empty clientTypes list when contact is offline

This is a temporary workaround for upstream bug [0] that
should be removed when they fix it.

[0] https://bugs.freedesktop.org/show_bug.cgi?id=55883

REVIEW: 106881

M  +12   -1    KTp/Models/contact-model-item.cpp
M  +1    -0    KTp/Models/contact-model-item.h

http://commits.kde.org/telepathy-common-internals/7157d9e94744ad6f4aabffd28958cd7c0b408df5