iiif_print 2.0.0 → 3.0.0

data/app/jobs/iiif_print/jobs/child_works_from_pdf_job.rb

@@ -87,7 +87,7 @@ module IiifPrint
         )
         BatchCreateJob.perform_later(user,
                                      @child_work_titles,
-                                     {},
+                                     @resource_types,
                                      @uploaded_files,
                                      attributes.merge!(model: child_model.to_s, split_from_pdf_id: @split_from_pdf_id).with_indifferent_access,
                                      operation)
@@ -99,6 +99,7 @@ module IiifPrint
       def prepare_import_data(original_pdf_path, image_files, user)
         @uploaded_files = []
         @child_work_titles = {}
+        @resource_types = {}
         number_of_pages_in_pdf = image_files.size
         image_files.each_with_index do |image_path, page_number|
           file_id = create_uploaded_file(user, image_path).to_s
@@ -111,6 +112,7 @@ module IiifPrint
             page_padding: number_of_digits(nbr: number_of_pages_in_pdf)
           )
 
+          @resource_types[file_id] = resource_types
           @uploaded_files << file_id
           @child_work_titles[file_id] = child_title
           # save child work info to create the member relationships
@@ -144,10 +146,14 @@ module IiifPrint
         uf.id
       end
 
-      # TODO: what attributes do we need to fill in from the parent work? What about AllinsonFlex?
       def attributes
         IiifPrint.config.child_work_attributes_function.call(parent_work: @parent_work, admin_set_id: @child_admin_set_id)
       end
+
+      # TODO: Does this method need to be configurable?
+      def resource_types
+        @parent_work.try(:resource_type)
+      end
     end
   end
 end

data/app/listeners/iiif_print/listener.rb

@@ -15,7 +15,7 @@ module IiifPrint
       file_set = event[:file_set]
       return false unless file_set
       return false unless file_set.file_set?
-      return false unless file_set.original_file.pdf?
+      return false unless file_set.original_file&.pdf?
 
       work = IiifPrint.parent_for(file_set)
       # A short-circuit to avoid fetching the underlying file.

data/app/models/concerns/iiif_print/solr_document_decorator.rb

@@ -0,0 +1,47 @@
+# frozen_string_literal: true
+
+module IiifPrint
+  module SolrDocumentDecorator
+    def digest_sha1
+      digest[/urn:sha1:([\w]+)/, 1]
+    end
+
+    def method_missing(method_name, *args, &block)
+      super unless iiif_print_solr_field_names.include? method_name.to_s
+      self[IiifPrint.solr_name(method_name.to_s)]
+    end
+
+    def respond_to_missing?(method_name, include_private = false)
+      iiif_print_solr_field_names.include?(method_name.to_s) || super
+    end
+
+    # @see https://github.com/samvera/hyrax/commit/7108409c619cd2ba4ae8c836b9f3b429a7e9837b
+    def file_set_ids
+      # Yes, this looks a little odd.  But the truth is the prior key (e.g. `file_set_ids_ssim`) was
+      # an alias of `member_ids_ssim`.
+      self['member_ids_ssim']
+    end
+
+    def any_highlighting?
+      response&.[]('highlighting')&.[](id)&.present?
+    end
+
+    def solr_document
+      self
+    end
+  end
+end
+
+SolrDocument.prepend(IiifPrint::SolrDocumentDecorator)
+SolrDocument.attribute :is_child, Hyrax::SolrDocument::Metadata::Solr::String, 'is_child_bsi'
+SolrDocument.attribute :split_from_pdf_id, Hyrax::SolrDocument::Metadata::Solr::String, 'split_from_pdf_id_ssi'
+SolrDocument.attribute :digest, Hyrax::SolrDocument::Metadata::Solr::String, 'digest_ssim'
+
+# @note These properties came from the newspaper_works gem.  They are configurable.
+SolrDocument.class_attribute :iiif_print_solr_field_names, default: %w[alternative_title genre
+                                                                       issn lccn oclcnum held_by text_direction
+                                                                       page_number section author photographer
+                                                                       volume issue_number geographic_coverage
+                                                                       extent publication_date height width
+                                                                       edition_number edition_name frequency preceded_by
+                                                                       succeeded_by]

data/app/models/iiif_print/iiif_search_decorator.rb

@@ -33,3 +33,4 @@ module IiifPrint
     end
   end
 end
