hyrax 5.2.0 → 5.3.0

data/lib/hyrax/transactions/container.rb

@@ -36,6 +36,8 @@ module Hyrax
       require 'hyrax/transactions/steps/add_to_parent'
       require 'hyrax/transactions/steps/apply_collection_type_permissions'
       require 'hyrax/transactions/steps/apply_permission_template'
+      require 'hyrax/transactions/steps/apply_permission_template_on_update'
+      require 'hyrax/transactions/steps/apply_workflow_on_admin_set_change'
       require 'hyrax/transactions/steps/change_depositor'
       require 'hyrax/transactions/steps/check_for_default_admin_set'
       require 'hyrax/transactions/steps/check_for_empty_admin_set'
@@ -49,6 +51,7 @@ module Hyrax
       require 'hyrax/transactions/steps/set_collection_type_gid'
       require 'hyrax/transactions/steps/remove_file_set_from_work'
       require 'hyrax/transactions/steps/remove_from_membership'
+      require 'hyrax/transactions/steps/remove_redirect_paths'
       require 'hyrax/transactions/steps/save'
       require 'hyrax/transactions/steps/save_access_control'
       require 'hyrax/transactions/steps/save_collection_banner'
@@ -58,6 +61,7 @@ module Hyrax
       require 'hyrax/transactions/steps/set_uploaded_date_unless_present'
       require 'hyrax/transactions/steps/set_user_as_creator'
       require 'hyrax/transactions/steps/set_user_as_depositor'
+      require 'hyrax/transactions/steps/sync_redirect_paths'
       require 'hyrax/transactions/steps/update_work_members'
       require 'hyrax/transactions/steps/validate'
 
@@ -245,6 +249,14 @@ module Hyrax
         ops.register 'save_collection_logo' do
           Steps::SaveCollectionLogo.new
         end
+
+        ops.register 'sync_redirect_paths' do
+          Steps::SyncRedirectPaths.new
+        end
+
+        ops.register 'remove_redirect_paths' do
+          Steps::RemoveRedirectPaths.new
+        end
       end
 
       namespace 'work_resource' do |ops| # Hyrax::Work resource
@@ -260,6 +272,14 @@ module Hyrax
           Steps::ApplyPermissionTemplate.new
         end
 
+        ops.register 'apply_permission_template_on_update' do
+          Steps::ApplyPermissionTemplateOnUpdate.new
+        end
+
+        ops.register 'apply_workflow_on_admin_set_change' do
+          Steps::ApplyWorkflowOnAdminSetChange.new
+        end
+
         ops.register 'change_depositor' do
           Steps::ChangeDepositor.new
         end
@@ -287,6 +307,14 @@ module Hyrax
         ops.register 'update_work_members' do
           Steps::UpdateWorkMembers.new
         end
+
+        ops.register 'sync_redirect_paths' do
+          Steps::SyncRedirectPaths.new
+        end
+
+        ops.register 'remove_redirect_paths' do
+          Steps::RemoveRedirectPaths.new
+        end
       end
       # rubocop:enable Metrics/BlockLength
     end

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

