diff options
| author | Rafael Mendonça França <rafaelmfranca@gmail.com> | 2012-01-17 23:01:48 -0300 |
|---|---|---|
| committer | Rafael Mendonça França <rafaelmfranca@gmail.com> | 2012-01-17 23:20:51 -0300 |
| commit | 014da68634c59a7df1950dd7b838fd0ba5181969 (patch) | |
| tree | e1659556d07e5f698225c0225fa1f923864bf463 /actionpack/lib | |
| parent | 336f973763dea0dc5d7f833fa0090e513503b19d (diff) | |
| download | rails-014da68634c59a7df1950dd7b838fd0ba5181969.tar.gz rails-014da68634c59a7df1950dd7b838fd0ba5181969.tar.bz2 rails-014da68634c59a7df1950dd7b838fd0ba5181969.zip | |
Use block syntax to avoid code duplication
Diffstat (limited to 'actionpack/lib')
| -rw-r--r-- | actionpack/lib/action_view/helpers/form_helper.rb | 6 |
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: |