aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/lib/action_view/helpers
diff options
context:
space:
mode:
authorRafael Mendonça França <rafaelmfranca@gmail.com>2012-01-17 23:01:48 -0300
committerRafael Mendonça França <rafaelmfranca@gmail.com>2012-01-17 23:20:51 -0300
commit014da68634c59a7df1950dd7b838fd0ba5181969 (patch)
treee1659556d07e5f698225c0225fa1f923864bf463 /actionpack/lib/action_view/helpers
parent336f973763dea0dc5d7f833fa0090e513503b19d (diff)
downloadrails-014da68634c59a7df1950dd7b838fd0ba5181969.tar.gz
rails-014da68634c59a7df1950dd7b838fd0ba5181969.tar.bz2
rails-014da68634c59a7df1950dd7b838fd0ba5181969.zip
Use block syntax to avoid code duplication
Diffstat (limited to 'actionpack/lib/action_view/helpers')
-rw-r--r--actionpack/lib/action_view/helpers/form_helper.rb6
1 files changed, 3 insertions, 3 deletions
diff --git a/actionpack/lib/action_view/helpers/form_helper.rb b/actionpack/lib/action_view/helpers/form_helper.rb
index 10277599a0..2d6103a350 100644
--- a/actionpack/lib/action_view/helpers/form_helper.rb
+++ b/actionpack/lib/action_view/helpers/form_helper.rb
@@ -376,9 +376,9 @@ module ActionView
builder = options[:parent_builder] = instantiate_builder(object_name, object, options, &proc)
fields_for = fields_for(object_name, object, options, &proc)
default_options = builder.multipart? ? { :multipart => true } : {}
- output = form_tag(options.delete(:url) || {}, default_options.merge!(options.delete(:html)))
- output << fields_for
- output.safe_concat('</form>')
+ default_options.merge!(options.delete(:html))
+
+ form_tag(options.delete(:url) || {}, default_options) { fields_for }
end
def apply_form_for_options!(record, object, options) #:nodoc: