[Bug 28200] TpBaseContactList

[bugzilla-daemon at freedesktop.org]

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

--- Comment #37 from Simon McVittie <simon.mcvittie at collabora.co.uk> 2010-07-22 04:14:15 PDT ---
(In reply to comment #36)
> > > Alternatively, CreateChannel and EnsureChannel could call
> > > set_group_members_async(group, []) or add_to_group_async(group, []) (which the
> > > subclass has to support anyway, because they can be called from D-Bus),
> > > removing the need for an explicit create method.
> > Is the create method something that's going to disappear in telepathy-1.0 or
> > something? If so, avoiding it entirely sounds like the way forwards.

It's currently necessary to be able to say to an implementation "create this
group, with no other side-effects", to implement CreateChannel and
EnsureChannel for GROUP channels. However, when we remove the old
Chan.T.ContactList channels, and just use these new Conn.I.ContactList and
Conn.I.ContactGroups interfaces, the need to do that will mostly go away.

There's currently no Conn.I.ContactGroups.CreateGroup(s: Group) in the D-Bus
API, because I don't think anyone will call it, and if someone does need to
create a group without members, it's trivial to call AddToGroup(group, []) or
SetGroupMembers(group, []) (depending on the behaviour they want if they race
with someone else creating the same group). See also the 42nd comment in Bug
#21787.

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