deac/jquery
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

jquery core: Simplifying isEmptyObject() and adding tests.

Browse source
This commit is contained in:
Ariel Flesler 2009-07-16 15:16:44 +00:00
parent 991d039b62
commit a38a5cd531
2 changed files with 13 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
src/core.js
View file

@ -292,9 +292,9 @@ jQuery.extend({
},
isEmptyObject: function( obj ) {
var name = "";
for(name in obj) break;
return !name;
for(var name in obj)
return false;
return true;
},
// check if an element is in a (or is an) XML document

10
test/unit/core.js
View file

@ -599,3 +599,13 @@ test("jQuery.makeArray", function(){
ok( jQuery.makeArray(document.getElementById('form')).length >= 13, "Pass makeArray a form (treat as elements)" );
});
test("jQuery.isEmptyObject", function(){
expect(2);
equals(true, jQuery.isEmptyObject({}), "isEmptyObject on empty object literal" );
equals(false, jQuery.isEmptyObject({a:1}), "isEmptyObject on non-empty object literal" );
// What about this ?
// equals(true, jQuery.isEmptyObject(null), "isEmptyObject on null" );
});