bulkrax 8.2.3 → 8.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: efd4c183d72156bd32a32eb0e160485f1dc46c67630f6835db761bb93ab3b16b
-  data.tar.gz: 05c58dccdae7e285db479602490dd3be7429405f931a18aede0accbd4eec3503
+  metadata.gz: 9a409c17f35bc09ef0a8c63d06a064d01fbff68a0e4c0b91c9bfaed45d6e8169
+  data.tar.gz: 7fad94124795ff133b83d05030c550f2a52f808ac017fdd3a436c17ce039cd01
 SHA512:
-  metadata.gz: e64eecf68566f81c2d105640b71b8b6834e4c32fc18a416f0c026883387a257f913ad82ad676826b6a744a352edbce4a21aa82a304b23f2583f2c15a97f72078
-  data.tar.gz: 9c4ce4f1b4d36c5dcb031ce0a8ecb6617c0114dbc891384d3de5fe6e5ee42b04761001d269d22d6b54c616aaf3cf793e461187d0e17fd20c3a025a907c20fb4e
+  metadata.gz: 65280762e81baa620e73d965d5e7c58ec4ccb40d303c00cf634e7f9b3b3850e541cc9964900ce7b9aec9f6be1a7fe81dd9781db5b74209a9f8661f5f8a96f595
+  data.tar.gz: e91e65f6de8edc1640dd9b6e75c246fff4fd0812a3f87f6b4600dd9c95afc095be6076c9a80c5bca50ccc73c3831fc0a30c1160b46f18c483a6d68277bf762a8

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

@@ -5,18 +5,20 @@ $(document).on('turbolinks:load ready', function() {
   $('button#err_toggle').click(function() {
     $('#error_trace').toggle();
   });
+  
   $('button#fm_toggle').click(function() {
     $('#field_mapping').toggle();
   });
+
   $('#bulkraxItemModal').on('show.bs.modal', function (event) {
-    var button = $(event.relatedTarget) // Button that triggered the modal
-    var recipient = button.data('entry-id') // Extract info from data-* attributes
+    var button = $(event.relatedTarget); // Button that triggered the modal
+    var recipient = button.data('entry-id'); // Extract info from data-* attributes
     // If necessary, you could initiate an AJAX request here (and then do the updating in a callback).
     // Update the modal's content. We'll use jQuery here, but you could use a data binding library or other methods instead.
-    var modal = $(this)
+    var modal = $(this);
     modal.find('a').each(function() {
-      this.href = this.href.replace(/\d+\?/, recipient + '?')
-    })
-    return true
-  })
+      this.href = this.href.replace(/\d+\?/, recipient + '?');
+    });
+    return true;
+  });
 });

data/app/assets/javascripts/bulkrax/importers.js.erb

