Hopiu/angular.js
1
0
Fork 0
mirror of https://github.com/Hopiu/angular.js.git synced 2026-05-10 07:44:43 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix JSON to match native browser behavior

Browse source
This commit is contained in:
Misko Hevery 2011-03-30 16:28:28 -07:00
parent f57536ddb6
commit 26e651996a
2 changed files with 6 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
src/JSON.js
View file

@ -128,9 +128,9 @@ function toJsonArray(buf, obj, pretty, stack) {
var childPretty = pretty ? pretty + " " : false; var childPretty = pretty ? pretty + " " : false;
var keys = []; var keys = [];
for(var k in obj) { for(var k in obj) {
if (obj[k] === undefined) if (obj.hasOwnProperty(k) && obj[k] !== undefined) {
continue; keys.push(k);
keys.push(k); }
} }
keys.sort(); keys.sort();
for ( var keyIndex = 0; keyIndex < keys.length; keyIndex++) { for ( var keyIndex = 0; keyIndex < keys.length; keyIndex++) {

5
test/JsonSpec.js
View file

@ -78,10 +78,11 @@ describe('json', function(){
expect(angular.toJson(obj)).toEqual('{"$a":"a"}'); expect(angular.toJson(obj)).toEqual('{"$a":"a"}');
}); });
it('should serialize inherited properties', function() { it('should NOT serialize inherited properties', function() {
// This is what native Browser does
var obj = inherit({p:'p'}); var obj = inherit({p:'p'});
obj.a = 'a'; obj.a = 'a';
expect(angular.toJson(obj)).toEqual('{"a":"a","p":"p"}'); expect(angular.toJson(obj)).toEqual('{"a":"a"}');
}); });
it('should serialize same objects multiple times', function() { it('should serialize same objects multiple times', function() {