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

[bugzilla-daemon at freedesktop.org]

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

--- Comment #8 from Xavier Claessens <xclaesse at gmail.com> 2012-05-14 06:36:32 UTC ---
(In reply to comment #7)
> (In reply to comment #6)
> > Eliminating avatar tokens might break MC's ability to set avatars nicely.
> 
> It doesn't, if the CM is required to provide a boolean indicator of whether
> avatars persist, and MC stores a boolean "avatar-changed-locally" in its
> Account data. Algorithm:

A "avatars-persist" property on Avatar iface makes perfect sense to me, good
idea.

> * when local user sets avatar, store it, store avatar-changed-locally = TRUE,
>   and (if online) call SetAvatar on the Connection
> * when connecting, if avatar-changed-locally || !avatars-persist, upload the
>   avatar to the CM with SetAvatar
> * when SetAvatar succeeds or fails (for either of those reasons), store
>   avatar-changed-locally = FALSE
> * when connecting, if !avatar-changed-locally && avatars-persist, download our
>   own avatar from the CM. Store it, and store avatar-changed-locally = FALSE.
> * when the CM signals that our own avatar has changed, also download it,
>   store it, and store avatar-changed-locally = FALSE.
> 
> I think that works?

That seems correct to me

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