@@ -2,205 +2,240 @@
 // All this logic will automatically be available in application.js.
 
 function prepBulkrax(event) {
-  var refresh_button = $('.refresh-set-source')
-  var base_url = $('#importer_parser_fields_base_url')
-  var initial_base_url = base_url.val()
-  var file_path_value = $('#importer_parser_fields_import_file_path').val()
-  handleFileToggle(file_path_value)
+  var refresh_button = $('.refresh-set-source');
+  var base_url = $('#importer_parser_fields_base_url');
+  var initial_base_url = base_url.val();
+  var file_path_value = $('#importer_parser_fields_import_file_path').val();
+  handleFileToggle(file_path_value);
+
+  // Initialize the uploader only if hyraxUploader is defined
+  if (typeof $.fn.hyraxUploader === 'function') {
+    // Initialize the uploader
+    $('.fileupload-bulkrax').hyraxUploader({ maxNumberOfFiles: 1 });
+
+    // Function to toggle 'required' attribute based on uploaded files
+    function toggleRequiredAttribute() {
+      const fileInput = $('#addfiles');
+      const uploadedFilesTable = $('.fileupload-bulkrax tbody.files');
+
+      if (uploadedFilesTable.find('tr.template-download').length > 0) {
+        // Remove 'required' if there are uploaded files
+        fileInput.removeAttr('required');
+      } else {
+        // Add 'required' if no uploaded files
+        fileInput.attr('required', 'required');
+      }
+    }
+
+    // Check the required attribute when a file is added or removed
+    $('#addfiles').on('change', function() {
+      toggleRequiredAttribute();
+    });
+
+    // Also check when an upload completes or is canceled
+    $('.fileupload-bulkrax').on('fileuploadcompleted fileuploaddestroyed', function() {
+      toggleRequiredAttribute();
+    });
+
+    // Ensure 'required' is only added if there are no files on form reset
+    $('#file-upload-cancel-btn').on('click', function() {
+      $('#addfiles').attr('required', 'required');
+      $('#addfiles').val(''); // Clear file input to ensure 'required' behavior resets
+    });
+
+    // Initial check in case files are already uploaded
+    toggleRequiredAttribute();
+  }
 
   // handle refreshing/loading of external sets via button click
   $('body').on('click', '.refresh-set-source', function(e) {
-    e.preventDefault()
+    e.preventDefault();
 
-    external_set_select = $("#importer_parser_fields_set")
-    handleSourceLoad(refresh_button, base_url, external_set_select)
-  })
+    external_set_select = $("#importer_parser_fields_set");
+    handleSourceLoad(refresh_button, base_url, external_set_select);
+  });
 
   // handle refreshing/loading of external sets via blur event for the base_url field
   $('body').on('blur', '#importer_parser_fields_base_url', function(e) {
-    e.preventDefault()
+    e.preventDefault();
 
     // retrieve the latest base_url
-    base_url = $('#importer_parser_fields_base_url')
+    base_url = $('#importer_parser_fields_base_url');
 
     // ensure we don't make another query if the value is the same -- this can be forced by clicking the refresh button
     if (initial_base_url != base_url.val()) {
-      external_set_select = $("#importer_parser_fields_set")
-      handleSourceLoad(refresh_button, base_url, external_set_select)
-      initial_base_url = base_url.val()
+      external_set_select = $("#importer_parser_fields_set");
+      handleSourceLoad(refresh_button, base_url, external_set_select);
+      initial_base_url = base_url.val();
     }
-  })
+  });
 
   // hide and show correct parser fields depending on klass setting
   $('body').on('change', '#importer_parser_klass', function(e) {
-    handleParserKlass()
-  })
-  handleParserKlass()
+    handleParserKlass();
+  });
+  handleParserKlass();
 
   // observer for cloud files being added
   var form = document.getElementById('new_importer');
   if (form == null) {
-    var form = document.getElementsByClassName('edit_importer')[0];
+    form = document.getElementsByClassName('edit_importer')[0];
   }
