Mercurial > pidgin.yaz
comparison libpurple/protocols/jabber/presence.c @ 17814:d495103dcf88
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.
author | Mark Doliner <mark@kingant.net> |
---|---|
date | Sun, 10 Jun 2007 15:58:23 +0000 |
parents | 3919c68eb5b7 |
children | 6e0961356343 |
comparison
equal
deleted
inserted
replaced
17812:f2869d5facfe | 17814:d495103dcf88 |
---|---|
606 /* if the message is blank, then there really isn't a message */ | 606 /* if the message is blank, then there really isn't a message */ |
607 if(formatted_msg && !*formatted_msg) | 607 if(formatted_msg && !*formatted_msg) |
608 formatted_msg = NULL; | 608 formatted_msg = NULL; |
609 | 609 |
610 if(formatted_msg) | 610 if(formatted_msg) |
611 purple_markup_html_to_xhtml(formatted_msg, NULL, msg); | 611 *msg = purple_markup_strip_html(formatted_msg); |
612 } | 612 } |
613 | 613 |
614 if(priority) | 614 if(priority) |
615 *priority = purple_status_get_attr_int(status, "priority"); | 615 *priority = purple_status_get_attr_int(status, "priority"); |
616 } | 616 } |