hyrax 5.1.0 → 5.3.0

data/app/indexers/hyrax/indexers/file_set_indexer.rb

@@ -8,8 +8,11 @@ module Hyrax
       include Hyrax::PermissionIndexer
       include Hyrax::VisibilityIndexer
       include Hyrax::ThumbnailIndexer
-      include Hyrax::Indexer(:core_metadata)
-      include Hyrax::Indexer(:file_set_metadata)
+      if Hyrax.config.file_set_include_metadata
+        include Hyrax::Indexer(:core_metadata)
+        include Hyrax::Indexer(:file_set_metadata)
+      end
+      check_if_flexible(Hyrax::FileSet)
 
       def to_solr # rubocop:disable Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/MethodLength, Metrics/PerceivedComplexity
         super.tap do |solr_doc| # rubocop:disable Metrics/BlockLength
@@ -21,6 +24,7 @@ module Hyrax
           solr_doc['extracted_text_id_ssi']        = resource.extracted_text_id.to_s
           solr_doc['hasRelatedMediaFragment_ssim'] = resource.representative_id.to_s
           solr_doc['hasRelatedImage_ssim']         = resource.thumbnail_id.to_s
+          solr_doc['transcript_ids_ssim']          = resource.transcript_ids&.map(&:to_s) if resource.respond_to?(:transcript_ids)
 
           index_lease(solr_doc)
           index_embargo(solr_doc)

data/app/indexers/hyrax/indexers/pcdm_collection_indexer.rb

@@ -9,7 +9,9 @@ module Hyrax
       include Hyrax::VisibilityIndexer
       include Hyrax::LocationIndexer
       include Hyrax::ThumbnailIndexer
-      include Hyrax::Indexer(:core_metadata)
+      include Hyrax::Indexer(:core_metadata) if Hyrax.config.collection_include_metadata?
+      include Hyrax::Indexers::RedirectsIndexer if Hyrax.config.redirects_enabled?
+      check_if_flexible(Hyrax::PcdmCollection)
 
       self.thumbnail_path_service = CollectionThumbnailPathService
 
@@ -20,8 +22,16 @@ module Hyrax
           index_document[:member_of_collection_ids_ssim] = resource.member_of_collection_ids.map(&:to_s)
           index_document[:depositor_ssim] = [resource.depositor]
           index_document[:depositor_tesim] = [resource.depositor]
+          index_document['thumbnail_alt_text_tesim'] = thumbnail_alt_text(resource.id.to_s)
         end
       end
+
+      private
+
+      def thumbnail_alt_text(collection_id)
+        branding = CollectionBrandingInfo.where(collection_id: collection_id, role: "thumbnail").first
+        branding&.alt_text.presence
+      end
     end
   end
 end

data/app/indexers/hyrax/indexers/pcdm_object_indexer.rb

@@ -9,7 +9,8 @@ module Hyrax
       include Hyrax::VisibilityIndexer
       include Hyrax::LocationIndexer
       include Hyrax::ThumbnailIndexer
-      include Hyrax::Indexer(:core_metadata)
+      include Hyrax::WorkflowIndexer
+      include Hyrax::Indexers::RedirectsIndexer if Hyrax.config.redirects_enabled?
 
       def to_solr # rubocop:disable Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/MethodLength
         super.tap do |solr_doc|
@@ -20,13 +21,15 @@ module Hyrax
           solr_doc['admin_set_sim']   = admin_set_label
           solr_doc['admin_set_tesim'] = admin_set_label
           solr_doc["#{Hyrax.config.admin_set_predicate.qname.last}_ssim"] = [resource.admin_set_id.to_s]
-          solr_doc['member_of_collection_ids_ssim'] = resource.member_of_collection_ids.map(&:to_s)
-          solr_doc['member_ids_ssim'] = resource.member_ids.map(&:to_s)
+          solr_doc['member_of_collection_ids_ssim'] = resource.member_of_collection_ids&.map(&:to_s)
+          solr_doc['member_of_collections_ssim'] = collection_titles_for(resource.member_of_collection_ids)
+          solr_doc['member_ids_ssim'] = resource.member_ids&.map(&:to_s)
           solr_doc['depositor_ssim'] = [resource.depositor]
           solr_doc['depositor_tesim'] = [resource.depositor]
           solr_doc['hasRelatedMediaFragment_ssim'] = [resource.representative_id.to_s]
           solr_doc['hasRelatedImage_ssim'] = [resource.thumbnail_id.to_s]
