Fix #4040, do not call .clone method on plain objects or arrays.

lib/utils.js

@@ -96,7 +96,7 @@ var Utils = module.exports = {
   cloneDeep: function(obj, fn) {
   cloneDeep: function(obj, fn) {
     return lodash.cloneDeep(obj, function (elem) {
     return lodash.cloneDeep(obj, function (elem) {
       // Preserve special data-types like `fn` across clones. _.get() is used for checking up the prototype chain
       // Preserve special data-types like `fn` across clones. _.get() is used for checking up the prototype chain
-      if (elem && typeof elem.clone === 'function') {return elem.clone(); }
+      if (elem && !Array.isArray(elem) && !lodash.isPlainObject(elem) && typeof elem.clone === 'function') {return elem.clone(); }
       // Unfortunately, lodash.cloneDeep doesn't preserve Buffer.isBuffer, which we have to rely on for binary data
       // Unfortunately, lodash.cloneDeep doesn't preserve Buffer.isBuffer, which we have to rely on for binary data
       if (Buffer.isBuffer(elem)) { return elem; }
       if (Buffer.isBuffer(elem)) { return elem; }
 
 

test/integration/utils.test.js

@@ -109,6 +109,43 @@ describe(Support.getTestDialectTeaser('Utils'), function() {
     });
     });
   });
   });
 
 
+  describe('cloneDeep', function() {
+    it('should clone objects', function() {
+      var obj = {foo: 1},
+          clone = Utils.cloneDeep(obj);
+
+      expect(obj).to.not.equal(clone);
+    });
+
+    it('should clone nested objects', function() {
+      var obj = {foo: {bar: 1}},
+          clone = Utils.cloneDeep(obj);
+
+      expect(obj.foo).to.not.equal(clone.foo);
+    });
+
+    it('should not call clone methods on plain objects', function() {
+      expect(function() {
+        Utils.cloneDeep({
+          clone: function() {
+            throw new Error('clone method called');
+          }
+        });
+      }).to.not.throw();
+    });
+
+    it('should not call clone methods on arrays', function() {
+      expect(function() {
+        var arr = [];
+        arr.clone = function() {
+          throw new Error('clone method called');
+        };
+
+        Utils.cloneDeep(arr);
+      }).to.not.throw();
+    });
+  });
+
   describe('validateParameter', function() {
   describe('validateParameter', function() {
     describe('method signature', function() {
     describe('method signature', function() {
       it('throws an error if the value is not defined', function() {
       it('throws an error if the value is not defined', function() {