@@ -0,0 +1,65 @@
+# frozen_string_literal: true
+module Hyrax
+  module Transactions
+    module Steps
+      ##
+      # A `dry-transaction` step that applies a permission template to a work
+      # when its admin set has changed during an update.
+      #
+      # Removes grants from the old admin set's permission template that are not
+      # present in the new template, then applies the new template's grants.
+      # Manually added permissions and the depositor's access are preserved.
+      #
+      # @since 5.1.0
+      class ApplyPermissionTemplateOnUpdate
+        include Dry::Monads[:result]
+
+        ##
+        # @param [Hyrax::Work] object
+        #
+        # @return [Dry::Monads::Result]
+        def call(object)
+          return Success(object) unless object.respond_to?(:previous_admin_set_id)
+
+          old_template = Hyrax::PermissionTemplate.find_by(source_id: object.previous_admin_set_id)
+          new_template = Hyrax::PermissionTemplate.find_by(source_id: object.admin_set_id)
+
+          remove_old_grants(object, old_template, new_template) if old_template
+
+          if new_template
+            Hyrax::PermissionTemplateApplicator.apply(new_template).to(model: object)
+          else
+            Hyrax.logger.warn("At update time, #{object} doesn't have a " \
+                              "PermissionTemplate for new AdministrativeSet " \
+                              "#{object.admin_set_id}. Continuing without " \
+                              "applying permissions.")
+          end
+
+          Success(object)
+        end
+
+        private
+
+        def remove_old_grants(object, old_template, new_template)
+          new_agents = agents_from_template(new_template)
+
+          object.edit_groups = object.edit_groups.to_a - (old_template.agent_ids_for(agent_type: 'group', access: 'manage') - new_agents[:manage_groups])
+          object.edit_users  = object.edit_users.to_a  - (old_template.agent_ids_for(agent_type: 'user',  access: 'manage') - new_agents[:manage_users])
+          object.read_groups = object.read_groups.to_a - (old_template.agent_ids_for(agent_type: 'group', access: 'view')   - new_agents[:view_groups])
+          object.read_users  = object.read_users.to_a  - (old_template.agent_ids_for(agent_type: 'user',  access: 'view')   - new_agents[:view_users])
+        end
+
+        def agents_from_template(template)
+          return { manage_groups: [], manage_users: [], view_groups: [], view_users: [] } if template.nil?
+
+          {
+            manage_groups: template.agent_ids_for(agent_type: 'group', access: 'manage').to_a,
+            manage_users: template.agent_ids_for(agent_type: 'user', access: 'manage').to_a,
+            view_groups: template.agent_ids_for(agent_type: 'group', access: 'view').to_a,
+            view_users: template.agent_ids_for(agent_type: 'user', access: 'view').to_a
+          }
+        end
+      end
+    end
+  end
+end

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

@@ -0,0 +1,97 @@
+# frozen_string_literal: true
+module Hyrax
+  module Transactions
+    module Steps
+      ##
+      # A `dry-transaction` step that moves a work's `Sipity::Entity` to the
+      # new admin set's active workflow when the admin set has changed during
+      # an update and both the old and new admin sets use workflow management.
+      #
+      # The entity's `workflow_state` is preserved by matching on state name:
+      # the new workflow's state with the same name is used when one exists,
+      # and the new workflow's initial state is used as a fallback (with a
+      # warning) when it does not.
+      #
+      # The depositor's existing `EntitySpecificResponsibility` is re-created
+      # against the new workflow's `depositing` role so it is not orphaned to
+      # a `WorkflowRole` from the prior workflow.
+      #
+      # No-ops when:
+      # * the admin set has not changed,
+      # * the new admin set has no active workflow (the entity is left in
+      #   place, pointing at the prior workflow), or
+      # * the work has no `Sipity::Entity` yet (the prior admin set was not
+      #   workflow-managed).
+      class ApplyWorkflowOnAdminSetChange
+        include Dry::Monads[:result]
+
+        ##
+        # @param [Hyrax::Work] object
+        #
+        # @return [Dry::Monads::Result]
+        def call(object)
+          return Success(object) unless object.respond_to?(:previous_admin_set_id)
+
+          entity = Sipity::Entity.find_by(proxy_for_global_id: Hyrax::GlobalID(object).to_s)
+          return Success(object) if entity.nil?
+
+          new_workflow = active_workflow_for(object.admin_set_id)
+          return Success(object) if new_workflow.nil?
+
+          move_entity_to_workflow(entity: entity, new_workflow: new_workflow, object: object)
+          reassign_depositor_responsibility(entity: entity, new_workflow: new_workflow, object: object)
+
+          Success(object)
+        end
+
+        private
+
+        def active_workflow_for(admin_set_id)
+          Sipity::Workflow.find_active_workflow_for(admin_set_id: admin_set_id)
+        rescue Sipity::Workflow::NoActiveWorkflowError
+          nil
+        end
+
+        def move_entity_to_workflow(entity:, new_workflow:, object:)
+          target_state = matching_state_on(new_workflow, entity.workflow_state_name) ||
+                         fallback_initial_state(new_workflow, entity, object)
+          entity.update!(workflow: new_workflow, workflow_state: target_state)
+        end
+
+        def matching_state_on(workflow, state_name)
+          return nil if state_name.blank?
+          workflow.workflow_states.find_by(name: state_name)
+        end
+
+        def fallback_initial_state(new_workflow, entity, object)
+          Hyrax.logger.warn("Workflow state #{entity.workflow_state_name.inspect} from prior workflow " \
+                            "does not exist on the new active workflow for AdministrativeSet " \
+                            "#{object.admin_set_id}. Falling back to the initial state for work " \
+                            "#{object.id}.")
+          new_workflow.initial_workflow_state
+        end
+
+        def reassign_depositor_responsibility(entity:, new_workflow:, object:)
+          depositor = ::User.find_by_user_key(object.depositor)
+          return if depositor.nil?
+
+          remove_stale_depositor_responsibility(entity: entity, new_workflow: new_workflow, depositor: depositor)
+
+          Hyrax::Workflow::PermissionGenerator.call(agents: depositor,
+                                                    entity: entity,
+                                                    roles: Hyrax::RoleRegistry::DEPOSITING,
+                                                    workflow: new_workflow)
+        end
+
+        def remove_stale_depositor_responsibility(entity:, new_workflow:, depositor:)
+          depositor_agent = Sipity::Agent(depositor)
+          entity.entity_specific_responsibilities
+                .joins(:workflow_role)
+                .where(agent_id: depositor_agent.id)
+                .where.not(sipity_workflow_roles: { workflow_id: new_workflow.id })
+                .destroy_all
+        end
+      end
+    end
+  end
+end

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

