paper_trail 5.0.0 → 5.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: a998b8383ac675662fac129fce0617b3c2f13351
-  data.tar.gz: c431eeedc5dec2b6882775a95ea6ef79df63f2a9
+  metadata.gz: 9cd678385e88cdbf152ecc4c2a4a1876afd7a175
+  data.tar.gz: 93e28bbba83f8be22cfb8db3ee3e1912818f86af
 SHA512:
-  metadata.gz: 1108f2949a0a98f6ad9e7f75eba6b2d40d21209de2beb6c856142930aa58b8f0f7c65fa1cf9fe73dd9fc3b70c22afc8886846581d992eb0ac9934504205db960
-  data.tar.gz: 1db8e0b3427d3801672829e014dcf1035f55ea75fecb54bebbeb5d6d54f0203d57bedce6ed5307fd5dfa8a68ebd971bc16ed4ad923fac4d9a46ff1c9f1aea0e5
+  metadata.gz: 19a6e0288279964bced8ca4d4ae4ccacd668e1750dbff133d8c6e3ee76514c150402669c808954b24efcfb0fa085b370e3c20d2afe667bd4385d4ea233f4449e
+  data.tar.gz: b26460e5948b60afb39fc6dc18c04f7c3337d449dc8a0401efc67d3cdabaa193fd3f23341d8d4f44f8ed726af4e5f81079d7be24372b3a6d8dbae683d852e44b

data/.github/CONTRIBUTING.md

@@ -98,6 +98,7 @@ DB=postgres bundle exec rake
   - update any other references to version number
 1. Commit and tag with `git tag -a -m "v5.0.0" "v5.0.0"`
 1. `git push --tags origin master`
-1. `gem push paper_trail`
+1. `gem build paper_trail.gemspec`
+1. `gem push paper_trail-5.0.0.gem`
 
 [1]: https://github.com/airblade/paper_trail/blob/master/doc/bug_report_template.rb

data/CHANGELOG.md

@@ -1,3 +1,21 @@
+## 5.0.1 (2016-05-04)
+
+### Breaking Changes
+
+- None
+
+### Added
+
+- None
+
+### Fixed
+
+- [#791](https://github.com/airblade/paper_trail/issues/791) -
+  A rare issue in applications that override `warn`.
+- [#789](https://github.com/airblade/paper_trail/issues/789) -
+  A potentially common issue, in applications with initializers that use
+  versioned models.
+
 ## 5.0.0 (2016-05-02)
 
 ### Breaking Changes

data/README.md

@@ -38,6 +38,8 @@ has been destroyed.
   - [Finding Out Who Was Responsible For A Change](#finding-out-who-was-responsible-for-a-change)
   - [Associations](#associations)
   - [Storing metadata](#storing-metadata)
+- ActiveRecord
+  - [Single Table Inheritance](#single-table-tnheritance)
 - Extensibility
   - [Custom Version Classes](#custom-version-classes)
   - [Custom Serializer](#custom-serializer)
@@ -980,6 +982,23 @@ end
 If you're using [strong_parameters][18] instead of [protected_attributes][17]
 then there is no need to use `attr_accessible`.
 
+## Single Table Inheritance (STI)
+
+PaperTrail supports [Single Table Inheritance][39], and even supports an
+un-versioned base model, as of 23ffbdc7e1.
+
+```ruby
+class Fruit < ActiveRecord::Base
+  # un-versioned base model
+end
+class Banana < Fruit
+  has_paper_trail
+end
+```
+
+However, there is a known issue when reifying [associations](#associations),
+see https://github.com/airblade/paper_trail/issues/594
+
 ## Custom Version Classes
 
 You can specify custom version subclasses with the `:class_name` option:
@@ -1554,3 +1573,5 @@ Released under the MIT licence.
 [36]: http://www.postgresql.org/docs/9.4/interactive/ddl.html
 [37]: https://github.com/ankit1910/paper_trail-globalid
 [38]: https://github.com/sferik/rails_admin
+[39]: http://api.rubyonrails.org/classes/ActiveRecord/Base.html#class-ActiveRecord::Base-label-Single+table+inheritance
+[40]: http://api.rubyonrails.org/classes/ActiveRecord/Associations/ClassMethods.html#module-ActiveRecord::Associations::ClassMethods-label-Polymorphic+Associations

data/lib/generators/paper_trail/install_generator.rb

@@ -40,7 +40,7 @@ module PaperTrail
     def add_paper_trail_migration(template)
       migration_dir = File.expand_path("db/migrate")
       if self.class.migration_exists?(migration_dir, template)
-        warn "Migration already exists: #{template}"
+        ::Kernel.warn "Migration already exists: #{template}"
       else
         migration_template "#{template}.rb", "db/migrate/#{template}.rb"
       end

data/lib/paper_trail.rb

@@ -176,6 +176,10 @@ unless PaperTrail.active_record_protected_attributes?
   end
 end
 
+ActiveSupport.on_load(:active_record) do
+  include PaperTrail::Model
+end
+
 # Require frameworks
 require "paper_trail/frameworks/sinatra"
 if defined?(::Rails) && ActiveRecord::VERSION::STRING >= "3.2"

data/lib/paper_trail/frameworks/active_record.rb

@@ -2,7 +2,3 @@
 # since otherwise the model(s) will get loaded in via the `Rails::Engine`.
 require "paper_trail/frameworks/active_record/models/paper_trail/version_association"
 require "paper_trail/frameworks/active_record/models/paper_trail/version"
-
-ActiveSupport.on_load(:active_record) do
-  include PaperTrail::Model
-end

data/lib/paper_trail/frameworks/rails/controller.rb

@@ -94,7 +94,7 @@ module PaperTrail
         user_present = user_for_paper_trail.present?
         whodunnit_blank = ::PaperTrail.whodunnit.blank?
         if enabled && user_present && whodunnit_blank && !@set_paper_trail_whodunnit_called
-          warn <<-EOS.strip_heredoc
+          ::Kernel.warn <<-EOS.strip_heredoc
             user_for_paper_trail is present, but whodunnit has not been set.
             PaperTrail no longer adds the set_paper_trail_whodunnit
             before_filter for you. Please add this before_filter to your

data/lib/paper_trail/frameworks/rails/engine.rb

@@ -5,7 +5,6 @@ module PaperTrail
       paths["app/models"] << "lib/paper_trail/frameworks/active_record/models"
       config.paper_trail = ActiveSupport::OrderedOptions.new
       initializer "paper_trail.initialisation" do |app|
-        ActiveRecord::Base.send :include, PaperTrail::Model
         PaperTrail.enabled = app.config.paper_trail.fetch(:enabled, true)
       end
     end

data/lib/paper_trail/version_number.rb

@@ -3,7 +3,7 @@ module PaperTrail
   module VERSION
     MAJOR = 5
     MINOR = 0
-    TINY = 0
+    TINY = 1
     PRE = nil
 
     STRING = [MAJOR, MINOR, TINY, PRE].compact.join(".").freeze

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: paper_trail
 version: !ruby/object:Gem::Version
-  version: 5.0.0
+  version: 5.0.1
 platform: ruby
 authors:
 - Andy Stewart
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-05-02 00:00:00.000000000 Z
+date: 2016-05-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord