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.

Clemens Kofler

Clemens Kofler

Not a member

Clemens Kofler's latest activity

Thursday, July 17 2008
641 FormHelper#fields_for has illogical (and unused) option was updated in Ruby on Rails
  • Clemens Kofler
    Clemens Kofler commented at 1:34 PM

    RSL: Good catch but that's not exactly true.

    As I said, all tests pass which means it doesn't break any existing functionality. Moreover, form_for only uses f...
  • Clemens Kofler
    Clemens Kofler created the ticket at 12:12 PM

    I've just found an option in FormHelper#fields_for that seems to be both, illogical and (therefore) unused.

    In the case statement, there's the following snipp...
638 DateHelper#date_or_time_select and DateHelper#default_time_from_options modif... was created in Ruby on Rails
  • Clemens Kofler
    Clemens Kofler created the ticket at 12:28 AM

    While working on a plugin that monkey patched some of Rails' DateHelper methods, I've stumbled across a small but potentially very confusing bug in date_or_tim...
Saturday, July 12 2008
606 Inconsistent functionality in date_helper.rb was created in Ruby on Rails
  • Clemens Kofler
    Clemens Kofler created the ticket at 8:51 PM

    I was working on some doc patches for the date/datetime helpers when I found out that there are some inconsistencies. IMO, all date helpers (both, object relat...
257 add branch/tag option for 'script plugin/install' with git was updated in Ruby on Rails
  • Clemens Kofler
    • Tag changed from enhancement, patch, tested to enhancement, patch, plugins, tested
    by Clemens Kofler at 6:01 PM

    +1 (although plugins should be replaced by gems anyway).

Friday, July 11 2008
593 Paragraphed anchors and notice layout in generators was updated in Ruby on Rails
  • Clemens Kofler
    • Tag changed from 2.1, generators, patch to 2.1, generators, patch, railties
    by Clemens Kofler at 10:57 AM

    All tests pass. Although I think the scaffold shouldn't be used anyways (or, rather, that you usually throw away the views), it kinda makes sense.

    • 1/2 ;-)
600 Print controller module into request log was updated in Ruby on Rails
  • Clemens Kofler
    Clemens Kofler commented at 10:47 AM

    Little note: A single test fails ->

    1) Failure:

    test_action_cache_conditional_options(ActionCacheTest)

    [./test/controller/caching_test.rb:290:in `test_actio...
  • Clemens Kofler
    • Tag changed from actionpack, controller, enhancement, patch to actionpack, controller, enhancement, patch, tested
    by Clemens Kofler at 10:14 AM

    +1 from me. It passes the tests and makes total sense.

601 Can't have a two letter model name was updated in Ruby on Rails
  • Clemens Kofler
    • Tag changed from model naming, 2.1 to 2.1, activerecord, bug
    by Clemens Kofler at 10:07 AM

    I can't confirm this.

    ./script/generate scaffold lp name:string

    rake db:migrate

    Works fine in ./script/console:

    Loading development environment (Rails 2.1....
Thursday, July 10 2008
458 record.to_xml(:methods => other_ids) yields incorrect representation was updated in Ruby on Rails
  • Clemens Kofler
    Clemens Kofler commented at 8:09 PM

    Oops, didn't mean to change the tags - that just happened ... (but I guess it's right that way anyways)