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.

This project is archived and is in readonly mode.

All changesets

Changeset [0fee3592786da833bcfbe20afc3a9483118c3c12] by Will Bryant

Revision 0fee3592786da833bcfbe20afc3a9483118c3c12, committed by Will Bryant · December 7th, 2010 @ 08:48 PM

Don't add non-new records back to the target array after loading targets on associations, as that makes destroy_all destroy any created records that don't match the scope destroy_all is called on

Signed-off-by: Michael Koziarski michael@koziarski.com
https://github.com/rails/rails/commit/0fee3592786da833bcfbe20afc3a9...

3 changed files

  • M activerecord/lib/active_record/associations/association_collection.rb
  • M activerecord/test/cases/associations/has_many_associations_test.rb
  • M activerecord/test/models/company.rb