Change all scope declarations to use class method syntax on the Version class. close #288

CHANGELOG.md

@@ -1,5 +1,7 @@
 ## 3.0.0 (Unreleased)
 
+  - [#288](https://github.com/airblade/paper_trail/issues/288) - Change all scope declarations to class methods on the `PaperTrail::Version`
+    class. Fixes usability when `PaperTrail::Version.abstract_class? == true`.
   - [#281](https://github.com/airblade/paper_trail/issues/281) - `Rails::Controller` helper will return `false` for the
     `paper_trail_enabled_for_controller` method if `PaperTrail.enabled? == false`.
   - [#280](https://github.com/airblade/paper_trail/pull/280) - Don't track virtual timestamp attributes.

lib/paper_trail/has_paper_trail.rb

@@ -69,7 +69,7 @@ module PaperTrail
             :class_name => self.version_class_name, :as => :item
         else
           has_many self.versions_association_name,
-            :class_name => version_class_name,
+            :class_name => self.version_class_name,
             :as         => :item,
             :order      => "#{PaperTrail.timestamp_field} ASC"
         end

lib/paper_trail/version.rb

@@ -27,19 +27,20 @@ module PaperTrail
     end
 
     # These methods accept a timestamp or a version and returns other versions that come before or after
-    scope :subsequent, lambda { |obj|
+    def self.subsequent(obj)
       obj = obj.send(PaperTrail.timestamp_field) if obj.is_a?(self)
       where("#{PaperTrail.timestamp_field} > ?", obj).order("#{PaperTrail.timestamp_field} ASC")
-    }
-    scope :preceding, lambda { |obj|
+    end
+
+    def self.preceding(obj)
       obj = obj.send(PaperTrail.timestamp_field) if obj.is_a?(self)
       where("#{PaperTrail.timestamp_field} < ?", obj).order("#{PaperTrail.timestamp_field} DESC")
-    }
+    end
 
-    scope :between, lambda { |start_time, end_time|
+    def self.between(start_time, end_time)
       where("#{PaperTrail.timestamp_field} > ? AND #{PaperTrail.timestamp_field} < ?", start_time, end_time).
         order("#{PaperTrail.timestamp_field} ASC")
-    }
+    end
 
     # Restore the item from this version.
     #

test/dummy/config/environments/development.rb

@@ -27,7 +27,7 @@ Dummy::Application.configure do
 
   # Log the query plan for queries taking more than this (works
   # with SQLite, MySQL, and PostgreSQL)
-  config.active_record.auto_explain_threshold_in_seconds = 0.5
+  # config.active_record.auto_explain_threshold_in_seconds = 0.5
 
   # Do not compress assets
   config.assets.compress = false