Changeset 1390

Show
Ignore:
Timestamp:
06/23/08 23:58:15 (2 months ago)
Author:
cederberg@gmail.com
Message:

Fixed escaping of vertical tab character in repr for strings (also affected MochiKit.Base.serializeJSON). Fixes #313.

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • mochikit/trunk/MochiKit/Base.js

    r1365 r1390  
    801801            ).replace(/[\n]/g, "\\n" 
    802802            ).replace(/[\t]/g, "\\t" 
     803            ).replace(/[\v]/g, "\\v" 
    803804            ).replace(/[\r]/g, "\\r"); 
    804805    }, 

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

    r1389 r1390  
    630630return "["+m.map(m.repr,o).join(", ")+"]"; 
    631631},reprString:function(o){ 
    632 return ("\""+o.replace(/(["\\])/g,"\\$1")+"\"").replace(/[\f]/g,"\\f").replace(/[\b]/g,"\\b").replace(/[\n]/g,"\\n").replace(/[\t]/g,"\\t").replace(/[\r]/g,"\\r"); 
     632return ("\""+o.replace(/(["\\])/g,"\\$1")+"\"").replace(/[\f]/g,"\\f").replace(/[\b]/g,"\\b").replace(/[\n]/g,"\\n").replace(/[\t]/g,"\\t").replace(/[\v]/g,"\\v").replace(/[\r]/g,"\\r"); 
    633633},reprNumber:function(o){ 
    634634return o+""; 

  • mochikit/trunk/tests/test_Base.js

    r1323 r1390  
    393393    t.is( typeof(res['&']), "undefined", "extra cruft in parseQueryString output"); 
    394394 
    395     t.is( serializeJSON("foo\n\r\b\f\t"), "\"foo\\n\\r\\b\\f\\t\"", "string JSON" ); 
     395    t.is( serializeJSON("foo\n\r\b\f\t\v"), "\"foo\\n\\r\\b\\f\\t\\v\"", "string JSON" ); 
    396396    t.is( serializeJSON(null), "null", "null JSON"); 
    397397    try {