+::BlacklightIiifSearch::IiifSearch.prepend(IiifPrint::IiifSearchDecorator)

data/app/models/iiif_print/iiif_search_response_decorator.rb

@@ -38,3 +38,4 @@ module IiifPrint
     end
   end
 end
+::BlacklightIiifSearch::IiifSearchResponse.prepend(IiifPrint::IiifSearchResponseDecorator)

data/app/presenters/iiif_print/file_set_presenter_decorator.rb

@@ -9,3 +9,4 @@ module IiifPrint
     end
   end
 end
+Hyrax::FileSetPresenter.prepend(IiifPrint::FileSetPresenterDecorator)

data/app/presenters/iiif_print/{iiif_manifest_presenter_behavior.rb → iiif_manifest_presenter/display_image_presenter_decorator.rb}

@@ -1,38 +1,9 @@
 # mixin to provide URL for IIIF Content Search service
 module IiifPrint
-  module IiifManifestPresenterBehavior
-    extend ActiveSupport::Concern
-
-    # Extending the presenter to the base url which includes the protocol.
-    # We need the base url to render the facet links and normalize the interface.
-    attr_accessor :base_url
-
-    def manifest_metadata
-      # ensure we are using a SolrDocument
-      @manifest_metadata ||= IiifPrint.manifest_metadata_from(work: model.solr_document, presenter: self)
-    end
-
-    def search_service
-      Rails.application.routes.url_helpers.solr_document_iiif_search_url(id, host: hostname)
-    end
-
-    # OVERRIDE: Hyrax 3x, avoid nil returning to IIIF Manifest gem
-    # @see https://github.com/samvera/iiif_manifest/blob/c408f90eba11bef908796c7236ba6bcf8d687acc/lib/iiif_manifest/v3/manifest_builder/record_property_builder.rb#L28
-    ##
-    # @return [Array<Hash{String => String}>]
-    def sequence_rendering
-      Array(try(:rendering_ids)).map do |file_set_id|
-        rendering = file_set_presenters.find { |p| p.id == file_set_id }
-        return [] unless rendering
-
-        { '@id' => Hyrax::Engine.routes.url_helpers.download_url(rendering.id, host: hostname),
-          'format' => rendering.mime_type.presence || I18n.t("hyrax.manifest.unknown_mime_text"),
-          'label' => I18n.t("hyrax.manifest.download_text") + (rendering.label || '') }
-      end.flatten
-    end
-
+  module IiifManifestPresenter
     # OVERRIDE: Hyrax v3.x
-    module DisplayImagePresenterBehavior
+    module DisplayImagePresenterDecorator
+      extend ActiveSupport::Concern
       # Extending the presenter to the base url which includes the protocol.
       # We need the base url to render the facet links and normalize the interface.
       attr_accessor :base_url
@@ -128,3 +99,4 @@ module IiifPrint
     end
   end
 end
+Hyrax::IiifManifestPresenter::DisplayImagePresenter.prepend(IiifPrint::IiifManifestPresenter::DisplayImagePresenterDecorator)

data/app/presenters/iiif_print/iiif_manifest_presenter_decorator.rb

@@ -0,0 +1,35 @@
+# mixin to provide URL for IIIF Content Search service
+module IiifPrint
+  module IiifManifestPresenterDecorator
+    extend ActiveSupport::Concern
+
+    # Extending the presenter to the base url which includes the protocol.
+    # We need the base url to render the facet links and normalize the interface.
+    attr_accessor :base_url
+
+    def manifest_metadata
+      # ensure we are using a SolrDocument
+      @manifest_metadata ||= IiifPrint.manifest_metadata_from(work: model.solr_document, presenter: self)
+    end
+
+    def search_service
+      Rails.application.routes.url_helpers.solr_document_iiif_search_url(id, host: hostname)
+    end
+
+    # OVERRIDE: Hyrax 3x, avoid nil returning to IIIF Manifest gem
+    # @see https://github.com/samvera/iiif_manifest/blob/c408f90eba11bef908796c7236ba6bcf8d687acc/lib/iiif_manifest/v3/manifest_builder/record_property_builder.rb#L28
+    ##
+    # @return [Array<Hash{String => String}>]
+    def sequence_rendering
+      Array(try(:rendering_ids)).map do |file_set_id|
+        rendering = file_set_presenters.find { |p| p.id == file_set_id }
+        return [] unless rendering
+
+        { '@id' => Hyrax::Engine.routes.url_helpers.download_url(rendering.id, host: hostname),
+          'format' => rendering.mime_type.presence || I18n.t("hyrax.manifest.unknown_mime_text"),
+          'label' => I18n.t("hyrax.manifest.download_text") + (rendering.label || '') }
+      end.flatten
+    end
+  end
+end
+Hyrax::IiifManifestPresenter.prepend(IiifPrint::IiifManifestPresenterDecorator)

