[Bug 24061] TpAccount, TpAccountManager: add convenience API similar to libempathy's

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Thu Sep 24 19:33:53 CEST 2009


http://bugs.freedesktop.org/show_bug.cgi?id=24061





--- Comment #12 from Simon McVittie <simon.mcvittie at collabora.co.uk>  2009-09-24 10:33:52 PST ---
(In reply to comment #11)
> > I removed it, and added a new property on TpAccount called default-presence.
>
> (Without looking at your implementation I don't even know what a
> default-presence property on TpAccount would do...)

Having looked at the implementation I'm pretty sure it's not useful...

> "the default presence that should be set on the account when it becomes enabled"

... since this is just RequestedPresence in disguise, but without syncing up
over D-Bus. (MC will allow RequestedPresence to be set on disabled accounts,
and will honour RequestedPresence when an account is Enabled.)

The feature that I think Sjoerd wants is Bug #24104, which needs extra D-Bus
API to do correctly, and isn't adequately covered by this default-presence
property anyway.

Until Bug #24104 is fixed, Empathy can emulate
tp_account_manager_get_requested_global_presence by having a global variable
(or something equivalent, like a member of a singleton) in which it stores its
idea of the global presence, and using that global variable as the presence to
request on newly created accounts. That doesn't work unless the account and
presence UIs are in the same process, but neither does the API you had
initially, so it's not a regression.


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



More information about the telepathy-bugs mailing list