Req #69079 [Asn->Csd]: enhancement for mb_substitute_character

Edit report at https://bugs.php.net/bug.php?id=69079&edit=1 ID: 69079 Updated by: krakjoe@php.net Reported by: masakielastic at gmail dot com Summary: enhancement for mb_substitute_character -Status: Assigned +Status: Closed Type: Feature/Change Request Package: mbstring related PHP Version: 5.6.5 Assigned To: yohgaki Block user comment: N Private report: N New Comment: The fix for this bug has been committed. 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/. For Windows: http://windows.php.net/snapshots/ Thank you for the report, and for helping us make PHP better. Previous Comments: ------------------------------------------------------------------------ [2015-02-19 11:44:39] masakielastic at gmail dot com Description: ------------ It is not enough to check the range of argument of mb_substitute_character. Consider UTF-8. For example, 0x1F600 (GRINNING FACE) is rejected. 0xD800 is accepted despite of high surrogate. Test script: --------------- var_dump( mb_substitute_character(0x1F600), mb_substitute_character(0xD800) ); Expected result: ---------------- bool(true) bool(false) Actual result: -------------- bool(false) bool(true) ------------------------------------------------------------------------ -- Edit this bug report at https://bugs.php.net/bug.php?id=69079&edit=1