@@ -15,7 +15,7 @@ module Hyrax
         # @return [Dry::Monads::Result] `Failure` if there is no `AdminSet` for
         #   the input; `Success(input)`, otherwise.
         def call(obj)
-          obj.admin_set_id ? Success(obj) : Failure(:no_admin_set_id)
+          obj.admin_set_id.to_s.present? ? Success(obj) : Failure(:no_admin_set_id)
         end
       end
     end

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

@@ -0,0 +1,35 @@
+# frozen_string_literal: true
+require 'dry/monads'
+
+module Hyrax
+  module Transactions
+    module Steps
+      # A `dry-transaction` step that removes a resource's rows from the
+      # `hyrax_redirect_paths` redirects table. Runs as part of the destroy
+      # transactions so deleted resources don't leave dangling claims on
+      # redirect paths.
+      #
+      # See documentation/redirects.md.
+      class RemoveRedirectPaths
+        include Dry::Monads[:result]
+
+        # @param [Valkyrie::Resource] resource
+        # @return [Dry::Monads::Result]
+        def call(resource)
+          return Success(resource) unless removable?(resource)
+          Hyrax::RedirectPath.where(resource_id: resource.id.to_s).delete_all
+          Success(resource)
+        rescue ActiveRecord::StatementInvalid => e
+          Hyrax.logger.error("[redirects] remove_redirect_paths failed: #{e.message}")
+          Failure([:redirect_path_remove_error, e.message])
+        end
+
+        private
+
+        def removable?(resource)
+          Hyrax.config.redirects_enabled? && resource.respond_to?(:id) && resource.id.present?
+        end
+      end
+    end
+  end
+end

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

@@ -35,6 +35,7 @@ module Hyrax
             valid_future_date?(change_set.lease, 'lease_expiration_date') if change_set.respond_to?(:lease) && change_set.lease
             valid_future_date?(change_set.embargo, 'embargo_release_date') if change_set.respond_to?(:embargo) && change_set.embargo
             new_collections = changed_collection_membership(change_set)
+            old_admin_set_id = changed_admin_set_id(change_set)
 
             unsaved = change_set.sync
             save_lease_or_embargo(unsaved)
@@ -50,6 +51,8 @@ module Hyrax
 
           user ||= ::User.find_by_user_key(saved.depositor)
 
+          saved.define_singleton_method(:previous_admin_set_id) { old_admin_set_id } if old_admin_set_id
+
           publish_changes(resource: saved, user: user, new: unsaved.new_record, new_collections: new_collections)
           Success(saved)
         end
@@ -83,6 +86,13 @@ module Hyrax
           change_set.member_of_collection_ids - change_set.model.member_of_collection_ids
         end
 
+        def changed_admin_set_id(change_set)
+          return nil unless change_set.respond_to?(:admin_set_id) &&
+                            change_set.changed?(:admin_set_id)
+
+          change_set.model.admin_set_id
+        end
+
         def publish_changes(resource:, user:, new: false, new_collections: [])
           if resource.collection?
             @publisher.publish('collection.metadata.updated', collection: resource, user: user)

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

@@ -16,7 +16,7 @@ module Hyrax
         #
         # @return [Dry::Monads::Result]
         def call(obj)
