bulk_dependency_eraser 1.4.2 → 1.4.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 734c542b83d1cf827cadf8a8a41c30d8a86ebe7f98e539e0bdce4777f917c2e9
-  data.tar.gz: fedb350237abe9898c2d54077db57c1c500b76c99c6913431c2445f5fd49293a
+  metadata.gz: a505cd8d1072193bb45a5896b9ec1882793c030552ff4a992f06f48b446deca9
+  data.tar.gz: 0061d59c8ccd8abe86eedf0bd78e7fb57f6ed2b14e03fd0ad2c247c136c1ffef
 SHA512:
-  metadata.gz: 5b69927dd186b6eac307ff85a5bd06783245c5173a6d890ff29a8abb42edbf8df105a3ed592cf3ae92f649c4547f32c289316b83ae7f02d19233a05d9a2a5d47
-  data.tar.gz: 9a4febdbdd779faf17bc332b8853c41bb727e722fea564ed45a8b871ee53f8e4677983c2459a4894a4304c90397a48fa927d41609e4558033ee24c43badbc4b7
+  metadata.gz: e6363cea019a2c86478e3ba5c79c6348db9868c64c40a02a326849827e404bae1542bb3c9686f1ad258717a432c52706b3ce2cbde58e820e520604743c7e7b12
+  data.tar.gz: 1af86850bdacf28110bfa80236ee188c8a5995e92214ee6e4e099696870385f3d62c504031501039747ac4c03cc43b95af7cb24aaae22825c6630e7740c143e4

data/lib/bulk_dependency_eraser/base.rb

@@ -5,6 +5,28 @@ module BulkDependencyEraser
     # Default Custom Scope for mapped-by-name classes, no effect.
     DEFAULT_KLASS_MAPPED_SCOPE_WRAPPER = ->(query) { query }
 
+    DEFAULT_DB_READ_WRAPPER  = ->(block) {
+      begin
+        ActiveRecord::Base.connected_to(role: :reading) do
+          block.call
+        end
+      rescue ActiveRecord::ConnectionNotEstablished
+        # No role: :reading setup, use regular connection
+        block.call
+      end
+    }
+    DEFAULT_DB_WRITE_WRAPPER = ->(block) {
+      begin
+        ActiveRecord::Base.connected_to(role: :writing) do
+          block.call
+        end
+      rescue ActiveRecord::ConnectionNotEstablished
+        # No role: :writing setup, use regular connection
+        block.call
+      end
+    }
+    DEFAULT_DB_BLANK_WRAPPER = ->(block) { block.call }
+
     DEFAULT_OPTS = {
       # Applied to all queries. Useful for taking advantage of specific indexes
       # - not indexed by klass name. Proc would handle the logic for that.
@@ -17,10 +39,6 @@ module BulkDependencyEraser
       proc_scopes_per_class_name: {},
     }.freeze
 
-
-    # Default Database wrapper, no effect.
-    DEFAULT_DB_WRAPPER = ->(block) { block.call }
-
     attr_reader :errors
 
     def initialize opts: {}

data/lib/bulk_dependency_eraser/builder.rb

@@ -7,7 +7,7 @@ module BulkDependencyEraser
       # - We would have to instantiate if we wanted to apply that scope filter.
       instantiate_if_assoc_scope_with_arity: false,
       # wraps around the DB reading
-      db_read_wrapper: self::DEFAULT_DB_WRAPPER,
+      db_read_wrapper: self::DEFAULT_DB_READ_WRAPPER,
       # Will parse these tables and their dependencies, but will remove the tables from the lists after parsing.
       ignore_tables: [],
       # Won't parse any table in this list
@@ -34,12 +34,6 @@ module BulkDependencyEraser
       reading_proc_scopes_per_class_name: {},
     }.freeze
 
-    DEFAULT_DB_WRAPPER = ->(block) do
-      ActiveRecord::Base.connected_to(role: :reading) do
-        block.call
-      end
-    end
-
     DEPENDENCY_NULLIFY = %i[
       nullify
     ].freeze

data/lib/bulk_dependency_eraser/deleter.rb

@@ -2,7 +2,7 @@ module BulkDependencyEraser
   class Deleter < Base
     DEFAULT_OPTS = {
       verbose: false,
-      db_delete_wrapper: self::DEFAULT_DB_WRAPPER,
+      db_delete_wrapper: self::DEFAULT_DB_WRITE_WRAPPER,
       # Set to true if you want 'ActiveRecord::InvalidForeignKey' errors raised during deletions
       enable_invalid_foreign_key_detection: false,
       disable_batching: false,
@@ -26,12 +26,6 @@ module BulkDependencyEraser
       deletion_proc_scopes_per_class_name: {},
     }.freeze
 
-    DEFAULT_DB_WRAPPER = ->(block) do
-      ActiveRecord::Base.connected_to(role: :writing) do
-        block.call
-      end
-    end
-
     def initialize class_names_and_ids: {}, opts: {}
       @class_names_and_ids = class_names_and_ids
       super(opts:)

data/lib/bulk_dependency_eraser/nullifier.rb

@@ -2,7 +2,7 @@ module BulkDependencyEraser
   class Nullifier < Base
     DEFAULT_OPTS = {
       verbose: false,
-      db_nullify_wrapper: self::DEFAULT_DB_WRAPPER,
+      db_nullify_wrapper: self::DEFAULT_DB_WRITE_WRAPPER,
       # Set to true if you want 'ActiveRecord::InvalidForeignKey' errors raised during nullifications
       # - I can't think of a use-case where a nullification would generate an invalid key error
       # - Not hurting anything to leave it in, but might remove it in the future.
@@ -28,12 +28,6 @@ module BulkDependencyEraser
       nullification_proc_scopes_per_class_name: {},
     }.freeze
 
-    DEFAULT_DB_WRAPPER = ->(block) do
-      ActiveRecord::Base.connected_to(role: :writing) do
-        block.call
-      end
-    end
-
     # @param class_names_columns_and_ids [Hash] - model names with columns to nullify pointing towards the record IDs that require the nullification.
     # - structure:
     #    {

data/lib/bulk_dependency_eraser/version.rb

@@ -1,3 +1,3 @@
 module BulkDependencyEraser
-  VERSION = "1.4.2".freeze
+  VERSION = "1.4.3".freeze
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: bulk_dependency_eraser
 version: !ruby/object:Gem::Version
-  version: 1.4.2
+  version: 1.4.3
 platform: ruby
 authors:
 - benjamin.dana.software.dev@gmail.com