-          solr_doc['hasFormat_ssim'] = resource.rendering_ids.map(&:to_s) if resource.rendering_ids.present?
+          solr_doc['thumbnail_alt_text_tesim'] = thumbnail_alt_text(resource.thumbnail_id)
+          solr_doc['hasFormat_ssim'] = resource.rendering_ids&.map(&:to_s) if resource.rendering_ids.present?
           index_embargo(solr_doc)
           index_lease(solr_doc)
         end
@@ -34,6 +37,14 @@ module Hyrax
 
       private
 
+      def thumbnail_alt_text(thumbnail_id)
+        return if thumbnail_id.blank?
+        file_set = Hyrax.query_service.find_by(id: thumbnail_id)
+        Array(file_set.try(:alt_text)).first.presence
+      rescue Valkyrie::Persistence::ObjectNotFoundError, Hyrax::ObjectNotFoundError
+        nil
+      end
+
       def suppressed?(resource)
         Hyrax::ResourceStatus.new(resource: resource).inactive?
       end
@@ -44,6 +55,16 @@ module Hyrax
         admin_set.title
       end
 
+      def collection_titles_for(collection_ids)
+        return [] if collection_ids.blank?
+        collection_ids.map do |id|
+          collection = Hyrax.query_service.find_by(id: id)
+          collection.title&.first
+        rescue Valkyrie::Persistence::ObjectNotFoundError
+          nil
+        end.compact
+      end
+
       def index_embargo(doc)
         if resource.embargo&.active?
           doc['embargo_release_date_dtsi'] = resource.embargo.embargo_release_date&.to_datetime

data/app/indexers/hyrax/indexers/redirects_indexer.rb

@@ -0,0 +1,29 @@
+# frozen_string_literal: true
+
+module Hyrax
+  module Indexers
+    ##
+    # Indexer mixin that emits the `redirects_path_tesim` Solr field for
+    # resources that carry a `redirects` attribute. The field is consumed
+    # by the show-page renderer via `SolrDocument#redirects_path`.
+    #
+    # @example
+    #   class WorkIndexer < Hyrax::Indexers::PcdmObjectIndexer
+    #     include Hyrax::Indexers::RedirectsIndexer
+    #   end
+    module RedirectsIndexer
+      def to_solr(*args)
+        super.tap do |document|
+          next document unless Hyrax.config.redirects_active?
+          next document unless resource.respond_to?(:redirects)
+          # Valkyrie's JSONValueMapper symbolizes hash keys on read; accept either.
+          # Paths are normalized at write time by Hyrax::RedirectsNormalization.
+          paths = Array(resource.redirects)
+                  .map { |entry| entry['path'] || entry[:path] }
+                  .reject(&:blank?)
+          document['redirects_path_tesim'] = paths
+        end
+      end
+    end
+  end
+end

data/app/indexers/hyrax/indexers/resource_indexer.rb

@@ -60,6 +60,10 @@ module Hyrax
       end
 
       class << self
+        def check_if_flexible(model)
+          include Hyrax::Indexer(model.to_s, index_loader: Hyrax::Schema.m3_schema_loader) if model.flexible?
+        end
+
         ##
         # @api public
         # @param resource [Valkyrie::Resource] an instance of a

data/app/indexers/hyrax/indexers.rb

@@ -11,6 +11,8 @@ module Hyrax
     def self.PcdmObjectIndexer(work_class) # rubocop:disable Naming/MethodName
       Class.new(Hyrax::Indexers::PcdmObjectIndexer) do
         @model_class = work_class
+        include Hyrax::Indexer(:core_metadata) if work_class.ancestors.detect { |k| k.inspect == "Hyrax::Schema(core_metadata)" }
+        check_if_flexible(work_class)
 
         class << self
           attr_reader :model_class

