@@ -279,6 +279,7 @@ module ActiveRecord other_array_valid = false unless delete(@target.select { |v| !other.include?(v) }) concat(other_array.select { |v| !current.include?(v) }) + @target = other_array # Rollback any database transactions if all the other objects aren't valid raise ReplacementArrayInvalidException.new unless other_array_valid activerecord/lib/active_record/associations/association_collection.rb 03c9375ebc5d3351e32ce85f1147244db10fa76b Gavin Stark g.stark@realdigitalmedia.com http://github.com/gstark/rails/commit/54ff471423037de41dc4e7fc4e2443a15dd8cdd8 54ff471423037de41dc4e7fc4e2443a15dd8cdd8 2008-07-20T14:23:42-07:00 2008-07-20T14:23:42-07:00 Forgot to reset the @target so that the association has the _new_ values so that the master object validation fails, and our values are the assigned (yet invalid) ones. 7557c24325501304c3cab35dc40537f4e00c0fde Gavin Stark g.stark@realdigitalmedia.com