data/app/presenters/iiif_print/{iiif_manifest_presenter_factory_behavior.rb → iiif_manifest_presenter_factory_decorator.rb}

@@ -1,5 +1,5 @@
 module IiifPrint
-  module IiifManifestPresenterFactoryBehavior
+  module IiifManifestPresenterFactoryDecorator
     # This will override Hyrax::IiifManifestPresenter::Factory#build and introducing
     # the expected behavior:
     # - child work images show as canvases in the parent work manifest
@@ -31,3 +31,4 @@ module IiifPrint
     end
   end
 end
+Hyrax::IiifManifestPresenter::Factory.prepend(IiifPrint::IiifManifestPresenterFactoryDecorator)

data/app/renderers/hyrax/renderers/faceted_attribute_renderer_decorator.rb

@@ -16,3 +16,4 @@ module Hyrax
     end
   end
 end
+Hyrax::Renderers::FacetedAttributeRenderer.prepend(Hyrax::Renderers::FacetedAttributeRendererDecorator)

data/app/services/iiif_print/derivative_rodeo_service.rb

@@ -366,7 +366,7 @@ module IiifPrint
                    else
                      # TODO: This is the fedora URL representing the file we uploaded; is that adequate?  Will we
                      # have access to this file?
-                     file_set.original_file.uri.to_s
+                     file_set.original_file&.uri&.to_s
                    end
     end
 

data/app/services/iiif_print/{manifest_builder_service_behavior.rb → manifest_builder_service_decorator.rb}

@@ -1,6 +1,6 @@
 module IiifPrint
   # rubocop:disable Metrics/ModuleLength
-  module ManifestBuilderServiceBehavior
+  module ManifestBuilderServiceDecorator
     def initialize(*args,
                    version: IiifPrint.config.default_iiif_manifest_version,
                    iiif_manifest_factory: iiif_manifest_factory_for(version),
@@ -154,3 +154,4 @@ module IiifPrint
   end
   # rubocop:enable Metrics/ClassLength
 end
+Hyrax::ManifestBuilderService.prepend(IiifPrint::ManifestBuilderServiceDecorator)

data/app/services/iiif_print/simple_schema_loader_decorator.rb

@@ -9,3 +9,4 @@ module IiifPrint
     end
   end
 end
+Hyrax::SimpleSchemaLoader.prepend(IiifPrint::SimpleSchemaLoaderDecorator)

data/app/transactions/hyrax/transactions/steps/delete_all_file_sets_decorator.rb

@@ -33,3 +33,5 @@ module Hyrax
     end
   end
 end
+
+"Hyrax::Transactions::Steps::DeleteAllFileSets".safe_constantize&.prepend(Hyrax::Transactions::Steps::DeleteAllFileSetsDecorator)

data/config/metadata/child_works_from_pdf_splitting.yaml

@@ -5,6 +5,7 @@ attributes:
     index_keys:
       - "is_child_bsi"
     form:
+      display: false
       required: false
       primary: false
       multiple: false
@@ -15,6 +16,7 @@ attributes:
     index_keys:
       - "split_from_pdf_id_ssi"
     form:
+      display: false
       required: false
       primary: false
       multiple: false

data/lib/iiif_print/base_derivative_service.rb

@@ -33,7 +33,7 @@ module IiifPrint
       # @TODO: verify if this works for ActiveFedora and if so, remove commented code.
       #        If not, modify to use adapter.
       # file_set.class.image_mime_types.include?(file_set.mime_type)
-      file_set.original_file.image?
+      file_set.original_file&.image?
     end
 
     def derivative_path_factory

data/lib/iiif_print/blacklight_iiif_search/annotation_decorator.rb

@@ -134,3 +134,4 @@ module IiifPrint
     end
   end
 end
+::BlacklightIiifSearch::IiifSearchAnnotation.prepend(IiifPrint::BlacklightIiifSearch::AnnotationDecorator)

data/lib/iiif_print/configuration.rb

@@ -212,7 +212,7 @@ module IiifPrint
             visibility: 'lease',
             visibility_after_lease: lease.visibility_after_lease,
             visibility_during_lease: lease.visibility_during_lease,
-            lease_release_date: lease.lease_release_date
+            lease_release_date: lease.lease_expiration_date
           }
         else
           visibility_params = { visibility: parent_work.visibility.to_s }

