Add json_last_error_msg() function

This replaces json_last_error(true) and is consistent with other custom
error handling functions.

diff --git a/ext/json/json.c b/ext/json/json.c
index 5e0351f3f1e0df130eab02e87bf89422e4adb0c9..4d29a66688640f9a12300a8817a10dd96de32940 100644 (file)
--- a/ext/json/json.c
+++ b/ext/json/json.c
@@ -34,6 +34,7 @@ static PHP_MINFO_FUNCTION(json);
 static PHP_FUNCTION(json_encode);
 static PHP_FUNCTION(json_decode);
 static PHP_FUNCTION(json_last_error);
+static PHP_FUNCTION(json_last_error_msg);
 
 static const char digits[] = "0123456789abcdef";
 
@@ -51,8 +52,10 @@ ZEND_BEGIN_ARG_INFO_EX(arginfo_json_decode, 0, 0, 1)
        ZEND_ARG_INFO(0, depth)
 ZEND_END_ARG_INFO()
 
-ZEND_BEGIN_ARG_INFO_EX(arginfo_json_last_error, 0, 0, 0)
-       ZEND_ARG_INFO(0, as_string)
+ZEND_BEGIN_ARG_INFO(arginfo_json_last_error, 0)
+ZEND_END_ARG_INFO()
+
+ZEND_BEGIN_ARG_INFO(arginfo_json_last_error_msg, 0)
 ZEND_END_ARG_INFO()
 /* }}} */
 
@@ -61,6 +64,7 @@ static const function_entry json_functions[] = {
        PHP_FE(json_encode, arginfo_json_encode)
        PHP_FE(json_decode, arginfo_json_decode)
        PHP_FE(json_last_error, arginfo_json_last_error)
+       PHP_FE(json_last_error_msg, arginfo_json_last_error_msg)
        PHP_FE_END
 };
 /* }}} */
