release for version 0.6.1

Author: sasaplus1

HISTORY.md

@@ -1,3 +1,7 @@
+# 0.6.1 / 2016-01-13
+
+  - fixed #9
+
 # 0.6.0 / 2015-12-29
 
   - fixed #7

build/deepcopy.js

@@ -354,8 +354,8 @@ return /******/ (function(modules) { // webpackBootstrap
 	  var key = undefined,
 	      value = undefined,
 	      index = undefined,
-	      result = undefined,
 	      resultCopy = undefined,
+	      result = undefined,
 	      ref = undefined;
 
 	  for (i = 0, len = keys.length; i < len; ++i) {
@@ -367,8 +367,10 @@ return /******/ (function(modules) { // webpackBootstrap
 	      resultCopy = _copy.copy(value, customizer);
 	      result = resultCopy !== null ? resultCopy : value;
 
-	      visited.push(value);
-	      reference.push(result);
+	      if (value !== null && /^(?:function|object)$/.test(typeof value)) {
+	        visited.push(value);
+	        reference.push(result);
+	      }
 	    } else {
 	      // circular reference
 	      ref = reference[index];

build/deepcopy.min.js

@@ -1,6 +1,6 @@
 {
   "name": "deepcopy",
-  "version": "0.6.0",
+  "version": "0.6.1",
   "author": "sasa+1 <[email protected]>",
   "contributors": [
     "kjirou <[email protected]>"
@@ -51,9 +51,5 @@
     "power-assert": "^1.2.0",
     "rimraf": "^2.4.4",
     "webpack": "^1.12.9"
-  },
-  "engines": {
-    "node": "^4.2.3",
-    "npm": "^2.14.7"
   }
 }

package.json

@@ -45,7 +45,7 @@ function recursiveCopy(target, customizer, clone, visited, reference) {
 
   let i, len;
 
-  let key, value, index, result, resultCopy, ref;
+  let key, value, index, resultCopy, result, ref;
 
   for (i = 0, len = keys.length; i < len; ++i) {
     key = keys[i];
@@ -56,8 +56,10 @@ function recursiveCopy(target, customizer, clone, visited, reference) {
       resultCopy = copy(value, customizer);
       result = (resultCopy !== null) ? resultCopy : value;
 
-      visited.push(value);
-      reference.push(result);
+      if (value !== null && /^(?:function|object)$/.test(typeof value)) {
+        visited.push(value);
+        reference.push(result);
+      }
     } else {
       // circular reference
       ref = reference[index];

src/index.js

@@ -295,6 +295,48 @@ describe('deepcopy', function() {
     assert(+copiedSymbolObject[a] === +copiedSymbolObject[b]);
   });
 
+  it('can recursively copy from Object in Array', function() {
+    const array = [
+      {
+        a: 1,
+        b: 2,
+      },
+      {
+        a: 1,
+        b: 3,
+      },
+    ];
+
+    const copiedArray = deepcopy(array);
+
+    assert(array !== copiedArray);
+    assert(array[0].a === copiedArray[0].a);
+    assert(array[0].b === copiedArray[0].b);
+    assert(array[1].a === copiedArray[1].a);
+    assert(array[1].b === copiedArray[1].b);
+  });
+
+  it('can recursively copy from Array in Objct', function() {
+    const object = {
+      a: [
+        1,
+        2,
+      ],
+      b: [
+        1,
+        3,
+      ],
+    };
+
+    const copiedObject = deepcopy(object);
+
+    assert(object !== copiedObject);
+    assert(object.a[0] === copiedObject.a[0]);
+    assert(object.b[0] === copiedObject.b[0]);
+    assert(object.a[1] === copiedObject.a[1]);
+    assert(object.b[1] === copiedObject.b[1]);
+  });
+
   it('can copy Class from Array and Object by customizer', function() {
     function MyClass(number) {
       this.number = +number;