lib/tasks/bulkrax_tasks.rake in bulkrax-4.2.1 vs lib/tasks/bulkrax_tasks.rake in bulkrax-4.3.0

@@ -1,30 +1,29 @@
 # frozen_string_literal: true
 
 namespace :bulkrax do
   desc "Remove old exported zips and create new ones with the new file structure"
   task rerun_all_exporters: :environment do
+    # delete the existing folders and zip files
+    Dir["tmp/exports/**"].each { |file| FileUtils.rm_rf(file) }
+
     if defined?(::Hyku)
       Account.find_each do |account|
-        puts "=============== updating #{account.name} ============"
         next if account.name == "search"
         switch!(account)
+        puts "=============== updating #{account.name} ============"
 
-        rerun_exporters_and_delete_zips
+        make_new_exports
 
         puts "=============== finished updating #{account.name} ============"
       end
     else
-      rerun_exporters_and_delete_zips
+      make_new_exports
     end
   end
 
-  def rerun_exporters_and_delete_zips
-    begin
-      Bulkrax::Exporter.all.each { |e| Bulkrax::ExporterJob.perform_later(e.id) }
-    rescue => e
-      puts "(#{e.message})"
-    end
-
-    Dir["tmp/exports/**.zip"].each { |zip_path| FileUtils.rm_rf(zip_path) }
+  def make_new_exports
+    Bulkrax::Exporter.all.each { |e| Bulkrax::ExporterJob.perform_later(e.id) }
+  rescue => e
+    puts "(#{e.message})"
   end
 end