Bug 24878 - Butterfly fails to advertise Text capabilities
Summary: Butterfly fails to advertise Text capabilities
Status: RESOLVED FIXED
Alias: None
Product: Telepathy
Classification: Unclassified
Component: butterfly (show other bugs)
Version: unspecified
Hardware: Other All
: medium normal
Assignee: Telepathy bugs list
QA Contact: Telepathy bugs list
URL:
Whiteboard:
Keywords:
: 26366 (view as bug list)
Depends on:
Blocks:
 
Reported: 2009-11-03 07:26 UTC by Olivier Crête
Modified: 2010-02-02 07:49 UTC (History)
3 users (show)

See Also:
i915 platform:
i915 features:


Attachments

Description Olivier Crête 2009-11-03 07:26:58 UTC
Which makes the N900 not show the "IM using MSN" button on some contacts.. which is fail

Nokia-N900-42-10:~# dbus-send --print-reply --dest=org.freedesktop.Telepathy.Connection.butterfly.msn.tester_40tester_2eca /org/freedesktop/Telepathy/Connection/butterfly/msn/tester_40tester_2eca org.freedesktop.Telepathy.Connection.Interface.Capabilities.GetCapabilities array:uint32:54
method return sender=:1.488 -> dest=:1.494 reply_serial=2
   array [
      struct {
         uint32 54
         string "org.freedesktop.Telepathy.Channel.Type.StreamedMedia"
         uint32 3
         uint32 7
      }
   ]
Nokia-N900-42-10:~#
Comment 1 Olivier Le Thanh Duong 2009-12-27 17:40:03 UTC
This branch should fix the problem
http://git.collabora.co.uk/?p=user/olethanh/telepathy-butterfly.git;a=shortlog;h=refs/heads/text_capabilities
Comment 2 Jonny Lamb 2009-12-28 05:19:47 UTC
(In reply to comment #1)
> This branch should fix the problem

Looks fine, except one thing:

add_text_capabilities takes a list of handles. In on_addressbook_contact_added, you give it a single handle. That'll raise an exception saying it can't iterate through a handle.

-            self.add_text_capabilities(handle)
+            self.add_text_capabilities([handle])

Other than that, looks fine. I guess you've tested it on the N900?
Comment 3 Olivier Le Thanh Duong 2009-12-29 08:58:23 UTC
Thanks for the review, I have fixed your remark in my branch.
I haven't been able to test is since I haven't got the issue in my n900 but the capabilities show up fine in tp-inspector and Tester said he would test it.
Comment 4 Olivier Le Thanh Duong 2010-01-19 08:28:22 UTC
Merged in master
Comment 5 avs 2010-02-02 07:49:29 UTC
*** Bug 26366 has been marked as a duplicate of this bug. ***


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.