Changeset 1323

Show
Ignore:
Timestamp:
12/14/07 12:10:52 (7 months ago)
Author:
bob@redivi.com
Message:

fix for parseQueryString bug #284

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • mochikit/trunk/MochiKit/Base.js

    r1319 r1323  
    11641164            ? encodedString.substring(1) 
    11651165            : encodedString; 
    1166         var pairs = qstr.replace(/\+/g, "%20").split(/(\&amp\;|\&\#38\;|\&|\&)/); 
     1166        var pairs = qstr.replace(/\+/g, "%20").split(/\&amp\;|\&\#38\;|\&|\&/); 
    11671167        var o = {}; 
    11681168        var decode; 

  • mochikit/trunk/packed/MochiKit/MochiKit.js

    r1321 r1323  
    901901},parseQueryString:function(_136,_137){ 
    902902var qstr=(_136.charAt(0)=="?")?_136.substring(1):_136; 
    903 var _139=qstr.replace(/\+/g,"%20").split(/(\&amp\;|\&\#38\;|\&|\&)/); 
     903var _139=qstr.replace(/\+/g,"%20").split(/\&amp\;|\&\#38\;|\&|\&/); 
    904904var o={}; 
    905905var _13b; 

  • mochikit/trunk/tests/test_Base.js

    r1303 r1323  
    390390    t.is( res["foo"], "2", "parseQueryString strip leading question mark"); 
    391391 
     392    var res = parseQueryString("x=1&y=2"); 
     393    t.is( typeof(res['&']), "undefined", "extra cruft in parseQueryString output"); 
     394 
    392395    t.is( serializeJSON("foo\n\r\b\f\t"), "\"foo\\n\\r\\b\\f\\t\"", "string JSON" ); 
    393396    t.is( serializeJSON(null), "null", "null JSON");