@@ -37,6 +37,7 @@ module Merger def merge! keep.class.transaction do associations! + duplicates.each {|duplicate| duplicate.send(:merged_with, keep) if duplicate.respond_to?(:merged_with) } duplicates.each(&:destroy) if options[:destroy] end end lib/merger/merge.rb @@ -74,4 +74,9 @@ class Merger::MergeTest < Test::Unit::TestCase Merger::Merge.new(people(:original), people(:duplicate), :destroy => false).merge! assert_not_nil Person.find_by_id(people(:duplicate).id) end + + def test_merged_with_callback + people(:duplicate).expects(:merged_with).with(people(:original)) + @merge.merge! + end end \ No newline at end of file test/merger/merge_test.rb @@ -2,6 +2,7 @@ $:.unshift(File.dirname(__FILE__) + '/../lib') require 'rubygems' require 'test/unit' +require 'mocha' require 'multi_rails_init' require 'active_record' require 'active_record/fixtures' test/test_helper.rb 07fa702d142f366a2faeff23d47711210bac6067 Jean-Francois Couture jfcouture@gmail.com http://github.com/collectiveidea/merger/commit/32f2fbc133daf31a849f3152e1c512895faad120 32f2fbc133daf31a849f3152e1c512895faad120 2009-04-24T06:07:47-07:00 2009-04-16T08:32:34-07:00 added simple callback merged_with: if the model has a merged_with(record), it will get called after the merge useful with the :destroy => false options if you want to keep the model for historical purpose or as a safety precaution 1d9891de9be5cc754434b5d74be436a7b683f5f7 Brandon Keepers brandon@collectiveidea.com