From 8e8bf37aa9ff4a32838ac477f97b458e9b99da7e Mon Sep 17 00:00:00 2001
From: David Heinemeier Hansson <david@loudthinking.com>
Date: Sun, 17 Apr 2005 09:44:28 +0000
Subject: Fixed DateHelper to return values on the option tags such that
 they'll work properly in IE with form_remote_tag #1024 [rscottmace@gmail.com]

git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@1184 5ecf4fe2-1ee6-0310-87b1-e25e094e27de
---
 actionpack/lib/action_view/helpers/form_options_helper.rb | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

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

diff --git a/actionpack/lib/action_view/helpers/form_options_helper.rb b/actionpack/lib/action_view/helpers/form_options_helper.rb
index fd8a347cb0..fe86accf25 100644
--- a/actionpack/lib/action_view/helpers/form_options_helper.rb
+++ b/actionpack/lib/action_view/helpers/form_options_helper.rb
@@ -92,7 +92,7 @@ module ActionView
           else
             is_selected = ( (selected.respond_to?(:include?) ? selected.include?(element) : element == selected) )
             is_selected = ( (selected.respond_to?(:include?) && !selected.is_a?(String) ? selected.include?(element) : element == selected) )
-            options << ((is_selected) ? "<option selected=\"selected\">#{html_escape(element.to_s)}</option>" : "<option>#{html_escape(element.to_s)}</option>")
+            options << ((is_selected) ? "<option value=\"#{html_escape(element.to_s)}\" selected=\"selected\">#{html_escape(element.to_s)}</option>" : "<option value=\"#{html_escape(element.to_s)}\">#{html_escape(element.to_s)}</option>")
           end
         end
 
@@ -168,7 +168,7 @@ module ActionView
 
         if priority_countries
           country_options += options_for_select(priority_countries, selected)
-          country_options += "<option>-------------</option>\n"
+          country_options += "<option value=\"\">-------------</option>\n"
         end
 
         if priority_countries && priority_countries.include?(selected)
@@ -205,7 +205,7 @@ module ActionView
 
         if priority_zones
           zone_options += options_for_select(convert_zones[priority_zones], selected)
-          zone_options += "<option>-------------</option>\n"
+          zone_options += "<option value=\"\">-------------</option>\n"
 
           zones = zones.reject { |z| priority_zones.include?( z ) }
         end
@@ -298,7 +298,7 @@ module ActionView
 
       private
         def add_blank_option(option_tags, add_blank)
-          add_blank ? "<option></option>\n" + option_tags : option_tags
+          add_blank ? "<option value=\"\"></option>\n" + option_tags : option_tags
         end
     end
   end
-- 
cgit v1.2.3