importo 3.0.17 → 3.0.18

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 889fe30458aad6a71cd035c536ccf89a954bfd9876695312da33ed3721008ece
-  data.tar.gz: 30bbe4ad60229d637580bbcdf5a272e8367caa800ab82bc9fcf73513914222fd
+  metadata.gz: 6377f43dbbb4b09f3700ceb910fad844598c0197c5f35857c2cd712cf37cdd4b
+  data.tar.gz: a22342e284e1df6b973920d4404c3641eba58ef0e47a79dd3ccf66b2db6e3b23
 SHA512:
-  metadata.gz: 1199d4f8b28371c6885ac5de06b1dfaee325a229b514995a88ff00faec5e4bb7fc3ea70ea5c20d54893e53e2aa2bb431f560e699d0144b6917f48ff81af44a28
-  data.tar.gz: ba7f56e0c96d24fbdc60ec9503a161ba356015b461ab5f6e961bf0b56e3252d26af091ebd8f83c49db6c02417adf13809982cf857067b89f233654777295582b
+  metadata.gz: 5fd684e69696971df33154b2e4554566ba1609869139863ff0b104dd4cad6ba322f89f0a13f2f8170c505c8befb68cc3a6460ecab88bfa98782a3b0661ce7596
+  data.tar.gz: 1d7cd156345cf52bd9ac0dd348f9057f00baf3d84312d6068b851f4bb62631cbb1c9a75b9fc11cdc43c683500d941e99ca7816f650839cf5df37937fd85c5eae

data/app/adapters/importo/sidekiq_batch_adapter.rb

@@ -8,10 +8,10 @@ module Importo
       @instance = Sidekiq::Batch.new
     end
 
-    delegate :description=, :bid, to: :@instance
+    delegate :description=, :status, to: :@instance
 
     def on_success(job)
-      @instance.on(:success, job.constantize, properties)
+      @instance.on(:complete, job.constantize, properties)
     end
 
     def add

data/app/importers/concerns/importable.rb

@@ -132,17 +132,16 @@ module Importable
     batch.on_success("Importo::ImportJobCallback")
 
     batch.add do
-      bid = batch.bid
       column_with_delay = columns.select { |k, v| v.delay.present? }
       loop_data_rows do |attributes, index|
         if column_with_delay.present?
-          delay = column_with_delay.map do |k, v|
+          delay = column_with_delay.filter_map do |k, v|
             next unless attributes[k].present?
             v.delay.call(attributes[k])
-          end.compact
+          end
         end
-        Importo::ImportJob.set(wait_until: (delay.max * index).seconds.from_now).perform_async(JSON.dump(attributes), index, import.id, bid) if delay.present?
-        Importo::ImportJob.perform_async(JSON.dump(attributes), index, import.id, bid) unless delay.present?
+        Importo::ImportJob.set(wait_until: (delay.max * index).seconds.from_now).perform_async(JSON.dump(attributes), index, import.id) if delay.present?
+        Importo::ImportJob.perform_async(JSON.dump(attributes), index, import.id) unless delay.present?
       end
     end
 

data/app/importers/importo/import_job_callback.rb

@@ -2,12 +2,18 @@ module Importo
   class ImportJobCallback
     include Rails.application.routes.url_helpers
 
-    def on_success(_status, options)
+    def on_complete(_status, options)
       options = options.deep_stringify_keys
       import = Import.find(options["import_id"])
       if import.present?
-        import.result.attach(io: import.importer.results_file, filename: import.importer.file_name("results"),
-          content_type: "application/vnd.openxmlformats-officedocument.spreadsheetml.sheet")
+        results_file = import.importer.results_file
+        if results_file.is_a?(StringIO)
+          import.result.attach(io: results_file, filename: import.importer.file_name("results"),
+                               content_type: "application/vnd.openxmlformats-officedocument.spreadsheetml.sheet")
+        else
+          import.result.attach(io: File.open(results_file), filename: import.importer.file_name("results"),
+                               content_type: "application/vnd.openxmlformats-officedocument.spreadsheetml.sheet")
+        end
 
         ActiveRecord::Base.uncached do
           import.result_message = I18n.t("importo.importers.result_message",

data/app/jobs/importo/import_job.rb

@@ -20,7 +20,7 @@ module Importo
       end
     end
 
-    def perform(attributes, index, import_id, bid)
+    def perform(attributes, index, import_id)
       self.class.execute_row(attributes, index, import_id, false, bid)
     end
 
@@ -33,7 +33,7 @@ module Importo
       batch = Importo::SidekiqBatchAdapter.find(bid)
 
       if !import.completed? && import.can_complete? && batch.finished?
-        ImportJobCallback.new.on_success(batch.status, {import_id: import_id})
+        ImportJobCallback.new.on_complete(batch.status, {import_id: import.id})
       end
     end
   end

data/lib/importo/test_helpers.rb

@@ -1,4 +1,4 @@
-# frozen_string_literal: true
+# frozen_string_literal: true-
 
 require "axlsx"
 require "roo/excelx"
@@ -30,7 +30,7 @@ module Importo
       import.save!
 
       ImportService.perform(import: import)
-      ImportJobCallback.new.on_success({import_id: import.id})
+      ImportJobCallback.new.on_complete("success", {import_id: import.id})
       import
     end
   end

data/lib/importo/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Importo
-  VERSION = "3.0.17"
+  VERSION = "3.0.18"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: importo
 version: !ruby/object:Gem::Version
-  version: 3.0.17
+  version: 3.0.18
 platform: ruby
 authors:
 - Andre Meij
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2024-10-01 00:00:00.000000000 Z
+date: 2024-10-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: caxlsx