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.

Peter Marklund

Peter Marklund

Rails Developer · Not a member

Peter Marklund's latest activity

Sunday, May 10 2009
2543 Make ActiveRecord::Base.exists? invoke find_initial to support :include scopes was updated in Ruby on Rails
  • Peter Marklund
    Peter Marklund commented at 9:41 PM

    Frederick, thanks for the input on performance implications of count. I just wanted to give a recap of the options discussed here. We have:

    1) The current imple...
Thursday, May 07 2009
2617 Global Ruby Methods Causing ActiveRecord Attribute Methods to Never be Defined was created in Ruby on Rails
  • Peter Marklund
    Peter Marklund created the ticket at 2:31 PM

    I ran into a clash between RSpec and the acts_as_taggable_on plugin. It turns out that a global method can cause the ActiveRecord attribute methods to never be ...
Tuesday, April 28 2009
2575 Not Eager Loading Application Classes Causes STI Breakage was created in Ruby on Rails
  • Peter Marklund
    Peter Marklund created the ticket at 2:25 PM

    It turns out that the fact that we don't typically eager load application classes in the development environment (since cache_classes is false) can break STI si...
Monday, April 27 2009
2543 Make ActiveRecord::Base.exists? invoke find_initial to support :include scopes was updated in Ruby on Rails
  • Peter Marklund
    Peter Marklund commented at 10:17 AM

    Thanks Michael! I think the count approach sounds like a really good. I uploaded a new patch.

    Cheers

    Peter
Wednesday, April 22 2009
2543 Make ActiveRecord::Base.exists? invoke find_initial to support :include scopes was updated in Ruby on Rails
  • Peter Marklund
    • Tag changed from 2-3-stable, 2.3.2, activerecord, exists, scope to 2-3-stable, 2.3.2, activerecord, exists, patch, scope
    by Peter Marklund at 12:34 PM
  • Peter Marklund
    Peter Marklund created the ticket at 12:32 PM

    Currently ActiveRecord::Base.exists? does not support scopes with the :include option and it breaks for example if you have a scope with :include and an :order ...
Tuesday, March 17 2009
2271 Reset request_params in TestRequest#recycle! was created in Ruby on Rails
  • Peter Marklund
    Peter Marklund created the ticket at 2:48 PM

    We used to do self.request_params = {} in TestRequest#recycle! but this has been taken out, I guess as part of the Rack rewrite. The consequence is that if you ...