Bug #69187 [Asn]: json_last_error return BC in PHP7

Edit report at https://bugs.php.net/bug.php?id=69187&edit=1 ID: 69187 Updated by: bukka@php.net Reported by: hdtfonseca at gmail dot com Summary: json_last_error return BC in PHP7 Status: Assigned Type: Bug Package: JSON related Operating System: linux PHP Version: master-Git-2015-03-04 (Git) Assigned To: bukka Block user comment: N Private report: N New Comment: Hi Adam, this is actually caused by your commit ( https://github.com/php/php-src/commit/a7b3abe4e6f5e2fdfd8d55b676c9ca6b3f9c8cc8 ) :). The reason is the null is converted to empty string... The same will be for false. I think that it makes sense to not emit error for null or false but only for empty string. json_decode already returns number when you pass number (json_decode(1)) so it makes sense to have it consistent. There is still a small inconsistency with boolean - json_decode(false) returns NULL and json_decode(true) returns 1. However I'm not sure if it's worthy it to break BC (even slightly)... I will apply the patch for NULL and FALSE in a couple days if no objections. Previous Comments: ------------------------------------------------------------------------ [2015-03-04 19:36:46] aharvey@php.net Jakub, is this expected, and if so, can you update UPGRADING with the situations where this will happen, please? ------------------------------------------------------------------------ [2015-03-04 18:13:44] hdtfonseca at gmail dot com Description: ------------ The output of the function json_last_error() is different in PHP master in some situations and this is causing a BC break. Test script: --------------- json_decode(null, true); echo json_last_error(); Expected result: ---------------- 0 Actual result: -------------- 4 ------------------------------------------------------------------------ -- Edit this bug report at https://bugs.php.net/bug.php?id=69187&edit=1