Hi,

Thanks for the patch.
I applyed it on the bugfixe branch and on the 0.78 one.

https://forge.indepnet.net/projects/glpi/repository/revisions/11400*

Regards

Julien



Denis Linvinus a écrit :
> Hello glpi-dev team!
> I have found bug in 0.72.4.
> After applying my patch for decoding email, previous decoding 
> mechanism also recode some emails.
>
> First decoding occur in mailgate.class.php in function get_part at
>
> if (count($structure->parameters)>0){
>                                   foreach ($structure->parameters as 
> $param){
>                                     if 
> ((strtoupper($param->attribute)=='CHARSET') && 
> function_exists('mb_convert_encoding') && strtoupper($param->value) != 
> 'UTF-8'){
>                                        $text = 
> mb_convert_encoding($text, 'utf-8',$param->value); // <<<<<first decoding
>                                        }
>                                    }
>                                 }
>
> and second in same file in function buildTicket at
>
>                 $body=$this->getBody($i);
>
>                
>                 // Do it before using charset variable
>                 
> $head['subject']=$this->decodeMimeString($head['subject']);
>
>                 if 
> (!empty($this->charset)&&function_exists('mb_convert_encoding')){
>                         $body=mb_convert_encoding($body, 
> 'utf-8',$this->charset); //<<<<<<<second recoding
>                 }
>
> because $this->charset filled from headers.
> in attach patch for glpi 0.72.4 it add boolean variable 
> "body_converted" to check decoding state.
> and second attached file, it is email with problem present.
>
> ------------------------------------------------------------------------
>
> _______________________________________________
> Glpi-dev mailing list
> Glpi-dev@gna.org
> https://mail.gna.org/listinfo/glpi-dev


_______________________________________________
Glpi-dev mailing list
Glpi-dev@gna.org
https://mail.gna.org/listinfo/glpi-dev

Reply via email to