[Telepathy] [PATCH] TelepathyQt4: Various bugs + patches to fix them
George Kiagiadakis
kiagiadakis.george at gmail.com
Mon Jun 15 16:13:30 PDT 2009
2009/6/15 Andre Moreira Magalhaes <andre.magalhaes at collabora.co.uk>:
> George Kiagiadakis wrote:
>> Patch 0002 makes ClientApproverAdaptor use the full name of the Connection
>> property ("org.freedesktop.Telepathy.ChannelDispatchOperation.Connection")
>> to read the Connection object path from the properties that are passed to
>> its AddDispatchOperation method, in order to compy with the spec. Before I
>> fix that, it would crash a while after the channel dispatcher had called
>> this method, because the Connection object was assumed to be valid, although
>> it wasn't.
>>
>>
>
> This is wrong, the property _is_ Connection as stated on the spec:
> |
> Properties| - |a{sv}| (Qualified_Property_Value_Map
> <http://telepathy.freedesktop.org/spec.html#type-Qualified_Property_Value_Map>)
>
> Properties of the channel dispatch operation. This MUST NOT include
> properties that could change, SHOULD include as many properties as
> possible given that constraint, and MUST include at least the
> Account
>
> <http://telepathy.freedesktop.org/spec.html#org.freedesktop.Telepathy.ChannelDispatchOperation.Account>,
> Connection
>
> <http://telepathy.freedesktop.org/spec.html#org.freedesktop.Telepathy.ChannelDispatchOperation.Connection>
> and PossibleHandlers
>
> <http://telepathy.freedesktop.org/spec.html#org.freedesktop.Telepathy.ChannelDispatchOperation.PossibleHandlers>
> properties.
> If your CD version is giving you a different property, it's wrong, and the
> CD must be fixed. What CD (MC) version are you using? We can discuss this on
> #telepathy at freenode if you prefer.
I am using mission-control 5.1.0 (git checkout from the 5.1.0 tag). I
put a debug() << properties; statement there and I saw that it passes
all those properties prefixed with the interface name. That is:
"org.freedesktop.Telepathy.ChannelDispatchOperation.Account" (not just
"Account"),
"org.freedesktop.Telepathy.ChannelDispatchOperation.Connection" (not
just "Connection")
etc...
That looked correct to me, but maybe I am interpreting wrong the spec.
If you need any more details, ping me on irc tomorrow, I should be
around nearly all day (have in mind I am on GMT+3).
> Next time it would be great if you could use a git tree somewhere, it's
> easier to review the patches and to apply them.
Ok, I will do that if I need to fix anything else in the future :)
Best Regards,
George
More information about the telepathy
mailing list