data/lib/iiif_print/data/fileset_helper.rb

@@ -13,7 +13,7 @@ module IiifPrint
       def first_fileset
         # if context is fileset id (e.g. caller is view partial) string,
         #   get the fileset from that id
-        return FileSet.find(@work) if @work.is_a?(String)
+        return Hyrax.query_service.find_by(id: @work) if @work.is_a?(String)
         # if "work" context is a FileSet, not actual work, return it
         return @work if @work.is_a?(Hyrax::FileSet) || @work.is_a?(FileSet)
         # in most cases, get from work's members:

data/lib/iiif_print/engine.rb

@@ -48,55 +48,14 @@ module IiifPrint
 
       Hyrax.publisher.subscribe(IiifPrint::Listener.new) if Hyrax.respond_to?(:publisher)
 
-      Hyrax::IiifManifestPresenter.prepend(IiifPrint::IiifManifestPresenterBehavior)
-      Hyrax::IiifManifestPresenter::Factory.prepend(IiifPrint::IiifManifestPresenterFactoryBehavior)
-      Hyrax::ManifestBuilderService.prepend(IiifPrint::ManifestBuilderServiceBehavior)
-      Hyrax::Renderers::FacetedAttributeRenderer.prepend(Hyrax::Renderers::FacetedAttributeRendererDecorator)
-      Hyrax::WorksControllerBehavior.prepend(IiifPrint::WorksControllerBehaviorDecorator)
-      "Hyrax::Transactions::Steps::DeleteAllFileSets".safe_constantize&.prepend(Hyrax::Transactions::Steps::DeleteAllFileSetsDecorator)
-      # Hyku::WorksControllerBehavior was introduced in Hyku v6.0.0+.  Yes we don't depend on Hyku,
-      # but this allows us to do minimal Hyku antics with IiifPrint.
-      'Hyku::WorksControllerBehavior'.safe_constantize&.prepend(IiifPrint::WorksControllerBehaviorDecorator)
-
-      Hyrax::FileSetPresenter.prepend(IiifPrint::FileSetPresenterDecorator)
-      Hyrax::WorkShowPresenter.prepend(IiifPrint::WorkShowPresenterDecorator)
-      Hyrax::IiifHelper.prepend(IiifPrint::IiifHelperDecorator)
-
-      if ActiveModel::Type::Boolean.new.cast(ENV.fetch('HYRAX_VALKYRIE', false))
-        # Newer versions of Hyrax favor `Hyrax::Indexers::FileSetIndexer` and deprecate
-        # `Hyrax::ValkyrieFileSetIndexer`.
-        'Hyrax::Indexers::FileSetIndexer'.safe_constantize&.prepend(IiifPrint::FileSetIndexer)
-
-        # Versions 3.0+ of Hyrax have `Hyrax::ValkyrieFileSetIndexer` so we want to decorate that as
-        # well.  We want to use the elsif construct because later on Hyrax::ValkyrieFileSetIndexer
-        # inherits from Hyrax::Indexers::FileSetIndexer and only implements:
-        # `def initialize(*args); super; end`
-        'Hyrax::ValkyrieFileSetIndexer'.safe_constantize&.prepend(IiifPrint::FileSetIndexer)
-
-        # Newer versions of Hyrax favor `Hyrax::Indexers::PcdmObjectIndexer` and deprecate
-        # `Hyrax::ValkyrieWorkIndexer`
-        indexers = Hyrax.config.curation_concerns.map do |concern|
-          "#{concern}ResourceIndexer".safe_constantize
-        end
-        indexers.each { |indexer| indexer.prepend(IiifPrint::ChildWorkIndexer) }
-
-        # Versions 3.0+ of Hyrax have `Hyrax::ValkyrieWorkIndexer` so we want to decorate that as
-        # well.  We want to use the elsif construct because later on Hyrax::ValkyrieWorkIndexer
-        # inherits from Hyrax::Indexers::PcdmObjectIndexer and only implements:
-        # `def initialize(*args); super; end`
-        'Hyrax::ValkyrieWorkIndexer'.safe_constantize&.prepend(IiifPrint::ChildWorkIndexer)
-      else
-        # The ActiveFedora::Base indexer for FileSets
-        Hyrax::FileSetIndexer.prepend(IiifPrint::FileSetIndexer)
-        # The ActiveFedora::Base indexer for Works
-        Hyrax::WorkIndexer.prepend(IiifPrint::ChildWorkIndexer)
+      # Allows us to use decorator files
+      Dir.glob(File.join(File.dirname(__FILE__), "../../app/**/*_decorator*.rb")).sort.each do |c|
+        Rails.configuration.cache_classes ? require(c) : load(c)
       end
 
