diff options
author | José Valim <jose.valim@plataformatec.com.br> | 2012-01-31 08:59:48 -0800 |
---|---|---|
committer | José Valim <jose.valim@plataformatec.com.br> | 2012-01-31 08:59:48 -0800 |
commit | 40c287c7983c20c498f6a8a2ea49e9a6455347f7 (patch) | |
tree | 04d39a5808e2b01eaf0a53e61e32672111167ea2 /actionpack/lib/action_view/helpers/form_options_helper.rb | |
parent | 3bf859daf6f821e8847a52f73dbb6e20cb7bab95 (diff) | |
parent | 4af62c02ae3dd0738965b167fbf07940e5d64daf (diff) | |
download | rails-40c287c7983c20c498f6a8a2ea49e9a6455347f7.tar.gz rails-40c287c7983c20c498f6a8a2ea49e9a6455347f7.tar.bz2 rails-40c287c7983c20c498f6a8a2ea49e9a6455347f7.zip |
Merge pull request #4802 from carlosantoniodasilva/collection-helpers-refactor
Refactor select helper and remove eval
Diffstat (limited to 'actionpack/lib/action_view/helpers/form_options_helper.rb')
-rw-r--r-- | actionpack/lib/action_view/helpers/form_options_helper.rb | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/actionpack/lib/action_view/helpers/form_options_helper.rb b/actionpack/lib/action_view/helpers/form_options_helper.rb index c8811e3b10..c4cdfef4a2 100644 --- a/actionpack/lib/action_view/helpers/form_options_helper.rb +++ b/actionpack/lib/action_view/helpers/form_options_helper.rb @@ -334,7 +334,7 @@ module ActionView end.join("\n").html_safe end - # Returns a string of option tags that have been compiled by iterating over the +collection+ and assigning + # Returns a string of option tags that have been compiled by iterating over the +collection+ and assigning # the result of a call to the +value_method+ as the option value and the +text_method+ as the option text. # Example: # options_from_collection_for_select(@people, 'id', 'name') @@ -418,9 +418,9 @@ module ActionView # wrap the output in an appropriate <tt><select></tt> tag. def option_groups_from_collection_for_select(collection, group_method, group_label_method, option_key_method, option_value_method, selected_key = nil) collection.map do |group| - group_label_string = eval("group.#{group_label_method}") + group_label_string = group.send(group_label_method) "<optgroup label=\"#{ERB::Util.html_escape(group_label_string)}\">" + - options_from_collection_for_select(eval("group.#{group_method}"), option_key_method, option_value_method, selected_key) + + options_from_collection_for_select(group.send(group_method), option_key_method, option_value_method, selected_key) + '</optgroup>' end.join.html_safe end |