ar_after_transaction 0.5.0 → 0.8.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6292167d9c1247b96be6e17c00316cd658030406264fe2274623b20a2ae6f491
-  data.tar.gz: 34f5526692f3e3aa8921aca3e77dc853834a6382878893666d352fa59fd613be
+  metadata.gz: 0cc72b7f26b842b9adfedb004133d2cc0748e0783b0fdfeb0aebe9e4c52f8b3a
+  data.tar.gz: c61ca0e181cc2a4eaacdb45b63aab20dc77bd7af72dcb066735adce2f989944b
 SHA512:
-  metadata.gz: 87bdce8967f2c902544b44ab4993c9598190d2ef30a9e0bfa9be21cb40eb732816594e94d83068d7e420a8a0cce1631a4c822589661f18174cbed77979d4429a
-  data.tar.gz: 12476735674f01a238b6188859bc4b29f501a1d49a833b8f876d89b884c87459bfddc8888a10b87e8da0e8cac1c7f324383e7687d4bc2f4b4b4ac816aa2426ae
+  metadata.gz: a5402cb5c03bbf9d67a8a2da4e30dde24c191e8ec403a8f4a0084d95a5d6d46f9d658cf3903f21f2947e51113df2a13eae51393bf6e8a9730e3bd8126a93cabc
+  data.tar.gz: 4e6556949152666f978bf152ac5c4c830dfbbaa4cb3a36da6004a4ec8a2273a989476116377ec81c3f58e4710265da911eaf2debf5565ca21718c6443c3904ca

data/Readme.md

