diff options
author | Adam Niedzielski <adam.niedzielski@goodylabs.com> | 2013-09-06 13:41:24 +0200 |
---|---|---|
committer | Adam Niedzielski <adam.niedzielski@goodylabs.com> | 2013-09-09 07:59:07 +0200 |
commit | e18f045b6518d555f14ac84f39aa4177ea51185d (patch) | |
tree | 9b9d89819106384f4656084bb32efd04a2aae1f0 /actionview/test | |
parent | 638adfc447609f498ac1461886507110704f4638 (diff) | |
download | rails-e18f045b6518d555f14ac84f39aa4177ea51185d.tar.gz rails-e18f045b6518d555f14ac84f39aa4177ea51185d.tar.bz2 rails-e18f045b6518d555f14ac84f39aa4177ea51185d.zip |
form_for - fix :namespace and :as options clash
:as option should not overwrite :namespace option when
generating html id attribute of the form element. id should be prefixed
by specified namespace even if :as option is present
Add test case showing the issue and code fixing it
Diffstat (limited to 'actionview/test')
-rw-r--r-- | actionview/test/template/form_helper_test.rb | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/actionview/test/template/form_helper_test.rb b/actionview/test/template/form_helper_test.rb index 8cca43d7ca..944884c9dd 100644 --- a/actionview/test/template/form_helper_test.rb +++ b/actionview/test/template/form_helper_test.rb @@ -1681,6 +1681,18 @@ class FormHelperTest < ActionView::TestCase assert_dom_equal expected, output_buffer end + def test_form_for_with_namespace_and_as_option + form_for(@post, namespace: 'namespace', as: 'custom_name') do |f| + concat f.text_field(:title) + end + + expected = whole_form('/posts/123', 'namespace_edit_custom_name', 'edit_custom_name', method: 'patch') do + "<input id='namespace_custom_name_title' name='custom_name[title]' type='text' value='Hello World' />" + end + + assert_dom_equal expected, output_buffer + end + def test_two_form_for_with_namespace form_for(@post, namespace: 'namespace_1') do |f| concat f.label(:title) |