data/app/inputs/controlled_vocabulary_input.rb

@@ -44,7 +44,13 @@ class ControlledVocabularyInput < MultiValueInput
   def hidden_id_field(value, index)
     name = name_for(attribute_name, index, 'id')
     id = id_for(attribute_name, index, 'id')
-    hidden_value = value.try(:node?) ? '' : value.rdf_subject
+    hidden_value = if value.try(:node?) || value.is_a?(ActionController::Parameters)
+                     ''
+                   elsif value.respond_to?(:rdf_subject)
+                     value.rdf_subject
+                   else
+                     value.to_s
+                   end
     @builder.hidden_field(attribute_name, name: name, id: id, value: hidden_value, data: { id: 'remote' })
   end
 

data/app/jobs/concerns/hyrax/queued_job_behavior.rb

@@ -0,0 +1,22 @@
+# frozen_string_literal: true
+module Hyrax
+  # Grants the user's edit access on the provided FileSet
+  module QueuedJobBehavior
+    extend ActiveSupport::Concern
+
+    included do
+      queue_as Hyrax.config.ingest_queue_name
+      cattr_accessor :requeue_frequency
+    end
+
+    private
+
+    def redis_queue
+      Valkyrie::IndexingAdapter.find(:redis_queue)
+    end
+
+    def requeue(**args)
+      self.class.set(wait_until: (self.class.requeue_frequency || 5.minutes).from_now).perform_later(**args)
+    end
+  end
+end

data/app/jobs/content_deposit_error_event_job.rb

@@ -0,0 +1,14 @@
+# frozen_string_literal: true
+# Log a concern deposit to activity streams
+class ContentDepositErrorEventJob < ContentEventJob
+  attr_accessor :reason
+
+  def perform(repo_object, depositor, reason: '')
+    self.reason = reason
+    super(repo_object, depositor)
+  end
+
+  def action
+    "User #{link_to_profile depositor} deposit of #{link_to repo_object.title.first, polymorphic_path(repo_object)} has failed for #{reason}"
+  end
+end

data/app/jobs/create_work_job.rb

@@ -2,6 +2,7 @@
 # This is a job spawned by the BatchCreateJob
 class CreateWorkJob < Hyrax::ApplicationJob
   queue_as Hyrax.config.ingest_queue_name
+  discard_on StandardError
 
   before_enqueue do |job|
     operation = job.arguments.last
@@ -28,6 +29,7 @@ class CreateWorkJob < Hyrax::ApplicationJob
 
     return operation.success! if status
     operation.fail!(errors.full_messages.join(' '))
+    raise StandardError, operation.message
   end
 
   private

data/app/jobs/hyrax/propagate_change_depositor_job.rb

@@ -26,7 +26,7 @@ module Hyrax
     def apply_depositor_metadata(resource, depositor)
       depositor_id = depositor.respond_to?(:user_key) ? depositor.user_key : depositor
       resource.depositor = depositor_id if resource.respond_to? :depositor=
-      Hyrax::AccessControlList.new(resource: resource).grant(:edit).to(::User.find_by_user_key(depositor_id)).save
+      resource.permission_manager.acl.grant(:edit).to(::User.find_by_user_key(depositor_id)).save
     end
   end
 end

data/app/jobs/hyrax/queued_delete_job.rb

@@ -0,0 +1,11 @@
+# frozen_string_literal: true
+module Hyrax
+  class QueuedDeleteJob < ApplicationJob
+    include QueuedJobBehavior
+
+    def perform(size: 200)
+      redis_queue.delete_queue(size: size)
+      requeue(size: size)
+    end
+  end
+end

data/app/jobs/hyrax/queued_indexing_job.rb

@@ -0,0 +1,11 @@
+# frozen_string_literal: true
+module Hyrax
+  class QueuedIndexingJob < ApplicationJob
+    include QueuedJobBehavior
+
+    def perform(size: 200)
+      redis_queue.index_queue(size: size)
+      requeue(size: size)
+    end
+  end
+end

