From 36d1a2f302817815e0e70334733d2daf1a14a4f5 Mon Sep 17 00:00:00 2001 From: Marcel Molina Date: Wed, 17 May 2006 05:12:53 +0000 Subject: [PATCH] Make Array#in_groups_of just return the grouped collection if a block isn't given. [Marcel Molina Jr.] git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@4345 5ecf4fe2-1ee6-0310-87b1-e25e094e27de --- activesupport/CHANGELOG | 2 ++ activesupport/lib/active_support/core_ext/array.rb | 6 +++++- activesupport/test/core_ext/array_ext_test.rb | 3 ++- 3 files changed, 9 insertions(+), 2 deletions(-) diff --git a/activesupport/CHANGELOG b/activesupport/CHANGELOG index a651cff958..670584b8eb 100644 --- a/activesupport/CHANGELOG +++ b/activesupport/CHANGELOG @@ -1,5 +1,7 @@ *SVN* +* Make Array#in_groups_of just return the grouped collection if a block isn't given. [Marcel Molina Jr.] + * Don't destroy a HashWithIndifferentAccess if symbolize_keys! or stringify_keys! is called on it. Closes #5076. [Marcel Molina Jr., guy.naor@famundo.com] * Document Module::delegate. #5002 [pergesu@gmail.com] diff --git a/activesupport/lib/active_support/core_ext/array.rb b/activesupport/lib/active_support/core_ext/array.rb index db6f5587da..243aeab423 100644 --- a/activesupport/lib/active_support/core_ext/array.rb +++ b/activesupport/lib/active_support/core_ext/array.rb @@ -16,7 +16,11 @@ class Array #:nodoc: require 'enumerator' collection = dup collection << fill_with until collection.size.modulo(number).zero? - collection.each_slice(number, &block) + grouped_collection = [] unless block_given? + collection.each_slice(number) do |group| + block_given? ? yield(group) : grouped_collection << group + end + grouped_collection unless block_given? end # Divide the array into one or more subarrays based on a delimiting +value+ diff --git a/activesupport/test/core_ext/array_ext_test.rb b/activesupport/test/core_ext/array_ext_test.rb index caf3af2345..0d5f58a9be 100644 --- a/activesupport/test/core_ext/array_ext_test.rb +++ b/activesupport/test/core_ext/array_ext_test.rb @@ -47,6 +47,7 @@ class ArrayExtGroupingTests < Test::Unit::TestCase end assert_equal [%w(a b c), %w(d e f), %w(g h i)], groups + assert_equal [%w(a b c), %w(d e f), %w(g h i)], ('a'..'i').to_a.in_groups_of(3) end def test_group_by_with_padding @@ -122,4 +123,4 @@ class ArrayToXmlTests < Test::Unit::TestCase assert xml.include?(%(Evergreen)) assert xml.include?(%(Jason)) end -end \ No newline at end of file +end