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.

David Röthlisberger's latest activity

Saturday, October 17 2009
1031 handle parameterized associations was updated in Ruby on Rails
Tuesday, October 14 2008
1127 :allow_nil option for delegate was updated in Ruby on Rails
  • David Röthlisberger
    David Röthlisberger commented at 4:46 PM

    How is it hiding the error any more than:

    
    def contact_name; contact && contact.name; end
    

    It's intended as a shortcut for the case where not having a contact...

  • David Röthlisberger
    David Röthlisberger commented at 2:07 PM

    (Um, that "1." in the comment above was supposed to be a "+1").

  • David Röthlisberger
    David Röthlisberger commented at 2:06 PM
    1. Useful, and the patch is very simple.

    If anything, I'd change the rdoc to say "If the object to which you delegate...".

Tuesday, September 16 2008
1058 Duplicate unit test names: test_nested_fields_for_with_index was created in Ruby on Rails
Thursday, July 31 2008
735 Documentation for ActiveRecord#find(:select) was created in Ruby on Rails
Thursday, July 17 2008
181 Incorrect input name with nested fields_for was updated in Ruby on Rails
  • David Röthlisberger
    David Röthlisberger commented at 2:01 AM

    I am swamped under at the moment, and the rails internals are still black magic to me, so it'll be at least a month before I get around to trying to fix it.

    I...

Tuesday, May 13 2008
181 Incorrect input name with nested fields_for was created in Ruby on Rails
  • David Röthlisberger
    David Röthlisberger created the ticket at 4:36 PM
    form_for('post[]', @post) do |f|
      f.fields_for('comment[]', @comment) do |c|
        c.text_field(:name)
      end
    end
    

    I expect:

    <input name="post[123][comment]...
Thursday, May 08 2008
145 collection_select doesn't honour :index option to fields_for was updated in Ruby on Rails
  • David Röthlisberger
    David Röthlisberger commented at 5:37 PM

    In the previous comment, the second test should be named "test_collection_select_under_fields_for_with_collection_and_index".

  • David Röthlisberger
    David Röthlisberger commented at 5:35 PM

    This is on edge rails.

    See the test cases in attached 145-001.patch:

    the first test (test_collection_select_under_fields_for_with_collection) passes;

    the se...