[Tickets #9567] Re: charset pb replying to message

[bugs at horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/9567
------------------------------------------------------------------------------
  Ticket             | 9567
  Updated By         | rsalmon at mbpgroup.com
  Summary            | charset pb replying to message
  Queue              | IMP
  Version            | Git master
  Type               | Bug
  State              | Feedback
  Priority           | 1. Low
  Milestone          |
  Patch              |
  Owners             | Michael Slusarz
+New Attachment     | Xss.tgz
------------------------------------------------------------------------------


rsalmon at mbpgroup.com (2011-03-07 09:22) wrote:

> So what you are saying is that BEFORE line 83, $charset is utf-8 and  
> $doc->encoding is iso-8859-1?  If that is the case, I don't see why  
> this isn't working... we are converting $text to ISO-8859-1 (from  
> UTF-8) and then sending to loadHTML.  So things should be fine.

I got mislead with editors charset. Depending on which one I was using  
(vi, nedit) I wasn't getting (seeing)  the same output, and I just  
realised that now.

So, this got me up to framework/Text_Filter/lib/Horde/Text/Filter/Xss.php

I've attached the log patch and horde log file. The log file is trace  
of replying to the message.

It looks like dom->saveXML returns UTF-8 characters.

If I change the last 'return' of function postProcess($text) like this
-        return Horde_String::convertCharset($text, $dom->encoding,  
$this->_params['charset']);
+        return $text;

Then accents look Ok!