hyrax 3.4.1 → 3.5.0

data/documentation/legacyREADME.md

@@ -50,7 +50,7 @@ The Samvera community is here to help. Please see our [support guide](./.github/
 # Getting started
 
 This document contains instructions specific to setting up an app with __Hyrax
-v3.4.1__. If you are looking for instructions on installing a different
+v3.5.0__. If you are looking for instructions on installing a different
 version, be sure to select the appropriate branch or tag from the drop-down
 menu above.
 
@@ -143,7 +143,7 @@ Hyrax requires Rails 5. We recommend the latest Rails 5.2 release.
 
 ```
 # If you don't already have Rails at your disposal...
-gem install rails -v 5.2.6
+gem install rails -v 5.2.8.1
 ```
 
 ### JavaScript runtime
@@ -159,7 +159,7 @@ NOTE: The steps need to be done in order to create a new Hyrax based app.
 Generate a new Rails application using the template.
 
 ```
-rails _5.2.6_ new my_app -m https://raw.githubusercontent.com/samvera/hyrax/v3.0.2/template.rb
+rails _5.2.8.1_ new my_app -m https://raw.githubusercontent.com/samvera/hyrax/v3.5.0/template.rb
 ```
 
 Generating a new Rails application using Hyrax's template above takes cares of a number of steps for you, including:

data/hyrax.gemspec

@@ -26,7 +26,7 @@ SUMMARY
 
   spec.required_ruby_version = '>= 2.5'
 
-  # Note: rails does not follow sem-ver conventions, it's
+  # NOTE: rails does not follow sem-ver conventions, it's
   # minor version releases can include breaking changes; see
   # http://guides.rubyonrails.org/maintenance_policy.html
   spec.add_dependency 'rails', '~> 5.0'
@@ -44,6 +44,7 @@ SUMMARY
   spec.add_dependency 'dry-events', '~> 0.2.0'
   spec.add_dependency 'dry-equalizer', '~> 0.2'
   spec.add_dependency 'dry-struct', '~> 1.0'
+  spec.add_dependency 'dry-monads', '< 1.5'
   spec.add_dependency 'dry-transaction', '~> 0.11'
   spec.add_dependency 'dry-validation', '~> 1.3'
   spec.add_dependency 'flipflop', '~> 2.3'
@@ -58,6 +59,7 @@ SUMMARY
   spec.add_dependency 'iiif_manifest', '>= 0.3', '< 2.0'
   spec.add_dependency 'jquery-datatables-rails', '~> 3.4'
   spec.add_dependency 'jquery-ui-rails', '~> 6.0'
+  spec.add_dependency 'json-ld', '< 3.2'
   spec.add_dependency 'json-schema' # for Arkivo
   # Pin more tightly because 0.x gems are potentially unstable
   spec.add_dependency 'kaminari_route_prefix', '~> 0.1.1'
@@ -70,10 +72,12 @@ SUMMARY
   spec.add_dependency 'oauth2', '~> 1.2'
   spec.add_dependency 'posix-spawn'
   spec.add_dependency 'power_converter', '~> 0.1', '>= 0.1.2'
+  spec.add_dependency 'psych', '~> 3.3'
   spec.add_dependency 'qa', '~> 5.5', '>= 5.5.1' # questioning_authority
   spec.add_dependency 'rails_autolink', '~> 1.1'
   spec.add_dependency 'rdf-rdfxml' # controlled vocabulary importer
   spec.add_dependency 'rdf-vocab', '~> 3.0'
+  spec.add_dependency 'redis', '~> 4.0'
   spec.add_dependency 'redis-namespace', '~> 1.5'
   spec.add_dependency 'redlock', '>= 0.1.2'
   spec.add_dependency 'reform', '~> 2.3'
@@ -82,7 +86,7 @@ SUMMARY
   spec.add_dependency 'samvera-nesting_indexer', '~> 2.0'
   spec.add_dependency 'select2-rails', '~> 3.5'
   spec.add_dependency 'signet'
-  spec.add_dependency 'tinymce-rails'
+  spec.add_dependency 'tinymce-rails', '~> 5.10'
   spec.add_dependency 'valkyrie', '~> 2', '>= 2.1.1'
 
   spec.add_development_dependency "capybara", '~> 3.29'
@@ -106,7 +110,7 @@ SUMMARY
   spec.add_development_dependency 'i18n_yaml_sorter'
   spec.add_development_dependency 'rails-controller-testing', '~> 1'
   # the hyrax style guide is based on `bixby`. see `.rubocop.yml`
-  spec.add_development_dependency 'bixby', '~> 3.0', ">= 3.0.2"
+  spec.add_development_dependency 'bixby', '~> 5.0', '>= 5.0.2' # bixby 5 briefly dropped Ruby 2.5
   spec.add_development_dependency 'shoulda-callback-matchers', '~> 1.1.1'
   spec.add_development_dependency 'shoulda-matchers', '~> 3.1'
   spec.add_development_dependency 'webdrivers', '~> 4.4'

data/lib/generators/hyrax/collection_resource/USAGE

@@ -0,0 +1,20 @@
+Description:
+  This generator creates the necessary files for a Hyrax::PcdmCollection.
+
+Example:
+  Usage: rails generate hyrax:collection_resource MyCollection [with_basic_metadata]
+
+  Argument:
+    with_basic_metadata (optional) - if present, the collection resource will include basic metadata
+
+  This will create:
+    app/models/my_collection.rb
+    app/forms/my_collection_form.rb
+    app/indexers/my_collection_indexer.rb
+    config/metadata/my_collection.yaml
+    spec/models/my_collection_spec.rb
+    spec/forms/my_collection_form_spec.rb
+    spec/indexers/my_collection_indexer_spec.rb
+
+  This will also:
+    set Hyrax.config.collection_model = "MyCollection"

data/lib/generators/hyrax/collection_resource/collection_resource_generator.rb

@@ -0,0 +1,133 @@
+# frozen_string_literal: true
+require 'rails/generators'
+require 'rails/generators/model_helpers'
+
+class Hyrax::CollectionResourceGenerator < Rails::Generators::NamedBase # rubocop:disable Metrics/ClassLength
+  # ActiveSupport can interpret models as plural which causes
+  # counter-intuitive route paths. Pull in ModelHelpers from
+  # Rails which warns users about pluralization when generating
+  # new models or scaffolds.
+  include Rails::Generators::ModelHelpers
+
+  source_root File.expand_path('../templates', __FILE__)
+
+  argument :with_basic_metadata, type: :string, default: "", banner: 'with_basic_metadata'
+
+  desc 'This generator makes the following changes to your application:
+   1. Creates a collection model and model spec, optionally including basic metadata.
+   2. Creates a collection form and spec.
+   3. Creates a collection indexer and spec.
+   4. Creates a collection metadata config.
+   5. Sets this to be the collection class.
+'
+
+  def self.exit_on_failure?
+    true
+  end
+
+  def validate_name
+    return unless name.strip.casecmp("collection").zero?
+    raise Thor::MalformattedArgumentError,
+          set_color("Error: A collection resource with the name '#{name}' would cause name-space clashes. "\
+                    "Please use a different name.", :red)
+  end
+
+  def banner
+    if revoking?
+      say_status("info", "DESTROYING VALKYRIE COLLECTION MODEL: #{class_name}", :blue)
+    else
+      say_status("info", "GENERATING VALKYRIE COLLECTION MODEL: #{class_name}", :blue)
+    end
+  end
+
+  def create_metadata_config
+    template('collection_metadata.yaml', File.join('config/metadata/', "#{file_name}.yaml"))
+  end
+
+  def create_model
+    filepath = File.join('app/models', "#{file_name}.rb")
+    template('collection.rb.erb', filepath)
+    return unless include_basic_metadata?
+    inject_into_file filepath, before: /include Hyrax::Schema/ do
+      "include Hyrax::Schema(:basic_metadata)\n  "
+    end
+  end
+
+  def create_model_spec
+    return unless rspec_installed?
+    filepath = File.join('spec/models/', "#{file_name}_spec.rb")
+    template('collection_spec.rb.erb', filepath)
+
+    return unless include_basic_metadata?
+    inject_into_file filepath, after: /it_behaves_like 'a Hyrax::PcdmCollection'/ do
+      "\n  it_behaves_like 'a model with basic metadata'"
+    end
+  end
+
+  def create_form
+    filepath = File.join('app/forms/', "#{file_name}_form.rb")
+    template('collection_form.rb.erb', filepath)
+    return unless include_basic_metadata?
+    inject_into_file filepath, before: /include Hyrax::FormFields/ do
+      "include Hyrax::FormFields(:basic_metadata)\n  "
+    end
+  end
+
+  # @todo If shared specs are expanded to test for basic metadata, inject calling that test here.
+  def create_form_spec
+    return unless rspec_installed?
+    template('collection_form_spec.rb.erb', File.join('spec/forms/', "#{file_name}_form_spec.rb"))
+  end
+
+  def create_indexer
+    filepath = File.join('app/indexers/', "#{file_name}_indexer.rb")
+    template('collection_indexer.rb.erb', filepath)
+    return unless include_basic_metadata?
+    inject_into_file filepath, before: /include Hyrax::Indexer/ do
+      "include Hyrax::Indexer(:basic_metadata)\n  "
+    end
+  end
+
+  def create_indexer_spec
+    return unless rspec_installed?
+    filepath = File.join('spec/indexers/', "#{file_name}_indexer_spec.rb")
+    template('collection_indexer_spec.rb.erb', filepath)
+
+    return unless include_basic_metadata?
+    inject_into_file filepath, after: /it_behaves_like 'a Hyrax::Resource indexer'/ do
+      "\n  it_behaves_like 'a Basic metadata indexer'"
+    end
+  end
+
+  # Inserts after the last registered work, or at the top of the config block
+  def set_as_the_collection_class
+    config = 'config/initializers/hyrax.rb'
+    lastmatch = nil
+    in_root do
+      File.open(config).each_line do |line|
+        lastmatch = line if line.match?(/config.collection_model = /)
+      end
+      content = "  # Injected via `rails g hyrax:collection_resource #{class_name}`\n" \
+                "  config.collection_model = '#{class_name}'\n"
+
+      anchor = lastmatch || "Hyrax.config do |config|\n"
+      inject_into_file config, after: anchor do
+        content
+      end
+    end
+  end
+
+  private
+
+  def include_basic_metadata?
+    with_basic_metadata.present? && with_basic_metadata == "with_basic_metadata"
+  end
+
+  def rspec_installed?
+    defined?(RSpec) && defined?(RSpec::Rails)
+  end
+
+  def revoking?
+    behavior == :revoke
+  end
+end

data/lib/generators/hyrax/collection_resource/templates/collection.rb.erb

@@ -0,0 +1,34 @@
+# frozen_string_literal: true
+
+# Generated via
+#  `rails generate hyrax:collection_resource <%= class_name %>`
+class <%= class_name %> < Hyrax::PcdmCollection
+  # @note Do not directly update `basic_metadata.yaml`.  It is also used by works.
+  #
+  # To change metadata for collections
+  # * extend by adding fields to `/config/metadata/<%= file_name %>.yaml`
+  # * remove all basic metadata
+  #   * if you generated `with_basic_metadata` and now don't want any basic metadata,
+  #       comment out or delete the schema include statement for `:basic_metadata`
+  #   * update form and indexer classes to also remove the `:basic_metadata` schema include
+  # * remove some basic metadata
+  #   * comment out or delete the schema include statement for `:basic_metadata`
+  #   * update form and indexer classes to also remove the `:basic_metadata` schema include
+  #   * copy fields you want to keep from `/config/metadata/basic_metadata.yaml`
+  #       to `/config/metadata/<%= file_name %>.yaml`
+  # * override some basic metadata
+  #   * fields are not allowed to repeat, so to override any basic metadata, you
+  #       must first comment out or delete the schema include statement for `:basic_metadata`
+  #   * update form and indexer classes to also remove the `:basic_metadata` schema include
+  #   * copy all fields you want to keep unchanged or to override from `/config/metadata/basic_metadata.yaml`
+  #       to `/config/metadata/<%= file_name %>.yaml`
+  #   * update them in `config/metadata/<%= file_name %>.yaml to have the desired
+  #       characteristics
+  #
+  # Alternative:
+  # * comment out or delete schema include statements
+  # * add Valkyrie attributes to this class
+  # * update form and indexer to process the attributes
+  #
+  include Hyrax::Schema(:<%= file_name %>)
+end

data/lib/generators/hyrax/collection_resource/templates/collection_form.rb.erb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+# Generated via
+#  `rails generate hyrax:collection_resource <%= class_name %>`
+class <%= class_name %>Form < Hyrax::Forms::PcdmCollectionForm
+  include Hyrax::FormFields(:<%= file_name %>)
+end

data/lib/generators/hyrax/collection_resource/templates/collection_form_spec.rb.erb

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+# Generated via
+#  `rails generate hyrax:collection_resource <%= class_name %>`
+require 'rails_helper'
+require 'valkyrie/specs/shared_specs'
+
+RSpec.describe <%= class_name %>Form do
+  let(:change_set) { described_class.new(resource) }
+  let(:resource)   { <%= class_name %>.new }
+
+  it_behaves_like 'a Valkyrie::ChangeSet'
+end

data/lib/generators/hyrax/collection_resource/templates/collection_indexer.rb.erb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+# Generated via
+#  `rails generate hyrax:collection_resource <%= class_name %>`
+class <%= class_name %>Indexer < Hyrax::PcdmCollectionIndexer
+  include Hyrax::Indexer(:<%= file_name %>)
+end

data/lib/generators/hyrax/collection_resource/templates/collection_indexer_spec.rb.erb

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+# Generated via
+#  `rails generate hyrax:collection_resource <%= class_name %>`
+require 'rails_helper'
+require 'hyrax/specs/shared_specs/indexers'
+
+RSpec.describe <%= class_name %>Indexer do
+  let(:indexer_class) { described_class }
+  let(:resource)      { <%= class_name %>.new }
+
+  it_behaves_like 'a Hyrax::Resource indexer'
+end

data/lib/generators/hyrax/collection_resource/templates/collection_metadata.yaml

@@ -0,0 +1,22 @@
+# Simple yaml config-driven schema which is used to define model attributes,
+# index key names, and form properties.
+#
+# Attributes must have a type but all other configuration options are optional.
+#
+# attributes:
+#   attribute_name:
+#     type: string
+#     multiple: false
+#     index_keys:
+#       - "attribute_name_sim"
+#     form:
+#       required: true
+#       primary: true
+#       multiple: false
+#
+# @see config/metadata/basic_metadata.yaml for an example configuration
+#
+# Generated via
+#  `rails generate hyrax:collection_resource <%= class_name %>`
+
+attributes: {}

data/lib/generators/hyrax/collection_resource/templates/collection_spec.rb.erb

@@ -0,0 +1,12 @@
+# frozen_string_literal: true
+
+# Generated via
+#  `rails generate hyrax:collection_resource <%= class_name %>`
+require 'rails_helper'
+require 'hyrax/specs/shared_specs/hydra_works'
+
+RSpec.describe <%= class_name %> do
+  subject(:collection) { described_class.new }
+
+  it_behaves_like 'a Hyrax::PcdmCollection'
+end

data/lib/generators/hyrax/install_generator.rb

@@ -35,6 +35,15 @@ module Hyrax
   18. Generates RIIIF image server implementation
          """
 
+    # browse-everything 1.2.0 is not compatible with Ruby 2.5
+    def pin_browse_everything
+      return unless Gem::Version.new(RUBY_VERSION) < Gem::Version.new('2.6')
+      gem 'browse-everything', '~> 1.1.2'
+      Bundler.with_clean_env do
+        run "bundle update browse-everything"
+      end
+    end
+
     def run_required_generators
       say_status('info', '[Hyrax] GENERATING BLACKLIGHT', :blue)
       generate 'blacklight:install --devise'

data/lib/generators/hyrax/templates/config/locales/hyrax.de.yml

@@ -50,7 +50,7 @@ de:
     directory:
       suffix: "@ Example.org"
     footer:
-      copyright_html: "<strong>Copyright © 2018 Samvera lizenziert</strong> unter der Apache Lizenz, Version 2.0"
+      copyright_html: "<strong>Copyright © 2022 Samvera lizenziert</strong> unter der Apache Lizenz, Version 2.0"
       service_html: Ein Dienst von <a href="http://samvera.org/" class="navbar-link" target="_blank">Samvera</a> .
     institution_name: Institution
     institution_name_full: Name des Instituts

data/lib/generators/hyrax/templates/config/locales/hyrax.en.yml

@@ -50,7 +50,7 @@ en:
     directory:
       suffix: "@example.org"
     footer:
-      copyright_html: "<strong>Copyright &copy; 2018 Samvera</strong> Licensed under the Apache License, Version 2.0"
+      copyright_html: "<strong>Copyright &copy; 2022 Samvera</strong> Licensed under the Apache License, Version 2.0"
       service_html: A service of <a href="http://samvera.org/" class="navbar-link" target="_blank">Samvera</a>.
     institution_name: Institution
     institution_name_full: The Institution Name

data/lib/generators/hyrax/templates/config/locales/hyrax.es.yml

@@ -50,7 +50,7 @@ es:
     directory:
       suffix: "@example.org"
     footer:
-      copyright_html: "<strong>Copyright &copy; 2018 Samvera</strong> bajo licencia de Apache, Version 2.0"
+      copyright_html: "<strong>Copyright &copy; 2022 Samvera</strong> bajo licencia de Apache, Version 2.0"
       service_html: Un servicio de <a href="http://samvera.org/" class="navbar-link" target="_blank">Samvera</a>.
     institution_name: institución
     institution_name_full: El nombre de la institución

data/lib/generators/hyrax/templates/config/locales/hyrax.fr.yml

@@ -50,7 +50,7 @@ fr:
     directory:
       suffix: "@ Example.org"
     footer:
-      copyright_html: "<strong>Copyright © 2018 Samvera</strong> Licence sous Licence Apache, Version 2.0"
+      copyright_html: "<strong>Copyright © 2022 Samvera</strong> Licence sous Licence Apache, Version 2.0"
       service_html: Un service de <a href="http://samvera.org/" class="navbar-link" target="_blank">Samvera</a> .
     institution_name: Institution
     institution_name_full: Nom de l'établissement

data/lib/generators/hyrax/templates/config/locales/hyrax.it.yml

@@ -50,7 +50,7 @@ it:
     directory:
       suffix: "@ example.org"
     footer:
-      copyright_html: "<strong>Copyright © 2018 Samvera</strong> Licenza sotto la licenza Apache, versione 2.0"
+      copyright_html: "<strong>Copyright © 2022 Samvera</strong> Licenza sotto la licenza Apache, versione 2.0"
       service_html: Un servizio di <a href="http://samvera.org/" class="navbar-link" target="_blank">Samvera</a> .
     institution_name: Istituzione
     institution_name_full: Nome dell'Istituzione

data/lib/generators/hyrax/templates/config/locales/hyrax.zh.yml

@@ -50,7 +50,7 @@ zh:
     directory:
       suffix: "@example.org"
     footer:
-      copyright_html: "<strong>版权所有 &copy; 2018 Samvera</strong> 根据Apache许可证2.0版许可"
+      copyright_html: "<strong>版权所有 &copy; 2022 Samvera</strong> 根据Apache许可证2.0版许可"
       service_html: 的服务<a href="http://samvera.org/" class="navbar-link" target="_blank">Samvera</a>.
     institution_name: 机构
     institution_name_full: 机构名称

data/lib/hyrax/active_fedora_dummy_model.rb

@@ -35,12 +35,25 @@ module Hyrax
       @id
     end
 
+    ##
+    # @api public
+    def to_key
+      [@id]
+    end
+
     ##
     # @api public
     def model_name
       @model.model_name
     end
 
+    ##
+    # @api public
+    # @return [String]
+    def human_readable_type
+      @model.human_readable_type
+    end
+
     ##
     # @api public
     #

data/lib/hyrax/configuration.rb

@@ -427,6 +427,18 @@ module Hyrax
       @branding_storage_adapter = Valkyrie::StorageAdapter.find(adapter.to_sym)
     end
 
+    ##
+    # @return [Valkyrie::StorageAdapter]
+    def derivatives_storage_adapter
+      @derivatives_storage_adapter ||= Valkyrie::StorageAdapter.find(:derivatives_disk)
+    end
+
+    ##
+    # @param [#to_sym] adapter
+    def derivatives_storage_adapter=(adapter)
+      @derivatives_storage_adapter = Valkyrie::StorageAdapter.find(adapter.to_sym)
+    end
+
     ##
     # @return [#save, #save_all, #delete, #wipe!] an indexing adapter
     def index_adapter
@@ -863,10 +875,17 @@ module Hyrax
                     end
     end
 
-    attr_accessor :nested_relationship_reindexer
+    def use_solr_graph_for_collection_nesting
+      ActiveModel::Type::Boolean.new.cast(ENV.fetch('HYRAX_USE_SOLR_GRAPH_NESTING', false))
+    end
 
+    attr_accessor :nested_relationship_reindexer
     def default_nested_relationship_reindexer
-      ->(id:, extent:) { Samvera::NestingIndexer.reindex_relationships(id: id, extent: extent) }
+      if use_solr_graph_for_collection_nesting
+        ->(id:, extent:) {}
+      else
+        ->(id:, extent:) { Samvera::NestingIndexer.reindex_relationships(id: id, extent: extent) }
+      end
     end
 
     attr_writer :solr_select_path

data/lib/hyrax/errors.rb

@@ -15,4 +15,6 @@ module Hyrax
   class SingleMembershipError < HyraxError; end
 
   class ObjectNotFoundError < ActiveFedora::ObjectNotFoundError; end
+
+  class ModelMismatchError < HyraxError; end
 end

data/lib/hyrax/publisher.rb

@@ -119,6 +119,10 @@ module Hyrax
     #     unique id), AND a `:user` (the ::User responsible for the update).
     register_event('collection.membership.updated')
 
+    # @since 3.5.0
+    # @macro a_registered_event
+    register_event('file.characterized')
+
     # @since 3.3.0
     # @macro a_registered_event
     register_event('file.downloaded')

data/lib/hyrax/specs/shared_specs/factories/strategies/valkyrie_resource.rb

@@ -3,6 +3,12 @@
 # @example
 #   let(:resource) { FactoryBot.valkyrie_create(:hyrax_work) }
 class ValkyrieCreateStrategy
+  def initialize
+    @strategy = FactoryBot.strategy_by_name(:create).new
+  end
+
+  delegate :association, to: :@strategy
+
   def result(evaluation)
     evaluation.notify(:after_build, evaluation.object)
     evaluation.notify(:before_create, evaluation.object)

data/lib/hyrax/specs/shared_specs/hydra_works.rb

@@ -135,7 +135,6 @@ RSpec.shared_examples 'a Hyrax::PcdmCollection' do
 
   it_behaves_like 'a Hyrax::Resource'
   it_behaves_like 'a model with core metadata'
-  it_behaves_like 'a model with basic metadata'
   it_behaves_like 'has members'
 
   describe '#collection_type_gid' do

data/lib/hyrax/specs/shared_specs/indexers.rb

@@ -266,6 +266,11 @@ RSpec.shared_examples 'a Collection indexer' do
         .to include(generic_type_sim: a_collection_containing_exactly('Collection'))
     end
 
+    it 'indexes member_of_collection_ids' do
+      expect(indexer.to_solr)
+        .to include(member_of_collection_ids_ssim: resource.member_of_collection_ids)
+    end
+
     it 'indexes depositor' do
       expect(indexer.to_solr)
         .to include(depositor_ssim: [resource.depositor],

data/lib/hyrax/transactions/collection_update.rb

@@ -9,6 +9,8 @@ module Hyrax
     # @since 3.2.0
     class CollectionUpdate < Transaction
       DEFAULT_STEPS = ['change_set.apply',
+                       'collection_resource.save_collection_banner',
+                       'collection_resource.save_collection_logo',
                        'collection_resource.save_acl'].freeze
 
       ##

data/lib/hyrax/transactions/container.rb

@@ -42,6 +42,8 @@ module Hyrax
       require 'hyrax/transactions/steps/remove_file_set_from_work'
       require 'hyrax/transactions/steps/save'
       require 'hyrax/transactions/steps/save_access_control'
+      require 'hyrax/transactions/steps/save_collection_banner'
+      require 'hyrax/transactions/steps/save_collection_logo'
       require 'hyrax/transactions/steps/set_default_admin_set'
       require 'hyrax/transactions/steps/set_modified_date'
       require 'hyrax/transactions/steps/set_uploaded_date_unless_present'
@@ -195,6 +197,14 @@ module Hyrax
         ops.register 'save_acl' do
           Steps::SaveAccessControl.new
         end
+
+        ops.register 'save_collection_banner' do
+          Steps::SaveCollectionBanner.new
+        end
+
+        ops.register 'save_collection_logo' do
+          Steps::SaveCollectionLogo.new
+        end
       end
 
       namespace 'work_resource' do |ops| # Hyrax::Work resource

data/lib/hyrax/transactions/steps/save_collection_banner.rb

@@ -0,0 +1,59 @@
+# frozen_string_literal: true
+module Hyrax
+  module Transactions
+    module Steps
+      ##
+      # Adds banner info via `ChangeSet`.
+      #
+      # During the update collection process this step is called to update the file
+      # to be used as a the banner for the collection.
+      #
+      class SaveCollectionBanner
+        include Dry::Transaction::Operation
+
+        ##
+        # @param [Hyrax::ChangeSet] change_set
+        # @param [Array<Integer>] update_banner_file_ids
+        # @param [Boolean] banner_unchanged_indicator
+        #
+        # @return [Dry::Monads::Result] `Failure` if the banner info fails to save;
+        #   `Success(input)`, otherwise.
+        def call(collection_resource, update_banner_file_ids: nil, banner_unchanged_indicator: true)
+          return Success(collection_resource) if ActiveModel::Type::Boolean.new.cast(banner_unchanged_indicator)
+          collection_id = collection_resource.id.to_s
+          process_banner_input(collection_id: collection_id, update_banner_file_ids: update_banner_file_ids)
+          Success(collection_resource)
+        end
+
+        private
+
+        def process_banner_input(collection_id:, update_banner_file_ids:)
+          remove_banner(collection_id: collection_id)
+          add_new_banner(collection_id: collection_id, uploaded_file_ids: update_banner_file_ids) if update_banner_file_ids
+        end
+
+        def remove_banner(collection_id:)
+          banner_info = CollectionBrandingInfo.where(collection_id: collection_id).where(role: "banner")
+          banner_info&.delete_all
+        end
+
+        def add_new_banner(collection_id:, uploaded_file_ids:)
+          f = uploaded_files(uploaded_file_ids).first
+          banner_info = CollectionBrandingInfo.new(
+            collection_id: collection_id,
+            filename: File.split(f.file_url).last,
+            role: "banner",
+            alt_txt: "",
+            target_url: ""
+          )
+          banner_info.save f.file_url
+        end
+
+        def uploaded_files(uploaded_file_ids)
+          return [] if uploaded_file_ids.empty?
+          UploadedFile.find(uploaded_file_ids)
+        end
+      end
+    end
+  end
+end