aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/lib/action_view/helpers/tags/collection_helpers.rb
diff options
context:
space:
mode:
authorCarlos Antonio da Silva <carlosantoniodasilva@gmail.com>2012-02-01 17:28:50 -0200
committerCarlos Antonio da Silva <carlosantoniodasilva@gmail.com>2012-02-02 09:40:25 -0200
commitb1619560614a49956828ec75aebaa4ad8b212684 (patch)
treeb8d52a1db5d7d5b2326404ec76f4ca826125fd4d /actionpack/lib/action_view/helpers/tags/collection_helpers.rb
parent36cb7150fd4087c22b61f536431c149d4c545586 (diff)
downloadrails-b1619560614a49956828ec75aebaa4ad8b212684.tar.gz
rails-b1619560614a49956828ec75aebaa4ad8b212684.tar.bz2
rails-b1619560614a49956828ec75aebaa4ad8b212684.zip
Move collection radio buttons / check boxes back to FormOptionsHelper
[Carlos Antonio da Silva + Rafael Mendonça França]
Diffstat (limited to 'actionpack/lib/action_view/helpers/tags/collection_helpers.rb')
-rw-r--r--actionpack/lib/action_view/helpers/tags/collection_helpers.rb14
1 files changed, 6 insertions, 8 deletions
diff --git a/actionpack/lib/action_view/helpers/tags/collection_helpers.rb b/actionpack/lib/action_view/helpers/tags/collection_helpers.rb
index 1c9ba8598f..4fc2293a8d 100644
--- a/actionpack/lib/action_view/helpers/tags/collection_helpers.rb
+++ b/actionpack/lib/action_view/helpers/tags/collection_helpers.rb
@@ -33,7 +33,6 @@ module ActionView
def instantiate_builder(builder_class, value, text, html_options)
builder_class.new(@template_object, @object_name, @method_name,
sanitize_attribute_name(value), text, value, html_options)
-
end
# Generate default options for collection helpers, such as :checked and
@@ -42,14 +41,13 @@ module ActionView
html_options = @html_options.dup
[:checked, :selected, :disabled].each do |option|
- next unless @options[option]
-
+ next unless current_value = @options[option]
- accept = if @options[option].respond_to?(:call)
- @options[option].call(item)
- else
- Array(@options[option]).include?(value)
- end
+ accept = if current_value.respond_to?(:call)
+ current_value.call(item)
+ else
+ Array(current_value).include?(value)
+ end
if accept
html_options[option] = true