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

jQuery.extend(true, Object, Object) copies custom objects correctly.

Browse source 
- Also update jQuery.isObject to handle this case correctly
This commit is contained in:
Yehuda Katz 2009-07-16 07:32:03 +00:00
parent 8d52c27808
commit c10f87120f
2 changed files with 25 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

14
test/unit/core.js
View file

@ -435,7 +435,7 @@ test("jQuery.merge()", function() {
});
test("jQuery.extend(Object, Object)", function() {
expect(21);
expect(23);
var settings = { xnumber1: 5, xnumber2: 7, xstring1: "peter", xstring2: "pan" },
options = { xnumber2: 1, xstring2: "x", xxx: "newstring" },
@ -463,9 +463,19 @@ test("jQuery.extend(Object, Object)", function() {
var empty = {};
var optionsWithLength = { foo: { length: -1 } };
jQuery.extend(true, empty, optionsWithLength);
isObj( empty.foo, optionsWithLength.foo, "The length property must copy correctly" );
empty = {};
var optionsWithDate = { foo: { date: new Date } };
jQuery.extend(true, empty, optionsWithDate);
isObj( empty.foo, optionsWithDate.foo, "Dates copy correctly" );
var myKlass = function() {};
empty = {};
var optionsWithCustomObject = { foo: { date: new myKlass } };
jQuery.extend(true, empty, optionsWithCustomObject);
isObj( empty.foo, optionsWithCustomObject.foo, "Custom objects copy correctly" );
var nullUndef;
nullUndef = jQuery.extend({}, options, { xnumber2: null });
ok( nullUndef.xnumber2 === null, "Check to make sure null values are copied");