Use include? instead of in? for Enumerable#without.

[egilburg]
rails · Mar 2, 2015 · 7c34548 · 7c34548
1 parent afd1a87
commit 7c34548
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/activesupport/lib/active_support/core_ext/enumerable.rb b/activesupport/lib/active_support/core_ext/enumerable.rb
@@ -69,7 +69,7 @@ def exclude?(object)
   #   {foo: 1, bar: 2, baz: 3}.without :bar
   #     => {foo: 1, baz: 3}
   def without(*elements)
-    reject { |element| element.in?(elements) }
+    reject { |element| elements.include?(element) }
   end
 end
 

diff --git a/activesupport/test/core_ext/enumerable_test.rb b/activesupport/test/core_ext/enumerable_test.rb
@@ -1,7 +1,6 @@
 require 'abstract_unit'
 require 'active_support/core_ext/array'
 require 'active_support/core_ext/enumerable'
-require 'active_support/core_ext/object/inclusion'
 
 Payment = Struct.new(:price)
 class SummablePayment < Payment