]> source.dussan.org Git - jquery.git/commitdiff
Made jQuery.extend(true, ...) to extend recursively only 'object literal' values.
authorRobert Katic <robert.katic@gmail.com>
Thu, 12 Nov 2009 05:48:45 +0000 (13:48 +0800)
committerJohn Resig <jeresig@gmail.com>
Thu, 12 Nov 2009 13:59:47 +0000 (21:59 +0800)
src/core.js
test/unit/core.js

index 1d5a25bd5dab645bc45c410ced0450df1c249a5e..b82133635fb2d19f0a34eec81f7daeac4e8ae6d0 100644 (file)
@@ -272,19 +272,10 @@ jQuery.extend = jQuery.fn.extend = function() {
                                        continue;
                                }
 
-                               // Recurse if we're merging object values
-                               if ( deep && copy && typeof copy === "object" && !copy.nodeType ) {
-                                       var clone;
-
-                                       if ( src ) {
-                                               clone = src;
-                                       } else if ( jQuery.isArray(copy) ) {
-                                               clone = [];
-                                       } else if ( jQuery.isObjectLiteral(copy) ) {
-                                               clone = {};
-                                       } else {
-                                               clone = copy;
-                                       }
+                               // Recurse if we're merging object literal values
+                               if ( deep && copy && jQuery.isObjectLiteral(copy) ) {
+                                       // Don't extend not object literals
+                                       var clone = src && jQuery.isObjectLiteral(src) ? src : {};
 
                                        // Never move original objects, clone them
                                        target[ name ] = jQuery.extend( deep, clone, copy );
index b355a8c40f35091ac449767443d3580e87555e27..9b43c4889439c027c03d0b23d4a5c1315ebaccc1 100644 (file)
@@ -553,16 +553,20 @@ test("jQuery.extend(Object, Object)", function() {
        same( empty.foo, optionsWithDate.foo, "Dates copy correctly" );
 
        var myKlass = function() {};
-       var optionsWithCustomObject = { foo: { date: new myKlass } };
+       var customObject = new myKlass();
+       var optionsWithCustomObject = { foo: { date: new customObject } };
        empty = {};
        jQuery.extend(true, empty, optionsWithCustomObject);
-       same( empty.foo, optionsWithCustomObject.foo, "Custom objects copy correctly (no methods)" );
+       ok( empty.foo && empty.foo.date && empty.foo.date === customObject, "Custom objects copy correctly (no methods)" );
        
        // Makes the class a little more realistic
        myKlass.prototype = { someMethod: function(){} };
        empty = {};
        jQuery.extend(true, empty, optionsWithCustomObject);
-       same( empty.foo, optionsWithCustomObject.foo, "Custom objects copy correctly" );
+       ok( empty.foo && empty.foo.date && empty.foo.date === customObject, "Custom objects copy correctly" );
+       
+       var ret = jQuery.extend(true, { foo: 4 }, { foo: new Number(5) } );
+       ok( ret.foo == 5, "Wrapped numbers copy correctly" );
 
        var nullUndef;
        nullUndef = jQuery.extend({}, options, { xnumber2: null });