+
   // only setup the observer on the new and edit importer pages
   if (form != null) {
     var config = { childList: true, attributes: true };
     var callback = function(mutationsList) {
       for(var mutation of mutationsList) {
-
         if (mutation.type == 'childList') {
           browseButton = document.getElementById('browse');
-          var exp = /selected_files\[[0-9]*\]\[url\]/
+          var exp = /selected_files\[[0-9]*\]\[url\]/;
           for (var node of mutation.addedNodes) {
             if (node.attributes != undefined) {
-              var name = node.attributes.name.value
+              var name = node.attributes.name.value;
               if (exp.test(name)) {
                 browseButton.innerHTML = 'Cloud Files Added';
                 browseButton.style.backgroundColor = 'green';
-                browseButton.after(document.createElement("br"), node.value.toString())
+                browseButton.after(document.createElement("br"), node.value.toString());
               }
             }
           }
         }
       }
     };
-    var observer = new MutationObserver (callback);
-    observer.observe (form, config);
+    var observer = new MutationObserver(callback);
+    observer.observe(form, config);
   }
 }
 
 function handleFileToggle(file_path) {
   if (file_path === undefined || file_path.length === 0) {
-    $('#file_path').hide()
-    $('#file_upload').hide()
-    $('#cloud').hide()
-    $('#existing_options').hide()
-    $('#file_path input').attr('required', null)
-    $('#file_upload input').attr('required', null)
+    $('#file_path').hide();
+    $('#file_upload').hide();
+    $('#cloud').hide();
+    $('#existing_options').hide();
+    $('#file_path input').attr('required', null);
+    $('#file_upload input').attr('required', null);
   } else {
-    $('#file_path').show()
-    $('#file_upload').hide()
-    $('#cloud').hide()
-    $('#existing_options').hide()
-    $('#file_path input').attr('required', 'required')
-    $('#file_upload input').attr('required', null)
-    $('#importer_parser_fields_file_style_specify_a_path_on_the_server').attr('checked', true)
+    $('#file_path').show();
+    $('#file_upload').hide();
+    $('#cloud').hide();
+    $('#existing_options').hide();
+    $('#file_path input').attr('required', 'required');
+    $('#file_upload input').attr('required', null);
+    $('#importer_parser_fields_file_style_specify_a_path_on_the_server').attr('checked', true);
   }
 
   $('#importer_parser_fields_file_style_upload_a_file').click(function(e){
-    $('#file_path').hide()
-    $('#file_upload').show()
-    $('#cloud').hide()
-    $('#existing_options').hide()
-    $('#file_path input').attr('required', null)
-    $('#file_upload input').attr('required', 'required')
-  })
+    $('#file_path').hide();
+    $('#file_upload').show();
+    $('#cloud').hide();
+    $('#existing_options').hide();
+    $('#file_path input').attr('required', null);
+    $('#file_upload input').attr('required', 'required');
+  });
   $('#importer_parser_fields_file_style_specify_a_path_on_the_server').click(function(e){
-    $('#file_path').show()
-    $('#file_upload').hide()
-    $('#cloud').hide()
-    $('#existing_options').hide()
-    $('#file_path input').attr('required', 'required')
-    $('#file_upload input').attr('required', null)
-  })
+    $('#file_path').show();
+    $('#file_upload').hide();
+    $('#cloud').hide();
+    $('#existing_options').hide();
+    $('#file_path input').attr('required', 'required');
+    $('#file_upload input').attr('required', null);
+  });
   $('#importer_parser_fields_file_style_add_cloud_file').click(function(e){
-    $('#file_path').hide()
-    $('#file_upload').hide()
-    $('#cloud').show()
-    $('#existing_options').hide()
-    $('#file_path input').attr('required', null)
-    $('#file_upload input').attr('required', null)
-  })
+    $('#file_path').hide();
+    $('#file_upload').hide();
+    $('#cloud').show();
+    $('#existing_options').hide();
+    $('#file_path input').attr('required', null);
+    $('#file_upload input').attr('required', null);
+  });
   $('#importer_parser_fields_file_style_existing_entries').click(function(e){
-    $('#file_path').hide()
-    $('#file_upload').hide()
-    $('#cloud').hide()
-    $('#existing_options').show()
-    $('#file_path input').attr('required', null)
-    $('#file_upload input').attr('required', null)
-  })
-
+    $('#file_path').hide();
+    $('#file_upload').hide();
+    $('#cloud').hide();
+    $('#existing_options').show();
+    $('#file_path input').attr('required', null);
+    $('#file_upload input').attr('required', null);
+  });
 }
 
 function handleParserKlass() {
   <% Bulkrax.parsers.map{ |p| p[:partial]}.uniq.each do |partial| %>
   if($('.<%= partial %>').length > 0) {
-    window.<%= partial %> = $('.<%= partial %>').detach()
+    window.<%= partial %> = $('.<%= partial %>').detach();
   }
   <% end %>
 
-  var parser_klass = $("#importer_parser_klass option:selected")
+  var parser_klass = $("#importer_parser_klass option:selected");
   if(parser_klass.length > 0 && parser_klass.data('partial') && parser_klass.data('partial').length > 0) {
-    $('.parser_fields').append(window[parser_klass.data('partial')])
+    $('.parser_fields').append(window[parser_klass.data('partial')]);
   }
 
-  handleBrowseEverything()
-  var file_path_value = $('#importer_parser_fields_import_file_path').val()
-  handleFileToggle(file_path_value)
+  handleBrowseEverything();
+  var file_path_value = $('#importer_parser_fields_import_file_path').val();
+  handleFileToggle(file_path_value);
 }
 
 function handleBrowseEverything(){
-  var button = $("button[data-toggle='browse-everything']")
+  var button = $("button[data-toggle='browse-everything']");
   if(button.length == 0) { return; }
   button.browseEverything({
     route: button.data('route'),
     target: button.data('target')
   }).done(function(data) {
     var evt = { isDefaultPrevented: function() { return false; } };
-    $('.ev-browser.show').removeClass('show')
+    $('.ev-browser.show').removeClass('show');
     if($('#fileupload').length > 0) {
-      var files = $.map(data, function(d) { return { name: d.file_name, size: d.file_size, id: d.url } });
+      var files = $.map(data, function(d) { return { name: d.file_name, size: d.file_size, id: d.url }; });
       $.blueimp.fileupload.prototype.options.done.call($('#fileupload').fileupload(), evt, { result: { files: files }});
     }
-    return true
-    // User has submitted files; data contains an array of URLs and their options
+    return true;
   }).cancel(function() {
-    $('.ev-browser.show').removeClass('show')
-    // User cancelled the browse operation
+    $('.ev-browser.show').removeClass('show');
   }).fail(function(status, error, text) {
-    $('.ev-browser.show').removeClass('show')
-    // URL retrieval experienced a technical failure
+    $('.ev-browser.show').removeClass('show');
   });
 }
 
 function handleSourceLoad(refresh_button, base_url, external_set_select) {
   if (base_url.val() == "") { // ignore empty base_url value
-    return
+    return;
   }
 
-  var initial_button_text = refresh_button.html()
+  var initial_button_text = refresh_button.html();
 
-  refresh_button.html('Refreshing...')
-  refresh_button.attr('disabled', true)
+  refresh_button.html('Refreshing...');
+  refresh_button.attr('disabled', true);
 
   $.post('/importers/external_sets', {
     base_url: base_url.val(),
   }, function(res) {
     if (!res.error) {
-      genExternalSetOptions(external_set_select, res.sets) // sets is [[name, spec]...]
+      genExternalSetOptions(external_set_select, res.sets);
     } else {
-      setError(external_set_select, res.error)
+      setError(external_set_select, res.error);
     }
 
-    refresh_button.html(initial_button_text)
-    refresh_button.attr('disabled', false)
-  })
+    refresh_button.html(initial_button_text);
+    refresh_button.attr('disabled', false);
+  });
 }
 
 function genExternalSetOptions(selector, sets) {
-  out = '<option value="">- Select One -</option>'
+  out = '<option value="">- Select One -</option>';
 
   out += sets.map(function(set) {
-    return '<option value="'+set[1]+'">'+set[0]+'</option>'
-  })
+    return '<option value="'+set[1]+'">'+set[0]+'</option>';
+  });
 