data/app/jobs/migrate_files_to_valkyrie_job.rb

@@ -2,6 +2,11 @@
 # Responsible for conditionally enqueuing the file and thumbnail migration
 # logic of an ActiveFedora object.
 class MigrateFilesToValkyrieJob < Hyrax::ApplicationJob
+  # Define a logger for this job
+  def logger
+    FileUtils.mkdir_p(Hyrax.config.working_path)
+    @logger ||= Logger.new(Hyrax.config.working_path.join('migrate_files_to_valkyrie_job.log'))
+  end
   ##
   #
   # @param resource [Hyrax::FileSet]
@@ -33,6 +38,7 @@ class MigrateFilesToValkyrieJob < Hyrax::ApplicationJob
     # @todo should we trigger a job if the member is a child work?
     paths = Hyrax::DerivativePath.derivatives_for_reference(resource)
     paths.each do |path|
+      next unless File.size?(path) # skip blank files
       container = container_for(path)
       mime_type = Marcel::MimeType.for(extension: File.extname(path))
       directives = { url: path, container: container, mime_type: mime_type }
@@ -50,27 +56,39 @@ class MigrateFilesToValkyrieJob < Hyrax::ApplicationJob
 
     files = Hyrax.custom_queries.find_many_file_metadata_by_ids(ids: resource.file_ids)
     files.each do |file|
-      # If it doesn't start with fedora, we've likely already migrated it.
-      next unless /^fedora:/.match?(file.file_identifier.to_s)
-      resource.file_ids.delete(file.id)
+      begin
+        # If it doesn't start with fedora, we've likely already migrated it.
+        next unless /^fedora:/.match?(file.file_identifier.to_s)
+
+        resource.file_ids.delete(file.id)
+
+        # We've observed cases where the extracted text (which doesn't get created in Valkyrie)
+        # does not have a filename and it overrides the original file.  The extracted text should
+        # be in the derivatives, not with the file set.
+        next if file.original_filename.blank?
 
-      Tempfile.create do |tempfile|
-        tempfile.binmode
-        tempfile.write(URI.open(file.file_identifier.to_s.gsub("fedora:", "http:")).read)
-        tempfile.rewind
+        Tempfile.create do |tempfile|
+          tempfile.binmode
+          tempfile.write(URI.open(file.file_identifier.to_s.gsub("fedora:", "http:")).read)
+          tempfile.rewind
 
-        # valkyrie_file = Hyrax.storage_adapter.upload(resource: resource, file: tempfile, original_filename: file.original_filename)
-        valkyrie_file = Hyrax::ValkyrieUpload.file(
-          filename: resource.label,
-          file_set: resource,
-          io: tempfile,
-          use: file.pcdm_use.select {|use| Hyrax::FileMetadata::Use.use_list.include?(use)},
-          user: User.find_or_initialize_by(User.user_key_field => resource.depositor),
-          mime_type: file.mime_type,
-          skip_derivatives: true
-        )
-        valkyrie_file = copy_attributes(valkyrie_file:, original_file: file)
-        Hyrax.persister.save(resource: valkyrie_file)
+          # valkyrie_file = Hyrax.storage_adapter.upload(resource: resource, file: tempfile, original_filename: file.original_filename)
+          valkyrie_file = Hyrax::ValkyrieUpload.file(
+            filename: file.original_filename,
+            file_set: resource,
+            io: tempfile,
+            use: file.pcdm_use.select {|use| Hyrax::FileMetadata::Use.use_list.include?(use)},
+            user: User.find_or_initialize_by(User.user_key_field => resource.depositor),
+            mime_type: file.mime_type,
+            skip_derivatives: true
+          )
+          valkyrie_file = copy_attributes(valkyrie_file:, original_file: file)
+          Hyrax.persister.save(resource: valkyrie_file)
+        end
+      rescue StandardError => e
+        # Log errors specific to file migration
+        logger.error("Error migrating file #{file.id} for resource #{resource.id}: #{e.message}")
+        logger.error(e.backtrace.join("\n"))
       end
     end
     # reindex the file set after migrating files to include characterization info
