fixing mistakenly-aliased contains ... should refer to include.

This commit is contained in:
Jeremy Ashkenas
2010-10-15 17:15:26 -04:00
parent b07c802766
commit 74d282ccc8
2 changed files with 5 additions and 4 deletions

View File

@@ -111,7 +111,7 @@ $(document).ready(function() {
test('collections: include', function() {
ok(_.include([1,2,3], 2), 'two is in the array');
ok(!_.include([1,3,9], 2), 'two is not in the array');
ok(_.contains({moe:1, larry:3, curly:9}, 3), '_.include on objects checks their values');
ok(_.contains({moe:1, larry:3, curly:9}, 3) === true, '_.include on objects checks their values');
ok(_([1,2,3]).include(2), 'OO-style include');
});

View File

@@ -174,7 +174,8 @@
};
// Determine if a given value is included in the array or object using `===`.
_.include = function(obj, target) {
// Aliased as `contains`.
_.include = _.contains = function(obj, target) {
if (nativeIndexOf && obj.indexOf === nativeIndexOf) return obj.indexOf(target) != -1;
var found = false;
each(obj, function(value) {
@@ -310,8 +311,8 @@
};
// Produce an array that contains every item shared between all the
// passed-in arrays. Aliased as `contains`.
_.intersect = _.contains = function(array) {
// passed-in arrays.
_.intersect = function(array) {
var rest = slice.call(arguments, 1);
return _.filter(_.uniq(array), function(item) {
return _.every(rest, function(other) {