diff options
author | Yves Senn <yves.senn@gmail.com> | 2013-07-01 13:39:04 -0700 |
---|---|---|
committer | Yves Senn <yves.senn@gmail.com> | 2013-07-01 13:39:04 -0700 |
commit | 53394420287009ac18ddb2831c981ea93e9491d0 (patch) | |
tree | 1ee7bb9dca7feba21d1faf8fb325003a3d4fa313 /actionview | |
parent | dc9ff7bb7cb8e33aa6ddc6c9960f3d6394ec6987 (diff) | |
parent | 38aafc037705aab2f09ec924bf634d105002e016 (diff) | |
download | rails-53394420287009ac18ddb2831c981ea93e9491d0.tar.gz rails-53394420287009ac18ddb2831c981ea93e9491d0.tar.bz2 rails-53394420287009ac18ddb2831c981ea93e9491d0.zip |
Merge pull request #11227 from vipulnsward/form_builder
Remove `FormBuilder` deprecation warning about block argument and tests
Diffstat (limited to 'actionview')
-rw-r--r-- | actionview/CHANGELOG.md | 4 | ||||
-rw-r--r-- | actionview/lib/action_view/helpers/form_helper.rb | 6 | ||||
-rw-r--r-- | actionview/test/template/form_helper_test.rb | 12 |
3 files changed, 5 insertions, 17 deletions
diff --git a/actionview/CHANGELOG.md b/actionview/CHANGELOG.md index a89070c910..957fb2fbcc 100644 --- a/actionview/CHANGELOG.md +++ b/actionview/CHANGELOG.md @@ -1,3 +1,7 @@ +* Remove deprecated block passing to FormBuilder#new. + + *Vipul A M* + * Pick `DateField` `DateTimeField` and `ColorField` values from stringified options allowing use of symbol keys with helpers. *Jon Rowe* diff --git a/actionview/lib/action_view/helpers/form_helper.rb b/actionview/lib/action_view/helpers/form_helper.rb index f64c0ca30b..8a4830d887 100644 --- a/actionview/lib/action_view/helpers/form_helper.rb +++ b/actionview/lib/action_view/helpers/form_helper.rb @@ -1237,11 +1237,7 @@ module ActionView self end - def initialize(object_name, object, template, options, block=nil) - if block - ActiveSupport::Deprecation.warn "Giving a block to FormBuilder is deprecated and has no effect anymore." - end - + def initialize(object_name, object, template, options) @nested_child_index = {} @object_name, @object, @template, @options = object_name, object, template, options @default_options = @options ? @options.slice(:index, :namespace) : {} diff --git a/actionview/test/template/form_helper_test.rb b/actionview/test/template/form_helper_test.rb index db44b00ef6..f05b6d0d94 100644 --- a/actionview/test/template/form_helper_test.rb +++ b/actionview/test/template/form_helper_test.rb @@ -2916,18 +2916,6 @@ class FormHelperTest < ActionView::TestCase assert_equal "fields", output end - def test_form_builder_block_argument_deprecation - builder_class = Class.new(ActionView::Helpers::FormBuilder) do - def initialize(object_name, object, template, options, block) - super - end - end - - assert_deprecated(/Giving a block to FormBuilder is deprecated and has no effect anymore/) do - builder_class.new(:foo, nil, nil, {}, proc {}) - end - end - def test_form_for_only_instantiates_builder_once initialization_count = 0 builder_class = Class.new(ActionView::Helpers::FormBuilder) do |