bulkrax 6.0.1 → 7.0.0

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

@@ -0,0 +1,18 @@
+<div class="modal fade" id="bulkraxItemModal" tabindex="-1" role="dialog" aria-labelledby="bulkraxItemModalLabel">
+  <div class="modal-dialog" role="document">
+    <div class="modal-content">
+      <div class="modal-body">
+        <h5>Options for Updating an Entry</h5>
+        <hr />
+        <p>Rebuild metadata and files.</p>
+        <%= link_to 'Build', item_entry_path(item, e), method: :patch, class: 'btn btn-primary'  %>
+        <hr />
+        <p>Remove existing work and then recreate the works metadata and files.</p>
+        <%= link_to 'Remove and then Build', item_entry_path(item, e, destroy_first: true), method: :patch, class: 'btn btn-primary'  %>
+      </div>
+      <div class="modal-footer">
+        <button type="button" class="btn btn-default" data-dismiss="modal"><%= t('helpers.action.cancel') %></button>
+      </div>
+    </div>
+  </div>
+</div>

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

@@ -13,69 +13,25 @@
 
 <div class="panel panel-default">
   <div class="panel-body">
-    <% if @importers.present? %>
-      <div class="table-responsive">
-        <table class="table table-striped datatable">
-          <thead>
-            <tr>
-              <th scope="col">Name</th>
-              <th scope="col">Status</th>
-              <th scope="col">Last Run</th>
-              <th scope="col">Next Run</th>
-              <th scope="col">Entries Enqueued</th>
-              <th scope="col">Entries Processed</th>
-              <th scope="col">Entries Failed</th>
-              <th scope="col">Entries Deleted Upstream</th>
-              <th scope="col">Total Collection Entries</th>
-              <th scope="col">Total Work Entries</th>
-              <th scope="col">Total File Set Entries</th>
-              <th scope="col"></th>
-              <th scope="col"></th>
-              <th scope="col"></th>
-            </tr>
-          </thead>
-          <tbody>
-            <% @importers.each do |importer| %>
-              <tr>
-                <th scope="row"><%= link_to importer.name, importer_path(importer) %></th>
-                <td><%= importer.status %></td>
-                <td><%= importer.last_imported_at.strftime("%b %d, %Y") if importer.last_imported_at %></td>
-                <td><%= importer.next_import_at.strftime("%b %d, %Y") if importer.next_import_at %></td>
-                <td><%= importer.last_run&.enqueued_records %></td>
-                <td><%= (importer.last_run&.processed_records || 0) %></td>
-                <td><%= (importer.last_run&.failed_records || 0) %></td>
-                <td><%= importer.last_run&.deleted_records %></td>
-                <td><%= importer.last_run&.total_collection_entries %></td>
-                <td><%= importer.last_run&.total_work_entries %></td>
-                <td><%= importer.last_run&.total_file_set_entries %></td>
-                <td><%= link_to raw('<span class="glyphicon glyphicon-info-sign"></span>'), importer_path(importer) %></td>
-                <td><%= link_to raw('<span class="glyphicon glyphicon-pencil"></span>'), edit_importer_path(importer) %></td>
-                <td><%= link_to raw('<span class="glyphicon glyphicon-remove"></span>'), importer, method: :delete, data: { confirm: 'Are you sure?' } %></td>
-              </tr>
-            <% end %>
-          </tbody>
-        </table>
-      </div>
-      <% else %>
-        <p>No importers have been created.</p>
-      <% end %>
+    <div class="table-responsive">
+      <table id='importers-table' class="table table-striped">
+        <thead>
+          <tr>
+            <th scope="col">Name</th>
+            <th scope="col">Status</th>
+            <th scope="col">Last Run</th>
+            <th scope="col">Next Run</th>
+            <th scope="col">Entries Enqueued</th>
+            <th scope="col">Entries Processed</th>
+            <th scope="col">Entries Failed</th>
+            <th scope="col">Entries Deleted Upstream</th>
+            <th scope="col">Total Collection Entries</th>
+            <th scope="col">Total Work Entries</th>
+            <th scope="col">Total File Set Entries</th>
+            <th scope="col">Actions</th>
+          </tr>
+        </thead>
+      </table>
+    </div>
   </div>
 </div>
