after_commit_everywhere 1.3.1 → 1.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6b63aab224d26cf40ba71632ebc055bd458bd3c72c87234715d1d8020f661247
-  data.tar.gz: b558770a87644ee346a8361419ea9d0e616062da4873427d260dc8e428d545b4
+  metadata.gz: 749225d1d1a6000f08b986f6c2704767339fc576648811f4421fb7344b864761
+  data.tar.gz: af075362eeaa19f6e893d5fcefadd751f8cf7817bd310ddea0a024e49482100e
 SHA512:
-  metadata.gz: 99d19cdc3ba272ad5a7c6a47b232d2cd6d5fa012889cd270763d154cf559d2b504b34ba650917a8618f073897a46e4e4fec79293092d05e66aa435d21179037a
-  data.tar.gz: 7169e814084435d36a231cc5b2a3c7b11268a3e24913826b00fefe87ceaddea2c2299208616938d51ee245eca60136a654f8ec02664e30c3466f59b43f82f62d
+  metadata.gz: c816318efea508b86763b07f69b6164eab40edb52f5c95953e05c6dd7df1b2432b1a933e0bfa68605b7b67c3e99dbee6a3e7c4c30389214c7ed9bab5d6afa704
+  data.tar.gz: 4b20e061dfbd60ca6caef4da43ba1866c92a4c3be3418016da1f1f768f01321c99d19f024020d48139dd4320b0c0d21854554e15ee67bf0e9a6a1f3b072e21ad

data/CHANGELOG.md

@@ -4,6 +4,19 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
 and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
 
+## 1.4.0 (2024-02-07)
+
+### Added
+
+ - Ability to prepend callbacks to the head of callback queue using `prepend: true` option.
+
+   ```ruby
+   AfterCommitEverywhere.after_commit { puts "I'm second!" }
+   AfterCommitEverywhere.after_commit(prepend: true) { puts "I'm first!" }
+   ```
+
+   See [Pull request #30](https://github.com/Envek/after_commit_everywhere/pull/30) by [@quentindemetz][] and [@A1090][].
+
 ## 1.3.1 (2023-06-21)
 
 ### Fixed
@@ -132,3 +145,5 @@ See [#11](https://github.com/Envek/after_commit_everywhere/issues/11) for discus
 [@stokarenko]: https://github.com/stokarenko "Sergey Tokarenko"
 [@lolripgg]: https://github.com/lolripgg "James Brewer"
 [@jpcamara]: https://github.com/jpcamara "JP Camara"
+[@quentindemetz]: https://github.com/quentindemetz "Quentin de Metz"
+[@A1090]: https://github.com/A1090 "Tabac Andreina"

data/README.md

@@ -111,7 +111,7 @@ Will be executed right after transaction in which it have been declared was roll
 
 If called outside transaction will raise an exception!
 
-Please keep in mind ActiveRecord's [limitations for rolling back nested transactions](http://api.rubyonrails.org/classes/ActiveRecord/Transactions/ClassMethods.html#module-ActiveRecord::Transactions::ClassMethods-label-Nested+transactions). See [`in_transaction`](#in_transaction) for a workaround to this limitation. 
+Please keep in mind ActiveRecord's [limitations for rolling back nested transactions](http://api.rubyonrails.org/classes/ActiveRecord/Transactions/ClassMethods.html#module-ActiveRecord::Transactions::ClassMethods-label-Nested+transactions). See [`in_transaction`](#in_transaction) for a workaround to this limitation.
 
 ### Available helper methods
 
@@ -139,7 +139,7 @@ class ServiceObjectBtw
 end
 ```
 
-Our service object can run its database operations safely when run in isolation. 
+Our service object can run its database operations safely when run in isolation.
 
 ```rb
 ServiceObjectBtw.new.call # This opens a new #transaction block
@@ -195,6 +195,8 @@ end
     - `:warn_and_execute` to print warning and execute immediately
     - `:raise` to raise an exception instead of executing
 
+- `prepend` puts the callback at the head of the callback chain, instead of at the end.
+
 ### FAQ
 
 #### Does it works with transactional_test or DatabaseCleaner

data/lib/after_commit_everywhere/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module AfterCommitEverywhere
-  VERSION = "1.3.1"
+  VERSION = "1.4.0"
 end

data/lib/after_commit_everywhere.rb

@@ -39,11 +39,13 @@ module AfterCommitEverywhere
     # @param callback   [#call] Callback to be executed
     # @return           void
     def after_commit(
+      prepend: false,
       connection: nil,
       without_tx: EXECUTE,
       &callback
     )
       register_callback(
+        prepend: prepend,
         connection: connection,
         name: __method__,
         callback: callback,
@@ -64,6 +66,7 @@ module AfterCommitEverywhere
     # @param callback   [#call] Callback to be executed
     # @return           void
     def before_commit(
+      prepend: false,
       connection: nil,
       without_tx: WARN_AND_EXECUTE,
       &callback
@@ -73,6 +76,7 @@ module AfterCommitEverywhere
       end
 
       register_callback(
+        prepend: prepend,
         connection: connection,
         name: __method__,
         callback: callback,
@@ -90,8 +94,9 @@ module AfterCommitEverywhere
     # @param callback   [#call] Callback to be executed
     # @return           void
     # @raise            [NotInTransaction] if called outside transaction.
-    def after_rollback(connection: nil, &callback)
+    def after_rollback(prepend: false, connection: nil, &callback)
       register_callback(
+        prepend: prepend,
         connection: connection,
         name: __method__,
         callback: callback,
@@ -100,7 +105,7 @@ module AfterCommitEverywhere
     end
 
     # @api private
-    def register_callback(connection: nil, name:, without_tx:, callback:)
+    def register_callback(prepend:, connection: nil, name:, without_tx:, callback:)
       raise ArgumentError, "Provide callback to #{name}" unless callback
 
       unless in_transaction?(connection)
@@ -119,7 +124,16 @@ module AfterCommitEverywhere
 
       connection ||= default_connection
       wrap = Wrap.new(connection: connection, "#{name}": callback)
-      connection.add_transaction_record(wrap)
+
+      if prepend
+        # Hacking ActiveRecord's transaction internals to prepend our callback
+        # See https://github.com/rails/rails/blob/f0d433bb46ac233ec7fd7fae48f458978908d905/activerecord/lib/active_record/connection_adapters/abstract/transaction.rb#L148-L156
+        records = connection.current_transaction.instance_variable_get(:@records)
+        records = connection.current_transaction.instance_variable_set(:@records, []) if records.nil?
+        records.unshift(wrap)
+      else
+        connection.add_transaction_record(wrap)
+      end
     end
 
     # Helper method to determine whether we're currently in transaction or not

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: after_commit_everywhere
 version: !ruby/object:Gem::Version
-  version: 1.3.1
+  version: 1.4.0
 platform: ruby
 authors:
 - Andrey Novikov
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2023-06-21 00:00:00.000000000 Z
+date: 2024-02-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -246,7 +246,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.6
+rubygems_version: 3.5.3
 signing_key: 
 specification_version: 4
 summary: Executes code after database commit wherever you want in your application