Ticket #3313: radio_button_generates_unique_id_with_negative_values.2.diff
| File radio_button_generates_unique_id_with_negative_values.2.diff, 1.8 kB (added by josh, 2 years ago) |
|---|
-
actionpack/test/template/form_helper_test.rb
old new 125 125 def test_radio_button_is_checked_with_integers 126 126 assert_dom_equal('<input checked="checked" id="post_secret_1" name="post[secret]" type="radio" value="1" />', 127 127 radio_button("post", "secret", "1") 128 ) 128 ) 129 assert_dom_equal('<input checked="checked" id="post_secret_1" name="post[secret]" type="radio" value="1" />', 130 radio_button("post", "secret", 1) 131 ) 132 assert_dom_equal('<input id="post_secret_-1" name="post[secret]" type="radio" value="-1" />', 133 radio_button("post", "secret", -1) 134 ) 129 135 end 130 136 131 137 def test_radio_button_respects_passed_in_id 132 138 assert_dom_equal('<input checked="checked" id="foo" name="post[secret]" type="radio" value="1" />', 133 139 radio_button("post", "secret", "1", :id=>"foo") -
actionpack/lib/action_view/helpers/form_helper.rb
old new 289 289 checked = self.class.radio_button_checked?(value(object), tag_value) 290 290 end 291 291 options["checked"] = "checked" if checked 292 pretty_tag_value = tag_value.to_s.gsub(/\s/, "_").gsub(/ \W/, "").downcase292 pretty_tag_value = tag_value.to_s.gsub(/\s/, "_").gsub(/(?!-)\W/, "").downcase 293 293 options["id"] ||= defined?(@auto_index) ? 294 294 "#{@object_name}_#{@auto_index}_#{@method_name}_#{pretty_tag_value}" : 295 295 "#{@object_name}_#{@method_name}_#{pretty_tag_value}"