From 064842f516513eab305cb8a3257e32164cda68cb Mon Sep 17 00:00:00 2001
From: Marcel Molina <marcel@vernix.org>
Date: Mon, 10 Oct 2005 21:34:51 +0000
Subject: Convert boolean form options form the tag_helper. Recloses #809.

git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@2523 5ecf4fe2-1ee6-0310-87b1-e25e094e27de
---
 .../lib/action_view/helpers/form_tag_helper.rb     | 33 ++++++----------------
 1 file changed, 9 insertions(+), 24 deletions(-)

(limited to 'actionpack/lib/action_view/helpers/form_tag_helper.rb')

diff --git a/actionpack/lib/action_view/helpers/form_tag_helper.rb b/actionpack/lib/action_view/helpers/form_tag_helper.rb
index dff7969da0..07cc62a527 100644
--- a/actionpack/lib/action_view/helpers/form_tag_helper.rb
+++ b/actionpack/lib/action_view/helpers/form_tag_helper.rb
@@ -47,7 +47,7 @@ module ActionView
       # Options:
       # * <tt>:multiple</tt> - If set to true the selection will allow multiple choices.
       def select_tag(name, option_tags = nil, options = {})
-        content_tag("select", option_tags, { "name" => name, "id" => name }.update(convert_options(options)))
+        content_tag("select", option_tags, { "name" => name, "id" => name }.update(options))
       end
 
       # Creates a standard text field.
@@ -59,7 +59,7 @@ module ActionView
       # 
       # A hash of standard HTML options for the tag.
       def text_field_tag(name, value = nil, options = {})
-        tag("input", { "type" => "text", "name" => name, "id" => name, "value" => value }.update(convert_options(options)))
+        tag("input", { "type" => "text", "name" => name, "id" => name, "value" => value }.update(options))
       end
 
       # Creates a hidden field.
@@ -80,14 +80,14 @@ module ActionView
       # The specified URL will then be passed a File object containing the selected file, or if the field 
       # was left blank, a StringIO object.
       def file_field_tag(name, options = {})
-        text_field_tag(name, nil, convert_options(options).update("type" => "file"))
+        text_field_tag(name, nil, options.update("type" => "file"))
       end
 
       # Creates a password field.
       #
       # Takes the same options as text_field_tag
       def password_field_tag(name = "password", value = nil, options = {})
-        text_field_tag(name, value, convert_options(options).update("type" => "password"))
+        text_field_tag(name, value, options.update("type" => "password"))
       end
 
       # Creates a text input area.
@@ -103,49 +103,34 @@ module ActionView
           options.delete("size")
         end
 
-        content_tag("textarea", content, { "name" => name, "id" => name }.update(convert_options(options)))
+        content_tag("textarea", content, { "name" => name, "id" => name }.update(options))
       end
 
       # Creates a check box.
       def check_box_tag(name, value = "1", checked = false, options = {})
-        html_options = { "type" => "checkbox", "name" => name, "id" => name, "value" => value }.update(convert_options(options))
+        html_options = { "type" => "checkbox", "name" => name, "id" => name, "value" => value }.update(options)
         html_options["checked"] = "checked" if checked
         tag("input", html_options)
       end
 
       # Creates a radio button.
       def radio_button_tag(name, value, checked = false, options = {})
-        html_options = { "type" => "radio", "name" => name, "id" => name, "value" => value }.update(convert_options(options))
+        html_options = { "type" => "radio", "name" => name, "id" => name, "value" => value }.update(options)
         html_options["checked"] = "checked" if checked
         tag("input", html_options)
       end
 
       # Creates a submit button with the text <tt>value</tt> as the caption.
       def submit_tag(value = "Save changes", options = {})
-        tag("input", { "type" => "submit", "name" => "commit", "value" => value }.update(convert_options(options)))
+        tag("input", { "type" => "submit", "name" => "commit", "value" => value }.update(options))
       end
       
       # Displays an image which when clicked will submit the form.
       #
       # <tt>source</tt> is passed to AssetTagHelper#image_path
       def image_submit_tag(source, options = {})
-        tag("input", { "type" => "image", "src" => image_path(source) }.update(convert_options(options)))
+        tag("input", { "type" => "image", "src" => image_path(source) }.update(options))
       end
-      
-      private
-        def convert_options(options)
-          options = options.stringify_keys
-          %w( disabled readonly multiple ).each { |a| boolean_attribute(options, a) }
-          options
-        end
-        
-        def boolean_attribute(options, attribute)
-          if options[attribute]
-            options[attribute] = attribute
-          else
-            options.delete attribute
-          end
-        end
     end
   end
 end
-- 
cgit v1.2.3