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.

There was a problem

You must be a member of this account.

Jeff Dean

Jeff Dean

Not a member

Jeff Dean's latest activity

Wednesday, June 09 2010
4803 [PATCH] remove_column should raise an ArgumentError when no columns are passed was updated in Ruby on Rails
  • Jeff Dean
    • Tag changed from activecord, migrations to activecord, migrations, patch
    by Jeff Dean at 6:02 AM
  • Jeff Dean
    Jeff Dean commented at 6:00 AM
  • Jeff Dean
    Jeff Dean created the ticket at 5:57 AM

    Currently in migrations you can write the following:

    remove_column :foo

    This will execute without errors, but will make no change to your database. I think th...
Saturday, May 15 2010
4559 sending :id => nil to form helpers does not remove the "id" html attribute was updated in Ruby on Rails
  • Jeff Dean
    Jeff Dean commented at 10:13 PM

    Awesome - thanks José!

    I just realized that I used the arity-2 form of fetch, instead of the block format. Here is a patch that will make this a bit more perfor...
  • Jeff Dean
    Jeff Dean commented at 9:26 PM

    Thanks Casey. "select" actually did work as expected, and the syntax is:

    select("post", "secret", [], {}, "id" => nil)

    I've reattached the patch with an extra...
  • Jeff Dean
    Jeff Dean commented at 9:15 PM

    I think it should be consistent. I'm adding support for that now, and will re-post a patch.
Monday, May 10 2010
4558 sending :alt => nil to an image tag should omit the images alt tag was updated in Ruby on Rails
  • Jeff Dean
    • Tag cleared.
    by Jeff Dean at 2:36 AM

    And thanks for Marc-Andre for introducing me to fetch last year! I forgot about the block syntax.
Sunday, May 09 2010
4559 sending :id => nil to form helpers does not remove the "id" html attribute was updated in Ruby on Rails
  • Jeff Dean
    Jeff Dean commented at 7:32 PM

    For me the biggest issue is inconsistency. If I pass :class => nil to a text_field, no class attribute is emitted. If I pass :id => nil, the default id is emitt...
  • Jeff Dean
    Jeff Dean commented at 7:55 AM

    Here's a patch with tests.
  • Jeff Dean
    Jeff Dean created the ticket at 7:51 AM

    I have several apps where I don't need the id for anything. Especially now that nested_attributes produce such long ids, I frequently want to omit the "id" html...