-      ::BlacklightIiifSearch::IiifSearchResponse.prepend(IiifPrint::IiifSearchResponseDecorator)
-      ::BlacklightIiifSearch::IiifSearchAnnotation.prepend(IiifPrint::BlacklightIiifSearch::AnnotationDecorator)
-      ::BlacklightIiifSearch::IiifSearch.prepend(IiifPrint::IiifSearchDecorator)
-      Hyrax::Actors::FileSetActor.prepend(IiifPrint::Actors::FileSetActorDecorator)
-      Hyrax::Actors::CleanupFileSetsActor.prepend(IiifPrint::Actors::CleanupFileSetsActorDecorator)
+      Dir.glob(File.join(File.dirname(__FILE__), "../../lib/**/*_decorator*.rb")).sort.each do |c|
+        Rails.configuration.cache_classes ? require(c) : load(c)
+      end
 
       Hyrax.config do |config|
         config.callback.set(:after_create_fileset) do |file_set, user|
@@ -104,12 +63,6 @@ module IiifPrint
         end
       end
     end
-
-    config.after_initialize do
-      IiifPrint::Solr::Document.decorate(SolrDocument)
-      Hyrax::IiifManifestPresenter::DisplayImagePresenter
-        .prepend(IiifPrint::IiifManifestPresenterBehavior::DisplayImagePresenterBehavior)
-    end
     # rubocop:enable Metrics/BlockLength
   end
 end

data/lib/iiif_print/persistence_layer/valkyrie_adapter.rb

@@ -91,7 +91,7 @@ module IiifPrint
       end
 
       def self.pdf?(file_set)
-        file_set.original_file.pdf?
+        file_set.original_file&.pdf?
       end
 
       ##

data/lib/iiif_print/text_extraction_derivative_service.rb

@@ -28,7 +28,10 @@ module IiifPrint
 
       ocr_derivatives.each do |extension, method_name|
         path = prepare_path(extension.to_s)
-        write(content: ocr.public_send(method_name), path: path, extension: extension)
+        content = ocr.public_send(method_name)
+        next if content.blank?
+
+        write(content: content, path: path, extension: extension)
       end
     end
 

data/lib/iiif_print/version.rb

@@ -1,3 +1,3 @@
 module IiifPrint
-  VERSION = '2.0.0'.freeze
+  VERSION = '3.0.0'.freeze
 end

data/lib/iiif_print/works_controller_behavior_decorator.rb

@@ -0,0 +1,13 @@
+module IiifPrint
+  module WorksControllerBehaviorDecorator
+    # Extending the presenter to the base url which includes the protocol.
+    # We need the base url to render the facet links.
+    def iiif_manifest_presenter
+      super.tap { |i| i.base_url = request.base_url }
+    end
+  end
+end
+Hyrax::WorksControllerBehavior.prepend(IiifPrint::WorksControllerBehaviorDecorator)
+# Hyku::WorksControllerBehavior was introduced in Hyku v6.0.0+.  Yes we don't depend on Hyku,
+# but this allows us to do minimal Hyku antics with IiifPrint.
+'Hyku::WorksControllerBehavior'.safe_constantize&.prepend(IiifPrint::WorksControllerBehaviorDecorator)

data/lib/iiif_print.rb

@@ -13,8 +13,6 @@ require "iiif_print/text_formats_from_alto_service"
 require "iiif_print/tiff_derivative_service"
 require "iiif_print/lineage_service"
 require "iiif_print/metadata"
-require "iiif_print/works_controller_behavior"
-require "iiif_print/blacklight_iiif_search/annotation_decorator"
 require "iiif_print/split_pdfs/base_splitter"
 require "iiif_print/split_pdfs/child_work_creation_from_pdf_service"
 require "iiif_print/split_pdfs/derivative_rodeo_splitter"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: iiif_print
 version: !ruby/object:Gem::Version
