Bug 50085

Summary: [next] Consider removing InspectHandles/RequestHandles
Product: Telepathy Reporter: Xavier Claessens <xclaesse>
Component: tp-specAssignee: Telepathy bugs list <telepathy-bugs>
Status: RESOLVED FIXED QA Contact: Telepathy bugs list <telepathy-bugs>
Severity: normal    
Priority: medium Keywords: patch
Version: unspecified   
Hardware: Other   
OS: All   
URL: http://cgit.collabora.com/git/user/xclaesse/telepathy-spec.git/log/?h=next-handles
i915 platform: i915 features:
Bug Depends on:    
Bug Blocks: 23148    

Description Xavier Claessens 2012-05-18 04:38:35 UTC
They are now useless:

For CONTACT handles, we always get handle+id pairs and we have GetContactByID, so we should never have to inspect/request a contact handle.

Other types (actually only ROOM is remaining) are used for channels. In that case again we can set TARGET_ID in channel requests, and when we receive a channel we have both handle+id in its immutable properties.

So I see no use-case for InspectHandles/RequestHandles. Removing them is also a step forward to remove handles completely, even if we are still far from that.
Comment 1 Jonny Lamb 2012-06-07 07:02:33 UTC
Ah let's just do this.

+ SHOULD return the same thing that would be returned by GetContactByID
+ on a connected <tp:dbus-ref namespace="im.telepathy1"
+ >Connection</tp:dbus-ref>.</p>

tp:dbus-ref the GetContactByID please.

+ <p>The normalized contact's identifier. Not necessarily the same string
+ as passed to GetContactByID. As a special case,

Same thing.
Comment 2 Xavier Claessens 2012-06-07 07:24:31 UTC
fixed and merged. thanks.

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.