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.

Marc-André Lafortune's latest activity

Sunday, March 06 2011
6536 [PATCH] Fix fixture default value for 'type' column was updated in Ruby on Rails
Saturday, March 05 2011
6536 [PATCH] Fix fixture default value for 'type' column was created in Ruby on Rails
  • Marc-André Lafortune
    Marc-André Lafortune created the ticket at 9:29 PM

    Fixtures created by the rails generators are usually valid, except if STI is used. Indeed, the default value "MyString" is not appropriate for a column of name ...

6534 [PATCH] Fixing inconsistencies with Time{WithZone}#{hash,eql?} was updated in Ruby on Rails
  • Marc-André Lafortune
    Marc-André Lafortune commented at 8:52 PM
  • Marc-André Lafortune
    Marc-André Lafortune created the ticket at 8:51 PM

    If foo.eql?(bar) returns true then so should bar.eql?(foo).
    Moreover, foo.hash must then be the same as bar.hash.
    Otherwise the Hash lookup will fail. TimeWithZ...

Thursday, March 03 2011
6510 [PATCH] core_ext/enumerable multiple fixes was created in Ruby on Rails
  • Marc-André Lafortune
    Marc-André Lafortune created the ticket at 5:11 AM

    I got burned by the fact that Enumerable#index_by does not return an Enumerator if called without block, so I took a moment to fix the issues I could find in th...

Wednesday, August 04 2010
5290 register_javascript_include_default was created in Ruby on Rails
Wednesday, May 26 2010
4555 active_support/core_ext/array/random_access.rb conflicts with standard Ruby l... was updated in Ruby on Rails
  • Marc-André Lafortune
    Marc-André Lafortune commented at 6:49 AM

    I'm surprised nobody pointed out that Array#sample is what it should be called, since that's what exists natively in Ruby 1.9. Ideally it should also support an...

Sunday, May 09 2010
4558 sending :alt => nil to an image tag should omit the images alt tag was updated in Ruby on Rails
  • Marc-André Lafortune
    Marc-André Lafortune commented at 9:19 PM

    Let's try the attachment again...

  • Marc-André Lafortune
    Marc-André Lafortune commented at 9:19 PM

    Hi Jeff!

    I'd recommend using the block form of fetch, to avoid an unnecessary calculation in case the :alt option is specified. Modified diff attached.

Saturday, May 08 2010
4457 find_or_create_by_foo when foo is protected attribute was updated in Ruby on Rails
  • Marc-André Lafortune
    Marc-André Lafortune commented at 5:20 AM

    Great! Thanks Santiago.

    I actually felt bad not taking the time to write a patch for this...