-
-<script>
-  $(function() {
-    $('#DataTables_Table_0').DataTable({
-      destroy: true, /* Reinitialize DataTable with config below */
-      'columnDefs': [
-          { 'orderable': true, 'targets': [...Array(10).keys()] },
-          { 'orderable': false, 'targets': [10, 11, 12] }
-      ],
-      'language': {
-        'info': 'Showing _START_ to _END_ of _TOTAL_ importers',
-        'infoEmpty': 'No importers to show',
-        'infoFiltered': '(filtered from _MAX_ total importers)',
-        'lengthMenu': 'Show _MENU_ importers'
-      }
-    })
-  })
-</script>

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

@@ -1,9 +1,10 @@
 <div class="col-xs-12 main-header">
   <h1><span class="fa fa-cloud-upload" aria-hidden="true"></span> Importer: <%= @importer.name %></h1>
-  <% if @importer.parser_klass == 'Bulkrax::CsvParser' && @work_entries.map(&:failed?).any? %>
+
+  <% if @importer.failed_entries? %>
     <div class="pull-right">
-      <%= link_to 'Export Errored Entries', importer_export_errors_path(@importer.id), class: 'btn btn-primary' %>
-      <%= link_to 'Upload Corrected Entries', importer_upload_corrected_entries_path(@importer.id), class: 'btn btn-primary' %>
+      <%= link_to 'Export Errored Entries', importer_export_errors_path(@importer.id), class: 'btn btn-primary', data: { turbolinks: false }%>
+      <%= link_to 'Upload Corrected Entries', importer_upload_corrected_entries_path(@importer.id), class: 'btn btn-primary' if @importer.parser.is_a?(Bulkrax::CsvParser) %>
     </div>
   <% end %>
 </div>
@@ -75,17 +76,10 @@
 
     <div class="bulkrax-nav-tab-bottom-margin">
       <!-- Nav tabs -->
-      <ul class="bulkrax-nav-tab-top-margin tab-nav nav nav-tabs" role="tablist">
-        <li role="presentation" class="nav-link active"><a href="#work-entries" aria-controls="work-entries" role="tab" data-toggle="tab"><%= t('bulkrax.importer.labels.work_entries') %></a></li>
-        <li role="presentation" class="nav-link"><a href="#collection-entries" aria-controls="collection-entries" role="tab" data-toggle="tab"><%= t('bulkrax.importer.labels.collection_entries') %></a></li>
-        <li role="presentation" class="nav-link"><a href="#file-set-entries" aria-controls="file-set-entries" role="tab" data-toggle="tab"><%= t('bulkrax.importer.labels.file_set_entries') %></a></li>
-      </ul>
-      <!-- Tab panes -->
-      <div class="tab-content outline">
-        <%= render partial: 'bulkrax/shared/work_entries_tab', locals: { item: @importer, entries: @work_entries } %>
-        <%= render partial: 'bulkrax/shared/collection_entries_tab', locals: { item: @importer, entries: @collection_entries } %>
-        <%= render partial: 'bulkrax/shared/file_set_entries_tab', locals: { item: @importer, entries: @file_set_entries } %>
+      <div class="outline">
+        <%= render partial: 'bulkrax/shared/entries_tab', locals: { item: @importer} %>
       </div>
+      <%= render partial: 'bulkrax/importers/edit_item_buttons', locals: { item: @importer, e: @first_entry } if @first_entry.present? %>
     </div>
 
     <p class="bulkrax-p-align">

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

@@ -0,0 +1,16 @@
+<div class="tab-pane bulkrax-nav-tab-table-left-align" >
+  <table id='importer-show-table' class='table table-striped'>
+    <thead>
+      <tr>
+        <th><%= t('bulkrax.table_header.labels.identifier') %></th>
+        <th><%= t('bulkrax.table_header.labels.entry_id') %></th>
+        <th><%= t('bulkrax.table_header.labels.status') %></th>
+        <th><%= t('bulkrax.table_header.labels.type') %></th>
+        <th><%= t('bulkrax.table_header.labels.updated_at') %></th>
+        <th><%= t('bulkrax.table_header.labels.errors') %></th>
+        <th><%= t('bulkrax.table_header.labels.actions') %></th>
+      </tr>
+    </thead>
+  </table>
+  <div id='importer-entry-classes' class='hidden'><%= [item.parser.entry_class.to_s, item.parser.collection_entry_class.to_s, item.parser.file_set_entry_class.to_s].compact.join('|') %></div>
+</div>

