Merge branch 'main' into more_decorators_than_hgtv

app/jobs/iiif_print/jobs/child_works_from_pdf_job.rb

@@ -87,7 +87,7 @@ def split_pdf(original_pdf_path, user, child_model, pdf_file_set)
         )
         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 @@ def split_pdf(original_pdf_path, user, child_model, pdf_file_set)
       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 @@ def prepare_import_data(original_pdf_path, image_files, user)
             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 @@ def create_uploaded_file(user, path)
         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

app/listeners/iiif_print/listener.rb

@@ -15,7 +15,7 @@ def on_file_characterized(event, service: IiifPrint::SplitPdfs::ChildWorkCreatio
       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.

app/services/iiif_print/derivative_rodeo_service.rb

@@ -366,7 +366,7 @@ def input_uri
                    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
 

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

iiif_print.gemspec

@@ -12,12 +12,10 @@ Gem::Specification.new do |spec|
   spec.email       = ['[email protected]', '[email protected]',
                       '[email protected]', '[email protected]', '[email protected]',
                       '[email protected]', '[email protected]']
-  spec.homepage    = 'https://github.com/samvera-labs/iiif_print'
+  spec.homepage    = 'https://github.com/scientist-softserv/iiif_print/'
   spec.description = 'Gem/Engine for IIIF Print works in Hyrax-based Samvera Application.'
   spec.summary     = <<-SUMMARY
-  iiif_print is a Rails Engine gem providing model and administrative
-  functions to Hyrax-based Samvera applications, for management of
-  (primarily scanned) content.
+  IiifPrint is a gem (Rails "engine") for Hyrax-based digital repository applications to support displaying parent/child works in the same viewer (Universal Viewer) and the ability to search OCR from the parent work to the child work(s). IiifPring was originally based off of the samvera-labs Newspaper gem.
 SUMMARY
   spec.license = 'Apache-2.0'
   spec.files = `git ls-files`.split($OUTPUT_RECORD_SEPARATOR).select { |f| File.dirname(f) !~ %r{\A"?spec\/?} && f != 'bin/rails' }

lib/iiif_print/base_derivative_service.rb

@@ -33,7 +33,7 @@ def valid?
       # @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

lib/iiif_print/configuration.rb

@@ -212,7 +212,7 @@ def child_work_attributes_function
             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 }

lib/iiif_print/persistence_layer/valkyrie_adapter.rb

@@ -91,7 +91,7 @@ def self.destroy_children_split_from(file_set:, work:, model:, user:)
       end
 
       def self.pdf?(file_set)
-        file_set.original_file.pdf?
+        file_set.original_file&.pdf?
       end
 
       ##

lib/iiif_print/version.rb

@@ -1,3 +1,3 @@
 module IiifPrint
-  VERSION = '1.0.0'.freeze
+  VERSION = '2.0.1'.freeze
 end