Probably I couldn't explain well.
When I tried to apply a fix to that text which I use (1.1.4), I couldn't
make it.
Those lines which shall be replaced according to a fix in 1.1.4 had
significantly other appearance. In particular, there was an additional
operator if.
But I understood the idea, thanks. If I deal still with this issue, I
will consider your words.
---
Best regards,
Vladimir Gorpenko
A.L.E.C писал 2016-09-27 18:49:
On 09/27/2016 05:37 PM, Vladimir Gorpenko wrote:
// return if encoding found, string matches encoding and
convert succeeded
if (in_array($mb_from, $mbstring_list) && in_array($mb_to,
$mbstring_list)) {
if (mb_check_encoding($str, $mb_from)) {
// Do the same as //IGNORE with iconv
mb_substitute_character('none');
$out = mb_convert_encoding($str, $mb_to,
$mb_from);
mb_substitute_character($mbstring_sch);
if ($out !== false) {
return $out;
}
}
}
I don't decide to adapt your fix to my rcube_charset version.
In general mb_list_encodings() and mb_check_encoding() is not used now.
I did some more test and indeed mb_check_encoding() does not work with
'GBK', but mb_convert_encoding() does (at least with sample text I've
got). So, I assume current git-master code will work for you as well.
_______________________________________________
Roundcube Development discussion mailing list
dev@lists.roundcube.net
http://lists.roundcube.net/mailman/listinfo/dev