data/config/routes.rb

@@ -3,15 +3,21 @@
 Bulkrax::Engine.routes.draw do
   resources :exporters do
     get :download
-    resources :entries, only: %i[show]
+    get :entry_table
+    collection do
+      get :exporter_table
+    end
+    resources :entries, only: %i[show update destroy]
   end
   resources :importers do
     put :continue
+    get :entry_table
     get :export_errors
     collection do
+      get :importer_table
       post :external_sets
     end
-    resources :entries, only: %i[show]
+    resources :entries, only: %i[show update destroy]
     get :upload_corrected_entries
     post :upload_corrected_entries_file
   end

data/db/migrate/20240208005801_denormalize_status_message.rb

@@ -0,0 +1,7 @@
+class DenormalizeStatusMessage < ActiveRecord::Migration[5.2]
+  def change
+    add_column :bulkrax_entries, :status_message, :string, default: 'Pending' unless column_exists?(:bulkrax_entries, :status_message)
+    add_column :bulkrax_importers, :status_message, :string, default: 'Pending' unless column_exists?(:bulkrax_importers, :status_message)
+    add_column :bulkrax_exporters, :status_message, :string, default: 'Pending' unless column_exists?(:bulkrax_exporters, :status_message)
+  end
+end

data/db/migrate/20240209070952_update_identifier_index.rb

@@ -0,0 +1,6 @@
+class UpdateIdentifierIndex < ActiveRecord::Migration[5.2]
+  def change
+    remove_index :bulkrax_entries, :identifier if index_exists?(:bulkrax_entries, :identifier )
+    add_index :bulkrax_entries, [:identifier, :importerexporter_id, :importerexporter_type], name: 'bulkrax_identifier_idx' unless index_exists?(:bulkrax_entries, [:identifier, :importerexporter_id, :importerexporter_type], name: 'bulkrax_identifier_idx')
+  end
+end

data/lib/bulkrax/engine.rb

@@ -13,6 +13,12 @@ module Bulkrax
       end
     end
 
+    initializer 'requires' do
+      require 'bulkrax/persistence_layer'
+      require 'bulkrax/persistence_layer/active_fedora_adapter' if defined?(ActiveFedora)
+      require 'bulkrax/persistence_layer/valkyrie_adapter' if defined?(Valkyrie)
+    end
+
     config.generators do |g|
       g.test_framework :rspec
       begin

data/lib/bulkrax/persistence_layer/active_fedora_adapter.rb

@@ -0,0 +1,27 @@
+# frozen_string_literal: true
+
+module Bulkrax
+  module PersistenceLayer
+    class ActiveFedoraAdapter < AbstractAdapter
+      def self.find(id)
+        ActiveFedora::Base.find(id)
+      rescue ActiveFedora::ObjectNotFoundError => e
+        raise PersistenceLayer::RecordNotFound, e.message
+      end
+
+      def self.query(q, **kwargs)
+        ActiveFedora::SolrService.query(q, **kwargs)
+      end
+
+      def self.clean!
+        super do
+          ActiveFedora::Cleaner.clean!
+        end
+      end
+
+      def self.solr_name(field_name)
+        ActiveFedora.index_field_mapper.solr_name(field_name)
+      end
+    end
+  end
+end

data/lib/bulkrax/persistence_layer/valkyrie_adapter.rb

@@ -0,0 +1,8 @@
+# frozen_string_literal: true
+
+module Bulkrax
+  module PersistenceLayer
+    class ValkyrieAdapter < AbstractAdapter
+    end
+  end
+end

data/lib/bulkrax/persistence_layer.rb

@@ -0,0 +1,38 @@
+# frozen_string_literal: true
+
+module Bulkrax
+  ##
+  # The target data layer where we write and read our imported {Bulkrax::Entry} objects.
+  module PersistenceLayer
+    # We're inheriting from an ActiveRecord exception as that is something we know will be here; and
+    # something that the main_app will be expect to be able to handle.
+    class ObjectNotFoundError < ActiveRecord::RecordNotFound
+    end
+
+    # We're inheriting from an ActiveRecord exception as that is something we know will be here; and
+    # something that the main_app will be expect to be able to handle.
+    class RecordInvalid < ActiveRecord::RecordInvalid
+    end
+
+    class AbstractAdapter
+      # @see ActiveFedora::Base.find
+      def self.find(id)
+        raise NotImplementedError, "#{self}.#{__method__}"
+      end
+
+      def self.solr_name(field_name)
+        raise NotImplementedError, "#{self}.#{__method__}"
+      end
+
+      # @yield when Rails application is running in test environment.
+      def self.clean!
+        return true unless Rails.env.test?
+        yield
+      end
+
+      def self.query(q, **kwargs)
+        raise NotImplementedError, "#{self}.#{__method__}"
+      end
+    end
+  end
+end