@@ -1,6 +1,5 @@
 [![Gem Version](https://badge.fury.io/rb/ar_after_transaction.png)](http://badge.fury.io/rb/ar_after_transaction)
-[![Build Status](https://travis-ci.org/grosser/ar_after_transaction.png)](https://travis-ci.org/grosser/ar_after_transaction)
-[![Dependency Status](https://gemnasium.com/grosser/ar_after_transaction.png)](https://gemnasium.com/grosser/ar_after_transaction)
+![CI](https://github.com/grosser/ar_after_transaction/workflows/CI/badge.svg)
 
 Do something only after the currently open transactions have finished.
 
@@ -9,36 +8,44 @@ Normally everything gets rolled back when a transaction fails, but you cannot ro
 Install
 =======
 
-    gem install ar_after_transaction
+```bash
+gem install ar_after_transaction
+```
 
 
 Usage
 =====
+
 ### just-in-time callbacks
-    class User
-      after_create :do_stuff, :oops
-
-      def do_stuff
-        after_transaction do
-          send_an_email # cannot be rolled back
-        end
-        comments.create(...) # will be rolled back
-      end
-
-      def oops
-        raise "do the rolback!"
-      end
+
+```ruby
+class User
+  after_create :do_stuff, :oops
+
+  def do_stuff
+    after_transaction do
+      send_an_email # cannot be rolled back
     end
+    comments.create(...) # will be rolled back
+  end
+
+  def oops
+    raise "do the rolback!"
+  end
+end
+```
 
 ### General 'this should be rolled back when in a transaction' code like jobs
 
-    class Resque
-      def revertable_enqueue(*args)
-        ActiveRecord::Base.after_transaction do
-          enqueue(*args)
-        end
-      end
+```ruby
+class Resque
+  def revertable_enqueue(*args)
+    ActiveRecord::Base.after_transaction do
+      enqueue(*args)
     end
+  end
+end
+```
 
 ### When not in a transaction
 after_transaction will perform the given block immediately
@@ -47,25 +54,31 @@ after_transaction will perform the given block immediately
 after_transaction assumes zero open transactions.<br/>
 If you use transactional fixtures you should change it in test mode.
 
-    # config/environments/test.rb
-    config.after_initialize do
-      ActiveRecord::Base.normally_open_transactions = 1
-    end
+`Rspec:`
+```ruby
+# spec/rails_helper.rb
+  config.before(:suite) do
+    ActiveRecord::Base.normally_open_transactions = 1
+  end
+```
 
 ### Rails 3: after_commit hook can replace the first usage example:
 
-    class User
-      after_commit :send_an_email :on=>:create
-      after_create :do_stuff, :oops
-      ...
-    end
+```ruby
+class User
+  after_commit :send_an_email on: :create
+  after_create :do_stuff, :oops
+  ...
+end
+```
 
 Alternative
 ===========
+
 Rails 3+
  - basic support is built in, use it if you can!
  - `after_commit :foo`
- - `after_commit :bar, :on => :create / :update`
+ - `after_commit :bar, on: :create / :update`
  - [after_commit everywhere](https://dev.to/evilmartians/rails-aftercommit-everywhere--4j9g)
 
 
@@ -86,6 +99,7 @@ Authors
  - [Michael Wu](https://github.com/michaelmwu)
  - [C.W.](https://github.com/compwron)
  - [Ben Weintraub](https://github.com/benweint)
+ - [Vladimir Temnikov](https://github.com/vladimirtemnikov)
 
 [Michael Grosser](http://grosser.it)<br/>
 michael@grosser.it<br/>

data/lib/ar_after_transaction/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ARAfterTransaction
-  VERSION = Version = '0.5.0'
-end
+  VERSION = '0.8.0'
+end

data/lib/ar_after_transaction.rb

@@ -1,72 +1,30 @@
+# frozen_string_literal: true
+
 require 'active_record'
 require 'ar_after_transaction/version'
 
 module ARAfterTransaction
   module ClassMethods
-    def self.extended( base )
-      base.class_eval do
-        class << self
-          alias_method :transaction_without_after, :transaction
-          alias_method :transaction, :transaction_with_after
-        end
-      end
-    end
-
-    def transaction_with_after(*args)
-      clean = true
-      transaction_without_after(*args) do
-        begin
-          yield
-        rescue ActiveRecord::Rollback
-          clean = false
-          raise
-        end
-      end
-    rescue Exception
-      clean = false
-      raise
-    ensure
-      unless transactions_open?
-        callbacks = delete_after_transaction_callbacks
-        callbacks.each(&:call) if clean
-      end
-    end
-
     def after_transaction(&block)
-      if transactions_open?
-        connection.after_transaction_callbacks ||= []
-        connection.after_transaction_callbacks << block
-      else
-        yield
-      end
+      connection.after_transaction(&block)
     end
 
     def normally_open_transactions
-      @@normally_open_transactions ||= 0
+      connection.normally_open_transactions ||= 0
     end
 
-    def normally_open_transactions=(value)
-      @@normally_open_transactions = value
-    end
+    delegate :normally_open_transactions=, to: :connection
 
     private
 
     def transactions_open?
-      pool = connection_pool
-      return false unless pool && pool.active_connection?
-      connection.open_transactions > normally_open_transactions
-    end
-
-    def delete_after_transaction_callbacks
-      result = connection.after_transaction_callbacks || []
-      connection.after_transaction_callbacks = []
-      result
+      connection.send :transactions_open?
     end
   end
 
   module InstanceMethods
     def after_transaction(&block)
-      self.class.after_transaction(&block)
+      self.class.connection.after_transaction(&block)
     end
   end
 end
@@ -76,10 +34,55 @@ module ARAfterTransactionConnection
     base.class_eval do
       attr_accessor :normally_open_transactions
       attr_accessor :after_transaction_callbacks
+
+      alias_method :transaction_without_after, :transaction
+      alias_method :transaction, :transaction_with_after
+    end
+  end
+
+  def transaction_with_after(**args)
+    clean = true
+    transaction_without_after(**args) do
+      yield
+    rescue ActiveRecord::Rollback
+      clean = false
+      raise
+    end
+  rescue StandardError
+    clean = false
+    raise
+  ensure
+    unless transactions_open?
+      callbacks = delete_after_transaction_callbacks
+      callbacks.each(&:call) if clean
+    end
+  end
+
+  def after_transaction(&block)
+    if transactions_open?
+      self.after_transaction_callbacks ||= []
+      self.after_transaction_callbacks << block
+    else
+      yield
     end
   end
+
+  private
+
+  def transactions_open?
+    return false unless active?
+
+    self.normally_open_transactions ||= 0
+    open_transactions > normally_open_transactions
+  end
+
+  def delete_after_transaction_callbacks
+    result = after_transaction_callbacks || []
+    self.after_transaction_callbacks = []
+    result
+  end
 end
 
-ActiveRecord::Base.send(:extend, ARAfterTransaction::ClassMethods)
-ActiveRecord::Base.send(:include, ARAfterTransaction::InstanceMethods)
-ActiveRecord::ConnectionAdapters::AbstractAdapter.send(:include, ARAfterTransactionConnection)
+ActiveRecord::Base.extend ARAfterTransaction::ClassMethods
+ActiveRecord::Base.include ARAfterTransaction::InstanceMethods
+ActiveRecord::ConnectionAdapters::AbstractAdapter.include ARAfterTransactionConnection

metadata

@@ -1,37 +1,37 @@
 --- !ruby/object:Gem::Specification
 name: ar_after_transaction
 version: !ruby/object:Gem::Version
-  version: 0.5.0
+  version: 0.8.0
 platform: ruby
 authors:
 - Michael Grosser
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-04-25 00:00:00.000000000 Z
+date: 2022-01-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 3.2.0
+        version: 5.2.0
     - - "<"
       - !ruby/object:Gem::Version
-        version: '5.3'
+        version: '7.1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 3.2.0
+        version: 5.2.0
     - - "<"
       - !ruby/object:Gem::Version
-        version: '5.3'
+        version: '7.1'
 - !ruby/object:Gem::Dependency
-  name: wwtd
+  name: bump
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -45,7 +45,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: bump
+  name: rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -78,14 +78,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2'
+        version: '3'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2'
+        version: '3'
 - !ruby/object:Gem::Dependency
   name: sqlite3
   requirement: !ruby/object:Gem::Requirement
@@ -101,7 +101,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rails
+  name: wwtd
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -135,15 +135,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.2.0
+      version: 2.6.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.6
+rubygems_version: 3.1.6
 signing_key: 
 specification_version: 4
 summary: Execute irreversible actions only when transactions are not rolled back