laserlemon-vestal_versions 0.5.0 → 0.6.0

data/VERSION

@@ -1 +1 @@
-0.5.0
+0.6.0

data/lib/vestal_versions.rb

@@ -8,7 +8,7 @@ module LaserLemon
 
     module ClassMethods
       def versioned
-        has_many :versions, :as => :versioned, :order => 'versions.number ASC', :dependent => :destroy do
+        has_many :versions, :as => :versioned, :order => 'versions.number ASC', :dependent => :delete_all do
           def between(from_value, to_value)
             from, to = number_at(from_value), number_at(to_value)
             return [] if from.nil? || to.nil?
@@ -37,7 +37,9 @@ module LaserLemon
           end
         end
 
-        after_save :create_version, :if => :needs_version?
+        after_create :create_initial_version
+        after_update :create_initial_version, :if => :needs_initial_version?
+        after_update :create_version, :if => :needs_version?
 
         include InstanceMethods
         alias_method_chain :reload, :versions
@@ -46,6 +48,10 @@ module LaserLemon
 
     module InstanceMethods
       private
+        def needs_initial_version?
+          versions.empty?
+        end
+
         def needs_version?
           !changed.empty?
         end
@@ -55,13 +61,12 @@ module LaserLemon
           @version = new_version
         end
 
-        def create_version
-          if versions.empty?
-            versions.create(:changes => attributes, :number => 1)
-          else
-            versions.create(:changes => changes, :number => (last_version + 1))
-          end
+        def create_initial_version
+          versions.create(:changes => nil, :number => 1)
+        end
 
+        def create_version
+          versions.create(:changes => changes, :number => (last_version + 1))
           reset_version
         end
 

data/test/changes_test.rb

@@ -6,8 +6,8 @@ class ChangesTest < Test::Unit::TestCase
       @user = User.create(:name => 'Steve Richert')
     end
 
-    should "initially equal its parent's attributes" do
-      assert_equal @user.attributes, @user.versions.first.changes
+    should "initially be blank" do
+      assert @user.versions.first.changes.blank?
     end
 
     should 'contain all changed attributes' do

data/vestal_versions.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{vestal_versions}
-  s.version = "0.5.0"
+  s.version = "0.6.0"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["laserlemon"]
-  s.date = %q{2009-09-03}
+  s.date = %q{2009-09-05}
   s.description = %q{Keep a DRY history of your ActiveRecord models' changes}
   s.email = %q{steve@laserlemon.com}
   s.extra_rdoc_files = [

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: laserlemon-vestal_versions
 version: !ruby/object:Gem::Version 
-  version: 0.5.0
+  version: 0.6.0
 platform: ruby
 authors: 
 - laserlemon
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2009-09-03 00:00:00 -07:00
+date: 2009-09-05 00:00:00 -07:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency