Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Merge branch 'bug_7413' of https://github.com/rwldrn/jquery into rwld…
…rn-bug_7413
  • Loading branch information
jeresig committed Dec 9, 2010
2 parents 8943b42 + fc563cc commit fb6c038
Show file tree
Hide file tree
Showing 2 changed files with 17 additions and 4 deletions.
6 changes: 6 additions & 0 deletions src/core.js
Expand Up @@ -532,6 +532,12 @@ jQuery.extend({
},

isEmptyObject: function( obj ) {

// Fixes #7413 Check to see if obj passes isPlainObject
if ( !jQuery.isPlainObject( obj ) ) {
return false;
}

for ( var name in obj ) {
return false;
}
Expand Down
15 changes: 11 additions & 4 deletions test/unit/core.js
Expand Up @@ -848,13 +848,20 @@ test("jQuery.makeArray", function(){
});

test("jQuery.isEmptyObject", function(){
expect(2);
expect(11);

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" );
equals(false, jQuery.isEmptyObject(1), "isEmptyObject on number (wrong argument type)");
equals(false, jQuery.isEmptyObject(0), "isEmptyObject on falsy number (wrong argument type)");
equals(false, jQuery.isEmptyObject("test"), "isEmptyObject on string (wrong argument type)");
equals(false, jQuery.isEmptyObject(""), "isEmptyObject on falsy string (wrong argument type)");
equals(false, jQuery.isEmptyObject([1,2,3]), "isEmptyObject on array (wrong argument type)");
equals(false, jQuery.isEmptyObject([]), "isEmptyObject on an empty array (wrong argument type)");
equals(false, jQuery.isEmptyObject(undefined), "isEmptyObject on undefined (wrong argument type)");
equals(false, jQuery.isEmptyObject(false), "isEmptyObject on undefined (wrong argument type)");
equals(false, jQuery.isEmptyObject(null), "isEmptyObject on null (wrong argument type)" );

});

test("jQuery.proxy", function(){
Expand Down

0 comments on commit fb6c038

Please sign in to comment.