effective_resources 1.9.5 → 1.9.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 83e99f559d60204f03679abaeacb94f0406596ca751221002c69adcb2cea52ee
-  data.tar.gz: 7ff7732c4b04ffeff143812216af2e03a81008830fba3681614c44d59552959b
+  metadata.gz: 144152b4742a10b3d574bbe0458c8f80db4490dd178d0dbe323a180a05a87b46
+  data.tar.gz: 533c8ef7927b1782d85efaea9affc6c2a16d56452ae3c8d0df45f1ec8535f795
 SHA512:
-  metadata.gz: 1884f93d5785dde2523add135e534d86a18880fe9cc1ded09d73a915ed1ce8c842907e63a0a9e1ac7aa829489b55d3f1eaf0de2be4add65cfaa806e0701fd422
-  data.tar.gz: 7f88e77db260d68330f44868d79f8aebfd7458e19a3b4a4a205811071d2e3abc32f69777dd2ac2c6b964c0df4e1efde54e7b0233705ee7fd6d906e95facf9d1f
+  metadata.gz: cdd68a72ad36d7745ea3178d9f9b640799e1c406f0f1dec2d2e7c376dccadc02724c7298c10a96198dc552c9de21591457b36c601b74a91b60222675d8569386
+  data.tar.gz: 1d126ded3ea4f19d8d4d2040bf32037eb7a0e0c3f7adbdc2e9514d31fa35f54f301bdf6bafb27c3b1e7757ba680577440b2c6a7110f2103d94a07c98f702518b

data/app/controllers/concerns/effective/crud_controller/actions.rb

@@ -228,7 +228,7 @@ module Effective
 
         # No attributes are assigned or saved. We purely call action! on the resource
 
-        ActiveRecord::Base.transaction do
+        EffectiveResources.transaction do
           successes = resources.select do |resource|
             begin
               resource.public_send("#{action}!") if EffectiveResources.authorized?(self, action, resource)

data/app/controllers/concerns/effective/crud_controller/save.rb

@@ -30,7 +30,7 @@ module Effective
 
         success = false
 
-        ActiveRecord::Base.transaction do
+        EffectiveResources.transaction(resource) do
           begin
             run_callbacks(:resource_before_save)
 

data/app/models/concerns/acts_as_statused.rb

@@ -125,16 +125,16 @@ module ActsAsStatused
       define_method("#{sym}?") { status == sym.to_s }
       define_method("was_#{sym}?") { send("#{sym}_at").present? }
 
-      unless has_attribute?("#{sym}_at")
+      #unless has_attribute?("#{sym}_at")
         define_method("#{sym}_at") { status_steps["#{sym}_at".to_sym] }
         define_method("#{sym}_at=") { |value| status_steps["#{sym}_at".to_sym] = value }
-      end
+      #end
 
-      unless has_attribute?("#{sym}_by_id")
+      #unless has_attribute?("#{sym}_by_id")
         define_method("#{sym}_by") { acts_as_statused_by_user(sym) }
         define_method("#{sym}_by_id") { status_steps["#{sym}_by".to_sym] }
         define_method("#{sym}_by_id=") { |value| status_steps["#{sym}_by".to_sym] = value }
-      end
+      #end
 
       # approved!
       define_method("#{sym}!") do |atts = {}|
@@ -180,4 +180,3 @@ module ActsAsStatused
   end
 
 end
-

data/app/models/concerns/effective_after_commit.rb

@@ -12,17 +12,17 @@ module EffectiveAfterCommit
 
   module Base
     def after_commit(connection: self.class.connection, &callback)
-      Effective::AfterCommit.register_callback(connection: connection, name: __method__, callback: callback, no_tx_action: :execute)
+      Effective::AfterCommit.register_callback(connection: connection, name: __method__, callback: callback)
     end
 
     def before_commit(connection: self.class.connection, &callback)
       raise(NotImplementedError, "#{__method__} works only with Rails 5.0+") if ActiveRecord::VERSION::MAJOR < 5
-      Effective::AfterCommit.register_callback(connection: connection, name: __method__, callback: callback, no_tx_action: :warn_and_execute)
+      Effective::AfterCommit.register_callback(connection: connection, name: __method__, callback: callback)
     end
 
     def after_rollback(connection: self.class.connection, &callback)
       raise('expected a block') unless block_given?
-      Effective::AfterCommit.register_callback(connection: connection, name: __method__, callback: callback, no_tx_action: :exception)
+      Effective::AfterCommit.register_callback(connection: connection, name: __method__, callback: callback)
     end
   end
 

data/app/models/effective/after_commit.rb

