[Bug 32874] Offline contacts does not have capabilities

[bugzilla-daemon at freedesktop.org]

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

--- Comment #3 from Jonny Lamb <jonny.lamb at collabora.co.uk> 2011-01-10 08:26:02 PST ---
(In reply to comment #2)
> +      DEBUG ("don't know %u's presence; assuming text chat caps.", handle);
> 
> I don't like telling you assume text caps there as it's a decision of
> im_factory. So if we change there, likely that debug will become wrong.

Well, right, I agree -- if the IM factory behaviour changes, this will break,
which is why I added the big NOTE in that function. I also think the test is
strict enough to only allow the text caps, isn't it?

> It is not unlikely that at some point Call cap will be added for offline
> contacts, for voicemail. AFAIK Skype does it already.

Sure. At that point of course we'll have to change this behaviour, but it's not
so bad now, is it? Perhaps we could add a
GabbleCapsChannelManagerClass->get_assumed_caps vfunc so we can iterate through
the channel managers and get text caps from im-factory and Call caps from
call-factory, but that can be done later, no?

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