field_set_tag should take an options hash

Shouldn't that be:

module ActionView

module Helpers

module FormTagHelper

def field_set_tag(legend = nil, options={}, &block)

content = capture(&block)

concat(tag(:fieldset, options, true), block.binding)

concat(content_tag(:legend, legend), block.binding) unless legend.blank?

concat(content, block.binding)

concat("", block.binding)

end

end

end

end

Good catch. However you need a closing fieldset tag

  def field_set_tag(legend = nil, options={}, &block)
    content = capture(&block)
    concat(tag(:fieldset, options, true), block.binding)
    concat(content_tag(:legend, legend), block.binding) unless legend.blank?
    concat(content, block.binding)
    concat("</fieldset>", block.binding) 
  end

Here's the rationale: http://github.com/rails/rails/co...>