[Bug 33410] Do on-disk avatar cache in CM

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Wed Jan 25 00:28:40 CET 2012


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

--- Comment #3 from Xavier Claessens <xclaesse at gmail.com> 2012-01-24 15:28:40 PST ---
Have spoken about this on IRC, the plan is:

1) Redefine the Avatar iface in 'next' branch because keeping backward compat
is difficult here.

2) Avatar token does not need to be exposed directly to clients, since it's
only useful for caching. But CM should make sure string comparison of the URI
is enough to know the avatar image has change. I suggest just reuse the
existing cache from tp-glib format.

3) URI is preferred over filename, even if we are using only file:// because
path are "ay" on dbus and encoding is plateform specific, etc...
g_file_new_for_path() and g_file_get_uri() does the right thing™ for us.

4) We can use the client-interest mechanism to tell CM we are interested by
avatars, and the CM should request the data if not found in its cache. Like
that clients does not have to request them manually.

5) Still a RequestAvatars method is needed to force the fetch; for example in
the case of offline XMPP contacts where we don't have the token, some clients
may want to force the fetch of avatar the first time the contact is seen and
then keep that in its database (N9(00) use case).

-- 
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.


More information about the telepathy-bugs mailing list