-  version: 2.0.0
+  version: 3.0.0
 platform: ruby
 authors:
 - Sean Upton
@@ -14,7 +14,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-05-30 00:00:00.000000000 Z
+date: 2024-07-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: blacklight_iiif_search
@@ -363,8 +363,8 @@ files:
 - app/helpers/iiif_print/iiif_helper_decorator.rb
 - app/helpers/iiif_print/iiif_print_helper_behavior.rb
 - app/helpers/iiif_print_helper.rb
-- app/indexers/concerns/iiif_print/child_work_indexer.rb
-- app/indexers/concerns/iiif_print/file_set_indexer.rb
+- app/indexers/concerns/iiif_print/child_work_indexer_decorator.rb
+- app/indexers/concerns/iiif_print/file_set_indexer_decorator.rb
 - app/jobs/iiif_print/jobs/application_job.rb
 - app/jobs/iiif_print/jobs/child_works_from_pdf_job.rb
 - app/jobs/iiif_print/jobs/create_relationships_job.rb
@@ -372,7 +372,7 @@ files:
 - app/listeners/iiif_print/listener.rb
 - app/mailers/iiif_print/application_mailer.rb
 - app/models/concerns/iiif_print/set_child_flag.rb
-- app/models/concerns/iiif_print/solr/document.rb
+- app/models/concerns/iiif_print/solr_document_decorator.rb
 - app/models/iiif_print/application_record.rb
 - app/models/iiif_print/derivative_attachment.rb
 - app/models/iiif_print/iiif_search_decorator.rb
@@ -380,15 +380,16 @@ files:
 - app/models/iiif_print/ingest_file_relation.rb
 - app/models/iiif_print/pending_relationship.rb
 - app/presenters/iiif_print/file_set_presenter_decorator.rb
-- app/presenters/iiif_print/iiif_manifest_presenter_behavior.rb
-- app/presenters/iiif_print/iiif_manifest_presenter_factory_behavior.rb
+- app/presenters/iiif_print/iiif_manifest_presenter/display_image_presenter_decorator.rb
+- app/presenters/iiif_print/iiif_manifest_presenter_decorator.rb
+- app/presenters/iiif_print/iiif_manifest_presenter_factory_decorator.rb
 - app/presenters/iiif_print/work_show_presenter_decorator.rb
 - app/renderers/hyrax/renderers/faceted_attribute_renderer_decorator.rb
 - app/search_builders/concerns/iiif_print/allinson_flex_fields.rb
 - app/search_builders/concerns/iiif_print/exclude_models.rb
 - app/search_builders/concerns/iiif_print/highlight_search_params.rb
 - app/services/iiif_print/derivative_rodeo_service.rb
-- app/services/iiif_print/manifest_builder_service_behavior.rb
+- app/services/iiif_print/manifest_builder_service_decorator.rb
 - app/services/iiif_print/pluggable_derivative_service.rb
 - app/services/iiif_print/simple_schema_loader_decorator.rb
 - app/transactions/hyrax/transactions/iiif_print_container_decorator.rb
@@ -401,7 +402,6 @@ files:
 - app/views/hyrax/file_sets/_show_actions.html.erb
 - config/fcrepo_wrapper_test.yml
 - config/initializers/assets.rb
-- config/initializers/simple_schema_loader.rb
 - config/locales/iiif_print.de.yml
 - config/locales/iiif_print.en.yml
 - config/locales/iiif_print.es.yml
@@ -483,7 +483,7 @@ files:
 - lib/iiif_print/text_formats_from_alto_service.rb
 - lib/iiif_print/tiff_derivative_service.rb
 - lib/iiif_print/version.rb
-- lib/iiif_print/works_controller_behavior.rb
+- lib/iiif_print/works_controller_behavior_decorator.rb
 - lib/samvera/derivatives.rb
 - lib/samvera/derivatives/configuration.rb
 - lib/samvera/derivatives/hyrax.rb
@@ -510,7 +510,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.3
+rubygems_version: 3.1.6
 signing_key:
 specification_version: 4
 summary: IiifPrint is a gem (Rails "engine") for Hyrax-based digital repository applications

data/app/indexers/concerns/iiif_print/child_work_indexer.rb

