hyrax 5.1.0 → 5.3.0

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

@@ -0,0 +1,41 @@
+# frozen_string_literal: true
+
+module Hyrax
+  module Test
+    ##
+    # A generic PCDM Work, with only Hyrax "core" (required) metadata.
+    #
+    # @example building with FactoryBot
+    #   work = FactoryBot.build(:hyrax_work, :public, title: ['Comet in Moominland'])
+    #
+    # @example creating with FactoryBot
+    #   work = FactoryBot.valkyrie_create(:hyrax_work, :public, title: ['Comet in Moominland'])
+    class SimpleWork < Hyrax::Work
+      if Hyrax.config.work_include_metadata?
+        include Hyrax::Schema(:core_metadata)
+        include Hyrax::Schema(:basic_metadata)
+        include Hyrax::Schema(:monograph)
+      end
+    end
+
+    class SimpleWorkLegacy < ActiveFedora::Base
+      include WorkBehavior
+      include CoreMetadata
+    end
+
+    class SimpleWorkSearchBuilder < Hyrax::WorkSearchBuilder
+      def work_types
+        [Hyrax::Test::SimpleWorkLegacy]
+      end
+    end
+  end
+end
+
+if defined?(Wings)
+  Wings::ModelRegistry.register(Hyrax::Test::SimpleWork, Hyrax::Test::SimpleWorkLegacy)
+elsif defined?(ActiveFedora) && ENV.key?('FCREPO_BASE_PATH') && Hyrax.config.valkyrie_transition
+  # We do not want to add the lazy migration for ActiveFedora to Valkyrie when we don't have a valid
+  # Fedora end-point.  Now what is the best way to see if we have a valid and configured fedora
+  # connection?
+  Hyrax::ValkyrieLazyMigration.migrating(Hyrax::Test::SimpleWork, from: Hyrax::Test::SimpleWorkLegacy)
+end

data/lib/hyrax/specs/shared_specs.rb

@@ -4,3 +4,4 @@ require 'hyrax/specs/shared_specs/indexers'
 require 'hyrax/specs/shared_specs/metadata'
 require 'hyrax/specs/shared_specs/hydra_works'
 require 'hyrax/specs/shared_specs/workflow_method'
+require 'hyrax/specs/shared_specs/simple_work'

data/lib/hyrax/transactions/collection_create.rb

@@ -13,7 +13,8 @@ module Hyrax
                        'change_set.add_to_collections',
                        'change_set.apply',
                        'collection_resource.apply_collection_type_permissions',
-                       'collection_resource.save_acl'].freeze
+                       'collection_resource.save_acl',
+                       'collection_resource.sync_redirect_paths'].freeze
 
       ##
       # @see Hyrax::Transactions::Transaction

data/lib/hyrax/transactions/collection_destroy.rb

@@ -11,6 +11,7 @@ module Hyrax
       # TODO: Add step that checks if collection is empty for collections of types that require it
       DEFAULT_STEPS = ['collection_resource.delete_acl',
                        'collection_resource.remove_from_membership',
+                       'collection_resource.remove_redirect_paths',
                        'collection_resource.delete',
                        'collection_resource.delete_permission_template'].freeze.freeze
 

data/lib/hyrax/transactions/collection_update.rb

@@ -11,7 +11,8 @@ module Hyrax
       DEFAULT_STEPS = ['change_set.apply',
                        'collection_resource.save_collection_banner',
                        'collection_resource.save_collection_logo',
-                       'collection_resource.save_acl'].freeze
+                       'collection_resource.save_acl',
+                       'collection_resource.sync_redirect_paths'].freeze
 
       ##
       # @see Hyrax::Transactions::Transaction

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/add_file_sets.rb

@@ -20,10 +20,11 @@ module Hyrax
         ##
         # @param [Hyrax::Work] obj
         # @param [Enumerable<UploadedFile>] uploaded_files
-        # @param [Enumerable<Hash>] file_set_params
+        # @param [Enumerable<Hash>] file_set_params or nil
         #
         # @return [Dry::Monads::Result]
         def call(obj, uploaded_files: [], file_set_params: [])
+          return Success(obj) if uploaded_files.empty? && file_set_params.blank? # Skip if no files to attach
           if @handler.new(work: obj).add(files: uploaded_files, file_set_params: file_set_params).attach
             file_sets = obj.member_ids.map do |member|
               Hyrax.query_service.find_by(id: member) if Hyrax.query_service.find_by(id: member).is_a? Hyrax::FileSet

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.1.0'
+  VERSION = '5.3.0'
 end