Array named properties are not included in json string

[Krinkle]

From Huixian.Cui on August 03, 2012 07:43:50

I changed the code:

if (Object.prototype.toString.call(o[k]) === '[object Array]') {
    var arrayItem = [];

    for (var i = 0; i < o[k].length; i++) {
       arrayItem.push($.toJSON(o[k][i]));
    }

    val = '[' + arrayItem.join(',') + ']';
    pairs.push(name + ':' + val);
}
else {
    val = $.toJSON(o[k]);
    pairs.push(name + ':' + val);
}

Original issue: http://code.google.com/p/jquery-json/issues/detail?id=62

[Krinkle]

From krinklemail on November 13, 2012 17:41:23

Please provide a test case for the test suite (which would fail against the current code).

[Krinkle]

From dzlygin on May 19, 2013 05:34:04

var conf = new Array();
conf['key1'] = {text: 'sometext', posx: 50, posy: 50};
conf['key2'] = {text: 'othertext', posx: 50, posy: 80};

var a = $.toJSON(conf);

this code fails

[Krinkle]

This is not a valid bug for the native JSON.stringify method exhibits the same behaviour and we're just polyfilling that. We can't take feature requests.

Per the ES5 specification, when serialising an Array object:

https://es5.github.io/#x15.12.3

For each value a property [that] has an array index property name. The properties are enumerated in the ascending array index order of their names.

https://es5.github.io/#JA

The representation of arrays includes only the elements between zero and array.length – 1 inclusive. Named properties are excluded from the stringification.

I recommend you do not be add named properties to array objects in your program. You're probably confusing arrays with objects (other programming languages sometimes call them arrays).

var conf = {};
conf.key1 = { text: 'sometext', posx: 50, posy: 50 };
conf.key2 = { text: 'othertext', posx: 50, posy: 80 };

var a = $.toJSON(conf);
// {"key1":{"text":"sometext","posx":50,"posy":50},"key2":{"text":"othertext","posx":50,"posy":80}}