Remove thisArg from _.spread. [closes #957]

This commit is contained in:
jdalton
2015-02-10 22:05:24 -08:00
parent 2efd6250fb
commit fcf18fb965
2 changed files with 8 additions and 12 deletions

View File

@@ -7643,15 +7643,14 @@
}
/**
* Creates a function that invokes `func` with the `this` binding of `thisArg`
* and the array of arguments provided to the created function much like
* [Function#apply](http://es5.github.io/#x15.3.4.3).
* Creates a function that invokes `func` with the `this` binding of the
* created function and the array of arguments provided to the created
* function much like [Function#apply](http://es5.github.io/#x15.3.4.3).
*
* @static
* @memberOf _
* @category Function
* @param {Function} The function to spread arguments over.
* @param {*} [thisArg] The `this` binding of `func`.
* @returns {*} Returns the new function.
* @example
*
@@ -7680,7 +7679,7 @@
throw new TypeError(FUNC_ERROR_TEXT);
}
return function(array) {
return func.apply(thisArg, array);
return func.apply(this, array);
};
}

View File

@@ -12446,10 +12446,6 @@
QUnit.module('lodash.spread');
(function() {
function add(x, y) {
return x + y;
}
test('should spread arguments to `func`', 1, function() {
var spread = _.spread(add);
strictEqual(spread([4, 2]), 6);
@@ -12470,12 +12466,13 @@
deepEqual(args, [4, 2]);
});
test('should support the `thisArg` argument', 1, function() {
test('should not set a `this` binding', 1, function() {
var spread = _.spread(function(x, y) {
return this[x] + this[y];
}, [4, 2]);
});
strictEqual(spread([0, 1]), 6);
var object = { 'spread': spread, 'x': 4, 'y': 2 };
strictEqual(object.spread(['x', 'y']), 6);
});
}());