]> source.dussan.org Git - jquery.git/commitdiff
Make sure leading whitespace is trimmed for parseJSON. Fixes #6031.
authorjeresig <jeresig@gmail.com>
Sat, 13 Feb 2010 07:14:23 +0000 (02:14 -0500)
committerjeresig <jeresig@gmail.com>
Sat, 13 Feb 2010 07:14:23 +0000 (02:14 -0500)
src/core.js
test/unit/core.js

index 02d8269d215a3547238cbb3d7b82a68a218b54bc..9fa311d506ccef41dc3bd47458fdeccd02452751 100644 (file)
@@ -480,6 +480,9 @@ jQuery.extend({
                if ( typeof data !== "string" || !data ) {
                        return null;
                }
+
+               // Make sure leading/trailing whitespace is removed (IE can't handle it)
+               data = jQuery.trim( data );
                
                // Make sure the incoming data is actual JSON
                // Logic borrowed from http://json.org/json2.js
index 4576ab8eb6a07e4baa2d480f02ecd5a2fb210409..d8aba16db0d856efb8a39950fac358161000f976 100644 (file)
@@ -807,7 +807,7 @@ test("jQuery.proxy", function(){
 });
 
 test("jQuery.parseJSON", function(){
-       expect(7);
+       expect(8);
        
        equals( jQuery.parseJSON(), null, "Nothing in, null out." );
        equals( jQuery.parseJSON( null ), null, "Nothing in, null out." );
@@ -815,6 +815,8 @@ test("jQuery.parseJSON", function(){
        
        same( jQuery.parseJSON("{}"), {}, "Plain object parsing." );
        same( jQuery.parseJSON('{"test":1}'), {"test":1}, "Plain object parsing." );
+
+       same( jQuery.parseJSON('\n{"test":1}'), {"test":1}, "Make sure leading whitespaces are handled." );
        
        try {
                jQuery.parseJSON("{a:1}");
@@ -829,4 +831,4 @@ test("jQuery.parseJSON", function(){
        } catch( e ) {
                ok( true, "Test malformed JSON string." );
        }
-});
\ No newline at end of file
+});