paper_trail 9.0.0 → 9.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 24616bf1e1df70302e7417a24d60141dc2a0d20f1c6e8ac3690dbd4abf890a82
-  data.tar.gz: 8f0ea37c0b65a0c15bfbf4096db430e81d848952f6d42ecbc09d5028fa042d3e
+  metadata.gz: 1c8b21b88ed51a36003e6df11297955cfba569f71d3c6de6be11715a94ec63c9
+  data.tar.gz: 11b1b625dd80e75f3218a047d9caf69ead79e18bf79dc7f7b5e597b0f4b7f7a4
 SHA512:
-  metadata.gz: 109af4466c2f6a61b42633bb94e46909203252757d7864250dd8e62cddfe82ef317419fc5ad8f784eecb453af387a02242b643b58b8eadd5778ce1e64c4b56a3
-  data.tar.gz: b840ca06183ff5411b0dcc493c8bdbc8abaa94060d8dfccaf1b0cbd0bbf69cd4bda4c2a98e3bd406d4760317e9949c89e306bd7b54e676237b162f3721f30497
+  metadata.gz: 6e024e2ad10466f53e07edc2d0683de403034cdedf49b2f6440c3fe8c59b37f90a46ec1f6ddcf1412b21dfa4c0369e3caefe701eb05c4c30e9701a2c6666d1ff
+  data.tar.gz: 6c98b6f10cc6a4e4e0b8d0b516d3d89eec54243624a544b4b7bea7963400b38c944bdbe2d438ead507bc43f57b71fada18b7fd0f5caa51338c6c357c59e66345

data/lib/generators/paper_trail/install_generator.rb

@@ -16,7 +16,7 @@ module PaperTrail
       "ActiveRecord::ConnectionAdapters::Mysql2Adapter"
     ].freeze
 
-    source_root File.expand_path("../templates", __FILE__)
+    source_root File.expand_path("templates", __dir__)
     class_option(
       :with_changes,
       type: :boolean,

data/lib/paper_trail.rb

@@ -121,7 +121,8 @@ module PaperTrail
     # word "request" right there in your application code will remind you that
     # these variables only affect the current request, not all threads.
     #
-    # Given a block, temporarily sets the given `options` and execute the block.
+    # Given a block, temporarily sets the given `options`, executes the block,
+    # and returns the value of the block.
     #
     # Without a block, this currently just returns `PaperTrail::Request`.
     # However, please do not use `PaperTrail::Request` directly. Currently,
@@ -135,7 +136,6 @@ module PaperTrail
         Request
       else
         Request.with(options, &block)
-        nil
       end
     end
 

data/lib/paper_trail/record_trail.rb

@@ -4,8 +4,10 @@ module PaperTrail
   # Represents the "paper trail" for a single record.
   class RecordTrail
     DPR_TOUCH_WITH_VERSION = <<-STR.squish.freeze
-      my_model_instance.paper_trail.touch_with_version is deprecated,
-      please use my_model_instance.touch
+      my_model.paper_trail.touch_with_version is deprecated, please use
+      my_model.paper_trail.save_with_version, which is slightly different. It's
+      a save, not a touch, so make sure you understand the difference by reading
+      the ActiveRecord documentation for both.
     STR
     DPR_WHODUNNIT = <<-STR.squish.freeze
       my_model_instance.paper_trail.whodunnit('John') is deprecated,
@@ -508,6 +510,26 @@ module PaperTrail
       record_update(force: true, in_after_callback: false)
     end
 
+    # Save, and create a version record regardless of options such as `:on`,
+    # `:if`, or `:unless`.
+    #
+    # Arguments are passed to `save`.
+    #
+    # This is an "update" event. That is, we record the same data we would in
+    # the case of a normal AR `update`.
+    #
+    # In older versions of PaperTrail, a method named `touch_with_version` was
+    # used for this purpose. `save_with_version` is not exactly the same.
+    # First, the arguments are different. It passes all arguments to `save`.
+    # Second, it doesn't set any timestamp attributes prior to the `save` the
+    # way `touch_with_version` did.
+    def save_with_version(*args)
+      ::PaperTrail.request(enabled: false) do
+        @record.save(*args)
+      end
+      record_update(force: true, in_after_callback: false)
+    end
+
     # Like the `update_column` method from `ActiveRecord::Persistence`, but also
     # creates a version to record those changes.
     # @api public
@@ -648,7 +670,7 @@ module PaperTrail
 
     def log_version_errors(version, action)
       version.logger&.warn(
-        "Unable to create version for #{action} of #{@record.class.name}" +
+        "Unable to create version for #{action} of #{@record.class.name}" \
           "##{@record.id}: " + version.errors.full_messages.join(", ")
       )
     end

data/lib/paper_trail/version_number.rb

@@ -9,7 +9,7 @@ module PaperTrail
   module VERSION
     MAJOR = 9
     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: 9.0.0
+  version: 9.0.1
 platform: ruby
 authors:
 - Andy Stewart
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-03-27 00:00:00.000000000 Z
+date: 2018-04-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -66,28 +66,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '9.1'
+        version: '10.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '9.1'
+        version: '10.0'
 - !ruby/object:Gem::Dependency
   name: ffaker
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.7'
+        version: '2.8'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.7'
+        version: '2.8'
 - !ruby/object:Gem::Dependency
   name: generator_spec
   requirement: !ruby/object:Gem::Requirement
@@ -130,26 +130,6 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 0.21.0
-- !ruby/object:Gem::Dependency
-  name: rack-test
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 0.6.3
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '0.9'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 0.6.3
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '0.9'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -184,28 +164,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.51.0
+        version: 0.54.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.51.0
+        version: 0.54.0
 - !ruby/object:Gem::Dependency
   name: rubocop-rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.19.0
+        version: 1.24.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.19.0
+        version: 1.24.0
 - !ruby/object:Gem::Dependency
   name: sqlite3
   requirement: !ruby/object:Gem::Requirement