[Bug 38248] Get attributes for contacts related to a channel

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=38248

--- Comment #3 from Xavier Claessens <xclaesse at gmail.com> 2011-06-27 04:37:07 PDT ---
I'm looking into this again, and we really need (handle,id) pairs to create
TpContact, IMO.

On Channel main interface we are good because we have (TargetHandle, TargetID)
and (InitiatorHandle, InitiatorID) pairs and they are immutable. So the best is
probably to not touch that interface at all.

On Group interface though, we have issues because none of those properties
gives the corresponding ids: HandleOwners, LocalPendingMembers, Members,
RemotePendingMembers and SelfHandle. Also SelfHandleChanged and
HandleOwnersChanged does not give ids. The good news is MembersChangedDetailed
already gives the ids.

>From this, how do we fix Group interface?

1) extend current properties: Add HandleOwnersIds, LocalPendingMembersIds
(contains also actors), MembersIds, RemotePendingMembersIds, SelfHandleId
properties (with type a{us}). Also add HandleOwnersChangedDetailed and
SelfHandleChangedDetailed which gives the ids via a a{us} or even a a{sv} just
like the details in MembersChangedDetailed.

2) like 1, but introduce only 1 new property ids:a{us} which gives the mapping
for all handles in all properties (including also local-pending actors).
HandleOwnersChangedDetailed and SelfHandleChangedDetailed would still be
needed.

3) deprecate current properties and create new ones that gives directly the
ids. For example Members2 : a{us}. Note that local-pending would also give id
for the actor.

4) resurect GetAllMembers but gives contact attributes, just like
GetContactListAttributes. Note that would still need a solution for SelfHandle
and HandleOwners.

5) suggestions?

if we goes for 3) or 4) and since we already have lots of deprecated stuff,
maybe create a new iface SimpleGroup ?

My personal preference is solution 4 (consistent with ContactList iface)
together with additional SelfID:s and OwndersID:a{us} properties (consistent
with Channel iface) and their respective Detailed signal which gives an a{sv}
(consistent with MembersChangedDetailed).

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.