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.

Sam Saffron's latest activity

Tuesday, February 09 2010
3908 Arel depends on a too recent version of Active Support was created in Ruby on Rails
  • Sam Saffron
    Sam Saffron created the ticket at 1:32 PM

    Arel in master depends on ActiveSupport 3, when in fact all the APIs it uses work fine in ActiveSupport 2.3.5 (classify, delegate, cattr_accessor)

    As a building...

Monday, January 25 2010
2202 IE7 does not respect "return false;" in onclick was updated in Ruby on Rails
  • Sam Saffron
    Sam Saffron commented at 5:12 AM

    which is fixed in 1.4.1 http://is.gd/6YChL

  • Sam Saffron
    Sam Saffron commented at 5:08 AM

    This happens due to a bug in jquery 1.4 release !! assuming it was around in other forms due to similar javascript hackery.

    http://dev.jquery.com/ticket/5835

  • Sam Saffron
    Sam Saffron commented at 4:26 AM

    Cowboys, I just lost 2 hours of my time to this stupid bug, I can confirm this happens in ie8. I can also confirm this patch fixes it.

    Please accept this patch.

Thursday, January 21 2010
3758 Rack middleware is exploding in Rails 3 was updated in Ruby on Rails
  • Sam Saffron
    • Tag set to middleware, rails3
    by Sam Saffron at 12:02 AM
3759 Rails 3 AR performance has regressed since Rails 2 was updated in Ruby on Rails
  • Sam Saffron
    • Tag set to active_record, performance, rails3
    by Sam Saffron at 12:01 AM
Wednesday, January 20 2010
3759 Rails 3 AR performance has regressed since Rails 2 was created in Ruby on Rails
  • Sam Saffron
    Sam Saffron created the ticket at 10:14 PM

    Using a trivial model on both Rails 2 and 3:

    script/generate scaffold post title:string body:text

    I'm noticing both insert and create performance are much slowe...

3758 Rack middleware is exploding in Rails 3 was created in Ruby on Rails
  • Sam Saffron
    Sam Saffron created the ticket at 9:54 PM

    I have the following middleware:

    class RequestCache
    
      def self.[](lookup)
        cache[lookup]
      end
    
      def self.[]=(lookup, val)
        cache[lookup] = val
      end
    
    ...
3739 Active record is getting confused with some complex has_many conditions was updated in Ruby on Rails
  • Sam Saffron
    Sam Saffron commented at 9:51 PM

    Pratik,

    Still getting this in trunk, I tried setting up an equivalent test with treasures and was not able to repro. Will continue trying to dig down.

Monday, January 18 2010
3739 Active record is getting confused with some complex has_many conditions was created in Ruby on Rails
  • Sam Saffron
    Sam Saffron created the ticket at 1:18 PM

    I have this has_many clause

    has_many :top_comments, :order => 'created_at asc', :conditions => {deleted => false}, :limit => Comment.comments_to_display , :as =...