[Pidgin] #4509: Support XMPP Invisibility
Pidgin
trac at pidgin.im
Mon Aug 17 18:39:57 EDT 2009
#4509: Support XMPP Invisibility
--------------------+-------------------------------------------------------
Reporter: js | Owner: deryni
Type: defect | Status: new
Milestone: | Component: XMPP
Version: 2.3.1 | Resolution:
Keywords: |
--------------------+-------------------------------------------------------
Comment(by deryni):
Enabling the shared status stuff means that pidgin status changes will not
be seen, as the server will rewrite all presence stanzas (at least
according to the documentation). Which means it will become impossible to
set a status message for the Google Talk account from pidgin, since that
will get rewritten by the server to the current shared message (why Google
Talk only overwrites the message and not the type I have no idea, but
that's what it looks like).
Also, it looks like this patch is going to break anyone who actually uses
the shared status stuff because according to the docs when you set a
shared status you need to push the status-list settings you last received
or you will clear them. What exactly those lists do I'm not at all sure as
the docs don't really say. They seem to indicate they are a list of
messages associated with the status but don't indicate how (or when) they
get used or why there are multiple entries.
Additionally, by way of patch specific comments. Manually setting <iq>
stanza id:s is a very bad idea, you shouldn't do it. It isn't clear to me
why you request the current shared status settings to then ignore (and
remove) most of them when you set them in the callback handler. And
lastly, jabber_send_status_request doesn't appear to be used at all.
--
Ticket URL: <http://developer.pidgin.im/ticket/4509#comment:29>
Pidgin <http://pidgin.im>
Pidgin
More information about the Tracker
mailing list