@@ -93,9 +111,9 @@ class MigrateFilesToValkyrieJob < Hyrax::ApplicationJob
   #
   # @param filename [String] the name of the derivative file: i.e. 'x-thumbnail.jpg'
   # @return [String]
-  def container_for(filename)
+  def container_for(path)
     # we want the portion between the '-' and the '.'
-    file_blob = File.basename(filename, '.*').split('-').last
+    file_blob = File.basename(path, '.*').split('-').last
 
     case file_blob
     when 'thumbnail'

data/app/jobs/migrate_sipity_entity_job.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+# migrates a resource's sipity entity so it can be found
+class MigrateSipityEntityJob < ApplicationJob
+  # input [String] id of a migrated resource
+  def perform(id:)
+    resource = Hyrax.query_service.find_by(id: id)
+    new_gid = Hyrax::GlobalID(resource).to_s
+    return if Sipity::Entity.find_by(proxy_for_global_id: new_gid)
+
+    work = resource.internal_resource.constantize.find(id)
+    original_gid = Hyrax::GlobalID(work).to_s
+    return if new_gid == original_gid
+    original_entity = Sipity::Entity.find_by(proxy_for_global_id: original_gid)
+    return if original_entity.nil?
+    original_entity.update(proxy_for_global_id: new_gid)
+  rescue ActiveFedora::ObjectNotFoundError
+    # this happens when the resource was never in Fedora so there is nothing to migrate.
+    # We don't want to retry the job so we don't raise an error.
+  end
+end

data/app/models/batch_upload_item.rb

@@ -18,4 +18,8 @@ class BatchUploadItem < ActiveFedora::Base
   def create_or_update
     raise "This is a read only record"
   end
+
+  def flexible?
+    false
+  end
 end

data/app/models/concerns/hyrax/ability/flexible_metadata_ability.rb

@@ -0,0 +1,10 @@
+# frozen_string_literal: true
+module Hyrax
+  module Ability
+    module FlexibleMetadataAbility
+      def flexible_metadata_abilities
+        can :manage, Hyrax::FlexibleSchema if admin? && Hyrax.config.flexible?
+      end
+    end
+  end
+end

data/app/models/concerns/hyrax/ability.rb

@@ -55,6 +55,7 @@ module Hyrax
       include Hyrax::Ability::PermissionTemplateAbility
       include Hyrax::Ability::ResourceAbility
       include Hyrax::Ability::SolrDocumentAbility
+      include Hyrax::Ability::FlexibleMetadataAbility
 
       class_attribute :admin_group_name, :registered_group_name, :public_group_name
       self.admin_group_name = Hyrax.config.admin_user_group_name
@@ -79,7 +80,8 @@ module Hyrax
                              :permission_template_abilities,
                              :resource_abilities,
                              :solr_document_abilities,
-                             :trophy_abilities]
+                             :trophy_abilities,
+                             :flexible_metadata_abilities]
     end
 
     # Samvera doesn't use download user/groups, so make it an alias to read
@@ -404,8 +406,10 @@ module Hyrax
                                     .select(:source_id)
                                     .distinct
                                     .pluck(:source_id)
-
-      Hyrax.custom_queries.find_ids_by_model(model: Hyrax::AdministrativeSet, ids: ids).any?
+      return false if ids.empty?
+      Hyrax::SolrQueryService.new.with_ids(ids: ids).query_result(rows: 1000)['response']['docs'].any? do |doc|
+        (Hyrax::ModelRegistry.admin_set_rdf_representations & doc['has_model_ssim']).present?
+      end
     end
 
     def registered_user?

data/app/models/concerns/hyrax/collection_behavior.rb

@@ -68,6 +68,10 @@ module Hyrax
       title.present? ? title.join(' | ') : 'No Title'
     end
 
+    def flexible?
+      false
+    end
+
     module ClassMethods
       # This governs which partial to draw when you render this type of object
       def _to_partial_path # :nodoc:

data/app/models/concerns/hyrax/file_set/transcripts.rb