data/lib/bulkrax/version.rb

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

data/lib/bulkrax.rb

@@ -3,7 +3,8 @@
 require "bulkrax/version"
 require "bulkrax/engine"
 require 'active_support/all'
-
+require 'coderay'
+require 'denormalize_fields'
 # rubocop:disable Metrics/ModuleLength
 module Bulkrax
   extend self # rubocop:disable Style/ModuleFunction
@@ -19,7 +20,6 @@ module Bulkrax
                   :export_path,
                   :field_mappings,
                   :file_model_class,
-                  :fill_in_blank_source_identifiers,
                   :generated_metadata_mapping,
                   :import_path,
                   :multi_value_element_join_on,
@@ -35,6 +35,86 @@ module Bulkrax
                   :reserved_properties,
                   :server_name
 
+    ##
+    # @return [#call] with arity 2.  The first parameter is a {Bulkrax::ApplicationParser} and the
+    #         second parameter is an Integer for the index of the record encountered in the import.
+    attr_accessor :fill_in_blank_source_identifiers
+
+    ##
+    # @param adapter [Class<Bulkrax::PersistenceLayer::AbstractAdapter>]
+    attr_writer :persistence_adapter
+
+    ##
+    # @param [String]
+    attr_writer :solr_key_for_member_file_ids
+
+    ##
+    # @return [String]
+    # @see https://github.com/samvera/hyrax/pull/6513
+    def solr_key_for_member_file_ids
+      return @solr_key_for_member_file_ids if @solr_key_for_member_file_ids.present?
+
+      return "member_ids_ssim" if defined?(Hyrax)
+
+      "#{file_model_class.name.to_s.underscore}_ids_ssim"
+    end
+
+    ##
+    # @param coercer [#call]
+    # @see Bulkrax::FactoryClassFinder
+    attr_writer :factory_class_name_coercer
+
+    ##
+    # A function responsible for converting the name of a factory class to the corresponding
+    # constant.
+    #
+    # @return [#call, Bulkrax::FactoryClassFinder::DefaultCoercer] an object responding to call,
+    #         with one positional parameter (e.g. arity == 1)
+    #
+    # @example
+    #   Bulkrax.factory_class_name_coercer.call("Work")
+    #   => Work
+    def factory_class_name_coercer
+      @factory_class_name_coercer || Bulkrax::FactoryClassFinder::DefaultCoercer
+    end
+
+    attr_writer :ingest_queue_name
+    ##
+    # @return [String, Proc]
+    def ingest_queue_name
+      return @ingest_queue_name if @ingest_queue_name.present?
+      return Hyrax.config.ingest_queue_name if defined?(Hyrax)
+      :import
+    end
+
+    ##
+    # Configure the persistence adapter used for persisting imported data.
+    #
+    # @return [Class<Bulkrax::PersistenceLayer::AbstractAdapter>]
+    # @see Bulkrax::PersistenceLayer
+    def persistence_adapter
+      @persistence_adapter || derived_persistence_adapter
+    end
+
+    def derived_persistence_adapter
+      if defined?(Hyrax)
+        # There's probably some configuration of Hyrax we could use to better refine this; but it's
+        # likely a reasonable guess.  The main goal is to not break existing implementations and
+        # maintain an upgrade path.
+        if Gem::Version.new(Hyrax::VERSION) >= Gem::Version.new('6.0.0')
+          Bulkrax::PersistenceLayer::ValkyrieAdapter
+        else
+          Bulkrax::PersistenceLayer::ActiveFedoraAdapter
+        end
+      elsif defined?(ActiveFedora)
+        Bulkrax::PersistenceLayer::ActiveFedoraAdapter
+      elsif defined?(Valkyrie)
+        Bulkrax::PersistenceLayer::ValkyrieAdapter
+      else
+        raise "Unable to derive a persistence adapter"
+      end
+    end
+
     attr_writer :use_locking
 
     def use_locking
