Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

This project is archived and is in readonly mode.

bug in ActionView::Helpers::FormBuilder

#531

in form_options_helper.rb, at the very end of the file, the class FormBuilder receives 4 new methods (select, etc.). The code forgets to merge the builder default options to the html_options. As a result, for instance, a :index will be ignored (that's how I spotted it). I fixed it on my local installation: replace

def select(method, ...)

@template.select(..., html_options)

end

with:

def select(method, ...)

@template.select(..., @default_options.merge(html_options))

end

Same for the 3 methods after this one.

Cheers

Reported by Thierry · July 1st, 2008 @ 10:19 PM

State: wontfix
Milestone: 2.x
Assigned to: nobody
Importance: none

Activity

  1. josh
    josh
    • State changed from new to wontfix
    • Tag changed from 2.1, actionpack, bug to 2.1, actionpack, bug

    October 5th, 2008 @ 03:16 PM