Bump to v4.17.2.

Author: jdalton

README.md

@@ -1,4 +1,4 @@
-# lodash v4.17.1
+# lodash v4.17.2
 
 The [Lodash](https://lodash.com/) library exported as [Node.js](https://nodejs.org/) modules.
 
@@ -28,7 +28,7 @@ var at = require('lodash/at');
 var curryN = require('lodash/fp/curryN');
 ```
 
-See the [package source](https://github.com/lodash/lodash/tree/4.17.1-npm) for more details.
+See the [package source](https://github.com/lodash/lodash/tree/4.17.2-npm) for more details.
 
 **Note:**<br>
 Install [n_](https://www.npmjs.com/package/n_) for Lodash use in the Node.js < 6 REPL.

_basePickBy.js

@@ -1,5 +1,6 @@
 var baseGet = require('./_baseGet'),
-    baseSet = require('./_baseSet');
+    baseSet = require('./_baseSet'),
+    castPath = require('./_castPath');
 
 /**
  * The base implementation of  `_.pickBy` without support for iteratee shorthands.
@@ -20,7 +21,7 @@ function basePickBy(object, paths, predicate) {
         value = baseGet(object, path);
 
     if (predicate(value, path)) {
-      baseSet(result, path, value);
+      baseSet(result, castPath(path, object), value);
     }
   }
   return result;

_basePullAt.js

@@ -1,8 +1,5 @@
-var castPath = require('./_castPath'),
-    isIndex = require('./_isIndex'),
-    last = require('./last'),
-    parent = require('./_parent'),
-    toKey = require('./_toKey');
+var baseUnset = require('./_baseUnset'),
+    isIndex = require('./_isIndex');
 
 /** Used for built-in method references. */
 var arrayProto = Array.prototype;
@@ -29,14 +26,8 @@ function basePullAt(array, indexes) {
       var previous = index;
       if (isIndex(index)) {
         splice.call(array, index, 1);
-      }
-      else {
-        var path = castPath(index, array),
-            object = parent(array, path);
-
-        if (object != null) {
-          delete object[toKey(last(path))];
-        }
+      } else {
+        baseUnset(array, index);
       }
     }
   }

_baseUnset.js

@@ -3,12 +3,6 @@ var castPath = require('./_castPath'),
     parent = require('./_parent'),
     toKey = require('./_toKey');
 
-/** Used for built-in method references. */
-var objectProto = Object.prototype;
-
-/** Used to check objects for own properties. */
-var hasOwnProperty = objectProto.hasOwnProperty;
-
 /**
  * The base implementation of `_.unset`.
  *
@@ -20,8 +14,7 @@ var hasOwnProperty = objectProto.hasOwnProperty;
 function baseUnset(object, path) {
   path = castPath(path, object);
   object = parent(object, path);
-  var key = toKey(last(path));
-  return !(object != null && hasOwnProperty.call(object, key)) || delete object[key];
+  return object == null || delete object[toKey(last(path))];
 }
 
 module.exports = baseUnset;

core.js

@@ -13,7 +13,7 @@
   var undefined;
 
   /** Used as the semantic version number. */
-  var VERSION = '4.17.1';
+  var VERSION = '4.17.2';
 
   /** Error message constants. */
   var FUNC_ERROR_TEXT = 'Expected a function';

core.min.js

@@ -1,6 +1,9 @@
 var mapping = require('./_mapping'),
     fallbackHolder = require('./placeholder');
 
+/** Built-in value reference. */
+var push = Array.prototype.push;
+
 /**
  * Creates a function, with an arity of `n`, that invokes `func` with the
  * arguments it receives.
@@ -61,6 +64,36 @@ function createCloner(func) {
   };
 }
 
+/**
+ * This function is like `_.spread` except that it includes arguments after those spread.
+ *
+ * @private
+ * @param {Function} func The function to spread arguments over.
+ * @param {number} start The start position of the spread.
+ * @returns {Function} Returns the new function.
+ */
+function spread(func, start) {
+  return function() {
+    var length = arguments.length,
+        args = Array(length);
+
+    while (length--) {
+      args[length] = arguments[length];
+    }
+    var array = args[start],
+        lastIndex = args.length - 1,
+        otherArgs = args.slice(0, start);
+
+    if (array) {
+      push.apply(otherArgs, array);
+    }
+    if (start != lastIndex) {
+      push.apply(otherArgs, args.slice(start + 1));
+    }
+    return func.apply(this, otherArgs);
+  };
+}
+
 /**
  * Creates a function that wraps `func` and uses `cloner` to clone the first
  * argument it receives.
@@ -141,7 +174,6 @@ function baseConvert(util, name, func, options) {
     'iteratee': util.iteratee,
     'keys': util.keys,
     'rearg': util.rearg,
-    'spread': util.spread,
     'toInteger': util.toInteger,
     'toPath': util.toPath
   };
@@ -155,7 +187,6 @@ function baseConvert(util, name, func, options) {
       isFunction = helpers.isFunction,
       keys = helpers.keys,
       rearg = helpers.rearg,
-      spread = helpers.spread,
       toInteger = helpers.toInteger,
       toPath = helpers.toPath;
 

fp/_baseConvert.js

@@ -9,7 +9,6 @@ module.exports = {
   'iteratee': require('../iteratee'),
   'keys': require('../_baseKeys'),
   'rearg': require('../rearg'),
-  'spread': require('../spread'),
   'toInteger': require('../toInteger'),
   'toPath': require('../toPath')
 };

fp/_util.js

@@ -12,7 +12,7 @@
   var undefined;
 
   /** Used as the semantic version number. */
-  var VERSION = '4.17.1';
+  var VERSION = '4.17.2';
 
   /** Used as the size to enable large array optimizations. */
   var LARGE_ARRAY_SIZE = 200;
@@ -3805,7 +3805,7 @@
             value = baseGet(object, path);
 
         if (predicate(value, path)) {
-          baseSet(result, path, value);
+          baseSet(result, castPath(path, object), value);
         }
       }
       return result;
@@ -3881,14 +3881,8 @@
           var previous = index;
           if (isIndex(index)) {
             splice.call(array, index, 1);
-          }
-          else {
-            var path = castPath(index, array),
-                object = parent(array, path);
-
-            if (object != null) {
-              delete object[toKey(last(path))];
-            }
+          } else {
+            baseUnset(array, index);
           }
         }
       }
@@ -4352,8 +4346,7 @@
     function baseUnset(object, path) {
       path = castPath(path, object);
       object = parent(object, path);
-      var key = toKey(last(path));
-      return !(object != null && hasOwnProperty.call(object, key)) || delete object[key];
+      return object == null || delete object[toKey(last(path))];
     }
 
     /**
@@ -10847,15 +10840,11 @@
       start = start === undefined ? 0 : nativeMax(toInteger(start), 0);
       return baseRest(function(args) {
         var array = args[start],
-            lastIndex = args.length - 1,
             otherArgs = castSlice(args, 0, start);
 
         if (array) {
           arrayPush(otherArgs, array);
         }
-        if (start != lastIndex) {
-          arrayPush(otherArgs, castSlice(args, start + 1));
-        }
         return apply(func, this, otherArgs);
       });
     }
@@ -13470,16 +13459,16 @@
       if (object == null) {
         return result;
       }
-      var bitmask = CLONE_FLAT_FLAG | CLONE_SYMBOLS_FLAG;
+      var isDeep = false;
       paths = arrayMap(paths, function(path) {
         path = castPath(path, object);
-        bitmask |= (path.length > 1 ? CLONE_DEEP_FLAG : 0);
+        isDeep || (isDeep = path.length > 1);
         return path;
       });
-
       copyObject(object, getAllKeysIn(object), result);
-      result = baseClone(result, bitmask);
-
+      if (isDeep) {
+        result = baseClone(result, CLONE_DEEP_FLAG | CLONE_FLAT_FLAG | CLONE_SYMBOLS_FLAG);
+      }
       var length = paths.length;
       while (length--) {
         baseUnset(result, paths[length]);
@@ -13600,8 +13589,8 @@
 
       // Ensure the loop is entered when path is empty.
       if (!length) {
-        object = undefined;
         length = 1;
+        object = undefined;
       }
       while (++index < length) {
         var value = object == null ? undefined : object[toKey(path[index])];