diff --git a/app/models/bulk_import.rb b/app/models/bulk_import.rb index e3e46d7b1c6..8435c245a23 100644 --- a/app/models/bulk_import.rb +++ b/app/models/bulk_import.rb @@ -53,7 +53,7 @@ class BulkImport < ApplicationRecord BulkImport.increment_counter(:processed_items, bulk_import_id) BulkImport.increment_counter(:imported_items, bulk_import_id) if imported - # Since the incrementation has been done atomically, concurrent access to `bulk_import` is now bening + # Since the incrementation has been done atomically, concurrent access to `bulk_import` is now benign bulk_import = BulkImport.find(bulk_import_id) bulk_import.update!(state: :finished, finished_at: Time.now.utc) if bulk_import.processed_items == bulk_import.total_items end diff --git a/app/models/form/import.rb b/app/models/form/import.rb index 3cc4af064ff..9d72de54129 100644 --- a/app/models/form/import.rb +++ b/app/models/form/import.rb @@ -67,9 +67,8 @@ class Form::Import return false unless valid? ApplicationRecord.transaction do - now = Time.now.utc @bulk_import = current_account.bulk_imports.create(type: type, overwrite: overwrite || false, state: :unconfirmed, original_filename: data.original_filename, likely_mismatched: likely_mismatched?) - nb_items = BulkImportRow.insert_all(parsed_rows.map { |row| { bulk_import_id: bulk_import.id, data: row, created_at: now, updated_at: now } }).length + nb_items = @bulk_import.rows.insert_all(parsed_rows.map { |row| { data: row } }, record_timestamps: true).length @bulk_import.update(total_items: nb_items) end end