release for version 0.6.3

HISTORY.md

@@ -1,3 +1,7 @@
+# 0.6.3 / 2016-06-13
+
+  - fixed #10, #11, #12
+
 # 0.6.2 / 2016-06-12
 
   - fixed #10

build/deepcopy.js

@@ -159,7 +159,7 @@ return /******/ (function(modules) { // webpackBootstrap
 	  var resultValue = copyValue(target);
 
 	  if (resultValue !== null) {
-	    return copyValue(target);
+	    return resultValue;
 	  }
 
 	  return copyCollection(target, customizer);
@@ -361,15 +361,17 @@ return /******/ (function(modules) { // webpackBootstrap
 	    value = target[key];
 	    index = (0, _polyfill.indexOf)(visited, value);
 
+	    resultCopy = void 0;
+	    result = void 0;
+	    ref = void 0;
+
 	    if (index === -1) {
 	      resultCopy = (0, _copy.copy)(value, customizer);
 	      result = resultCopy !== null ? resultCopy : value;
 
 	      if (value !== null && /^(?:function|object)$/.test(typeof value)) {
 	        visited.push(value);
 	        reference.push(result);
-	      } else {
-	        ref = result;
 	      }
 	    } else {
 	      // circular reference

package.json

@@ -1,6 +1,6 @@
 {
   "name": "deepcopy",
-  "version": "0.6.2",
+  "version": "0.6.3",
   "author": "sasa+1 <sasaplus1@gmail.com>",
   "contributors": [
     "kjirou <kjirou.web@gmail.com>"

src/copy.js

@@ -6,7 +6,7 @@ function copy(target, customizer) {
   const resultValue = copyValue(target);
 
   if (resultValue !== null) {
-    return copyValue(target);
+    return resultValue;
   }
 
   return copyCollection(target, customizer);

src/index.js

@@ -61,15 +61,17 @@ function recursiveCopy(target, customizer, clone, visited, reference) {
     value = target[key];
     index = indexOf(visited, value);
 
+    resultCopy = undefined;
+    result = undefined;
+    ref = undefined;
+
     if (index === -1) {
       resultCopy = copy(value, customizer);
       result = (resultCopy !== null) ? resultCopy : value;
 
       if (value !== null && /^(?:function|object)$/.test(typeof value)) {
         visited.push(value);
         reference.push(result);
-      } else {
-        ref = result;
       }
     } else {
       // circular reference

test/index.js

@@ -385,6 +385,58 @@ describe('deepcopy', function() {
     assert(functionInArrayInArray.a[1][1] === copiedFunctionInArrayInArray.a[1][1]);
   });
 
+  it('can copy deeper Object and Array', function() {
+    const array = [
+      1,
+      2,
+      [
+        3,
+        4,
+        [
+          function() {},
+          6,
+          7
+        ]
+      ]
+    ];
+
+    const copiedArray = deepcopy(array);
+
+    assert(array !== copiedArray);
+    assert(array[0] === copiedArray[0]);
+    assert(array[1] === copiedArray[1]);
+    assert(array[2] !== copiedArray[2]);
+    assert(array[2][0] === copiedArray[2][0]);
+    assert(array[2][1] === copiedArray[2][1]);
+    assert(array[2][2] !== copiedArray[2][2]);
+    assert(array[2][2][0] !== copiedArray[2][2][0]);
+    assert(array[2][2][1] === copiedArray[2][2][1]);
+    assert(array[2][2][2] === copiedArray[2][2][2]);
+
+    const object = {
+      a: 'a',
+      b: 'b',
+      c: {
+        d: function() {},
+        e: {
+          f: 'f',
+          g: 'g'
+        }
+      }
+    };
+
+    const copiedObject = deepcopy(object);
+
+    assert(object !== copiedObject);
+    assert(object.a === copiedObject.a);
+    assert(object.b === copiedObject.b);
+    assert(object.c !== copiedObject.c);
+    assert(object.c.d !== copiedObject.c.d);
+    assert(object.c.e !== copiedObject.c.e);
+    assert(object.c.e.f === copiedObject.c.e.f);
+    assert(object.c.e.g === copiedObject.c.e.g);
+  });
+
   it('can copy Class from Array and Object by customizer', function() {
     function MyClass(number) {
       this.number = +number;