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.

Joe Van Dyk's latest activity

Tuesday, April 26 2011
6746 test/cases/finder_test.rb won't run on its own was updated in Ruby on Rails
  • Joe Van Dyk
    Joe Van Dyk commented at 5:23 AM

    Bah, here is the correct patch!

  • Joe Van Dyk
    Joe Van Dyk created the ticket at 5:21 AM

    On 2-3-stable, activerecord/test/cases/finder_test.rb is missing "require 'models/project'", causing it to fail if ran by itself. (Line 124 refers to it).

    Not a...

Friday, February 04 2011
6187 PostgreSQL and Rails 3.0.3 migrations fail with index name length > 64 chars was updated in Ruby on Rails
  • Joe Van Dyk
    Joe Van Dyk commented at 4:37 PM

    I'm on pgsql 9 with Rails 2.3:


    SQL (1.4ms) BEGIN Index name 'index_ratings_on_rating_type_and_rating_id_and_rated_type_and_rating' on table 'ratings' is too lo...

Tuesday, November 23 2010
4498 Rails loses precision when deserializing timestamps from Postgresql was updated in Ruby on Rails
  • Joe Van Dyk
    Joe Van Dyk commented at 9:36 PM

    I'm sort of confused by your test case.

    After updating all the records to now(), why would you expect them to match an earlier time?

    If you're not inside a tran...

Monday, December 28 2009
3626 Inflector.underscore does unnecessary string copies was updated in Ruby on Rails
  • Joe Van Dyk
    Joe Van Dyk commented at 7:54 PM

    Exact patch, but fixed the author's name and email in the commit log.

  • Joe Van Dyk
    Joe Van Dyk created the ticket at 7:15 PM

    The method uses gsub multiple times, which does unnecessary string copies.

    Patch fixes this. No tests need updating. Benchmarks shows the new method is about tw...

1853 Add ability to get multiple memcached keys at the same time. was updated in Ruby on Rails
  • Joe Van Dyk
    Joe Van Dyk commented at 7:11 PM

    It looks like this was merged in Feb 2009 in commit ad857712. Mark as resolved?

Sunday, September 27 2009
2989 Setting the id of a belongs_to object does not update the reference to the ob... was updated in Ruby on Rails
  • Joe Van Dyk
    Joe Van Dyk commented at 3:33 AM

    Elad, good catch about the foreign key.

  • Joe Van Dyk
    • Tag changed from active_record, associations, belongs_to, bugmash to 2-3-stable, active_record, associations, belongs_to, bugmash
    by Joe Van Dyk at 1:34 AM

    2.3 stable, sorry.

    Elad, that was my thinking. Also, the association should only be reloaded if the foreign key was changed.

1948 Transaction block sets model ID to non-existent row was updated in Ruby on Rails
  • Joe Van Dyk
    Joe Van Dyk commented at 1:43 AM

    +1 verified on sqlite3 as well.