From 53ede82ce2e8a1226fbb16ec67eb092cdcd12082 Mon Sep 17 00:00:00 2001 From: Teddy Wing Date: Mon, 12 Jun 2017 11:18:21 +0200 Subject: TableBuilderHelper: Change `#collect` calls to `#map` Use `#map` here because the name makes more sense to me in this context. Also, the `#thead` method uses `columns.map`, so we can do the same thing in these two places for consistency. Refs #3479 --- app/helpers/table_builder_helper.rb | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/app/helpers/table_builder_helper.rb b/app/helpers/table_builder_helper.rb index 586842d46..b3b5e3cde 100644 --- a/app/helpers/table_builder_helper.rb +++ b/app/helpers/table_builder_helper.rb @@ -54,7 +54,7 @@ module TableBuilderHelper def tbody(collection, columns, selectable, links) # TODO: refactor content_tag :tbody do - collection.collect do |item| + collection.map do |item| content_tag :tr do bcont = [] @@ -96,7 +96,7 @@ module TableBuilderHelper end menu = content_tag :ul, class: 'dropdown-menu' do - actions.collect do |action| + actions.map do |action| polymorph_url = [] unless [:show, :delete].include? action -- cgit v1.2.3