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.

Jeff Kreeftmeijer's latest activity

Saturday, March 12 2011
6555 delete_all doesn't work when chained with joins was updated in Ruby on Rails
  • Jeff Kreeftmeijer
    • Tag changed from activerecord rails3, bug, delete_all, edge, patch, tested to activerecord, delete_all, patch
    by Jeff Kreeftmeijer at 12:03 PM
  • Jeff Kreeftmeijer
    • Assigned user set to Jeff Kreeftmeijer
    by Jeff Kreeftmeijer at 12:02 PM

    Akira's patch fixes the issue for me, but the attached test fails because Pet.joins(:toys).where(:toys => {:name => 'Bone'}).count returns 0.

    Also, I'm not a gr...

Wednesday, March 09 2011
6259 validate_presence_of does not work with :allow_nil => true was updated in Ruby on Rails
Tuesday, March 08 2011
6259 validate_presence_of does not work with :allow_nil => true was updated in Ruby on Rails
  • Jeff Kreeftmeijer
    Jeff Kreeftmeijer commented at 11:46 AM

    While adding a note to the guides telling users not to use allow_nil or allow_blank, I found this:

    Using +:allow_nil+ with +validates_presence_of+ allows for +...
  • Jeff Kreeftmeijer
    Jeff Kreeftmeijer commented at 10:46 AM

    :allow_nil and allow_blank now both et ignored on presence validations. Andrew's right, if we want to do some kind of zero-length validation that allows nil for...

Monday, March 07 2011
6259 validate_presence_of does not work with :allow_nil => true was updated in Ruby on Rails
  • Jeff Kreeftmeijer
    • State changed from new to open
    • Assigned user set to Jeff Kreeftmeijer
    • Importance changed from to Low
    by Jeff Kreeftmeijer at 5:58 PM

    I did a quick test and it seems you're right: https://gist.github.com/858816

    allow_blank doesn't do anything when validating presence, which seems sensible to m...

Sunday, March 06 2011
6510 [PATCH] core_ext/enumerable multiple fixes was updated in Ruby on Rails
  • Jeff Kreeftmeijer
    • Importance changed from to Low
    by Jeff Kreeftmeijer at 9:16 AM

    Hi Marc-André,

    I cleaned up your ticket a bit. Please be sure to wrap inline code in backticks, since starting a line with a # will turn it into a header. :)

Saturday, March 05 2011
6447 reload! doesn't reload was updated in Ruby on Rails
  • Jeff Kreeftmeijer
    • State changed from open to wontfix
    by Jeff Kreeftmeijer at 12:03 PM

    Hi William,

    After talking to Josh about this, we came to this conclusion:

    The u object doesn't get reloaded because it already existed before the model changed ...

Tuesday, March 01 2011
6447 reload! doesn't reload was updated in Ruby on Rails
  • Jeff Kreeftmeijer
    • State changed from needs-more-info to open
    by Jeff Kreeftmeijer at 6:45 PM

    Thanks Will, I tried to reproduce again:

    # app/models/user.rb
    
    class User < ActiveRecord::Base
      def reloaded?
        false
      end
    end
    
    $ script/rails c
    Loading d...