@@ -607,21 +611,25 @@ static PHP_FUNCTION(json_decode)
 /* }}} */
 
 /* {{{ proto int json_last_error()
-   Returns the error code of the last json_decode(). */
+   Returns the error code of the last json_encode() or json_decode() call. */
 static PHP_FUNCTION(json_last_error)
 {
-       zend_bool as_string = 0;
-
-       if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "|b", &as_string) == FAILURE) {
+       if (zend_parse_parameters_none() == FAILURE) {
                return;
        }
 
-       /* return error code (JSON_ERROR_* constants) */
-       if (!as_string) {
-               RETURN_LONG(JSON_G(error_code));
+       RETURN_LONG(JSON_G(error_code));
+}
+/* }}} */
+
+/* {{{ proto string json_last_error_msg()
+   Returns the error string of the last json_encode() or json_decode() call. */
+static PHP_FUNCTION(json_last_error_msg)
+{
+       if (zend_parse_parameters_none() == FAILURE) {
+               return;
        }
 
-       /* return error message (for debugging purposes) */
        switch(JSON_G(error_code)) {
                case PHP_JSON_ERROR_NONE:
                        RETURN_STRING("No error", 1);
@@ -644,6 +652,7 @@ static PHP_FUNCTION(json_last_error)
                default:
                        RETURN_STRING("Unknown error", 1);
        }
+
 }
 /* }}} */
 

diff --git a/ext/json/tests/003.phpt b/ext/json/tests/003.phpt
index 71874525a701b2f1b88774f87830af354b42a133..4ce5b0fde9d0c161b6bf945e3a9af9c0c47c265f 100644 (file)
--- a/ext/json/tests/003.phpt
+++ b/ext/json/tests/003.phpt
@@ -13,14 +13,12 @@ var_dump($a);
 echo "\n";
 
 var_dump(json_encode($a));
-var_dump(json_last_error());
-var_dump(json_last_error(true));
+var_dump(json_last_error(), json_last_error_msg());
 
 echo "\n";
 
 var_dump(json_encode($a, JSON_PARTIAL_OUTPUT_ON_ERROR));
-var_dump(json_last_error());
-var_dump(json_last_error(true));
+var_dump(json_last_error(), json_last_error_msg());
 
 echo "Done\n";
 ?>

diff --git a/ext/json/tests/004.phpt b/ext/json/tests/004.phpt
index 49c543edcae6dcb06d83bf85dd7eb1ad4e2b5d1d..70ef3ffd1b80ff3f1670eea2bbbb7d037fa1f6b1 100644 (file)
--- a/ext/json/tests/004.phpt
+++ b/ext/json/tests/004.phpt
@@ -13,14 +13,12 @@ var_dump($a);
 echo "\n";
 
 var_dump(json_encode($a));
-var_dump(json_last_error());
-var_dump(json_last_error(true));
+var_dump(json_last_error(), json_last_error_msg());
 
 echo "\n";
 
 var_dump(json_encode($a, JSON_PARTIAL_OUTPUT_ON_ERROR));
-var_dump(json_last_error());
-var_dump(json_last_error(true));
+var_dump(json_last_error(), json_last_error_msg());
 
 echo "Done\n";
 ?>

diff --git a/ext/json/tests/007.phpt b/ext/json/tests/007.phpt
index 9ee190a24cdc9059fa29d33a3103c837ced04b35..7557ac9ed7497c916c50b43ccb92354f61965ac3 100644 (file)
--- a/ext/json/tests/007.phpt
+++ b/ext/json/tests/007.phpt
@@ -5,15 +5,15 @@ json_last_error() tests
 --FILE--
 <?php
 var_dump(json_decode("[1]"));
-var_dump(json_last_error());
+var_dump(json_last_error(), json_last_error_msg());
 var_dump(json_decode("[[1]]", false, 2));
-var_dump(json_last_error());
+var_dump(json_last_error(), json_last_error_msg());
 var_dump(json_decode("[1}"));
-var_dump(json_last_error());
+var_dump(json_last_error(), json_last_error_msg());
 var_dump(json_decode('["' . chr(0) . 'abcd"]'));
-var_dump(json_last_error());
+var_dump(json_last_error(), json_last_error_msg());
 var_dump(json_decode("[1"));
-var_dump(json_last_error());
+var_dump(json_last_error(), json_last_error_msg());
 
 
 echo "Done\n";
@@ -24,13 +24,17 @@ array(1) {
   int(1)
 }
 int(0)
+string(8) "No error"
 NULL
 int(1)
+string(28) "Maximum stack depth exceeded"
 NULL
 int(2)
+string(42) "State mismatch (invalid or malformed JSON)"
 NULL
 int(3)
+string(53) "Control character error, possibly incorrectly encoded"
 NULL
 int(4)
+string(12) "Syntax error"
 Done
-

diff --git a/ext/json/tests/bug54058.phpt b/ext/json/tests/bug54058.phpt
index 2c2304578e21361a1984886622148b238f1c23ac..df1b3130f8422a82e9dcdb724d0a46477ed5ef52 100644 (file)
--- a/ext/json/tests/bug54058.phpt
+++ b/ext/json/tests/bug54058.phpt
@@ -8,29 +8,25 @@ Bug #54058 (json_last_error() invalid UTF-8 produces wrong error)
 $bad_utf8 = quoted_printable_decode('=B0');
 
 json_encode($bad_utf8);
-var_dump(json_last_error());
-var_dump(json_last_error(true));
+var_dump(json_last_error(), json_last_error_msg());
 
 $a = new stdclass;
 $a->foo = quoted_printable_decode('=B0');
 json_encode($a);
-var_dump(json_last_error());
-var_dump(json_last_error(true));
+var_dump(json_last_error(), json_last_error_msg());
 
 $b = new stdclass;
 $b->foo = $bad_utf8;
 $b->bar = 1;
 json_encode($b);
-var_dump(json_last_error());
-var_dump(json_last_error(true));
+var_dump(json_last_error(), json_last_error_msg());
 
 $c = array(
     'foo' => $bad_utf8,
     'bar' => 1
 );
 json_encode($c);
-var_dump(json_last_error());
-var_dump(json_last_error(true));
+var_dump(json_last_error(), json_last_error_msg());
 
 ?>
 --EXPECTF--

diff --git a/ext/json/tests/bug61537.phpt b/ext/json/tests/bug61537.phpt
index f6bb02bae451dc81932bb4042286cebd309d26cb..80ed051c9a6d50c4e9e7eb7682cce3406d8f55ac 100644 (file)
--- a/ext/json/tests/bug61537.phpt
+++ b/ext/json/tests/bug61537.phpt
@@ -7,20 +7,20 @@ Bug #61537 (json_encode() incorrectly truncates/discards information)
 $invalid_utf8 = "\x9f";
 
 var_dump(json_encode($invalid_utf8));
-var_dump(json_last_error(), json_last_error(true));
+var_dump(json_last_error(), json_last_error_msg());
 
 var_dump(json_encode($invalid_utf8, JSON_PARTIAL_OUTPUT_ON_ERROR));
-var_dump(json_last_error(), json_last_error(true));
+var_dump(json_last_error(), json_last_error_msg());
 
 echo "\n";
 
 $invalid_utf8 = "an invalid sequen\xce in the middle of a string";
 
 var_dump(json_encode($invalid_utf8));
-var_dump(json_last_error(), json_last_error(true));
+var_dump(json_last_error(), json_last_error_msg());
 
 var_dump(json_encode($invalid_utf8, JSON_PARTIAL_OUTPUT_ON_ERROR));
-var_dump(json_last_error(), json_last_error(true));
+var_dump(json_last_error(), json_last_error_msg());
 
 ?>
 --EXPECTF--

diff --git a/ext/json/tests/inf_nan_error.phpt b/ext/json/tests/inf_nan_error.phpt
index f12e902d9f0e226f4fda969751cc9fd2d93ef41c..f9deecc4699551e7675850066c43794970ef767c 100644 (file)
--- a/ext/json/tests/inf_nan_error.phpt
+++ b/ext/json/tests/inf_nan_error.phpt
@@ -1,5 +1,7 @@
 --TEST--
 An error is thrown when INF or NaN are encoded
+--SKIPIF--
+<?php if (!extension_loaded("json")) print "skip"; ?>
 --FILE--
 <?php
 
@@ -8,10 +10,10 @@ $inf = INF;
 var_dump($inf);
 
 var_dump(json_encode($inf));
-var_dump(json_last_error(), json_last_error(true));
+var_dump(json_last_error(), json_last_error_msg());
 
 var_dump(json_encode($inf, JSON_PARTIAL_OUTPUT_ON_ERROR));
-var_dump(json_last_error(), json_last_error(true));
+var_dump(json_last_error(), json_last_error_msg());
 
 echo "\n";
 
@@ -20,10 +22,10 @@ $nan = NAN;
 var_dump($nan);
 
 var_dump(json_encode($nan));
-var_dump(json_last_error(), json_last_error(true));
+var_dump(json_last_error(), json_last_error_msg());
 
 var_dump(json_encode($nan, JSON_PARTIAL_OUTPUT_ON_ERROR));
-var_dump(json_last_error(), json_last_error(true));
+var_dump(json_last_error(), json_last_error_msg());
 ?>
 --EXPECTF--
 float(INF)

diff --git a/ext/json/tests/unsupported_type_error.phpt b/ext/json/tests/unsupported_type_error.phpt
index f36afb44a5e90872dfae0781580e6ac004da2977..45a167a5ac0b158a3c06fe3bd769a1a0030f8a66 100644 (file)
--- a/ext/json/tests/unsupported_type_error.phpt
+++ b/ext/json/tests/unsupported_type_error.phpt
@@ -1,5 +1,7 @@
 --TEST--
 An error is thrown when an unsupported type is encoded
+--SKIPIF--
+<?php if (!extension_loaded("json")) print "skip"; ?>
 --FILE--
 <?php
 
@@ -8,10 +10,10 @@ $resource = fopen(__FILE__, "r");
 var_dump($resource);
 
 var_dump(json_encode($resource));
-var_dump(json_last_error(), json_last_error(true));
+var_dump(json_last_error(), json_last_error_msg());
 
 var_dump(json_encode($resource, JSON_PARTIAL_OUTPUT_ON_ERROR));
-var_dump(json_last_error(), json_last_error(true));
+var_dump(json_last_error(), json_last_error_msg());
 
 ?>
 --EXPECTF--