-  selector.html(out)
-  selector.attr('disabled', false)
+  selector.html(out);
+  selector.attr('disabled', false);
 }
 
 function setError(selector, error) {
-  selector.html('<option value="none">Error - Please enter Base URL and try again</option>')
-  selector.attr('disabled', true)
+  selector.html('<option value="none">Error - Please enter Base URL and try again</option>');
+  selector.attr('disabled', true);
 }
 
-$(document).on({'ready': prepBulkrax, 'turbolinks:load': prepBulkrax})
+$(document).on({'ready': prepBulkrax, 'turbolinks:load': prepBulkrax});

data/app/controllers/bulkrax/importers_controller.rb

@@ -78,11 +78,13 @@ module Bulkrax
 
     # POST /importers
     # rubocop:disable Metrics/MethodLength
+    # rubocop:disable Metrics/AbcSize
     def create
       # rubocop:disable Style/IfInsideElse
       if api_request?
         return return_json_response unless valid_create_params?
       end
+      uploads = Hyrax::UploadedFile.find(params[:uploaded_files]) if params[:uploaded_files].present?
       file = file_param
       cloud_files = cloud_params
 
@@ -93,7 +95,7 @@ module Bulkrax
       # on a new import otherwise it only gets updated during the update path
       @importer.parser_fields['update_files'] = true if params[:commit] == 'Create and Import'
       if @importer.save
-        files_for_import(file, cloud_files)
+        files_for_import(file, cloud_files, uploads)
         if params[:commit] == 'Create and Import'
           Bulkrax::ImporterJob.send(@importer.parser.perform_method, @importer.id)
           render_request('Importer was successfully created and import has been queued.')
@@ -112,6 +114,7 @@ module Bulkrax
       end
       # rubocop:enable Style/IfInsideElse
     end
+    # rubocop:enable Metrics/AbcSize
 
     # PATCH/PUT /importers/1
     # # @todo refactor so as to not need to disable rubocop
@@ -120,7 +123,7 @@ module Bulkrax
       if api_request?
         return return_json_response unless valid_update_params?
       end
-
+      uploads = Hyrax::UploadedFile.find(params[:uploaded_files]) if params[:uploaded_files].present?
       file = file_param
       cloud_files = cloud_params
 