@@ -63,6 +143,8 @@ module Bulkrax
                  :default_work_type=,
                  :export_path,
                  :export_path=,
+                 :factory_class_name_coercer,
+                 :factory_class_name_coercer=,
                  :field_mappings,
                  :field_mappings=,
                  :file_model_class,
@@ -81,6 +163,8 @@ module Bulkrax
                  :object_factory=,
                  :parsers,
                  :parsers=,
+                 :persistence_adapter,
+                 :persistence_adapter=,
                  :qa_controlled_properties,
                  :qa_controlled_properties=,
                  :related_children_field_mapping,
@@ -97,6 +181,8 @@ module Bulkrax
                  :reserved_properties=,
                  :server_name,
                  :server_name=,
+                 :solr_key_for_member_file_ids,
+                 :solr_key_for_member_file_ids=,
                  :use_locking,
                  :use_locking=,
                  :use_locking?

data/lib/tasks/bulkrax_tasks.rake

@@ -1,6 +1,18 @@
 # frozen_string_literal: true
 
 namespace :bulkrax do
+  desc 'Update all status messages from the latest status. This is to refresh the denormalized field'
+  task update_status_messages: :environment do
+    @progress = ProgressBar.create(total: Bulkrax::Status.latest_by_statusable.count,
+                                   format: "%a %b\u{15E7}%i %c/%C %p%% %t",
+                                   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)
+      @progress.increment
+    end
+  end
+
   # Usage example: rails bulkrax:generate_test_csvs['5','100','GenericWork']
   desc 'Generate CSVs with fake data for testing purposes'
   task :generate_test_csvs, [:num_of_csvs, :csv_rows, :record_type] => :environment do |_t, args|

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bulkrax
 version: !ruby/object:Gem::Version
-  version: 6.0.1
+  version: 7.0.0
 platform: ruby
 authors:
 - Rob Kaufman
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-12-11 00:00:00.000000000 Z
+date: 2024-03-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -52,6 +52,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: denormalize_fields
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: iso8601
   requirement: !ruby/object:Gem::Requirement
@@ -210,6 +224,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: dry-monads
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: sqlite3
   requirement: !ruby/object:Gem::Requirement
@@ -283,6 +311,7 @@ files:
 - app/assets/config/bulkrax_manifest.js
 - app/assets/javascripts/bulkrax/application.js
 - app/assets/javascripts/bulkrax/bulkrax.js
+- app/assets/javascripts/bulkrax/datatables.js
 - app/assets/javascripts/bulkrax/entries.js
 - app/assets/javascripts/bulkrax/exporters.js
 - app/assets/javascripts/bulkrax/importers.js.erb
@@ -296,6 +325,7 @@ files:
 - app/controllers/bulkrax/exporters_controller.rb
 - app/controllers/bulkrax/importers_controller.rb
 - app/controllers/concerns/bulkrax/api.rb
+- app/controllers/concerns/bulkrax/datatables_behavior.rb
 - app/controllers/concerns/bulkrax/download_behavior.rb
 - app/factories/bulkrax/object_factory.rb
 - app/helpers/bulkrax/application_helper.rb
@@ -304,6 +334,10 @@ files:
 - app/helpers/bulkrax/validation_helper.rb
 - app/jobs/bulkrax/application_job.rb
 - app/jobs/bulkrax/create_relationships_job.rb
+- app/jobs/bulkrax/delete_and_import_collection_job.rb
+- app/jobs/bulkrax/delete_and_import_file_set_job.rb
+- app/jobs/bulkrax/delete_and_import_job.rb
+- app/jobs/bulkrax/delete_and_import_work_job.rb
 - app/jobs/bulkrax/delete_collection_job.rb
 - app/jobs/bulkrax/delete_file_set_job.rb
 - app/jobs/bulkrax/delete_job.rb
@@ -313,6 +347,7 @@ files:
 - app/jobs/bulkrax/exporter_job.rb
 - app/jobs/bulkrax/import_collection_job.rb
 - app/jobs/bulkrax/import_file_set_job.rb
+- app/jobs/bulkrax/import_job.rb
 - app/jobs/bulkrax/import_work_job.rb
 - app/jobs/bulkrax/importer_job.rb
 - app/jobs/bulkrax/schedule_relationships_job.rb