@@ -25,20 +25,11 @@ module Effective
       @handlers[:after_rollback]&.call
     end
 
-    def self.register_callback(connection:, name:, no_tx_action:, callback:)
+    def self.register_callback(connection:, name:, callback:)
       raise ArgumentError, "#{name} expected a block" unless callback
 
-      unless (connection.transaction_open? && connection.current_transaction.joinable?)
-        case no_tx_action
-        when :warn_and_execute
-          warn "#{name}: No transaction open. Executing callback immediately."
-          return callback.call
-        when :execute
-          return callback.call
-        when :exception
-          raise("#{name} is useless outside transaction")
-        end
-      end
+      raise("#{name} is useless outside transaction") unless connection.transaction_open?
+      raise("#{name} is useless outside transaction") unless connection.current_transaction.joinable?
 
       after_commit = Effective::AfterCommit.new("#{name}": callback)
       connection.add_transaction_record(after_commit)

data/app/models/effective/resources/actions.rb

@@ -12,10 +12,21 @@ module Effective
         @controller_path ||= route_name #[namespace, plural_name].compact * '/')
       end
 
+      def route_engines
+        if tenant? && Tenant.current.present?
+          [Rails.application, Tenant.Engine] + Rails::Engine.subclasses.reverse.reject do |klass|
+            tenant_engines_blacklist.any? { |name| klass.name.start_with?(name) }
+          end
+        else
+          [Rails.application] + Rails::Engine.subclasses.reverse
+        end
+      end
+
       def routes
         @routes ||= begin
           routes = nil
-          engines = [Rails.application] + Rails::Engine.subclasses.reverse
+
+          engines = route_engines()
 
           # Check from controller_path. This is generally correct.
           engines.each do |engine|

data/app/models/effective/resources/associations.rb

@@ -150,7 +150,7 @@ module Effective
 
         is_scope = false
 
-        ActiveRecord::Base.transaction do
+        EffectiveResources.transaction(klass) do
           begin
             relation = klass.public_send(name).kind_of?(ActiveRecord::Relation)
           rescue => e

data/app/models/effective/resources/naming.rb

@@ -32,11 +32,14 @@ module Effective
 
       def route_name_fallbacks
         mod = class_name.split('::').first.to_s.downcase
+        admin = ('admin' if namespace.present? && namespace.include?('/admin'))
 
         matches = [
           route_name.singularize,
           [*namespace, plural_name].join('/'),
+          [*admin, plural_name].join('/'),
           [*namespace, name].join('/'),
+          [*admin, name].join('/'),
           [*mod, *namespace, plural_name].join('/'),
           [*mod, *namespace, name].join('/')
         ]

data/app/models/effective/resources/tenants.rb

@@ -15,6 +15,10 @@ module Effective
         name if Rails.application.config.tenants[name].present?
       end
 
+      def tenant_engines_blacklist
+        return [] unless tenant?
+        Rails.application.config.tenants.map { |name, _| name.to_s.classify }
+      end
     end
   end
 end

data/lib/effective_resources/version.rb

@@ -1,3 +1,3 @@
 module EffectiveResources
-  VERSION = '1.9.5'.freeze
+  VERSION = '1.9.9'.freeze
 end

data/lib/effective_resources.rb

@@ -51,6 +51,32 @@ module EffectiveResources
     klass
   end
 
+  def self.transaction(resource = nil, &block)
+    connection = (resource if resource.respond_to?(:transaction))
+    connection ||= (resource.class if resource.class.respond_to?(:transaction))
+    connection ||= '::ApplicationRecord'.safe_constantize
+    connection ||= 'ActiveRecord::Base'.safe_constantize
+
+    raise('unable to determine transaction class') unless connection.present?
+
+    connection.transaction { yield }
+  end
+
+  # Used by streaming CSV export in datatables
+  def self.with_resource_enumerator(&block)
+    raise('expected a block') unless block_given?
+
+    tenant = Tenant.current if defined?(Tenant)
+
+    if tenant
+      Enumerator.new do |enumerator|
+        Tenant.as(tenant) { yield(enumerator) }
+      end
+    else
+      Enumerator.new { |enumerator| yield(enumerator) }
+    end
+  end
+
   def self.truthy?(value)
     if defined?(::ActiveRecord::ConnectionAdapters::Column::TRUE_VALUES)  # Rails <5
       ::ActiveRecord::ConnectionAdapters::Column::TRUE_VALUES.include?(value)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: effective_resources
 version: !ruby/object:Gem::Version
-  version: 1.9.5
+  version: 1.9.9
 platform: ruby
 authors:
 - Code and Effect
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-11-16 00:00:00.000000000 Z
+date: 2021-12-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails