#46496 [Opn->Csd]: [PATCH] wddx_serialize_value() treats input as ISO-8859-1

ID: 46496 Updated by: iliaa@php.net Reported By: mark at hell dot ne dot jp -Status: Open +Status: Closed Bug Type: WDDX related Operating System: Linux PHP Version: 5.2.6 New Comment: This bug has been fixed in CVS. Snapshots of the sources are packaged every three hours; this change will be in the next snapshot. You can grab the snapshot at http://snaps.php.net/. Thank you for the report, and for helping us make PHP better. Previous Comments: ------------------------------------------------------------------------ [2008-11-19 17:04:32] pajoye@php.net Reopen and add a note about the patch being committed to 5.2 (reviewed by Andrei). MFB will follow shortly. ------------------------------------------------------------------------ [2008-11-07 05:43:01] mark at hell dot ne dot jp Updated version of the patch, with a test fixed (test for bug #37569 depended on wddx_* functions accepting ISO-8859-1). http://ookoo.org/svn/snip/php-5.2.7rc3_wddx_utf8_resolved.patch compile runs nicely, make test does not report anything wrong, and the test for bug #37569 now also checks for this bug (ie. it checks that wddx_* functions indeed handle utf-8 as expected). ------------------------------------------------------------------------ [2008-11-06 05:00:49] mark at hell dot ne dot jp Here's a patch against PHP 5.2.7rc2 to fix this issue. The real problem is about WDDX always considering input is ISO-8859-1. This is not consistent with PHP <5.2.5, not consistent with the XML api, and not consistent with the documentation. http://ookoo.org/svn/snip/php-5.2.7rc2_wddx_utf8_resolved.patch ------------------------------------------------------------------------ [2008-11-06 04:54:54] mark at hell dot ne dot jp Tested and reproduced with PHP 5.2.7rc2 ------------------------------------------------------------------------ [2008-11-06 04:36:12] mark at hell dot ne dot jp Hello, I'd like this bug to be reopened. The &#xxx; are due to the PHP's bugtracker unability to display unicode characters. My report was initially written with korean string. Here's a screenshot of an UTF-8 terminal with the same test: http://beta.magicaltux.net/php5_bug_utf8_terminal.png ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at http://bugs.php.net/46496 -- Edit this bug report at http://bugs.php.net/?id=46496&edit=1