[Telepathy] I'm unable to create a channel

[Olli Salli]

On Mon, Nov 8, 2010 at 1:40 PM,  <mikhail.zabaluev at nokia.com> wrote:
> Hi,
>
>> -----Original Message-----
>> From: telepathy-
>> bounces+mikhail.zabaluev=nokia.com at lists.freedesktop.org
>> [mailto:telepathy-
>> bounces+mikhail.zabaluev=nokia.com at lists.freedesktop.org] On Behalf Of
>> ext Simon McVittie
>> Sent: Monday, November 08, 2010 1:36 PM
>> To: telepathy at lists.freedesktop.org
>> Subject: Re: [Telepathy] I'm unable to create a channel
>>
>> On Mon, 08 Nov 2010 at 16:29:20 +0530, Abhijit Bera wrote:
>> >
>> map.insert("org.freedesktop.Telepathy.Channel.ChannelType",QVariant("or
>> g.freedeskop.Telepathy.Channel.Type.ContactList"));
>>
>> There's a constant for ...ContactList, and there should also be one for
>> ...ChannelType I think?
>
> Indeed there should be: the code above has a typo in the interface name, which probably explains why it doesn't work.
>
> Regards,
>  Mikhail
> _______________________________________________
> telepathy mailing list
> telepathy at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/telepathy
>

(bah, failed to send this to the list earlier)

There is a constant for the interface name
(TP_QT4_IFACE_CHANNEL_TYPE_CONTACT_LIST in this case). We consider
property names so low-level that we don't generate string constants
for them - instead, you'll have to do TP_QT4_IFACE_CHANNEL +
QLatin1String(".ChannelType"). Low-level, because we've made it
unnecessary to build e.g. channel class or request props variant maps
by hand by virtue of having the methods like ensureTextChat for the
channel types for which this makes sense - which isn't true for
ContactList, as previously mentioned.b

And indeed, I can confirm you shouldn't attempt to use ContactList
channels directly. Tp::ContactManager abstracts them in a such a way
that your application will probably still work without code
modification underneath using the improved (more efficient)
Connection.Interface.ContactList API when we port Tp::ContactManager
to support it.

Br,
Olli Salli