@@ -358,6 +393,7 @@ files:
 - app/parsers/bulkrax/oai_qualified_dc_parser.rb
 - app/parsers/bulkrax/parser_export_record_set.rb
 - app/parsers/bulkrax/xml_parser.rb
+- app/services/bulkrax/factory_class_finder.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
@@ -372,6 +408,7 @@ files:
 - app/views/bulkrax/importers/_browse_everything.html.erb
 - 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/_form.html.erb
 - app/views/bulkrax/importers/_oai_fields.html.erb
 - app/views/bulkrax/importers/_xml_fields.html.erb
@@ -382,9 +419,7 @@ files:
 - app/views/bulkrax/importers/upload_corrected_entries.html.erb
 - app/views/bulkrax/shared/_bulkrax_errors.html.erb
 - app/views/bulkrax/shared/_bulkrax_field_mapping.html.erb
-- app/views/bulkrax/shared/_collection_entries_tab.html.erb
-- app/views/bulkrax/shared/_file_set_entries_tab.html.erb
-- app/views/bulkrax/shared/_work_entries_tab.html.erb
+- app/views/bulkrax/shared/_entries_tab.html.erb
 - app/views/hyrax/dashboard/sidebar/_bulkrax_sidebar_additions.html.erb
 - app/views/hyrax/dashboard/sidebar/_repository_content.html.erb
 - app/views/layouts/bulkrax/application.html.erb
@@ -425,9 +460,14 @@ files:
 - db/migrate/20220413180915_add_generated_metadata_to_bulkrax_exporters.rb
 - db/migrate/20220609001128_rename_bulkrax_importer_run_to_importer_run.rb
 - db/migrate/20230608153601_add_indices_to_bulkrax.rb
+- db/migrate/20240208005801_denormalize_status_message.rb
+- db/migrate/20240209070952_update_identifier_index.rb
 - lib/bulkrax.rb
 - lib/bulkrax/engine.rb
 - lib/bulkrax/entry_spec_helper.rb
+- lib/bulkrax/persistence_layer.rb
+- lib/bulkrax/persistence_layer/active_fedora_adapter.rb
+- lib/bulkrax/persistence_layer/valkyrie_adapter.rb
 - lib/bulkrax/version.rb
 - lib/generators/bulkrax/install_generator.rb
 - lib/generators/bulkrax/templates/README
@@ -457,7 +497,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.6
+rubygems_version: 3.4.10
 signing_key:
 specification_version: 4
 summary: Import and export tool for Hyrax and Hyku

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

@@ -1,39 +0,0 @@
-<div role="tabpanel" class="tab-pane bulkrax-nav-tab-table-left-align" id="collection-entries">
-  <table class='table table-striped'>
-    <thead>
-      <tr>
-        <th><%= t('bulkrax.table_header.labels.identifier') %></th>
-        <th><%= t('bulkrax.table_header.labels.entry_id') %></th>
-        <th><%= t('bulkrax.table_header.labels.status') %></th>
-        <th><%= t('bulkrax.table_header.labels.errors') %></th>
-        <th><%= t('bulkrax.table_header.labels.status_set_at') %></th>
-        <th><%= t('bulkrax.table_header.labels.actions') %></th>
-      </tr>
-    </thead>
-    <tbody>
-      <% 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, entry_path %></td>
-          <td><%= e.id %></td>
-          <% if e.status == "Complete" %>
-            <td><span class="glyphicon glyphicon-ok" style="color: green;"></span> <%= e.status %></td>
-          <% elsif e.status == "Pending" %>
-            <td><span class="glyphicon glyphicon-option-horizontal" style="color: blue;"></span> <%= e.status %></td>
-          <% else %>
-            <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"), entry_path %></td>
-          <% else %>
-            <td></td>
-          <% end %>
-          <td><%= e.status_at %></td>
-          <td><%= link_to raw('<span class="glyphicon glyphicon-info-sign"></span>'), entry_path %></td>
-        </tr>
-      <% end %>
-    </tbody>
-  </table>
-  <%= page_entries_info(entries) %><br />
-  <%= paginate(entries, theme: 'blacklight', param_name: :collections_entries_page, params: { anchor: 'collection-entries' }) %>
-</div>