aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/test
diff options
context:
space:
mode:
authorRyan McGeary <ryan@mcgeary.org>2013-04-05 14:51:15 -0400
committerRyan McGeary <ryan@mcgeary.org>2013-04-05 15:35:20 -0400
commit44a9aedd7b8d65517b15bbbb7729f3f16991e23f (patch)
tree21873c3e66e6b40a4855e510b47c0df1f1a965d3 /actionpack/test
parentfe588537654adc583d54157b1a5db48786452adf (diff)
downloadrails-44a9aedd7b8d65517b15bbbb7729f3f16991e23f.tar.gz
rails-44a9aedd7b8d65517b15bbbb7729f3f16991e23f.tar.bz2
rails-44a9aedd7b8d65517b15bbbb7729f3f16991e23f.zip
Fix explicit names on multiple file fields
If a file field tag is passed the multiple option, it is turned into an array field (appending "[]"), but if the file field is passed an explicit name as an option, leave the name alone (do not append "[]"). Fixes #9830
Diffstat (limited to 'actionpack/test')
-rw-r--r--actionpack/test/template/form_helper_test.rb10
1 files changed, 10 insertions, 0 deletions
diff --git a/actionpack/test/template/form_helper_test.rb b/actionpack/test/template/form_helper_test.rb
index 7b35424ec7..22af39add4 100644
--- a/actionpack/test/template/form_helper_test.rb
+++ b/actionpack/test/template/form_helper_test.rb
@@ -301,6 +301,16 @@ class FormHelperTest < ActionView::TestCase
assert_dom_equal expected, file_field("user", "avatar")
end
+ def test_file_field_with_multiple_behavior
+ expected = '<input id="import_file" multiple="multiple" name="import[file][]" type="file" />'
+ assert_dom_equal expected, file_field("import", "file", :multiple => true)
+ end
+
+ def test_file_field_with_multiple_behavior_and_explicit_name
+ expected = '<input id="import_file" multiple="multiple" name="custom" type="file" />'
+ assert_dom_equal expected, file_field("import", "file", :multiple => true, :name => "custom")
+ end
+
def test_hidden_field
assert_dom_equal '<input id="post_title" name="post[title]" type="hidden" value="Hello World" />',
hidden_field("post", "title")