-          obj.admin_set_id ||= Hyrax::EnsureWellFormedAdminSetService.call
+          obj.admin_set_id = Hyrax::EnsureWellFormedAdminSetService.call if obj.admin_set_id.to_s.blank?
 
           Success(obj)
         end

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

@@ -0,0 +1,141 @@
+# frozen_string_literal: true
+require 'dry/monads'
+
+module Hyrax
+  module Transactions
+    module Steps
+      # A `dry-transaction` step that mirrors a saved resource's `redirects`
+      # entries into the `hyrax_redirect_paths` redirects table. The unique
+      # index on `from_path` enforces global uniqueness at the DB level — if
+      # a concurrent save already claimed a path, the insert raises
+      # ActiveRecord::RecordNotUnique and this step returns Failure, which
+      # short-circuits the enclosing transaction.
+      #
+      # Every row carries the resource's canonical UUID URL in
+      # `permalink_path`. Column semantics:
+      #
+      # - `from_path` — the URL the visitor entered (an alias or the UUID
+      #   URL itself).
+      # - `to_path` — the URL the address bar should display after the
+      #   resolver routes the request. For the display row, equals its own
+      #   `from_path` (the visitor stays at the display URL). For
+      #   non-display rows, points at the display row's `from_path` (the
+      #   visitor lands at the user-facing display URL). When no entry is
+      #   marked, every alias's `to_path` is the UUID URL.
+      # - `permalink_path` — the resource's canonical UUID URL. Constant
+      #   per resource across all rows.
+      #
+      # When a display URL is set, the sync step also writes an extra row
+      # with `from_path = permalink_path` so visitors hitting the bare
+      # UUID URL are routed to the display alias.
+      #
+      # No-op when the redirects feature is off (config or Flipflop) or when
+      # the resource doesn't carry the redirects attribute.
+      #
+      # See documentation/redirects.md.
+      class SyncRedirectPaths
+        include Dry::Monads[:result]
+
+        # @param [Valkyrie::Resource] object the saved resource (must have an id)
+        # @return [Dry::Monads::Result]
+        def call(object)
+          return Success(object) unless syncable?(object)
+          replace_rows(object, build_rows(object))
+          Success(object)
+        rescue ActiveRecord::RecordNotUnique => e
+          Failure([:redirect_path_collision, e.message])
+        rescue ActiveRecord::StatementInvalid => e
+          Hyrax.logger.error("[redirects] sync_redirect_paths failed: #{e.message}")
+          Failure([:redirect_path_sync_error, e.message])
+        end
+
+        private
+
+        def syncable?(object)
+          return false unless Hyrax.config.redirects_active?
+          object.respond_to?(:redirects) && object.respond_to?(:id) && object.id.present?
+        end
+
+        def build_rows(object)
+          permalink = Hyrax::PermalinkPath.call(object)
+          entries = entries_for(object)
+          display_path = entries.find { |e| e[:is_display_url] }&.dig(:from_path)
+          resource_id = object.id.to_s
+          now = Time.current
+          alias_rows = entries.map { |entry| build_alias_row(entry, resource_id, permalink, display_path, now) }
+          alias_rows << build_permalink_row(resource_id, permalink, display_path, now) if display_path
+          alias_rows
+        end
+
+        # Valkyrie's JSONValueMapper symbolizes hash keys on read; accept either.
+        # Paths are normalized at write time by Hyrax::RedirectsNormalization.
+        def entries_for(object)
+          seen = Set.new
+          Array(object.redirects).each_with_object([]) do |entry, acc|
+            path = entry['path'] || entry[:path]
+            next if path.blank? || seen.include?(path)
+            seen << path
+            acc << { from_path: path, is_display_url: display_url_flag(entry) }
+          end
+        end
+
+        # Boolean-cast the stored value so importer/console writes that leave a
+        # string like "false" or "0" in the JSONB hash don't end up as truthy.
+        # Returns false when the key is absent.
+        def display_url_flag(entry)
+          raw = if entry.key?('is_display_url')
+                  entry['is_display_url']
+                elsif entry.key?(:is_display_url)
+                  entry[:is_display_url]
+                end
+          ActiveModel::Type::Boolean.new.cast(raw) || false
+        end
+
+        def build_alias_row(entry, resource_id, permalink, display_path, now)
+          to_path = if entry[:is_display_url]
+                      entry[:from_path]
+                    elsif display_path
+                      display_path
+                    else
+                      permalink
+                    end
+          { from_path: entry[:from_path],
+            to_path: to_path,
+            permalink_path: permalink,
+            resource_id: resource_id,
+            is_display_url: entry[:is_display_url],
+            created_at: now, updated_at: now }
+        end
+
+        def build_permalink_row(resource_id, permalink, display_path, now)
+          { from_path: permalink,
+            to_path: display_path,
+            permalink_path: permalink,
+            resource_id: resource_id,
+            is_display_url: false,
+            created_at: now, updated_at: now }
+        end
+
+        # Replaces the resource's rows when the set differs from what's
+        # persisted. Skips the rewrite when the desired and existing rows
+        # match (the common case on an update that didn't touch the
+        # redirects attribute), to preserve `created_at` and avoid an
+        # unnecessary DB transaction.
+        def replace_rows(object, rows)
+          desired = rows.map { |r| [r[:from_path], r[:to_path], r[:is_display_url]] }.sort
+
+          Hyrax::RedirectPath.transaction do
+            existing = Hyrax::RedirectPath.where(resource_id: object.id.to_s)
+                                          .pluck(:from_path, :to_path, :is_display_url)
+            return if desired == existing.sort
+
+            Hyrax::RedirectPath.where(resource_id: object.id.to_s).delete_all
+            # rubocop:disable Rails/SkipsModelValidations -- the DB unique index on `from_path` is the validation we rely on; bulk insert is intentional
+            Hyrax::RedirectPath.insert_all!(rows) if rows.any?
+            # rubocop:enable Rails/SkipsModelValidations
+          end
+        end
+      end
+    end
+  end
+end

