bulkrax 4.3.1 → 4.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: '072159be48b2862e910e9269862e43abb05d0a8203cd7173decf38363dcbd935'
-  data.tar.gz: f9caa42432e49cc8614c4e0c79cd1d9d6ec8274925bea5e3abb804a27ac87bdb
+  metadata.gz: d53012e0252f8033f5da334d5336c134deb3a6221040cfe4c1c6c01bb473d617
+  data.tar.gz: e986a3506c073aa533c4ea303a123e8866d2bf7ac6d8ab6df3949682f61b6c05
 SHA512:
-  metadata.gz: f273c9f50b2a7ca2deb04e072f43195f0826b3b9ba77f8fe17ebd1d6bb87afa70b7af30c27bed3538d9b1975f1a026ea0cee473a30b5c59416600c9b4b411fc0
-  data.tar.gz: 53df275372cfe59b5f825144641549a4f2e55af62c34a962d710b14f71a8a28a1c3ee83ad21e6080cac6b38b3bbb51a12d3454ab9f5b2f74bf489c04bca0f3b3
+  metadata.gz: 8ea20593d1164f62fdfda2dbe84bd8cc4e199a65d7e85eea0b68b27e71aeff7579f1b0059042f55eec1c7ab36f8dec9db406ad58cc3a315b29b8e4900f7cb450
+  data.tar.gz: 89a03cd842d855d48f8c4952b6e9a6daea757e80cb8c0f9ee370b59f90e434b39dfb548deb1757f36463914bd5cccbcf1df6c5ffd9102bb5f0d54b5b7ea6476f

data/app/assets/javascripts/bulkrax/exporters.js

