when merging post versions, delete old version rather than updating it

app/models/post.rb

@@ -1326,10 +1326,10 @@ class Post < ActiveRecord::Base
     def create_version(force = false)
       if new_record? || rating_changed? || source_changed? || parent_id_changed? || tag_string_changed? || force
         if merge_version?
-          merge_version
+          delete_previous_version
-        else
-          create_new_version
         end
+
+        create_new_version
       end
     end
 
@@ -1348,16 +1348,9 @@ class Post < ActiveRecord::Base
       )
     end
 
-    def merge_version
+    def delete_previous_version
       prev = versions.last
-      if prev
+      prev.destroy
-        prev.update_attributes(
-          :rating => rating,
-          :source => source,
-          :tags => tag_string,
-          :parent_id => parent_id
-        )
-      end
     end
 
     def revert_to(target)

test/unit/post_version_test.rb

@@ -59,6 +59,20 @@ class PostVersionTest < ActiveSupport::TestCase
       end
     end
 
+    context "that should be merged" do
+      setup do
+        @parent = FactoryGirl.create(:post)
+        @post = FactoryGirl.create(:post, :tag_string => "aaa bbb ccc", :rating => "q", :source => "xyz")
+      end
+
+      should "delete the previous version" do
+        assert_equal(1, @post.versions.count)
+        @post.update_attributes(:tag_string => "bbb ccc xxx", :source => "")
+        @post.reload
+        assert_equal(1, @post.versions.count)
+      end
+    end
+
     context "that has been updated" do
       setup do
         @parent = FactoryGirl.create(:post)