@@ -128,7 +131,7 @@ module Bulkrax
       field_mapping_params if params[:importer][:parser_fields].present?
 
       if @importer.update(importer_params)
-        files_for_import(file, cloud_files) unless file.nil? && cloud_files.nil?
+        files_for_import(file, cloud_files, uploads)
         # do not perform the import
         unless params[:commit] == 'Update Importer'
           set_files_parser_fields
@@ -218,8 +221,9 @@ module Bulkrax
 
     private
 
-    def files_for_import(file, cloud_files)
-      return if file.blank? && cloud_files.blank?
+    def files_for_import(file, cloud_files, uploads)
+      return if file.blank? && cloud_files.blank? && uploads.blank?
+
       @importer[:parser_fields]['import_file_path'] = @importer.parser.write_import_file(file) if file.present?
       if cloud_files.present?
         @importer[:parser_fields]['cloud_file_paths'] = cloud_files
@@ -229,6 +233,13 @@ module Bulkrax
         target = @importer.parser.retrieve_cloud_files(cloud_files, @importer)
         @importer[:parser_fields]['import_file_path'] = target if target.present?
       end
+
+      if uploads.present?
+        uploads.each do |upload|
+          @importer[:parser_fields]['import_file_path'] = @importer.parser.write_import_file(upload.file.file)
+        end
+      end
+
       @importer.save
     end
 

data/app/controllers/concerns/bulkrax/datatables_behavior.rb

@@ -132,7 +132,7 @@ module Bulkrax
           status_message: status_message_for(e),
           type: e.type,
           updated_at: e.updated_at,
-          errors: e.status_message == 'Failed' ? view_context.link_to(e.error_class, view_context.item_entry_path(item, e)) : "",
+          errors: e.latest_status&.error_class&.present? ? view_context.link_to(e.latest_status.error_class, view_context.item_entry_path(item, e), title: e.latest_status.error_message) : "",
           actions: entry_util_links(e, item)
         }
       end

data/app/factories/bulkrax/object_factory.rb

@@ -165,10 +165,7 @@ module Bulkrax
     end
 
     def delete(_user)
-      obj = find
-      return false unless obj
-
-      obj.delete(eradicate: true)
+      find&.delete(eradicate: true)
     end
 
     private

data/app/jobs/bulkrax/export_work_job.rb

@@ -26,7 +26,7 @@ module Bulkrax
         end
         # rubocop:enable Rails/SkipsModelValidations
       end
-      return entry if exporter_run.reload.enqueued_records.positive?
+      return entry if exporter_run.enqueued_records.positive?
 
       if exporter_run.failed_records.positive?
         exporter_run.exporter.set_status_info('Complete (with failures)')

data/app/models/bulkrax/status.rb

@@ -2,7 +2,7 @@
 
 module Bulkrax
   class Status < ApplicationRecord
-    belongs_to :statusable, polymorphic: true, denormalize: { fields: %i[status_message error_class], if: :latest? }
+    belongs_to :statusable, polymorphic: true, denormalize: { fields: %i[status_message], if: :latest? }
     belongs_to :runnable, polymorphic: true
     serialize :error_backtrace, Array
 

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

@@ -26,15 +26,11 @@ module Bulkrax
 
     # Prepend the file_set id to ensure a unique filename and also one that is not longer than 255 characters
     def filename(file_set)
+      # NOTE: Will this work with Valkyrie?
       return if file_set.original_file.blank?
-      if file_set.original_file.respond_to?(:original_filename) # valkyrie
-        fn = file_set.original_file.original_filename
-        mime = ::Marcel::MimeType.for(file_set.original_file.file.io)
-      else # original non valkyrie version
-        fn = file_set.original_file.file_name.first
-        mime = ::Marcel::MimeType.for(declared_type: file_set.original_file.mime_type)
-      end
-      ext_mime = ::Marcel::MimeType.for(name: fn)
+      fn = file_set.original_file.file_name.first
+      mime = ::Marcel::MimeType.for(file_set.original_file.mime_type)
+      ext_mime = ::Marcel::MimeType.for(file_set.original_file.file_name)
       if fn.include?(file_set.id) || importerexporter.metadata_only?
         filename = "#{fn}.#{mime.to_sym}"
         filename = fn if mime.to_s == ext_mime.to_s

data/app/parsers/bulkrax/csv_parser.rb

@@ -244,10 +244,7 @@ module Bulkrax
       record = Bulkrax.object_factory.find(identifier)
       return unless record
 