data/lib/hyrax/transactions/work_create.rb

@@ -16,7 +16,8 @@ module Hyrax
                        'work_resource.save_acl',
                        'work_resource.add_file_sets',
                        'work_resource.change_depositor',
-                       'work_resource.add_to_parent'].freeze
+                       'work_resource.add_to_parent',
+                       'work_resource.sync_redirect_paths'].freeze
 
       ##
       # @see Hyrax::Transactions::Transaction

data/lib/hyrax/transactions/work_destroy.rb

@@ -10,6 +10,7 @@ module Hyrax
     class WorkDestroy < Transaction
       DEFAULT_STEPS = ['work_resource.delete_all_file_sets',
                        'work_resource.delete_acl',
+                       'work_resource.remove_redirect_paths',
                        'work_resource.delete'].freeze
 
       ##

data/lib/hyrax/transactions/work_update.rb

@@ -5,9 +5,12 @@ module Hyrax
     # @since 3.4.0
     class WorkUpdate < Transaction
       DEFAULT_STEPS = ['change_set.apply',
+                       'work_resource.apply_permission_template_on_update',
+                       'work_resource.apply_workflow_on_admin_set_change',
                        'work_resource.save_acl',
                        'work_resource.add_file_sets',
-                       'work_resource.update_work_members'].freeze
+                       'work_resource.update_work_members',
+                       'work_resource.sync_redirect_paths'].freeze
 
       ##
       # @see Hyrax::Transactions::Transaction

data/lib/hyrax/version.rb

@@ -1,4 +1,4 @@
 # frozen_string_literal: true
 module Hyrax
-  VERSION = '5.2.0'
+  VERSION = '5.3.0'
 end

data/lib/hyrax.rb

@@ -17,6 +17,7 @@ require 'browse-everything'
 require 'hydra/works'
 require 'hyrax/engine'
 require 'hyrax/version'
+require 'hyrax/deprecation'
 require 'hyrax/inflections'
 require 'hyrax/name'
 require 'hyrax/valkyrie_can_can_adapter'
@@ -48,6 +49,8 @@ module Hyrax
     autoload :Zotero
     autoload :Listeners
     autoload :Workflow
+    autoload :SchemaLoader
+    autoload :M3SchemaLoader
     autoload :SimpleSchemaLoader
     autoload :VirusScanner
     autoload :DerivativeBucketedStorage
@@ -146,4 +149,62 @@ module Hyrax
   def self.custom_queries
     query_service.custom_queries
   end
