Change method to abort customization of plain objects/arrays first

lib/utils.js

@@ -95,16 +95,14 @@ var Utils = module.exports = {
   },
   cloneDeep: function(obj, fn) {
     return lodash.cloneDeep(obj, function (elem) {
+      // Do not try to customize cloning of plain objects and strings
+      if (Array.isArray(elem) || lodash.isPlainObject(elem)) {
+        return undefined;
+      }
       // Preserve special data-types like `fn` across clones. _.get() is used for checking up the prototype chain
-      if (elem) {
-        // Allow objects to implement their own clone method
-        if (typeof elem.clone === 'function') {
-          // However do not mistakenly call clone methods on arrays and plain objects
-          if (!Array.isArray(elem) && !lodash.isPlainObject(elem)) {
+      if (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
       if (Buffer.isBuffer(elem)) { return elem; }