@@ -0,0 +1,16 @@
+# frozen_string_literal: true
+module Hyrax
+  class FileSet
+    module Transcripts
+      extend ActiveSupport::Concern
+
+      included do
+        if Hyrax.config.file_set_include_metadata?
+          property :transcript_ids, predicate: ::RDF::URI.new('http://vocabulary.samvera.org/ns#transcriptIds'), multiple: true do |index|
+            index.as :stored_sortable
+          end
+        end
+      end
+    end
+  end
+end

data/app/models/concerns/hyrax/file_set_behavior.rb

@@ -14,6 +14,7 @@ module Hyrax
     include Hyrax::FileSet::Indexing
     include Hyrax::FileSet::BelongsToWorks
     include Hyrax::FileSet::Querying
+    include Hyrax::FileSet::Transcripts
     include HumanReadableType
     include CoreMetadata
     include Hyrax::BasicMetadata
@@ -38,5 +39,13 @@ module Hyrax
       Deprecation.warn "Method #to_presenter will be removed in Hyrax 5.0. Use Hyrax::FileSetsController#presenter.solr_document or `@presenter.solr_document` from a view instead."
       Blacklight::SearchService.new(config: CatalogController.blacklight_config).fetch(id).last
     end
+
+    def self.flexible?
+      false
+    end
+
+    def flexible?
+      false
+    end
   end
 end

data/app/models/concerns/hyrax/flexibility.rb

@@ -0,0 +1,107 @@
+# frozen_string_literal: true
+
+module Hyrax
+  module Flexibility
+    extend ActiveSupport::Concern
+    included do
+      attribute :schema_version,       Valkyrie::Types::String
+      attribute :contexts,          Valkyrie::Types::Set.of(Valkyrie::Types::String)
+    end
+
+    class_methods do
+      def try(term)
+        # This logic is necessary to handle method :try on a class
+        # otherwise load(attributes) tries to use the term as a schema to load and errors
+        return nil unless self.respond_to?(term)
+        self.send(term)
+      end
+
+      def flexible?
+        true
+      end
+
+      ## Override dry-struct 1.6.0 to enable redefining schemas on the fly
+      def attributes(new_schema)
+        keys = new_schema.keys.map { |k| k.to_s.chomp("?").to_sym }
+        schema_location = singleton_class? ? self.superclass : self
+        schema schema_location.schema.schema(new_schema)
+
+        define_accessors(keys)
+
+        @attribute_names = nil
+
+        direct_descendants = descendants&.select { |d| d.superclass == self }
+        direct_descendants&.each do |d|
+          inherited_attrs = new_schema.reject { |k, _| d.has_attribute?(k.to_s.chomp("?").to_sym) }
+          d.attributes(inherited_attrs)
+        end
+
+        new_schema.each_key do |key|
+          key = key.to_s.chomp('?')
+          next if instance_methods.include?("#{key}=".to_sym)
+
+          class_eval(<<-RUBY)
+          def #{key}=(value)
+            set_value("#{key}".to_sym, value)
+          end
+        RUBY
+        end
+
+        self
+      end
+
+      ## Override dry-struct 1.6.0 to filter attributes after schema reload happens
+      def new(attributes = default_attributes, safe = false, &block) # rubocop:disable Style/OptionalBooleanParameter
+        if attributes.is_a?(Struct)
+          if equal?(attributes.class)
+            attributes
+          else
+            # This implicit coercion is arguable but makes sense overall
+            # in cases there you pass child struct to the base struct constructor
+            # User.new(super_user)
+            #
+            # We may deprecate this behavior in future forcing people to be explicit
+            new(attributes.to_h, safe, &block)
+          end
+        else
+          load(attributes, safe)
+        end
+      rescue Dry::Types::CoercionError => e
+        raise RuntimeError, "[#{self}.new] #{e.class}: #{e.message}", e.backtrace
+      end
+
+      ## Read the schema from the database and load the correct schemas for the instance in to the class
+      def load(attributes, safe = false)
+        attributes[:schema_version] ||=  Hyrax::FlexibleSchema.order('id DESC').pick(:id)
+        struct = allocate
+        schema_version = attributes[:schema_version]
+        contexts = attributes[:contexts] || []
+        struct.singleton_class.attributes(Hyrax::Schema(self, schema_loader: Hyrax::Schema.m3_schema_loader, schema_version:, contexts:).attributes)
+        clean_attributes = safe ? struct.singleton_class.schema.call_safe(attributes) { |output = attributes| return yield output } : struct.singleton_class.schema.call_unsafe(attributes)
+        struct.__send__(:initialize, clean_attributes)
+        struct
+      end
+    end
+
+    def contexts=(value)
+      val = Array.wrap(value).map { |v| v.split }.flatten
+      @attributes[:contexts] = val
+    end
+
+    # Override set_value from valkyrie 3.1.1 to enable dynamic schema loading
+    def set_value(key, value)
+      @attributes[key.to_sym] = self.singleton_class.schema.key(key.to_sym).type.call(value)
+    end
+
+    def flexible?
+      self.class.flexible?
+    end
+
+    # Override inspect from dry-struct 1.6.0 to enable dynamic schema loading
+    def inspect
+      klass = self.singleton_class
+      attrs = klass.attribute_names.map { |key| " #{key}=#{@attributes[key].inspect}" }.join
+      "#<#{klass.name || klass.inspect}#{attrs}>"
+    end
+  end
+end