-      file_sets = Array.wrap(record) if record.file_set?
-      if file_sets.nil? # for valkyrie
-        file_sets = record.respond_to?(:file_sets) ? record.file_sets : record.members&.select(&:file_set?)
-      end
+      file_sets = record.file_set? ? Array.wrap(record) : record.file_sets
       file_sets << record.thumbnail if exporter.include_thumbnails && record.thumbnail.present? && record.work?
       file_sets.each do |fs|
         path = File.join(exporter_export_path, folder_count, 'files')
@@ -255,7 +252,7 @@ module Bulkrax
         file = filename(fs)
         next if file.blank? || fs.original_file.blank?
 
-        io = fs.original_file.respond_to?(:uri) ? open(fs.original_file.uri) : fs.original_file.file.io
+        io = open(fs.original_file.uri)
         File.open(File.join(path, file), 'wb') do |f|
           f.write(io.read)
           f.close

data/app/parsers/bulkrax/parser_export_record_set.rb

@@ -173,7 +173,7 @@ module Bulkrax
       # @see https://github.com/samvera/hyrax/blob/64c0bbf0dc0d3e1b49f040b50ea70d177cc9d8f6/app/indexers/hyrax/work_indexer.rb#L15-L18
       def file_sets
         @file_sets ||= ParserExportRecordSet.in_batches(candidate_file_set_ids) do |batch_of_ids|