@@ -1,10 +1,12 @@
 function hideUnhide(field) {
   var allSources = $('body').find('.export-source-option')
+  removeRequired(allSources)
   hide(allSources)
 
   if (field.length > 0) {
     var selectedSource = $('.' + field)
     unhideSelected(selectedSource)
+    addRequired(selectedSource)
   }
 
   if (field === 'collection') {
@@ -12,6 +14,16 @@ function hideUnhide(field) {
   }
 };
 
+function addRequired(selectedSource) {  
+  selectedSource.addClass('required').attr('required', 'required');
+  selectedSource.parent().addClass('required');
+}
+
+function removeRequired(allSources) {
+  allSources.removeClass('required').removeAttr('required');
+  allSources.parent().removeClass('required').removeAttr('required')
+};
+
 // hide all export_source
 function hide(allSources) {
   allSources.addClass('hidden');

data/app/controllers/bulkrax/importers_controller.rb

@@ -276,7 +276,7 @@ module Bulkrax
     def setup_client(url)
       return false if url.nil?
       headers = { from: Bulkrax.server_name }
-      @client ||= OAI::Client.new(url, headers: headers, parser: 'libxml', metadata_prefix: 'oai_dc')
+      @client ||= OAI::Client.new(url, headers: headers, parser: 'libxml')
     end
 
     # Download methods

data/app/factories/bulkrax/object_factory.rb

@@ -159,28 +159,55 @@ module Bulkrax
       file_set_attrs = attrs.slice(*object.attributes.keys)
       object.assign_attributes(file_set_attrs)
 
-      attrs['uploaded_files'].each do |uploaded_file_id|
+      attrs['uploaded_files']&.each do |uploaded_file_id|
         uploaded_file = ::Hyrax::UploadedFile.find(uploaded_file_id)
         next if uploaded_file.file_set_uri.present?
 
-        actor = ::Hyrax::Actors::FileSetActor.new(object, @user)
-        uploaded_file.update(file_set_uri: actor.file_set.uri)
-        actor.file_set.permissions_attributes = work_permissions
-        actor.create_metadata
-        actor.create_content(uploaded_file)
-        actor.attach_to_work(work)
+        create_file_set_actor(attrs, work, work_permissions, uploaded_file)
+      end
+      attrs['remote_files']&.each do |remote_file|
+        create_file_set_actor(attrs, work, work_permissions, nil, remote_file)
       end
 
       object.save!
     end
 
+    def create_file_set_actor(attrs, work, work_permissions, uploaded_file, remote_file = nil)
+      actor = ::Hyrax::Actors::FileSetActor.new(object, @user)
+      uploaded_file&.update(file_set_uri: actor.file_set.uri)
+      actor.file_set.permissions_attributes = work_permissions
+      actor.create_metadata(attrs)
+      actor.create_content(uploaded_file) if uploaded_file
+      actor.attach_to_work(work, attrs)
+      handle_remote_file(remote_file: remote_file, actor: actor, update: false) if remote_file
+    end
+
     def update_file_set(attrs)
       file_set_attrs = attrs.slice(*object.attributes.keys)
       actor = ::Hyrax::Actors::FileSetActor.new(object, @user)
-
+      attrs['remote_files']&.each do |remote_file|
+        handle_remote_file(remote_file: remote_file, actor: actor, update: true)
+      end
       actor.update_metadata(file_set_attrs)
     end
 
+    def handle_remote_file(remote_file:, actor:, update: false)
+      actor.file_set.label = remote_file['file_name']
+      actor.file_set.import_url = remote_file['url']
+
+      url = remote_file['url']
+      tmp_file = Tempfile.new(remote_file['file_name'].split('.').first)
+      tmp_file.binmode
+
+      URI.open(url) do |url_file|
+        tmp_file.write(url_file.read)
+      end
+
+      tmp_file.rewind
+      update == true ? actor.update_content(tmp_file) : actor.create_content(tmp_file, from_url: true)
+      tmp_file.close
+    end
+
     def clean_attrs(attrs)
       # avoid the "ArgumentError: Identifier must be a string of size > 0 in order to be treeified" error
       # when setting object.attributes
@@ -200,7 +227,6 @@ module Bulkrax
     def transform_attributes(update: false)
       @transform_attributes = attributes.slice(*permitted_attributes)
       @transform_attributes.merge!(file_attributes(update_files)) if with_files
-      @transform_attributes.transform_values! { |v| v == [""] ? [] : v }
       update ? @transform_attributes.except(:id) : @transform_attributes
     end
 

data/app/jobs/bulkrax/create_relationships_job.rb

@@ -81,7 +81,7 @@ module Bulkrax
     # This is adding the reverse relationship, from the child to the parent
     def collection_parent_work_child
       child_work_ids = child_records[:works].map(&:id)
-      parent_record.reindex_extent = Hyrax::Adapters::NestingIndexAdapter::LIMITED_REINDEX
+      parent_record.try(:reindex_extent=, Hyrax::Adapters::NestingIndexAdapter::LIMITED_REINDEX)
 
       parent_record.add_member_objects(child_work_ids)
       ImporterRun.find(importer_run_id).increment!(:processed_relationships, child_work_ids.count) # rubocop:disable Rails/SkipsModelValidations

data/app/jobs/bulkrax/import_work_job.rb

@@ -5,25 +5,27 @@ module Bulkrax
     queue_as :import
 
     # rubocop:disable Rails/SkipsModelValidations
-    def perform(*args)
-      entry = Entry.find(args[0])
+    def perform(entry_id, run_id, *)
+      entry = Entry.find(entry_id)
+      importer_run = ImporterRun.find(run_id)
       entry.build
       if entry.status == "Complete"
-        ImporterRun.find(args[1]).increment!(:processed_records)
-        ImporterRun.find(args[1]).increment!(:processed_works)
-        ImporterRun.find(args[1]).decrement!(:enqueued_records) unless ImporterRun.find(args[1]).enqueued_records <= 0 # rubocop:disable Style/IdenticalConditionalBranches
+        importer_run.increment!(:processed_records)
+        importer_run.increment!(:processed_works)
       else
         # do not retry here because whatever parse error kept you from creating a work will likely
         # keep preventing you from doing so.
-        ImporterRun.find(args[1]).increment!(:failed_records)
-        ImporterRun.find(args[1]).increment!(:failed_works)
-        ImporterRun.find(args[1]).decrement!(:enqueued_records) unless ImporterRun.find(args[1]).enqueued_records <= 0 # rubocop:disable Style/IdenticalConditionalBranches
+        importer_run.increment!(:failed_records)
+        importer_run.increment!(:failed_works)
       end
+      # Regardless of completion or not, we want to decrement the enqueued records.
+      importer_run.decrement!(:enqueued_records) unless importer_run.enqueued_records <= 0
+
       entry.save!
-      entry.importer.current_run = ImporterRun.find(args[1])
+      entry.importer.current_run = importer_run
       entry.importer.record_status
     rescue Bulkrax::CollectionsCreatedError
-      reschedule(args[0], args[1])
+      reschedule(entry_id, run_id)
     end
     # rubocop:enable Rails/SkipsModelValidations
 

data/app/matchers/bulkrax/application_matcher.rb

@@ -30,7 +30,7 @@ module Bulkrax
 
     def process_split
       if self.split.is_a?(TrueClass)
-        @result = @result.split(/\s*[:;|]\s*/) # default split by : ; |
+        @result = @result.split(Bulkrax.multi_value_element_split_on)
       elsif self.split
         result = @result.split(Regexp.new(self.split))
         @result = result.map(&:strip)

data/app/models/bulkrax/csv_entry.rb

@@ -18,7 +18,7 @@ module Bulkrax
       raise StandardError, 'CSV path empty' if path.blank?
       CSV.read(path,
         headers: true,
-        header_converters: :symbol,
+        header_converters: ->(h) { h.to_sym },
         encoding: 'utf-8')
     end
 
@@ -81,7 +81,7 @@ module Bulkrax
     def add_file
       self.parsed_metadata['file'] ||= []
       if record['file']&.is_a?(String)
-        self.parsed_metadata['file'] = record['file'].split(/\s*[;|]\s*/)
+        self.parsed_metadata['file'] = record['file'].split(Bulkrax.multi_value_element_split_on)
       elsif record['file'].is_a?(Array)
         self.parsed_metadata['file'] = record['file']
       end
@@ -176,7 +176,7 @@ module Bulkrax
       data = hyrax_record.send(key.to_s)
       if data.is_a?(ActiveTriples::Relation)
         if value['join']
-          self.parsed_metadata[key_for_export(key)] = data.map { |d| prepare_export_data(d) }.join(' | ').to_s # TODO: make split char dynamic
+          self.parsed_metadata[key_for_export(key)] = data.map { |d| prepare_export_data(d) }.join(Bulkrax.multi_value_element_join_on).to_s
         else
           data.each_with_index do |d, i|
             self.parsed_metadata["#{key_for_export(key)}_#{i + 1}"] = prepare_export_data(d)
@@ -236,7 +236,7 @@ module Bulkrax
 
     def handle_join_on_export(key, values, join)
       if join
-        parsed_metadata[key] = values.join(' | ') # TODO: make split char dynamic
+        parsed_metadata[key] = values.join(Bulkrax.multi_value_element_join_on)
       else
         values.each_with_index do |value, i|
           parsed_metadata["#{key}_#{i + 1}"] = value
@@ -260,7 +260,7 @@ module Bulkrax
       return [] unless parent_field_mapping.present? && record[parent_field_mapping].present?
 
       identifiers = []
-      split_references = record[parent_field_mapping].split(/\s*[;|]\s*/)
+      split_references = record[parent_field_mapping].split(Bulkrax.multi_value_element_split_on)
       split_references.each do |c_reference|
         matching_collection_entries = importerexporter.entries.select do |e|
           (e.raw_metadata&.[](source_identifier) == c_reference) &&

data/app/models/bulkrax/importer.rb

@@ -58,17 +58,26 @@ module Bulkrax
 
     # If field_mapping is empty, setup a default based on the export_properties
     def mapping
+      # rubocop:disable Style/IfUnlessModifier
       @mapping ||= if self.field_mapping.blank? || self.field_mapping == [{}]
                      if parser.import_fields.present? || self.field_mapping == [{}]
-                       ActiveSupport::HashWithIndifferentAccess.new(
-                         parser.import_fields.reject(&:nil?).map do |m|
-                           Bulkrax.default_field_mapping.call(m)
-                         end.inject(:merge)
-                       )
+                       default_field_mapping
                      end
                    else
-                     self.field_mapping
+                     default_field_mapping.merge(self.field_mapping)
                    end
+
+      # rubocop:enable Style/IfUnlessModifier
+    end
+
+    def default_field_mapping
+      return self.field_mapping if parser.import_fields.nil?
+
+      ActiveSupport::HashWithIndifferentAccess.new(
+        parser.import_fields.reject(&:nil?).map do |m|
+          Bulkrax.default_field_mapping.call(m)
+        end.inject(:merge)
+      )
     end
 
     def parser_fields
@@ -143,17 +152,13 @@ module Bulkrax
       import_objects(['relationship'])
     end
 
+    DEFAULT_OBJECT_TYPES = %w[collection work file_set relationship].freeze
+
     def import_objects(types_array = nil)
       self.only_updates ||= false
-      types = types_array || %w[collection work file_set relationship]
-      if parser.class == Bulkrax::CsvParser
-        parser.create_objects(types)
-      else
-        types.each do |object_type|
-          self.save if self.new_record? # Object needs to be saved for statuses
-          parser.send("create_#{object_type.pluralize}")
-        end
-      end
+      self.save if self.new_record? # Object needs to be saved for statuses
+      types = types_array || DEFAULT_OBJECT_TYPES
+      parser.create_objects(types)
     rescue StandardError => e
       status_info(e)
     end

data/app/models/bulkrax/oai_entry.rb

@@ -59,12 +59,11 @@ module Bulkrax
     def find_collection_ids
       return self.collection_ids if collections_created?
       if sets.blank? || parser.collection_name != 'all'
-        # c = Collection.where(Bulkrax.system_identifier_field => importerexporter.unique_collection_identifier(parser.collection_name)).first
         collection = find_collection(importerexporter.unique_collection_identifier(parser.collection_name))
         self.collection_ids << collection.id if collection.present? && !self.collection_ids.include?(collection.id)
       else # All - collections should exist for all sets
         sets.each do |set|
-          c = Collection.find_by(work_identifier => importerexporter.unique_collection_identifier(set.content))
+          c = find_collection(importerexporter.unique_collection_identifier(set.content))
           self.collection_ids << c.id if c.present? && !self.collection_ids.include?(c.id)
         end
       end

data/app/models/concerns/bulkrax/file_set_entry_behavior.rb

@@ -6,7 +6,14 @@ module Bulkrax
       ::FileSet
     end
 
+    def file_reference
+      return 'file' if parsed_metadata&.[]('file')&.map(&:present?)&.any?
+      return 'remote_files' if parsed_metadata&.[]('remote_files')&.map(&:present?)&.any?
+    end
+
     def add_path_to_file
+      return unless file_reference == 'file'
+
       parsed_metadata['file'].each_with_index do |filename, i|
         next if filename.blank?
 
@@ -22,7 +29,7 @@ module Bulkrax
     end
 
     def validate_presence_of_filename!
-      return if parsed_metadata&.[]('file')&.map(&:present?)&.any?
+      return if parsed_metadata&.[](file_reference)&.map(&:present?)&.any?
 
       raise StandardError, 'File set must have a filename'
     end

data/app/models/concerns/bulkrax/import_behavior.rb

@@ -165,15 +165,16 @@ module Bulkrax
     end
 
     def factory
-      @factory ||= Bulkrax::ObjectFactory.new(attributes: self.parsed_metadata,
-                                              source_identifier_value: identifier,
-                                              work_identifier: parser.work_identifier,
-                                              related_parents_parsed_mapping: parser.related_parents_parsed_mapping,
-                                              replace_files: replace_files,
-                                              user: user,
-                                              klass: factory_class,
-                                              importer_run_id: importerexporter.last_run.id,
-                                              update_files: update_files)
+      of = Bulkrax.object_factory || Bulkrax::ObjectFactory
+      @factory ||= of.new(attributes: self.parsed_metadata,
+                          source_identifier_value: identifier,
+                          work_identifier: parser.work_identifier,
+                          related_parents_parsed_mapping: parser.related_parents_parsed_mapping,
+                          replace_files: replace_files,
+                          user: user,
+                          klass: factory_class,
+                          importer_run_id: importerexporter.last_run.id,
+                          update_files: update_files)
     end
 
     def factory_class

data/app/parsers/bulkrax/application_parser.rb

@@ -2,6 +2,9 @@
 require 'zip'
 
 module Bulkrax
+  # An abstract class that establishes the API for Bulkrax's import and export parsing.
+  #
+  # @abstract Subclass the Bulkrax::ApplicationParser to create a parser that handles a specific format (e.g. CSV, Bagit, XML, etc).
   class ApplicationParser # rubocop:disable Metrics/ClassLength
     attr_accessor :importerexporter, :headers
     alias importer importerexporter
@@ -12,14 +15,21 @@ module Bulkrax
       :exporter_export_path, :exporter_export_zip_path, :importer_unzip_path, :validate_only,
       to: :importerexporter
 
+    # @todo Convert to `class_attribute :parser_fiels, default: {}`
     def self.parser_fields
       {}
     end
 
+    # @return [TrueClass,FalseClass] this parser does or does not support exports.
+    #
+    # @todo Convert to `class_attribute :export_supported, default: false, instance_predicate: true` and `self << class; alias export_supported? export_supported; end`
     def self.export_supported?
       false
     end
 
+    # @return [TrueClass,FalseClass] this parser does or does not support imports.
+    #
+    # @todo Convert to `class_attribute :import_supported, default: false, instance_predicate: true` and `self << class; alias import_supported? import_supported; end`
     def self.import_supported?
       true
     end
@@ -29,49 +39,70 @@ module Bulkrax
       @headers = []
     end
 
-    # @api
+    # @api public
+    # @abstract Subclass and override {#entry_class} to implement behavior for the parser.
     def entry_class
-      raise StandardError, 'must be defined'
+      raise NotImplementedError, 'must be defined'
     end
 
-    # @api
+    # @api public
+    # @abstract Subclass and override {#collection_entry_class} to implement behavior for the parser.
     def collection_entry_class
-      raise StandardError, 'must be defined'
+      raise NotImplementedError, 'must be defined'
     end
 
-    # @api
+    # @api public
+    # @abstract Subclass and override {#records} to implement behavior for the parser.
     def records(_opts = {})
-      raise StandardError, 'must be defined'
+      raise NotImplementedError, 'must be defined'
     end
 
+    # @return [Symbol] the name of the identifying property in the source system from which we're
+    # importing (e.g. is *not* this application that mounts *this* Bulkrax engine).
+    #
+    # @see #work_identifier
+    # @see https://github.com/samvera-labs/bulkrax/wiki/CSV-Importer#source-identifier Bulkrax Wiki regarding source identifier
     def source_identifier
       @source_identifier ||= get_field_mapping_hash_for('source_identifier')&.values&.first&.[]('from')&.first&.to_sym || :source_identifier
     end
 
+    # @return [Symbol] the name of the identifying property for the system which we're importing
+    #         into (e.g. the application that mounts *this* Bulkrax engine)
+    # @see #source_identifier
     def work_identifier
       @work_identifier ||= get_field_mapping_hash_for('source_identifier')&.keys&.first&.to_sym || :source
     end
 
+    # @return [String]
     def generated_metadata_mapping
       @generated_metadata_mapping ||= 'generated'
     end
 
+    # @return [String, NilClass]
+    # @see #related_parents_raw_mapping
     def related_parents_raw_mapping
       @related_parents_raw_mapping ||= get_field_mapping_hash_for('related_parents_field_mapping')&.values&.first&.[]('from')&.first
     end
 
+    # @return [String]
+    # @see #related_parents_field_mapping
     def related_parents_parsed_mapping
       @related_parents_parsed_mapping ||= (get_field_mapping_hash_for('related_parents_field_mapping')&.keys&.first || 'parents')
     end
 
+    # @return [String, NilClass]
+    # @see #related_children_parsed_mapping
     def related_children_raw_mapping
       @related_children_raw_mapping ||= get_field_mapping_hash_for('related_children_field_mapping')&.values&.first&.[]('from')&.first
     end
 
+    # @return [String]
+    # @see #related_children_raw_mapping
     def related_children_parsed_mapping
       @related_children_parsed_mapping ||= (get_field_mapping_hash_for('related_children_field_mapping')&.keys&.first || 'children')
     end
 
+    # @api private
     def get_field_mapping_hash_for(key)
       return instance_variable_get("@#{key}_hash") if instance_variable_get("@#{key}_hash").present?
 
@@ -85,6 +116,7 @@ module Bulkrax
       instance_variable_get("@#{key}_hash")
     end
 
+    # @return [Array<String>]
     def model_field_mappings
       model_mappings = Bulkrax.field_mappings[self.class.to_s]&.dig('model', :from) || []
       model_mappings |= ['model']
@@ -92,6 +124,7 @@ module Bulkrax
       model_mappings
     end
 
+    # @return [String]
     def perform_method
       if self.validate_only
         'perform_now'
@@ -100,29 +133,55 @@ module Bulkrax
       end
     end
 
+    # The visibility of the record.  Acceptable values are: "open", "embaro", "lease", "authenticated", "restricted".  The default is "open"
+    #
+    # @return [String]
+    # @see https://github.com/samvera/hydra-head/blob/main/hydra-access-controls/app/models/concerns/hydra/access_controls/access_right.rb Hydra::AccessControls::AccessRight for details on the range of values.
+    # @see https://github.com/samvera/hyrax/blob/bd2bcffc33e183904be2c175367648815f25bc2b/app/services/hyrax/visibility_intention.rb Hyrax::VisibilityIntention for how we process the visibility.
     def visibility
       @visibility ||= self.parser_fields['visibility'] || 'open'
     end
 
+    # @api public
+    #
+    # @param types [Array<Symbol>] the types of objects that we'll create.
+    #
+    # @see Bulkrax::Importer::DEFAULT_OBJECT_TYPES
+    # @see #create_collections
+    # @see #create_works
+    # @see #create_file_sets
+    # @see #create_relationships
+    def create_objects(types = [])
+      types.each do |object_type|
+        parser.send("create_#{object_type.pluralize}")
+      end
+    end
+
+    # @abstract Subclass and override {#create_collections} to implement behavior for the parser.
     def create_collections
-      raise StandardError, 'must be defined' if importer?
+      raise NotImplementedError, 'must be defined' if importer?
     end
 
+    # @abstract Subclass and override {#create_works} to implement behavior for the parser.
     def create_works
-      raise StandardError, 'must be defined' if importer?
+      raise NotImplementedError, 'must be defined' if importer?
     end
 
+    # @abstract Subclass and override {#create_file_sets} to implement behavior for the parser.
     def create_file_sets
-      raise StandardError, 'must be defined' if importer?
+      raise NotImplementedError, 'must be defined' if importer?
     end
 
+    # @abstract Subclass and override {#create_relationships} to implement behavior for the parser.
     def create_relationships
-      raise StandardError, 'must be defined' if importer?
+      raise NotImplementedError, 'must be defined' if importer?
     end
 
     # Optional, define if using browse everything for file upload
     def retrieve_cloud_files(files); end
 
+    # @param file [#path, #original_filename] the file object that with the relevant data for the
+    #        import.
     def write_import_file(file)
       path = File.join(path_for_import, file.original_filename)
       FileUtils.mv(
@@ -133,6 +192,8 @@ module Bulkrax
     end
 
     # Base path for imported and exported files
+    # @param [String]
+    # @return [String] the base path for files that this parser will "parse"
     def base_path(type = 'import')
       # account for multiple versions of hyku
       is_multitenant = ENV['HYKU_MULTITENANT'] == 'true' || ENV['SETTINGS__MULTITENANCY__ENABLED'] == 'true'
@@ -141,41 +202,48 @@ module Bulkrax
 
     # Path where we'll store the import metadata and files
     #  this is used for uploaded and cloud files
+    # @return [String]
     def path_for_import
       @path_for_import = File.join(base_path, importerexporter.path_string)
       FileUtils.mkdir_p(@path_for_import) unless File.exist?(@path_for_import)
       @path_for_import
     end
 
+    # @abstract Subclass and override {#setup_export_file} to implement behavior for the parser.
     def setup_export_file
-      raise StandardError, 'must be defined' if exporter?
+      raise NotImplementedError, 'must be defined' if exporter?
     end
 
+    # @abstract Subclass and override {#write_files} to implement behavior for the parser.
     def write_files
-      raise StandardError, 'must be defined' if exporter?
+      raise NotImplementedError, 'must be defined' if exporter?
     end
 
+    # @return [TrueClass,FalseClass]
     def importer?
       importerexporter.is_a?(Bulkrax::Importer)
     end
 
+    # @return [TrueClass,FalseClass]
     def exporter?
       importerexporter.is_a?(Bulkrax::Exporter)
     end
 
     # @param limit [Integer] limit set on the importerexporter
     # @param index [Integer] index of current iteration
-    # @return [boolean]
+    # @return [TrueClass,FalseClass]
     def limit_reached?(limit, index)
       return false if limit.nil? || limit.zero? # no limit
       index >= limit
     end
 
     # Override to add specific validations
+    # @return [TrueClass,FalseClass]
     def valid_import?
       true
     end
 
+    # @return [TrueClass,FalseClass]
     def record_has_source_identifier(record, index)
       if record[source_identifier].blank?
         if Bulkrax.fill_in_blank_source_identifiers.present?
@@ -199,6 +267,7 @@ module Bulkrax
     end
     # rubocop:enable Rails/SkipsModelValidations
 
+    # @return [Array<String>]
     def required_elements
       if Bulkrax.fill_in_blank_source_identifiers
         ['title']
@@ -287,12 +356,14 @@ module Bulkrax
     end
 
     # Path for the import
+    # @return [String]
     def import_file_path
       @import_file_path ||= real_import_file_path
     end
 
     private
 
+    # @return [String]
     def real_import_file_path
       return importer_unzip_path if file? && zip?
       parser_fields['import_file_path']

data/app/parsers/bulkrax/csv_parser.rb

@@ -180,7 +180,7 @@ module Bulkrax
     end
 
     def current_work_ids
-      ActiveSupport::Deprication.warn('Bulkrax::CsvParser#current_work_ids will be replaced with #current_record_ids in version 3.0')
+      ActiveSupport::Deprecation.warn('Bulkrax::CsvParser#current_work_ids will be replaced with #current_record_ids in version 3.0')
       current_record_ids
     end
 
@@ -440,7 +440,7 @@ module Bulkrax
         file_mapping = Bulkrax.field_mappings.dig(self.class.to_s, 'file', :from)&.first&.to_sym || :file
         next if r[file_mapping].blank?
 
-        r[file_mapping].split(/\s*[:;|]\s*/).map do |f|
+        r[file_mapping].split(Bulkrax.multi_value_element_split_on).map do |f|
           file = File.join(path_to_files, f.tr(' ', '_'))
           if File.exist?(file) # rubocop:disable Style/GuardClause
             file
@@ -468,7 +468,7 @@ module Bulkrax
       entry_uid ||= if Bulkrax.fill_in_blank_source_identifiers.present?
                       Bulkrax.fill_in_blank_source_identifiers.call(self, records.find_index(collection_hash))
                     else
-                      collection_hash[:title].split(/\s*[;|]\s*/).first
+                      collection_hash[:title].split(Bulkrax.multi_value_element_split_on).first
                     end
 
       entry_uid

data/app/parsers/bulkrax/oai_dc_parser.rb

@@ -13,8 +13,7 @@ module Bulkrax
     def client
       @client ||= OAI::Client.new(importerexporter.parser_fields['base_url'],
                                   headers: headers,
-                                  parser: 'libxml',
-                                  metadata_prefix: importerexporter.parser_fields['metadata_prefix'])
+                                  parser: 'libxml')
     rescue StandardError
       raise OAIError
     end
@@ -32,6 +31,7 @@ module Bulkrax
     end
 
     def records(opts = {})
+      opts[:metadata_prefix] ||= importerexporter.parser_fields['metadata_prefix']
       opts[:set] = collection_name unless collection_name == 'all'
 
       opts[:from] = importerexporter&.last_imported_at&.strftime("%Y-%m-%d") if importerexporter.last_imported_at && only_updates

data/app/services/bulkrax/remove_relationships_for_importer.rb

@@ -0,0 +1,107 @@
+# frozen_string_literal: true
+module Bulkrax
+  # This module is rather destructive; it will break relationships between the works, file sets, and
+  # collections that were imported via an importer.  You probably don't want to run this on your
+  # data, except in the case where you have been testing a Bulkrax::Importer, the parsers and
+  # mappings.  Then, you might have relationships that you want to remove.
+  #
+  # tl;dr - Caution this will break things!
+  class RemoveRelationshipsForImporter
+    # @api public
+    #
+    # Remove the relationships of the works and collections for all of the Bulkrax::Entry records
+    # associated with the given Bulkrax::Importer.
+    #
+    # @param importer [Bulkrax::Importer]
+    # @param with_progress_bar [Boolean]
+    def self.break_relationships_for!(importer:, with_progress_bar: false)
+      entries = importer.entries.select(&:succeeded?)
+      progress_bar = build_progress_bar_for(with_progress_bar: with_progress_bar, entries: entries)
+      new(progress_bar: progress_bar, entries: entries).break_relationships!
+    end
+
+    # @api private
+    #
+    # A null object that conforms to this class's use of a progress bar.
+    module NullProgressBar
+      def self.increment; end
+    end
+
+    # @api private
+    #
+    # @return [#increment]
+    def self.build_progress_bar_for(with_progress_bar:, entries:)
+      return NullProgressBar unless with_progress_bar
+
+      begin
+        require 'ruby-progressbar'
+        ProgessBar.create(total: entries.count)
+      rescue LoadError
+        Rails.logger.info("Using NullProgressBar because ProgressBar is not available due to a LoadError.")
+      end
+    end
+
+    # @param entries [#each]
+    # @param progress_bar [#increment]
+    def initialize(entries:, progress_bar:)
+      @progress_bar = progress_bar
+      @entries = entries
+    end
+
+    attr_reader :entries, :progress_bar
+
+    def break_relationships!
+      entries.each do |entry|
+        progress_bar.increment
+
+        obj = entry.factory.find
+        next if obj.is_a?(FileSet) # FileSets must be attached to a Work
+
+        if obj.is_a?(Collection)
+          remove_relationships_from_collection(obj)
+        else
+          remove_relationships_from_work(obj)
+        end
+
+        obj.try(:reindex_extent=, Hyrax::Adapters::NestingIndexAdapter::LIMITED_REINDEX)
+        obj.save!
+      end
+    end
+
+    def remove_relationships_from_collection(collection)
+      # Remove child work relationships
+      collection.member_works.each do |work|
+        change = work.member_of_collections.delete(collection)
+        work.save! if change.present?
+      end
+
+      # Remove parent collection relationships
+      collection.member_of_collections.each do |parent_col|
+        Hyrax::Collections::NestedCollectionPersistenceService
+          .remove_nested_relationship_for(parent: parent_col, child: collection)
+      end
+
+      # Remove child collection relationships
+      collection.member_collections.each do |child_col|
+        Hyrax::Collections::NestedCollectionPersistenceService
+          .remove_nested_relationship_for(parent: collection, child: child_col)
+      end
+    end
+
+    def remove_relationships_from_work(work)
+      # Remove parent collection relationships
+      work.member_of_collections = []
+
+      # Remove parent work relationships
+      work.member_of_works.each do |parent_work|
+        parent_work.members.delete(work)
+        parent_work.save!
+      end
+
+      # Remove child work relationships
+      work.member_works.each do |child_work|
+        work.member_works.delete(child_work)
+      end
+    end
+  end
+end

data/app/views/bulkrax/exporters/_form.html.erb

@@ -29,7 +29,7 @@
 
   <%= form.input :export_source_importer,
     label: t('bulkrax.exporter.labels.importer'),
-    # required: true,
+    required: true,
     prompt: 'Select from the list',
     label_html: { class: 'importer export-source-option hidden' },
     input_html: { class: 'importer export-source-option hidden' },
@@ -38,7 +38,7 @@
   <%= form.input :export_source_collection,
     prompt: 'Start typing ...',
     label: t('bulkrax.exporter.labels.collection'),
-    # required: true,
+    required: true,
     placeholder: @collection&.title&.first,
     label_html: { class: 'collection export-source-option hidden' },
     input_html: {
@@ -52,7 +52,7 @@
 
   <%= form.input :export_source_worktype,
     label: t('bulkrax.exporter.labels.worktype'),
-    # required: true,
+    required: true,
     prompt: 'Select from the list',
     label_html: { class: 'worktype export-source-option hidden' },
     input_html: { class: 'worktype export-source-option hidden' },

data/app/views/bulkrax/shared/_file_set_entries_tab.html.erb

@@ -14,7 +14,7 @@
       <% entries.each do |e| %>
         <% entry_path = item.class.to_s.include?('Importer') ? bulkrax.importer_entry_path(item.id, e.id) : bulkrax.exporter_entry_path(item.id, e.id) %>
         <tr>
-          <td><%= link_to e.identifier, bulkrax.importer_entry_path(item.id, e.id) %></td>
+          <td><%= link_to e.identifier, entry_path %></td>
           <td><%= e.id %></td>
           <% if e.status == "Complete" %>
             <td><span class="glyphicon glyphicon-ok" style="color: green;"></span> <%= e.status %></td>
@@ -24,12 +24,12 @@
             <td><span class="glyphicon glyphicon-remove" style="color: <%= e.status == 'Deleted' ? 'green' : 'red' %>;"></span> <%= e.status %></td>
           <% end %>
           <% if e.last_error.present? %>
-            <td><%= link_to e.last_error.dig("error_class"), bulkrax.importer_entry_path(item.id, e.id) %></td>
+            <td><%= link_to e.last_error.dig("error_class"), entry_path %></td>
           <% else %>
             <td></td>
           <% end %>
           <td><%= e.status_at %></td>
-          <td><%= link_to raw('<span class="glyphicon glyphicon-info-sign"></span>'), bulkrax.importer_entry_path(item.id, e.id) %></td>
+          <td><%= link_to raw('<span class="glyphicon glyphicon-info-sign"></span>'), entry_path %></td>
         </tr>
       <% end %>
     </tbody>

data/app/views/hyrax/dashboard/sidebar/_bulkrax_sidebar_additions.html.erb

@@ -1,6 +1,8 @@
-<%= menu.nav_link(bulkrax.importers_path) do %>
-  <span class="fa fa-cloud-upload" aria-hidden="true"></span> <span class="sidebar-action-text"><%= t('bulkrax.admin.sidebar.importers') %></span>
+<% if ENV.fetch('HYKU_BULKRAX_ENABLED', 'true') == 'true' %>
+  <%= menu.nav_link(bulkrax.importers_path) do %>
+    <span class="fa fa-cloud-upload" aria-hidden="true"></span> <span class="sidebar-action-text"><%= t('bulkrax.admin.sidebar.importers') %></span>
+  <% end %>
+  <%= menu.nav_link(bulkrax.exporters_path) do %>
+    <span class="fa fa-cloud-download" aria-hidden="true"></span> <span class="sidebar-action-text"><%= t('bulkrax.admin.sidebar.exporters') %></span>
+  <% end %>
 <% end %>
-<%= menu.nav_link(bulkrax.exporters_path) do %>
-  <span class="fa fa-cloud-download" aria-hidden="true"></span> <span class="sidebar-action-text"><%= t('bulkrax.admin.sidebar.exporters') %></span>
-<% end %>

data/app/views/hyrax/dashboard/sidebar/_repository_content.html.erb

@@ -1,19 +1,27 @@
-<li class="h5"><%= t('hyrax.admin.sidebar.repository_objects') %></li>
+  <li class="h5 nav-item"><%= t('hyrax.admin.sidebar.repository_objects') %></li>
 
-<%= menu.nav_link(hyrax.my_collections_path,
-                  also_active_for: hyrax.dashboard_collections_path) do %>
-  <span class="fa fa-folder-open" aria-hidden="true"></span> <span class="sidebar-action-text"><%= t('hyrax.admin.sidebar.collections') %></span>
-<% end %>
+  <%= menu.nav_link(hyrax.my_collections_path,
+                    class: "nav-link",
+                    onclick: "dontChangeAccordion(event);",
+                    also_active_for: hyrax.dashboard_collections_path) do %>
+    <span class="fa fa-folder-open" aria-hidden="true"></span> <span class="sidebar-action-text"><%= t('hyrax.admin.sidebar.collections') %></span>
+  <% end %>
 
-<%= menu.nav_link(hyrax.my_works_path,
-                  also_active_for: hyrax.dashboard_works_path) do %>
-  <span class="fa fa-file" aria-hidden="true"></span> <span class="sidebar-action-text"><%= t('hyrax.admin.sidebar.works') %></span>
-<% end %>
+  <%= menu.nav_link(hyrax.my_works_path,
+                    class: "nav-link",
+                    onclick: "dontChangeAccordion(event);",
+                    also_active_for: hyrax.dashboard_works_path) do %>
+    <span class="fa fa-file" aria-hidden="true"></span> <span class="sidebar-action-text"><%= t('hyrax.admin.sidebar.works') %></span>
+  <% end %>
 
-<%= menu.nav_link(bulkrax.importers_path) do %>
-  <span class="fa fa-cloud-upload" aria-hidden="true"></span> <span class="sidebar-action-text"><%= t('bulkrax.admin.sidebar.importers') %></span>
-<% end %>
+  <% if ::Hyrax::DashboardController&.respond_to?(:sidebar_partials) %>
+  <%= render 'hyrax/dashboard/sidebar/menu_partials', menu: menu, section: :repository_content %>
+  <% else %>
+  <%= menu.nav_link(bulkrax.importers_path) do %>
+    <span class="fa fa-cloud-upload" aria-hidden="true"></span> <span class="sidebar-action-text"><%= t('bulkrax.admin.sidebar.importers') %></span>
+  <% end %>
 
-<%= menu.nav_link(bulkrax.exporters_path) do %>
-  <span class="fa fa-cloud-download" aria-hidden="true"></span> <span class="sidebar-action-text"><%= t('bulkrax.admin.sidebar.exporters') %></span>
-<% end %>
+  <%= menu.nav_link(bulkrax.exporters_path) do %>
+    <span class="fa fa-cloud-download" aria-hidden="true"></span> <span class="sidebar-action-text"><%= t('bulkrax.admin.sidebar.exporters') %></span>
+  <% end %>
+  <% end %>

data/db/migrate/20211203195233_rename_children_counters_to_relationships.rb

@@ -1,4 +1,4 @@
-class RenameChildrenCountersToRelationships < ActiveRecord::Migration[5.2]
+class RenameChildrenCountersToRelationships < ActiveRecord::Migration[5.1]
   def change
     rename_column :bulkrax_importer_runs, :processed_children, :processed_relationships unless column_exists?(:bulkrax_importer_runs, :processed_relationships)
     rename_column :bulkrax_importer_runs, :failed_children, :failed_relationships unless column_exists?(:bulkrax_importer_runs, :failed_relationships)

data/db/migrate/20211220195027_add_file_set_counters_to_importer_runs.rb

@@ -1,4 +1,4 @@
-class AddFileSetCountersToImporterRuns < ActiveRecord::Migration[5.2]
+class AddFileSetCountersToImporterRuns < ActiveRecord::Migration[5.1]
   def change
     add_column :bulkrax_importer_runs, :processed_file_sets, :integer, default: 0 unless column_exists?(:bulkrax_importer_runs, :processed_file_sets)
     add_column :bulkrax_importer_runs, :failed_file_sets, :integer, default: 0 unless column_exists?(:bulkrax_importer_runs, :failed_file_sets)

data/db/migrate/20220118001339_add_import_attempts_to_entries.rb

@@ -1,4 +1,4 @@
-class AddImportAttemptsToEntries < ActiveRecord::Migration[5.2]
+class AddImportAttemptsToEntries < ActiveRecord::Migration[5.1]
   def change
     add_column :bulkrax_entries, :import_attempts, :integer, default: 0 unless column_exists?(:bulkrax_entries, :import_attempts)
   end

data/db/migrate/20220119213325_add_work_counters_to_importer_runs.rb

@@ -1,4 +1,4 @@
-class AddWorkCountersToImporterRuns < ActiveRecord::Migration[5.2]
+class AddWorkCountersToImporterRuns < ActiveRecord::Migration[5.1]
   def change
     add_column :bulkrax_importer_runs, :processed_works, :integer, default: 0 unless column_exists?(:bulkrax_importer_runs, :processed_works)
     add_column :bulkrax_importer_runs, :failed_works, :integer, default: 0 unless column_exists?(:bulkrax_importer_runs, :failed_works)

data/db/migrate/20220301001839_create_bulkrax_pending_relationships.rb

@@ -1,4 +1,4 @@
-class CreateBulkraxPendingRelationships < ActiveRecord::Migration[5.2]
+class CreateBulkraxPendingRelationships < ActiveRecord::Migration[5.1]
   def change
     unless table_exists?(:bulkrax_pending_relationships)
       create_table :bulkrax_pending_relationships do |t|

data/db/migrate/20220303212810_add_order_to_bulkrax_pending_relationships.rb

@@ -1,4 +1,4 @@
-class AddOrderToBulkraxPendingRelationships < ActiveRecord::Migration[5.2]
+class AddOrderToBulkraxPendingRelationships < ActiveRecord::Migration[5.1]
   def change
     add_column :bulkrax_pending_relationships, :order, :integer, default: 0 unless column_exists?(:bulkrax_pending_relationships, :order)
   end

data/db/migrate/20220412233954_add_include_thumbnails_to_bulkrax_exporters.rb

@@ -1,4 +1,4 @@
-class AddIncludeThumbnailsToBulkraxExporters < ActiveRecord::Migration[5.2]
+class AddIncludeThumbnailsToBulkraxExporters < ActiveRecord::Migration[5.1]
   def change
     add_column :bulkrax_exporters, :include_thumbnails, :boolean, default: false unless column_exists?(:bulkrax_exporters, :include_thumbnails)
   end

data/db/migrate/20220413180915_add_generated_metadata_to_bulkrax_exporters.rb

@@ -1,4 +1,4 @@
-class AddGeneratedMetadataToBulkraxExporters < ActiveRecord::Migration[5.2]
+class AddGeneratedMetadataToBulkraxExporters < ActiveRecord::Migration[5.1]
   def change
     add_column :bulkrax_exporters, :generated_metadata, :boolean, default: false unless column_exists?(:bulkrax_exporters, :generated_metadata)
   end

data/db/migrate/20220609001128_rename_bulkrax_importer_run_to_importer_run.rb

@@ -1,4 +1,4 @@
-class RenameBulkraxImporterRunToImporterRun < ActiveRecord::Migration[5.2]
+class RenameBulkraxImporterRunToImporterRun < ActiveRecord::Migration[5.1]
   def up
     if column_exists?(:bulkrax_pending_relationships, :bulkrax_importer_run_id)
       remove_foreign_key :bulkrax_pending_relationships, :bulkrax_importer_runs

data/lib/bulkrax/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Bulkrax
-  VERSION = '4.3.1'
+  VERSION = '4.4.0'
 end

data/lib/bulkrax.rb

@@ -1,25 +1,29 @@
 # frozen_string_literal: true
 
+require "bulkrax/version"
 require "bulkrax/engine"
 require 'active_support/all'
 
 module Bulkrax
   class << self
-    mattr_accessor :parsers,
-                   :default_work_type,
+    mattr_accessor :api_definition,
                    :default_field_mapping,
+                   :default_work_type,
+                   :export_path,
+                   :field_mappings,
                    :fill_in_blank_source_identifiers,
                    :generated_metadata_mapping,
+                   :import_path,
+                   :multi_value_element_join_on,
+                   :multi_value_element_split_on,
+                   :object_factory,
+                   :parsers,
+                   :qa_controlled_properties,
                    :related_children_field_mapping,
                    :related_parents_field_mapping,
-                   :reserved_properties,
-                   :qa_controlled_properties,
-                   :field_mappings,
-                   :import_path,
-                   :export_path,
                    :removed_image_path,
-                   :server_name,
-                   :api_definition
+                   :reserved_properties,
+                   :server_name
 
     self.parsers = [
       { name: "OAI - Dublin Core", class_name: "Bulkrax::OaiDcParser", partial: "oai_fields" },
@@ -29,8 +33,8 @@ module Bulkrax
       { name: "XML", class_name: "Bulkrax::XmlParser", partial: "xml_fields" }
     ]
 
-    self.import_path = 'tmp/imports'
-    self.export_path = 'tmp/exports'
+    self.import_path = Bulkrax.import_path || 'tmp/imports'
+    self.export_path = Bulkrax.export_path || 'tmp/exports'
     self.removed_image_path = Bulkrax::Engine.root.join('spec', 'fixtures', 'removed.png').to_s
     self.server_name = 'bulkrax@example.com'
 
@@ -137,6 +141,29 @@ module Bulkrax
     )
   end
 
+  DEFAULT_MULTI_VALUE_ELEMENT_JOIN_ON = ' | '
+  # Specify the delimiter for joining an attribute's multi-value array into a string.
+  #
+  # @note the specific delimeter should likely be present in the multi_value_element_split_on
+  #       expression.
+  def self.multi_value_element_join_on
+    @multi_value_element_join_on ||= DEFAULT_MULTI_VALUE_ELEMENT_JOIN_ON
+  end
+
+  DEFAULT_MULTI_VALUE_ELEMENT_SPLIT_ON = /\s*[:;|]\s*/.freeze
+  # @return [RegexClass] the regular express to use to "split" an attribute's values.  If set to
+  # `true` use the DEFAULT_MULTI_VALUE_ELEMENT_JOIN_ON.
+  #
+  # @note The "true" value is to preserve backwards compatibility.
+  # @see DEFAULT_MULTI_VALUE_ELEMENT_JOIN_ON
+  def self.multi_value_element_split_on
+    if @multi_value_element_join_on.is_a?(TrueClass)
+      DEFAULT_MULTI_VALUE_ELEMENT_SPLIT_ON
+    else
+      @multi_value_element_split_on ||= DEFAULT_MULTI_VALUE_ELEMENT_SPLIT_ON
+    end
+  end
+
   # this function maps the vars from your app into your engine
   def self.setup
     yield self

data/lib/generators/bulkrax/templates/config/initializers/bulkrax.rb

@@ -10,6 +10,9 @@ Bulkrax.setup do |config|
   # Default is the first returned by Hyrax.config.curation_concerns
   # config.default_work_type = MyWork
 
+  # Factory Class to use when generating and saving objects
+  config.object_factory = Bulkrax::ObjectFactory
+
   # Path to store pending imports
   # config.import_path = 'tmp/imports'
 
@@ -67,6 +70,13 @@ Bulkrax.setup do |config|
   # is controlled by the active terms in config/authorities/rights_statements.yml
   # Defaults: 'rights_statement' and 'license'
   # config.qa_controlled_properties += ['my_field']
+
+  # Specify the delimiter regular expression for splitting an attribute's values into a multi-value array.
+  # config.multi_value_element_split_on = //\s*[:;|]\s*/.freeze
+
+  # Specify the delimiter for joining an attribute's multi-value array into a string.  Note: the
+  # specific delimeter should likely be present in the multi_value_element_split_on expression.
+  # config.multi_value_element_join_on = ' | '
 end
 
 # Sidebar for hyrax 3+ support

data/lib/tasks/reset.rake

@@ -0,0 +1,65 @@
+# frozen_string_literal: true
+
+namespace :hyrax do
+  namespace :reset do
+    desc "Delete Bulkrax importers in addition to the reset below"
+    task importers_et_all: :works_and_collections do
+      # sometimes re-running an existing importer causes issues
+      # and you may need to create new ones or delete the existing ones
+      Bulkrax::Importer.delete_all
+    end
+
+    desc 'Reset fedora / solr and corresponding database tables w/o clearing other active record tables like users'
+    task works_and_collections: [:environment] do
+      confirm('You are about to delete all works and collections, this is not reversable!')
+      require 'active_fedora/cleaner'
+      ActiveFedora::Cleaner.clean!
+      Hyrax::PermissionTemplateAccess.delete_all
+      Hyrax::PermissionTemplate.delete_all
+      Bulkrax::PendingRelationship.delete_all
+      Bulkrax::Entry.delete_all
+      Bulkrax::ImporterRun.delete_all
+      Bulkrax::Status.delete_all
+      # Remove sipity methods, everything but sipity roles
+      Sipity::Workflow.delete_all
+      Sipity::EntitySpecificResponsibility.delete_all
+      Sipity::Comment.delete_all
+      Sipity::Entity.delete_all
+      Sipity::WorkflowRole.delete_all
+      Sipity::WorkflowResponsibility.delete_all
+      Sipity::Agent.delete_all
+      Mailboxer::Receipt.destroy_all
+      Mailboxer::Notification.delete_all
+      Mailboxer::Conversation::OptOut.delete_all
+      Mailboxer::Conversation.delete_all
+      AccountElevator.switch!('single.tenant.default')
+      # we need to wait till Fedora is done with its cleanup
+      # otherwise creating the admin set will fail
+      while AdminSet.exists?(AdminSet::DEFAULT_ID)
+        puts 'waiting for delete to finish before reinitializing Fedora'
+        sleep 20
+      end
+
+      Hyrax::CollectionType.find_or_create_default_collection_type
+      Hyrax::CollectionType.find_or_create_admin_set_type
+      AdminSet.find_or_create_default_admin_set_id
+
+      collection_types = Hyrax::CollectionType.all
+      collection_types.each do |c|
+        next unless c.title.match?(/^translation missing/)
+        oldtitle = c.title
+        c.title = I18n.t(c.title.gsub("translation missing: en.", ''))
+        c.save
+        puts "#{oldtitle} changed to #{c.title}"
+      end
+    end
+
+    def confirm(action)
+      return if ENV['RESET_CONFIRMED'].present?
+      confirm_token = rand(36**6).to_s(36)
+      STDOUT.puts "#{action} Enter '#{confirm_token}' to confirm:"
+      input = STDIN.gets.chomp
+      raise "Aborting. You entered #{input}" unless input == confirm_token
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bulkrax
 version: !ruby/object:Gem::Version
-  version: 4.3.1
+  version: 4.4.0
 platform: ruby
 authors:
 - Rob Kaufman
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-04-14 00:00:00.000000000 Z
+date: 2022-12-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -328,6 +328,7 @@ files:
 - app/parsers/bulkrax/oai_dc_parser.rb
 - app/parsers/bulkrax/oai_qualified_dc_parser.rb
 - app/parsers/bulkrax/xml_parser.rb
+- app/services/bulkrax/remove_relationships_for_importer.rb
 - app/views/bulkrax/entries/_parsed_metadata.html.erb
 - app/views/bulkrax/entries/_raw_metadata.html.erb
 - app/views/bulkrax/entries/show.html.erb
@@ -404,6 +405,7 @@ files:
 - lib/generators/bulkrax/templates/config/bulkrax_api.yml
 - lib/generators/bulkrax/templates/config/initializers/bulkrax.rb
 - lib/tasks/bulkrax_tasks.rake
+- lib/tasks/reset.rake
 homepage: https://github.com/samvera-labs/bulkrax
 licenses:
 - Apache-2.0
@@ -423,7 +425,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
+rubygems_version: 3.0.3
 signing_key:
 specification_version: 4
 summary: Import and export tool for Hyrax and Hyku