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.

Luca Guidi

Luca Guidi

Not a member

Luca Guidi's latest activity

Wednesday, June 10 2009
2738 [PATCH] Fix zero length stylesheet cache files bug in case of missing CSS files was updated in Ruby on Rails
  • Luca Guidi
    Luca Guidi commented at 10:22 AM

    You shouldn't use File#readable? because it doesn't make sense on Windows, in fact that method is used to check if the file is readable by the effective user id...
2251 AssociationCollection#destroy should only delete join table records was updated in Ruby on Rails
  • Luca Guidi
    Luca Guidi commented at 10:04 AM

    Ok, if hmt is only a convenient accessor, we should revert my original patch from master too. Please look at the description of this ticket (Post, Tagging and T...
Monday, June 08 2009
2251 AssociationCollection#destroy should only delete join table records was updated in Ruby on Rails
  • Luca Guidi
    Luca Guidi commented at 10:02 AM

    Ok, I believe we should focus our attention on the nature of hmt.

    I think it was introduced as an habmt on steroids association, look at my latest example above...
Wednesday, June 03 2009
2251 AssociationCollection#destroy should only delete join table records was updated in Ruby on Rails
  • Luca Guidi
    Luca Guidi commented at 2:36 PM

    I found a contradictory behavior, if look at [has_many_association_test.rb:858](http://github.com/rails/rails/blob/971e2438d98326c994ec6d3ef8e37b7e868ed6e2/acti...
  • Luca Guidi
    Luca Guidi commented at 11:08 AM

    I gave a look at the Olly's gist, I don't know if it's a proper way to use an hmt relation, please look at the Contact class, it uses both belongs_to and has_ma...
Sunday, May 24 2009
2251 AssociationCollection#destroy should only delete join table records was updated in Ruby on Rails
  • Luca Guidi
    Luca Guidi commented at 6:17 PM

    It works fine for me.
    Olly, please look at this gist (http://bit.ly/siWGR) and tell me if I'm missing something.

    I vendored Rails, then checked out the this com...
Friday, May 22 2009
2251 AssociationCollection#destroy should only delete join table records was updated in Ruby on Rails
  • Luca Guidi
    Luca Guidi commented at 3:49 PM

    I'll verify it ASAP
Thursday, April 09 2009
2251 AssociationCollection#destroy should only delete join table records was updated in Ruby on Rails
Tuesday, March 31 2009
2383 SqlBypass session store doesn't use ActiveRecord::Base connection was created in Ruby on Rails
  • Luca Guidi
    Luca Guidi created the ticket at 3:52 PM

    All the methods in ActiveRecord::SessionStore::SqlBypass use @@connection class var, instead of the homonym class method. This of course doesn't not guarantee t...
Monday, March 16 2009
2251 AssociationCollection#destroy should only delete join table records was updated in Ruby on Rails
  • Luca Guidi
    • Tag changed from activerecord, associations, destroy, has_many_through to activerecord, associations, destroy, has_many_through, patch
    by Luca Guidi at 12:32 PM

    Ok, I changed the order. Previously I was calling super before, just to perform the raise_on_type_mismatch check. Now I wrapped the operation in a transaction, ...