# HG changeset patch # User ivan.komarov@soc.pidgin.im # Date 1272919754 0 # Node ID 984cb1d0ea5d8c5fa90afe2bb464ef86576ccfec # Parent bb6e0a21823ec691d9a1d9b51fb27648b1543086 Two small fixes to improve sending/receiving HTML-formatted messages over ICQ. * Treat all incoming messages as HTML because a) this is what the official client does and b) we don't have a reliable way to check if a message is HTML anyway. * Wrap outgoing HTML in proper HTML tags to make ICQ6 happy. This was tested with ICQ 6/7, Miranda, Trillian, QIP, and iChat. diff -r bb6e0a21823e -r 984cb1d0ea5d libpurple/protocols/oscar/oscar.c --- a/libpurple/protocols/oscar/oscar.c Sat May 01 18:32:43 2010 +0000 +++ b/libpurple/protocols/oscar/oscar.c Mon May 03 20:49:14 2010 +0000 @@ -1234,7 +1234,7 @@ else { if (purple_account_get_bool(account, "use_clientlogin", OSCAR_DEFAULT_USE_CLIENTLOGIN)) - { + {3 ClientInfo aiminfo = CLIENTINFO_PURPLE_AIM; ClientInfo icqinfo = CLIENTINFO_PURPLE_ICQ; flap_connection_send_version_with_cookie_and_clientinfo(od, @@ -2486,32 +2486,6 @@ tmp = g_string_free(message, FALSE); /* - * If the message is from an ICQ user and to an ICQ user then escape any HTML, - * because HTML is not sent over ICQ as a means to format a message. - * So any HTML we receive is intended to be displayed. Also, \r\n must be - * replaced with
- * - * Note: There *may* be some clients which send messages as HTML formatted - - * they need to be special-cased somehow. - * - * Update: Newer ICQ clients have started sending IMs as HTML. We can - * distinguish HTML IMs from non-HTML IMs by looking at the features. If - * the features are "0x 01 06" then the message is plain text. If the - * features are "0x 01" then the message is HTML. - */ - if (od->icq && oscar_util_valid_name_icq(userinfo->bn) - && (args->featureslen != 1 || args->features[0] != 0x01)) - { - /* being recevied by ICQ from ICQ - escape HTML so it is displayed as sent */ - gchar *tmp2 = g_markup_escape_text(tmp, -1); - g_free(tmp); - tmp = tmp2; - tmp2 = purple_strreplace(tmp, "\r\n", "
"); - g_free(tmp); - tmp = tmp2; - } - - /* * Convert iChat color tags to normal font tags. */ if (purple_markup_find_tag("body", tmp, &start, &end, &attribs)) @@ -4787,7 +4761,8 @@ tmp2 = purple_markup_strip_html(tmp1); is_html = FALSE; } else { - tmp2 = g_strdup(tmp1); + /* ICQ 6 wants its HTML wrapped in these tags. Oblige it. */ + tmp2 = g_strdup_printf("%s", tmp1); is_html = TRUE; } g_free(tmp1);