-          fsq = "has_model_ssim:\"#{Bulkrax.file_model_internal_resource.demodulize}\" AND id:(\"" + batch_of_ids.join('" OR "') + "\")"
+          fsq = "has_model_ssim:#{Bulkrax.file_model_internal_resource} AND id:(\"" + batch_of_ids.join('" OR "') + "\")"
           fsq += extra_filters if extra_filters.present?
           Bulkrax.object_factory.query(
             fsq,

data/app/views/bulkrax/importers/_bagit_fields.html.erb

@@ -41,9 +41,16 @@
   <p>File upload and Cloud File upload must be a Zip file containing a single BagIt Bag, or a folder containing multiple BagIt Bags.</p>
   <p>The Server Path can point to a BagIt Bag, a folder containing BagIt Bags, or a zip file containing either.</p>
 
-  <%= fi.input :file_style, collection: ['Upload a File', 'Specify a Path on the Server', 'Add Cloud File'], as: :radio_buttons, label: false %>
+<%= fi.input :file_style,
+  collection: ['Upload a File', 'Specify a Path on the Server'] +
+              (defined?(::Hyrax) && Hyrax.config.browse_everything? ? ['Add Cloud File'] : []),
+  as: :radio_buttons, label: false %>
   <div id='file_upload'>
-    <%= fi.input 'file', as: :file, input_html: {accept: 'application/zip'} %><br />
+    <% if defined?(::Hyrax) %>
+      <%= render 'bulkrax/importers/file_uploader', accepted_file_types: 'application/zip' %>
+    <% else %>
+      <%= fi.input 'file', as: :file, input_html: {accept: 'application/zip'} %><br />
+    <% end %>
   </div>
   <div id='file_path'>
     <%= fi.input :import_file_path, as: :string, input_html: { value: importer.parser_fields['import_file_path'] } %>
@@ -53,4 +60,4 @@
       <%= render 'browse_everything', form: form %>
     <% end %>
   </div>
-</div>
+</div>

data/app/views/bulkrax/importers/_csv_fields.html.erb

@@ -1,5 +1,4 @@
 <div class='csv_fields'>
-
   <%= fi.input :visibility,
     label: 'Default Visibility',
     collection: [
@@ -11,7 +10,6 @@
     input_html: { class: 'form-control' },
     hint: 'If your CSV includes the visibility field, it will override the default setting.'
   %>
-
   <% if defined?(::Hyrax) %>
     <% rights_statements = Hyrax.config.rights_statement_service_class.new %>
     <%= fi.input :rights_statement,
@@ -24,26 +22,32 @@
           %>
     <%= fi.input :override_rights_statement, as: :boolean, hint: 'If checked, always use the selected rights statment. If unchecked, use rights or rights_statement from the record and only use the provided value if dc:rights is blank.', input_html: { checked: (importer.parser_fields['override_rights_statement'] == "1") } %>
   <% end %>
-  <h4>Add CSV File to Import:</h4>
+  <h4>Add CSV or ZIP File to Import:</h4>
   <%# accept a single file upload; data files and bags will need to be added another way %>
-
   <% file_style_list = ['Upload a File', 'Specify a Path on the Server'] %>
   <% file_style_list << 'Existing Entries' unless importer.new_record? %>
   <%= fi.input :file_style, collection: file_style_list, as: :radio_buttons, label: false %>
+  
   <div id='file_upload'>
-    <%= fi.input 'file', as: :file, input_html: { accept: 'text/csv,application/zip,application/gzip' } %><br />
+    <% if defined?(::Hyrax) %>
+      <%= render 'bulkrax/importers/file_uploader', accepted_file_types: 'text/csv,application/zip,application/gzip' %>
+    <% else %>
+      <%= fi.input 'file', as: :file, input_html: { accept: 'text/csv,application/zip,application/gzip' } %><br />
+    <% end %>
   </div>
+  
   <div id='file_path'>
     <%= fi.input :import_file_path, as: :string, input_html: { value: importer.parser_fields['import_file_path'] } %>
   </div>
+  
   <div id='existing_options'>
     <%= fi.collection_check_boxes :entry_statuses, [['Failed'], ['Pending'], ['Skipped'], ['Deleted'], ['Complete']], :first, :first %>
   </div>
-
+  
   <% if defined?(::Hyrax) && Hyrax.config.browse_everything? %>
-	  <h4>Add Files to Import:</h4>
-	  <p>Choose files to upload. The filenames must be unique, and the filenames must be referenced in a column called 'file' in the accompanying CSV file.</p>
+    <h4>Add Files to Import:</h4>
+    <p>Choose files to upload. The filenames must be unique, and the filenames must be referenced in a column called 'file' in the accompanying CSV file.</p>
     <%= render 'browse_everything', form: form %>
   <% end %>
   <br />
-</div>
+</div>

data/app/views/bulkrax/importers/_file_uploader.html.erb

@@ -0,0 +1,37 @@
+<div class="fileupload-bulkrax">
+  <noscript><input type="hidden" name="redirect" value="<%= main_app.root_path %>" /></noscript>
+  <table role="presentation" class="table table-striped"><tbody class="files"></tbody></table>
+  <div class="fileupload-buttonbar">
+    <div class="row">
+      <div class="col-12">
+        <div class="fileinput-button" id="add-files">
+          <input id="addfiles" type="file" style="display:none;" name="files[]"
+                 accept="<%= accepted_file_types || 'text/csv,application/zip,application/gzip' %>" multiple />
+          <button type="button" class="btn btn-success" onclick="document.getElementById('addfiles').click();">
+            <span class="fa fa-plus"></span>
+            <span>Add Files</span>
+          </button>
+        </div>
+        <button type="reset" id="file-upload-cancel-btn" class="btn btn-warning cancel">
+          <span class="fa fa-ban"></span>
+          <span>Cancel Upload</span>
+        </button>
+        <span class="fileupload-process"></span>
+      </div>
+    </div>
+    <div class="row">
+      <div class="col-12">
+        <div class="fileupload-progress fade">
+          <div class="progress" role="progressbar" aria-valuemin="0" aria-valuemax="100">
+            <div class="progress-bar progress-bar-striped progress-bar-animated bg-success" style="width:0%;"></div>
+          </div>
+          <div class="progress-extended">&nbsp;</div>
+        </div>
+      </div>
+    </div>
+  </div>
+  <div class="dropzone">
+    Drop files here to upload
+  </div>
+  <%= render 'hyrax/uploads/js_templates' %>
+</div>

data/app/views/bulkrax/importers/_xml_fields.html.erb

@@ -47,16 +47,22 @@
   <p>File upload and Cloud File upload MUST be a either a single XML file (for metadata only import) OR a Zip file containing the XML files and data files, each in a separate folder.</p>
   <p>The Server Path can point to a folder containing XML files and data files to import, or direct to the XML file itself.</p>
 
-  <%= fi.input :file_style, collection: ['Upload a File', 'Specify a Path on the Server', 'Add Cloud File'], as: :radio_buttons, label: false %>
+<%= fi.input :file_style,
+  collection: ['Upload a File', 'Specify a Path on the Server'] +
+              (defined?(::Hyrax) && Hyrax.config.browse_everything? ? ['Add Cloud File'] : []),
+  as: :radio_buttons, label: false %>
   <div id='file_upload'>
-    <%= fi.input 'file', as: :file, input_html: {accept: ['application/zip', 'application/xml']} %><br />
+    <% if defined?(::Hyrax) %>
+      <%= render 'bulkrax/importers/file_uploader', accepted_file_types: 'application/zip,application/xml' %>
+    <% else %>
+      <%= fi.input 'file', as: :file, input_html: {accept: ['application/zip', 'application/xml']} %><br />
+    <% end%>
   </div>
   <div id='file_path'>
     <%= fi.input :import_file_path, as: :string, input_html: { value: importer.parser_fields['import_file_path'] } %>
   </div>
   <div id='cloud'>
     <% if defined?(::Hyrax) && Hyrax.config.browse_everything? %>
-      <%= render 'browse_everything', form: form %>
     <% end %>
   </div>
-</div>
+</div>

data/lib/bulkrax/version.rb

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

data/lib/tasks/bulkrax_tasks.rake

@@ -9,7 +9,7 @@ namespace :bulkrax do
                                    progress_mark: ' ',
                                    remainder_mark: "\u{FF65}")
     Bulkrax::Status.latest_by_statusable.includes(:statusable).find_each do |status|
-      status.statusable.update(status_message: status.status_message, error_class: status.error_class)
+      status.statusable.update(status_message: status.status_message)
       @progress.increment
     end
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bulkrax
 version: !ruby/object:Gem::Version
-  version: 8.2.3
+  version: 8.3.0
 platform: ruby
 authors:
 - Rob Kaufman
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-12-20 00:00:00.000000000 Z
+date: 2024-12-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -30,14 +30,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.6.0
+        version: 0.4.6
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.6.0
+        version: 0.4.6
 - !ruby/object:Gem::Dependency
   name: coderay
   requirement: !ruby/object:Gem::Requirement
@@ -428,6 +428,7 @@ files:
 - app/views/bulkrax/importers/_csv_fields.html.erb
 - app/views/bulkrax/importers/_edit_form_buttons.html.erb
 - app/views/bulkrax/importers/_edit_item_buttons.html.erb
+- app/views/bulkrax/importers/_file_uploader.html.erb
 - app/views/bulkrax/importers/_form.html.erb
 - app/views/bulkrax/importers/_oai_fields.html.erb
 - app/views/bulkrax/importers/_xml_fields.html.erb
@@ -486,8 +487,6 @@ files:
 - db/migrate/20240823173525_add_error_tracking_to_pending_relationships.rb
 - db/migrate/20240916182737_add_last_imported_at_to_bulkrax_importers.rb
 - db/migrate/20240916182823_add_next_import_at_to_bulkrax_importers.rb
-- db/migrate/20241203010707_entry_error_denormalization.rb
-- db/migrate/20241205212513_faster_first_entry.rb
 - lib/bulkrax.rb
 - lib/bulkrax/engine.rb
 - lib/bulkrax/entry_spec_helper.rb
@@ -520,7 +519,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.4
+rubygems_version: 3.4.20
 signing_key:
 specification_version: 4
 summary: Import and export tool for Hyrax and Hyku

data/db/migrate/20241203010707_entry_error_denormalization.rb

@@ -1,7 +0,0 @@
-class EntryErrorDenormalization < ActiveRecord::Migration[5.1]
-  def change
-    add_column :bulkrax_entries, :error_class, :string unless column_exists?(:bulkrax_entries, :error_class)
-    add_column :bulkrax_importers, :error_class, :string unless column_exists?(:bulkrax_importers, :error_class)
-    add_column :bulkrax_exporters, :error_class, :string unless column_exists?(:bulkrax_exporters, :error_class)
-  end
-end

data/db/migrate/20241205212513_faster_first_entry.rb

@@ -1,7 +0,0 @@
-# frozen_string_literal: true
-class FasterFirstEntry < ActiveRecord::Migration[5.2]
-  def change
-    add_index :bulkrax_entries, [:importerexporter_id, :importerexporter_type, :id], name: 'index_bulkrax_entries_on_importerexporter_id_type_and_id' unless index_exists?(:bulkrax_entries, [:importerexporter_id, :importerexporter_type, :id],
-      name: 'index_bulkrax_entries_on_importerexporter_id_type_and_id')
-  end
-end