bulk_dependency_eraser 2.0.0 → 2.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a7d8f49a22e2103cef90bfb21c415d0dd687040cf31961d199ff5041d942de68
-  data.tar.gz: 5d264a97b2a00f479c13d8f69a2b00218464700465b2f8cf6f7703d37742b4a5
+  metadata.gz: e4bda32570c3319238a5ea03f6a71cbae1e20015caf5fe7bdba78cab3c7883a7
+  data.tar.gz: 9e371750cd1a05b54bfe724873daaf2c7e9aa66942f168d6417505a1427d46c1
 SHA512:
-  metadata.gz: cdfe1a03baab1a7d0bbd1a6984723815790dca786564df1cdf322ace227fc4734db60ae5c11d42d0b0d1f9f507fc6483091b42a641927acfe12cd87b5cf5eedb
-  data.tar.gz: 1c80246cc7315d5250fdbf0aa75fb199f0b1cdedd510282e45885e476caaff5ecee2896ef12a5053060858a300d860cb1ac2ace462907d0a021efed7dfa7cc9f
+  metadata.gz: 33130d09c651a9757af826cbbd5e1363a8f136a3d6e3bf1e2a128dad776185b544fd75a493013cd28d279690094e90cc3dc0f3b23597704df9842eb80e215808
+  data.tar.gz: 27f4e6014155961a1286f2d1f490a7ca2709fd2b9594a7c8ab3fb3164c26bbdf99e8609c5494e15e8595b0db9eee94d485a20a772f5a37d58554b497eaee5154

data/lib/bulk_dependency_eraser/deleter.rb

@@ -1,7 +1,29 @@
 module BulkDependencyEraser
   class Deleter < Base
+    DEFAULT_DB_DELETE_ALL_WRAPPER = ->(block) do
+      begin
+        block.call
+      rescue StandardError => e
+        report_error("Issue attempting to delete '#{current_class_name}': #{e.class.name} - #{e.message}")
+      end
+    end
+
     DEFAULT_OPTS = {
       verbose: false,
+      # Runs once, all deletions occur within it
+      # - useful if you wanted to implement a rollback:
+      #   - i.e:
+          # db_delete_all_wrapper: lambda do |block|
+          #   ActiveRecord::Base.transaction do
+          #     begin
+          #       block.call
+          #     rescue StandardError => e
+          #       report_error("Issue attempting to delete '#{current_class_name}': #{e.class.name} - #{e.message}")
+          #       raise ActiveRecord::Rollback
+          #     end
+          #   end
+          # end
+      db_delete_all_wrapper: self::DEFAULT_DB_DELETE_ALL_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,
@@ -37,8 +59,7 @@ module BulkDependencyEraser
       end
 
       current_class_name = 'N/A'
-      begin
-        # IDs should have already been reversed in builder
+      delete_all_in_db do
         class_names_and_ids.keys.reverse.each do |class_name|
           current_class_name = class_name
           ids = class_names_and_ids[class_name].reverse
@@ -57,8 +78,6 @@ module BulkDependencyEraser
             end
           end
         end
-      rescue StandardError => e
-        report_error("Issue attempting to delete '#{current_class_name}': #{e.class.name} - #{e.message}")
       end
 
       return errors.none?
@@ -116,6 +135,13 @@ module BulkDependencyEraser
     def delete_in_db(&block)
       puts "Deleting from DB..." if opts_c.verbose
       opts_c.db_delete_wrapper.call(block)
+      puts "Deleting from DB complete." if opts_c.verbose
+    end
+
+    def delete_all_in_db(&block)
+      puts "Deleting all from DB..." if opts_c.verbose
+      opts_c.db_delete_all_wrapper.call(block)
+      puts "Deleting all from DB complete." if opts_c.verbose
     end
   end
 end

data/lib/bulk_dependency_eraser/nullifier.rb

@@ -1,7 +1,29 @@
 module BulkDependencyEraser
   class Nullifier < Base
+    DEFAULT_DB_NULLIFY_ALL_WRAPPER = lambda do |block|
+      begin
+        block.call
+      rescue StandardError => e
+        report_error("Issue attempting to nullify '#{current_class_name}' column '#{current_column}': #{e.class.name} - #{e.message}")
+      end
+    end
+
     DEFAULT_OPTS = {
       verbose: false,
+      # Runs once, all deletions occur within it
+      # - useful if you wanted to implement a rollback:
+      #   - i.e:
+      #     db_nullify_all_wrapper: lambda do |block|
+      #       ActiveRecord::Base.transaction do
+      #         begin
+      #           block.call
+      #         rescue StandardError => e
+      #           report_error("Issue attempting to nullify '#{current_class_name}': #{e.class.name} - #{e.message}")
+      #           raise ActiveRecord::Rollback
+      #         end
+      #       end
+      #     end
+      db_nullify_all_wrapper: self::DEFAULT_DB_NULLIFY_ALL_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
@@ -91,7 +113,7 @@ module BulkDependencyEraser
     def execute
       current_class_name = 'N/A'
       current_column = 'N/A'
-      begin
+      nullify_all_in_db do
         # column_and_ids should have already been reversed in builder
         class_names_columns_and_ids.keys.reverse.each do |class_name|
           current_class_name = class_name
@@ -117,8 +139,6 @@ module BulkDependencyEraser
             end
           end
         end
-      rescue StandardError => e
-        report_error("Issue attempting to nullify '#{current_class_name}' column '#{current_column}': #{e.class.name} - #{e.message}")
       end
 
       return errors.none?
@@ -145,6 +165,9 @@ module BulkDependencyEraser
       opts_c.disable_nullify_batching.nil? ? opts_c.disable_batching : opts_c.disable_nullify_batching
     end
 
+    # @param klass   [ActiveRecord::Base]
+    # @param columns [Symbol | String | Array<String | Symbol>]
+    # @param ids     [Array[String | Integer]]
     def nullify_by_klass_column_and_ids klass, columns, ids
       query = klass.unscoped
       query = custom_scope_for_query(query)
@@ -183,6 +206,13 @@ module BulkDependencyEraser
     def nullify_in_db(&block)
       puts "Nullifying from DB..." if opts_c.verbose
       opts_c.db_nullify_wrapper.call(block)
+      puts "Nullifying from DB complete." if opts_c.verbose
+    end
+
+    def nullify_all_in_db(&block)
+      puts "Nullifying all from DB..." if opts_c.verbose
+      opts_c.db_nullify_all_wrapper.call(block)
+      puts "Nullifying all from DB complete." if opts_c.verbose
     end
 
   end

data/lib/bulk_dependency_eraser/version.rb

@@ -1,3 +1,3 @@
 module BulkDependencyEraser
-  VERSION = "2.0.0".freeze
+  VERSION = "2.1.0".freeze
 end

metadata

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