@@ -1,27 +0,0 @@
-# frozen_string_literal: true
-
-module IiifPrint
-  module ChildWorkIndexer
-    def to_solr
-      super.tap do |index_document|
-        index_solr_doc(index_document)
-      end
-    end
-
-    def generate_solr_document
-      super.tap do |solr_doc|
-        index_solr_doc(solr_doc)
-      end
-    end
-
-    private
-
-    def index_solr_doc(solr_doc)
-      object ||= @object || resource
-      solr_doc['is_child_bsi'] ||= object.try(:is_child)
-      solr_doc['split_from_pdf_id_ssi'] ||= object.try(:split_from_pdf_id)
-      solr_doc['is_page_of_ssim'] = iiif_print_lineage_service.ancestor_ids_for(object)
-      solr_doc['member_ids_ssim'] = iiif_print_lineage_service.descendent_member_ids_for(object)
-    end
-  end
-end

data/app/models/concerns/iiif_print/solr/document.rb

@@ -1,63 +0,0 @@
-module IiifPrint::Solr::Document
-  # @note Why decorate?  We want to avoid including this module via generator.  And the generator
-  #       previously did two things: 1) include `IiifPrint::Solr::Document` in `SolrDocument`; 2)
-  #       add the `attribute :is_child` field to the SolrDocument.  We can't rely on `included do`
-  #       block to handle that.
-  #
-  # This method is responsible for configuring the SolrDocument for a Hyrax/Hyku application.  It
-  # does three things:
-  #
-  # 1. Adds instance methods to the SolrDocument (see implementation below)
-  # 2. Adds the `is_child` attribute to the SolrDocument
-  # 3. Adds a class attribute (e.g. `iiif_print_solr_field_names`) to allow further customization.
-  #
-  # @note These `iiif_print_solr_field_names` came from the newspaper_works implementation and are
-  #       carried forward without much consideration, except to say "Make it configurable!"
-  #
-  # @param base [Class<SolrDocument>]
-  # @return [Class<SolrDocument>]
-  def self.decorate(base)
-    base.prepend(self)
-    base.send(:attribute, :is_child, Hyrax::SolrDocument::Metadata::Solr::String, 'is_child_bsi')
-    base.send(:attribute, :split_from_pdf_id, Hyrax::SolrDocument::Metadata::Solr::String, 'split_from_pdf_id_ssi')
-    base.send(:attribute, :digest, Hyrax::SolrDocument::Metadata::Solr::String, 'digest_ssim')
-
-    # @note These properties came from the newspaper_works gem.  They are configurable.
-    base.class_attribute :iiif_print_solr_field_names, default: %w[alternative_title genre
-                                                                   issn lccn oclcnum held_by text_direction
-                                                                   page_number section author photographer
-                                                                   volume issue_number geographic_coverage
-                                                                   extent publication_date height width
-                                                                   edition_number edition_name frequency preceded_by
-                                                                   succeeded_by]
-    base
-  end
-
-  def digest_sha1
-    digest[/urn:sha1:([\w]+)/, 1]
-  end
-
-  def method_missing(method_name, *args, &block)
-    super unless iiif_print_solr_field_names.include? method_name.to_s
-    self[IiifPrint.solr_name(method_name.to_s)]
-  end
-
-  def respond_to_missing?(method_name, include_private = false)
-    iiif_print_solr_field_names.include?(method_name.to_s) || super
-  end
-
-  # @see https://github.com/samvera/hyrax/commit/7108409c619cd2ba4ae8c836b9f3b429a7e9837b
-  def file_set_ids
-    # Yes, this looks a little odd.  But the truth is the prior key (e.g. `file_set_ids_ssim`) was
-    # an alias of `member_ids_ssim`.
-    self['member_ids_ssim']
-  end
-
-  def any_highlighting?
-    response&.[]('highlighting')&.[](id)&.present?
-  end
-
-  def solr_document
-    self
-  end
-end

data/config/initializers/simple_schema_loader.rb

@@ -1 +0,0 @@
-"Hyrax::SimpleSchemaLoader".safe_constantize&.prepend(IiifPrint::SimpleSchemaLoaderDecorator)

data/lib/iiif_print/works_controller_behavior.rb

@@ -1,9 +0,0 @@
-module IiifPrint
-  module WorksControllerBehaviorDecorator
-    # Extending the presenter to the base url which includes the protocol.
-    # We need the base url to render the facet links.
-    def iiif_manifest_presenter
-      super.tap { |i| i.base_url = request.base_url }
-    end
-  end
-end