+
+  ##
+  # Returns the schema for a resource class, optionally scoped by admin set contexts.
+  #
+  # When +admin_set_id+ is present and flexible schema is enabled, the admin set's
+  # contexts are used so flexible resources get a context-aware schema. If the
+  # admin set is not found, falls back to the base schema without raising.
+  #
+  # @param klass [Class] a Valkyrie resource class (e.g. Hyrax::Work)
+  # @param admin_set_id [String, nil] optional admin set id to resolve contexts from
+  # @return [Dry::Types::Schema, Array] the schema (keys/types or enumerable of keys)
+  def self.schema_for(klass:, admin_set_id: nil)
+    contexts = if admin_set_id.blank? || !config.flexible?
+                 []
+               else
+                 schema_contexts_for(admin_set_id)
+               end
+    resolve_schema(klass, contexts)
+  rescue Valkyrie::Persistence::ObjectNotFoundError
+    fallback_schema(klass)
+  end
+
+  ##
+  # @api private
+  #
+  # @param admin_set_id [String]
+  # @return [Array<String>] context identifiers from the admin set, or []
+  def self.schema_contexts_for(admin_set_id)
+    return [] if admin_set_id.blank?
+    admin_set = query_service.find_by(id: admin_set_id)
+    admin_set.respond_to?(:contexts) ? Array(admin_set.contexts) : []
+  end
+  private_class_method :schema_contexts_for
+
+  ##
+  # @api private
+  #
+  # @param klass [Class]
+  # @param contexts [Array<String>]
+  # @return [Dry::Types::Schema, Array] the resolved schema for the class and contexts
+  def self.resolve_schema(klass, contexts)
+    if contexts.present? && klass.respond_to?(:flexible?) && klass.flexible?
+      klass.new(contexts: contexts).singleton_class.schema || klass.schema
+    else
+      klass.new.singleton_class.schema || klass.schema
+    end
+  end
+  private_class_method :resolve_schema
+
+  ##
+  # @api private
+  #
+  # @param klass [Class]
+  # @return [Dry::Types::Schema, Array] the base schema when admin set lookup fails
+  def self.fallback_schema(klass)
+    klass.new.singleton_class.schema || klass.schema
+  end
+  private_class_method :fallback_schema
 end

data/lib/tasks/m3_profile_cleanup.rake

@@ -0,0 +1,35 @@
+# frozen_string_literal: true
+
+require 'yaml'
+require 'pathname'
+
+namespace :hyrax do
+  namespace :metadata do
+    desc "Clean up display_label attributes in M3 profile YAML file"
+    task :update_labels, [:profile_path] => :environment do |_task, args|
+      # Force i18n to load
+      I18n.t('blacklight.search.fields.show.title_tesim')
+
+      profile_editor = Hyrax::M3ProfileEditor.new(Rails.root.join(args[:profile_path]) || Rails.root.join('config', 'metadata_profiles', 'm3_profile.yaml'))
+
+      profile_editor.profile_data['properties'].each do |property_name, property_data|
+        default_label = case property_data['display_label']
+                        when String
+                          property_data['display_label']
+                        when Hash
+                          property_data['display_label']['default']
+                        else
+                          property_name.humanize
+                        end
+        existing_hash = property_data['view']&.fetch('label', nil)
+        existing_hash = nil if !existing_hash.is_a?(Hash) || existing_hash.keys.size <= 1
+        existing_hash ||= property_data['display_label'] if property_data['display_label'].is_a?(Hash) && property_data['display_label'].keys.size > 1
+
+        profile_editor.profile_data['properties'][property_name]['display_label'] = existing_hash.presence || {}
+        profile_editor.profile_data['properties'][property_name]['display_label']['default'] = profile_editor.find_i18n(default_label)
+        profile_editor.profile_data['properties'][property_name]['view'].delete('label') if profile_editor.profile_data['properties'][property_name]['view'].present?
+      end
+      profile_editor.save
+    end
+  end
+end

data/template.rb

@@ -4,6 +4,9 @@ insert_into_file 'config/application.rb', after: /config\.load_defaults [0-9.]+$
   "\n    config.add_autoload_paths_to_load_path = true"
 end
 
-gem 'hyrax', '5.2.0'
+# In order to test app generation with local code, un-comment the line below and comment out the other gem definition
+# Then run hyrax generation with the flag `-m /path/to/my/local/code/template.rb`
+# gem 'hyrax', path: __dir__
+gem 'hyrax', '5.3.0'
 run 'bundle install'
 generate 'hyrax:install', '-f'