[Bug 28200] TpBaseContactList

[bugzilla-daemon at freedesktop.org]

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

--- Comment #9 from Simon McVittie <simon.mcvittie at collabora.co.uk> 2010-06-03 11:29:42 PDT ---
I've pushed more fixes to smcv/contact-list-manager.

If you're feeling really enthusiastic about reviewing,
smcv/contact-list-new-signals and smcv/contact-list-new-methods additionally
implement the new APIs.

"""
AuthorizePublication (au: Contacts) → nothing

For contacts with publish=No, [...] it merely records the fact that presence
publication to those contacts is allowed; if any of those contacts ask to
receive the local user's presence later [...] the connection SHOULD immediately
allow them to do so, changing their publish attribute directly from No to Yes.
"""

Note to self: I haven't implemented those semantics in the example yet. When I
do, it should probably be rebased in ahead of the new signals and methods.

Other things still to do:

The various accessor methods are inconsistent about whether virtual methods can
raise an error. I think they should all never fail, tbh.

It might be good to move various "compliance levels" of groups of virtual
methods onto GInterfaces:

  * can list contacts (basic, required functionality - put in the class?)
  * can alter ContactList (all methods mandatory)
  * can list groups (mandatory getters; optional disjoint_groups)
  * can alter ContactGroups (mandatory set/add/remove and get_group_storage;
optional rename)
  * can alter Blocking (there seems no point in having a separate read-only
version)

That would mean we wouldn't necessarily need the various _implement_...()
methods, and tomeu would be happy.

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