paper_trail 2.6.2 → 2.6.3

data/README.md

@@ -347,7 +347,7 @@ You can specify custom version subclasses with the `:class_name` option:
 
     class PostVersion < Version
       # custom behaviour, e.g:
-      set_table_name :post_versions
+      self.table_name = :post_versions
     end
 
     class Post < ActiveRecord::Base
@@ -718,7 +718,8 @@ Many thanks to:
 * [Jason Noble](https://github.com/jasonnoble)
 * [Jared Mehle](https://github.com/jrmehle)
 * [Eric Schwartz](https://github.com/emschwar)
-
+* [Ben Woosley](https://github.com/Empact)
+* [Philip Arndt](https://github.com/parndt)
 
 
 ## Inspirations

data/lib/paper_trail/version.rb

@@ -7,12 +7,24 @@ class Version < ActiveRecord::Base
     scoped(:conditions => { :item_type => item_type, :item_id => item_id })
   end
 
+  def self.creates
+    where :event => 'create'
+  end
+
+  def self.updates
+    where :event => 'update'
+  end
+
+  def self.destroys
+    where :event => 'destroy'
+  end
+
   scope :subsequent, lambda { |version|
-    where(["#{self.primary_key} > ?", version.is_a?(self) ? version.send(self.primary_key) : version]).order("#{self.primary_key} ASC")
+    where(["#{self.primary_key} > ?", version]).order("#{self.primary_key} ASC")
   }
 
   scope :preceding, lambda { |version|
-    where(["#{self.primary_key} < ?", version.is_a?(self) ? version.send(self.primary_key) : version]).order("#{self.primary_key} DESC")
+    where(["#{self.primary_key} < ?", version]).order("#{self.primary_key} DESC")
   }
 
   scope :following, lambda { |timestamp|
@@ -68,9 +80,9 @@ class Version < ActiveRecord::Base
         end
 
         attrs.each do |k, v|
-          begin
-            model.send :write_attribute, k.to_sym , v
-          rescue NoMethodError
+          if model.respond_to?("#{k}=")
+            model.send :write_attribute, k.to_sym, v
+          else
             logger.warn "Attribute #{k} does not exist on #{item_type} (Version id: #{id})."
           end
         end

data/lib/paper_trail/version_number.rb

@@ -1,3 +1,3 @@
 module PaperTrail
-  VERSION = '2.6.2'
+  VERSION = '2.6.3'
 end

data/paper_trail.gemspec

@@ -15,10 +15,11 @@ Gem::Specification.new do |s|
   s.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
   s.require_paths = ['lib']
 
-  s.add_dependency 'rails', '~> 3'
+  s.add_dependency 'railties', '~> 3.0'
+  s.add_dependency 'activerecord', '~> 3.0'
 
   s.add_development_dependency 'rake'
   s.add_development_dependency 'shoulda',      '2.10.3'
-  s.add_development_dependency 'sqlite3-ruby', '~> 1.2'
+  s.add_development_dependency 'sqlite3',      '~> 1.2'
   s.add_development_dependency 'capybara',     '~> 1.0.0'
 end

data/test/dummy/config/application.rb

@@ -4,7 +4,6 @@ require "active_model/railtie"
 require "active_record/railtie"
 require "action_controller/railtie"
 require "action_view/railtie"
-require "action_mailer/railtie"
 
 Bundler.require
 require 'paper_trail'

data/test/dummy/config/environments/test.rb

@@ -23,7 +23,7 @@ Dummy::Application.configure do
   # Tell Action Mailer not to deliver emails to the real world.
   # The :test delivery method accumulates sent emails in the
   # ActionMailer::Base.deliveries array.
-  config.action_mailer.delivery_method = :test
+  config.action_mailer.delivery_method = :test if config.respond_to?(:action_mailer)
 
   # Use SQL instead of Active Record's schema dumper when creating the test database.
   # This is necessary if your schema can't be completely dumped by the schema dumper,

data/test/unit/version_test.rb

@@ -0,0 +1,43 @@
+require 'test_helper'
+
+class VersionTest < ActiveSupport::TestCase
+  setup {
+    change_schema
+    @article = Animal.create
+    assert Version.creates.present?
+  }
+
+  context "Version.creates" do
+    should "return only create events" do
+      Version.creates.each do |version|
+        assert_equal "create", version.event
+      end
+    end
+  end
+
+  context "Version.updates" do
+    setup {
+      @article.update_attributes(:name => 'Animal')
+      assert Version.updates.present?
+    }
+
+    should "return only update events" do
+      Version.updates.each do |version|
+        assert_equal "update", version.event
+      end
+    end
+  end
+
+  context "Version.destroys" do
+    setup {
+      @article.destroy
+      assert Version.destroys.present?
+    }
+
+    should "return only destroy events" do
+      Version.destroys.each do |version|
+        assert_equal "destroy", version.event
+      end
+    end
+  end
+end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: paper_trail
 version: !ruby/object:Gem::Version 
-  hash: 19
+  hash: 17
   prerelease: 
   segments: 
   - 2
   - 6
-  - 2
-  version: 2.6.2
+  - 3
+  version: 2.6.3
 platform: ruby
 authors: 
 - Andy Stewart
@@ -15,27 +15,43 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2012-03-12 00:00:00 +01:00
+date: 2012-03-26 00:00:00 +02:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
-  name: rails
+  name: railties
   prerelease: false
   version_requirements: &id001 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ~>
       - !ruby/object:Gem::Version 
-        hash: 5
+        hash: 7
         segments: 
         - 3
-        version: "3"
+        - 0
+        version: "3.0"
   type: :runtime
   requirement: *id001
 - !ruby/object:Gem::Dependency 
-  name: rake
+  name: activerecord
   prerelease: false
   version_requirements: &id002 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - ~>
+      - !ruby/object:Gem::Version 
+        hash: 7
+        segments: 
+        - 3
+        - 0
+        version: "3.0"
+  type: :runtime
+  requirement: *id002
+- !ruby/object:Gem::Dependency 
+  name: rake
+  prerelease: false
+  version_requirements: &id003 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ">="
@@ -45,11 +61,11 @@ dependencies:
         - 0
         version: "0"
   type: :development
-  requirement: *id002
+  requirement: *id003
 - !ruby/object:Gem::Dependency 
   name: shoulda
   prerelease: false
-  version_requirements: &id003 !ruby/object:Gem::Requirement 
+  version_requirements: &id004 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - "="
@@ -61,11 +77,11 @@ dependencies:
         - 3
         version: 2.10.3
   type: :development
-  requirement: *id003
+  requirement: *id004
 - !ruby/object:Gem::Dependency 
-  name: sqlite3-ruby
+  name: sqlite3
   prerelease: false
-  version_requirements: &id004 !ruby/object:Gem::Requirement 
+  version_requirements: &id005 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ~>
@@ -76,11 +92,11 @@ dependencies:
         - 2
         version: "1.2"
   type: :development
-  requirement: *id004
+  requirement: *id005
 - !ruby/object:Gem::Dependency 
   name: capybara
   prerelease: false
-  version_requirements: &id005 !ruby/object:Gem::Requirement 
+  version_requirements: &id006 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ~>
@@ -92,7 +108,7 @@ dependencies:
         - 0
         version: 1.0.0
   type: :development
-  requirement: *id005
+  requirement: *id006
 description: Track changes to your models' data.  Good for auditing or versioning.
 email: boss@airbladesoftware.com
 executables: []
@@ -181,6 +197,7 @@ files:
 - test/unit/inheritance_column_test.rb
 - test/unit/model_test.rb
 - test/unit/timestamp_test.rb
+- test/unit/version_test.rb
 has_rdoc: true
 homepage: http://github.com/airblade/paper_trail
 licenses: []
@@ -278,3 +295,4 @@ test_files:
 - test/unit/inheritance_column_test.rb
 - test/unit/model_test.rb
 - test/unit/timestamp_test.rb
+- test/unit/version_test.rb