data/app/models/concerns/hyrax/redirects_normalization.rb

@@ -0,0 +1,38 @@
+# frozen_string_literal: true
+
+module Hyrax
+  # Normalizes redirect path entries on assignment so every write path —
+  # form submissions, console writes, importers, change-set saves —
+  # produces canonical data. Read-side consumers can trust the persisted
+  # shape and skip their own normalization.
+  #
+  # Included next to Hyrax::Schema(:redirects) on Hyrax::Work and
+  # Hyrax::PcdmCollection. The override sits on Valkyrie's set_value
+  # primitive so it fires under both flex modes (the simple schema's
+  # generated setter and the m3 singleton-class setter both route through
+  # set_value).
+  #
+  # See documentation/redirects.md.
+  module RedirectsNormalization
+    extend ActiveSupport::Concern
+
+    def set_value(key, value)
+      value = normalize_redirects(value) if key.to_sym == :redirects
+      super
+    end
+
+    private
+
+    def normalize_redirects(value)
+      Array(value).map { |entry| normalize_entry(entry) }
+    end
+
+    def normalize_entry(entry)
+      return entry unless entry.is_a?(Hash)
+      normalized = entry.dup
+      path_key = normalized.key?(:path) ? :path : 'path'
+      normalized[path_key] = Hyrax::RedirectPathNormalizer.call(normalized[path_key])
+      normalized
+    end
+  end
+end

data/app/models/concerns/hyrax/solr_document/characterization.rb

@@ -133,7 +133,7 @@ module Hyrax
       end
 
       def original_checksum
-        self["original_checksum_tesim"]
+        self["original_checksum_tesim"] || self["checksum_tesim"]
       end
 
       def alpha_channels

data/app/models/concerns/hyrax/solr_document/metadata.rb

@@ -40,11 +40,20 @@ module Hyrax
         end
       end
 
+      def alt_text_for_view
+        return alt_text.first.presence || title.first if file_set?
+
+        thumbnail_alt_text.first || "#{title.first} #{I18n.t('hyrax.homepage.admin_sets.thumbnail')}"
+      end
+
       included do
         attribute :alternative_title, Solr::Array, "alternative_title_tesim"
+        attribute :alt_text, Solr::Array, "alt_text_tesim"
+        attribute :thumbnail_alt_text, Solr::Array, "thumbnail_alt_text_tesim"
         attribute :identifier, Solr::Array, "identifier_tesim"
         attribute :based_near, Solr::Array, "based_near_tesim"
         attribute :based_near_label, Solr::Array, "based_near_label_tesim"
+        attribute :redirects_path, Solr::Array, "redirects_path_tesim"
         attribute :related_url, Solr::Array, "related_url_tesim"
         attribute :resource_type, Solr::Array, "resource_type_tesim"
         attribute :edit_groups, Solr::Array, ::Ability.edit_group_field