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.

Mike Breen

Mike Breen

Not a member

Mike Breen's latest activity

Saturday, May 15 2010
3419 Allow ActiveRecord's find to take a Range was updated in Ruby on Rails
  • Mike Breen
    Mike Breen commented at 5:40 PM

    Updated the 2.3x patch with Jeremy's recommendations.
Saturday, April 24 2010
2884 reverse_merge!() modifies the hash outside the method scope was updated in Ruby on Rails
  • Mike Breen
    • Assigned user set to Ryan Bigg
    by Mike Breen at 6:54 PM

    Ruby passes params by reference not by value so I'm not sure why we would expect Rails to behave differently.

    What you probably want to do here is to use the no...
4043 Oracle or Sybase? was updated in Ruby on Rails
  • Mike Breen
    • Assigned user set to Ryan Bigg
    by Mike Breen at 6:15 PM

    This question is better suited for the mailing list or irc.
Thursday, December 03 2009
1211 reset_counter_cache was updated in Ruby on Rails
  • Mike Breen
    Mike Breen commented at 7:23 PM

    I think how you just put it is perfect :)

    Thanks for taking the time to clarify what exactly the worse case scenario is.
  • Mike Breen
    Mike Breen commented at 1:59 PM

    +1 for the new patch.

    Although the original ticket was solving a different use case (something that you would run infrequently, possibly during a migration) thi...
Wednesday, October 28 2009
1878 a generated plugin's tests are not run by 'rake test' was updated in Ruby on Rails
Monday, October 26 2009
3419 Allow ActiveRecord's find to take a Range was updated in Ruby on Rails
  • Mike Breen
    • Tag changed from 2, 2-3-stable, active_record, find, patch to 2-3-stable, active_record, find, patch
    by Mike Breen at 8:50 PM
  • Mike Breen
    Mike Breen commented at 1:56 PM

    @Matt Jones - Thanks for the feedback.

    This seems like a bad idea - what about people who are using non-numeric primary keys (GUIDs, for instance)?

    You're cor...
Sunday, October 25 2009
3419 Allow ActiveRecord's find to take a Range was updated in Ruby on Rails
  • Mike Breen
    Mike Breen commented at 11:16 AM

    @MatthewRudy - that's much nicer. Thanks!

    I've attached an updated patch with Matthew's changes.