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.

FormBuilder date helpers ignore html_options

#506

It turned out that the form builder (f.) helpers of date_select, datetime_select and time_select ignore the html_options you pass to it.

They do work for the normal helper from DateHelpers class however, the problem is that the html_options hash simply gets not passed through to them in their form builder definitions.

Some Rails developers apparently need more coffee. ;-)

Reported by Pascal Ehlert · June 28th, 2008 @ 04:15 PM

State: resolved
Milestone: 2.x
Assigned to: nobody
Importance: Low

Activity

  1. Pratik
    Pratik
    • State changed from new to incomplete

    Hey Pascal,

    Could you please add tests as well ?

    Thanks.

    June 28th, 2008 @ 06:59 PM

  2. Pascal Ehlert
    Pascal Ehlert
    • Tag changed from 2.1, actionpack, forms, helper, patch to 2.1, actionpack, forms, helper, patch, tests

    Sorry for not including them in the first patch, I was a bit in a hurry earlier.

    I have attached an updated version with tests. Hope this makes it into trunk soon.

    June 28th, 2008 @ 07:43 PM

  3. Pratik
    Pratik
    • State changed from incomplete to open

    June 28th, 2008 @ 08:24 PM

  4. Repository
    Repository

    (from [029a7455846cde2654958358a6fe354f236ade35]) Ensure FormBuilder date helpers respects html_options. [#506 state:resolved] [Pascal Ehlert]

    Signed-off-by: Pratik Naik

    http://github.com/rails/rails/co...

    June 29th, 2008 @ 12:51 AM

  5. Repository
    Repository

    (from [b416c6880bba538799513f4afd4125a0be8c2fb5]) Ensure FormBuilder date helpers respects html_options. [#506 state:resolved] [Pascal Ehlert]

    Signed-off-by: Pratik Naik

    Conflicts:

    actionpack/test/template/date_helper_test.rb

    http://github.com/rails/rails/co...

    June 29th, 2008 @ 12:52 AM

  6. Pascal Ehlert
    Pascal Ehlert

    Thanks for fixing, Pratik!

    Shouldn't this ticket be closed now?

    July 2nd, 2008 @ 10:43 AM

  7. Pratik
    Pratik
    • State changed from open to resolved

    July 2nd, 2008 @ 12:50 PM