[patch] [ccla] "Strict" JSON parsing in Rhino, IE6-7 can't handle empty strings

[andybalaam]

When I add this simple test to the tests/json.js test, it fails on Rhino:

diff --git a/tests/json.js b/tests/json.js
index eb7c5f8..192b487 100644
--- a/tests/json.js
+++ b/tests/json.js
@@ -54,7 +54,9 @@ define(["../main", "doh/main", "../json"], function(dojo, doh, JSON){
 
                },
                /*Apparently Firefox doesn't pass the key to the toJSON method*/
-               function serializeToJSON(t){ t.is('{"foo":{"name":"value"}}', JSON.stringify({foo:{toJSON:function(key){return {name:"value"}; }}})); }
+               function serializeToJSON(t){ t.is('{"foo":{"name":"value"}}', JSON.stringify({foo:{toJSON:function(key){return {name:"value"}; }}})); },
+               /*Strict parsing was failing when an empty string was used*/
+               function emptyString(t){ t.is("", JSON.parse('{"foo":""}', true).foo); }
        ]);
 
 var smallDataSet = {

with an error like this:

$ java -jar util/shrinksafe/js.jar dojo/dojo.js baseUrl=file://$PWD/dojo load=doh test=dojo/tests/json

------------------------------------------------------------
The Dojo Unit Test Harness, $Rev: 23869 $
Copyright (c) 2011, The Dojo Foundation, All Rights Reserved
loading test dojo/tests/json
------------------------------------------------------------ 

37 tests to run in 2 groups
------------------------------------------------------------
GROUP "tests.json" has 31 tests to run
	Error: Invalid characters in JSON
	ERROR IN:
		 (function emptyString(t) {t.is("", JSON.parse("{\"foo\":\"\"}", true).foo);})
------------------------------------------------------------
GROUP "tests.json.performance" has 6 tests to run
------------------------------------------------------------
| TEST SUMMARY:
------------------------------------------------------------
	 37 tests in 2 groups
	 1 errors
	 0 failures

[andybalaam]

Note that due to ticket ​http://bugs.dojotoolkit.org/ticket/16254 I was unable to run this test at all with Rhino until I commented out the existing test serializeCircular().

[andybalaam]

[patch][ccla] allow-empty-strings-in-strict-json Andy Balaam (IBM, CCLA)

[bill]

I can't wrap my head around that regex, but anyway Kris should check it in; it's his code (from #8111).

[bill]

PS: I guess it doesn't have anything specifically to do w/Rhino, but fails on any system where JSON.parse isn't available?

[andybalaam]

Hi Bill, yes - this fails on IE7 as well.

[dylan]

Kris, can you review and get this into 1.9 please? Since this is likely the last IE 6/7 release, it would be nice to stabilize wherever possible.

[Kris Zyp]

In [30704]:

Allow empty strings in strict JSON parsing, fixes #16255 !strict