Revision 49fd2d3743c303820668e5566f9513c3a16bca71

markdoliner at pidgin.im markdoliner at pidgin.im
Sun Jun 10 11:59:36 EDT 2007


-----------------------------------------------------------------
Revision: 49fd2d3743c303820668e5566f9513c3a16bca71
Ancestor: 8f74e2d74748121b6c3c039a04e295abd94b35a0
Author: markdoliner at pidgin.im
Date: 2007-06-10T15:58:23
Branch: im.pidgin.pidgin

Modified files:
        libpurple/protocols/jabber/libxmpp.c
        libpurple/protocols/jabber/presence.c

ChangeLog: 

Two questions:
1. XMPP <status> messages inside <presence> broadcasts should be
   plaintext and not HTML, right?
2. 'stripped = purple_markup_strip_html(str)' is basically the same as
   'purple_markup_html_to_xhtml(str, NULL, &stripped)', right?

This fixes a bug Emily found where setting an XMPP status message with
an embedded link would drop the link completely.  For example, in
Pidgin you set your away message and insert a link so that the message
body is <a href="http://www.example.com/">Example</a>.  When we set
this message for an XMPP account it just strips the HTML and sets the
message to "Example".  This change causes the message to be
"Example (http://www.example.com/)".  It's the same thing we do when
displaying links in tooltips in the blist.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 49fd2d3743c303820668e5566f9513c3a16bca71.diff
Type: text/x-diff
Size: 1080 bytes
Desc: not available
Url : http://pidgin.im/pipermail/commits/attachments/20070610/439453a1/attachment-0001.diff 


More information about the Commits mailing list