Summary: | Skype appear in list of available protocols even though it is not available | ||
---|---|---|---|
Product: | [Unmaintained] telepathy | Reporter: | David Edmundson <kde> |
Component: | accounts-kcm | Assignee: | Telepathy Bugs <kde-telepathy-bugs> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | andreaswuest, m.wege, mklapetek |
Priority: | NOR | Keywords: | junior-jobs |
Version: | git-latest | ||
Target Milestone: | 0.5.1 | ||
Platform: | unspecified | ||
OS: | Linux | ||
Latest Commit: | 900824a2f8ea5eb35a483e40498a593842bdb7ce | Version Fixed In: | 0.5.1 |
Sentry Crash Report: |
Description
David Edmundson
2012-06-18 12:37:08 UTC
May be it has something to do with a 32/64bit missmatch? I have 64bit version of Kubuntu. I know that skype with version2 only provided a 32bit package. The new version of Skype for Linux comes with a 32bit packages which is disguised as a 64bit-package. So may be that is the reason why KDE telepathy has problems (but: It works perfect with pidgin). May be a related upstream bug: https://code.google.com/p/skype4pidgin/issues/detail?id=193 Found the bug: when generating the list based on profiles we simply check the connectionmanager (haze) exists, however, not that this connection manager supports the relevant protocol void ProfileListModel::populateList() { ... //don't include profiles which we don't have a CM for if (! m_connectionManagerNames.contains(profile->cmName())) { continue; } .. } This logic needs to be improved. *** Bug 305503 has been marked as a duplicate of this bug. *** > Git commit 900824a2f8ea5eb35a483e40498a593842bdb7ce by David Edmundson. > Committed on 20/09/2012 at 22:37. > Pushed by davidedmundson into branch 'kde-telepathy-0.5'. > > Check connection manager supports protocol listed in profile. > > This should fix the issue of skype appearing in the list when telpeathy-haze was installed but pidgin-skype was not. > ConnectionManagers require having becomeReady() called before the list of protocols they support can be used, hence converting from a list of CMs to a hash of CmPtrs > > REVIEW: 106515 > > M +40 -7 src/KCMTelepathyAccounts/profile-list-model.cpp > M +3 -1 src/KCMTelepathyAccounts/profile-list-model.h |