valkyrie 3.3.0 → 3.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3bd57af32d07a78e2ac9a9a1c6217b8b601018413c1a08a67762565f92da91cd
-  data.tar.gz: 58e0073f9ef70694e77d016cf08869d90b920f2fca3a7c42e7141289a8b7121d
+  metadata.gz: 7ba113a4df1fa120076006223d629be9d186f96736fbb363781bc78f6c038013
+  data.tar.gz: 89e485a082def7836f884354b136c081496b5a9f36e98c2f9f31127dc053ab28
 SHA512:
-  metadata.gz: 939dbb151e2cb770e9bfbac0cedbfc78a160ace0f3f1c482560659f1a6cd16ad0984ae1eb15caf49d4ef5926f3a01721e032f7c8363ea815e935064eb547f7c0
-  data.tar.gz: d2b295e1881cc0d8d45e1c69e6255fab7b5be4d3e277d3e01eaa53498ca7346131ac1c2180d73f675c93b9b4a03d73316b9f6a8256b6b40a7492dda8c088d16a
+  metadata.gz: 3d3537379fb136ec2ca843e2dd13d85a22c46a9c74d94f1ca3cfb1ab32f4a193394ab1ff9deae649939e3260b0ba9e9d8f423f3fb87d40043d107ff646fb74bc
+  data.tar.gz: 4a908d4d2574d5017bc81af1683d714b028b5d1ab17e086ff0f8ea6d0a764348f3efd3f308aeedac4623956ccfce915a5c5d611ab84d7e9e2d08c22d5afe623a

data/.circleci/config.yml

@@ -9,7 +9,7 @@ jobs:
         type: "string"
       ruby:
         description: "Ruby version"
-        default: "2.7.7"
+        default: "3.1.6"
         type: "string"
     docker:
       - image: cimg/ruby:<< parameters.ruby >>
@@ -28,7 +28,7 @@ jobs:
         environment:
           CATALINA_OPTS: "-Djava.awt.headless=true -Dfile.encoding=UTF-8 -server -Xms512m -Xmx1024m -XX:NewSize=256m -XX:MaxNewSize=256m -XX:PermSize=256m -XX:MaxPermSize=256m -XX:+DisableExplicitGC"
           JAVA_OPTIONS: "-Djetty.http.port=8998 -Dfcrepo.dynamic.jms.port=61618 -Dfcrepo.dynamic.stomp.port=61614"
-      - image: fcrepo/fcrepo:6.4.0
+      - image: fcrepo/fcrepo:6.5.1-RC3-tomcat9
         environment:
           CATALINA_OPTS: "-Djava.awt.headless=true -Dfile.encoding=UTF-8 -server -Xms512m -Xmx1024m -XX:NewSize=256m -XX:MaxNewSize=256m -XX:PermSize=256m -XX:MaxPermSize=256m -XX:+DisableExplicitGC -Dorg.apache.tomcat.util.buf.UDecoder.ALLOW_ENCODED_SLASH=true"
           JAVA_OPTS: "-Djetty.http.port=8978 -Dfcrepo.dynamic.jms.port=61619 -Dfcrepo.dynamic.stomp.port=61615 -Dorg.apache.tomcat.util.buf.UDecoder.ALLOW_ENCODED_SLASH=true"
@@ -96,57 +96,45 @@ workflows:
   build:
     jobs:
       - build:
-          gemfile: "gemfiles/activerecord_7_0.gemfile"
-          ruby: 3.2.0
-          name: "Ruby3-2_rails7-0"
+          gemfile: "gemfiles/activerecord_7_2.gemfile"
+          ruby: 3.3.5
+          name: "Ruby3-3_rails7-2"
       - build:
-          gemfile: "gemfiles/activerecord_6_0.gemfile"
-          ruby: 3.2.0
-          name: "Ruby3-2_Rails6-0"
+          gemfile: "gemfiles/activerecord_7_1.gemfile"
+          ruby: 3.3.5
+          name: "Ruby3-3_rails7-1"
       - build:
           gemfile: "gemfiles/activerecord_7_0.gemfile"
-          ruby: 3.1.3
-          name: "Ruby3-1_rails7-0"
+          ruby: 3.3.5
+          name: "Ruby3-3_rails7-0"
       - build:
-          gemfile: "gemfiles/activerecord_6_0.gemfile"
-          ruby: 3.1.3
-          name: "Ruby3-1_Rails6-0"
+          gemfile: "gemfiles/activerecord_7_2.gemfile"
+          ruby: 3.2.5
+          name: "Ruby3-2_rails7-2"
+      - build:
+          gemfile: "gemfiles/activerecord_7_1.gemfile"
+          ruby: 3.2.5
+          name: "Ruby3-2_rails7-1"
       - build:
           gemfile: "gemfiles/activerecord_7_0.gemfile"
-          ruby: 3.0.5
-          name: "Ruby3-0_rails7-0"
+          ruby: 3.2.5
+          name: "Ruby3-2_rails7-0"
       - build:
-          gemfile: "gemfiles/activerecord_6_0.gemfile"
-          ruby: 3.0.5
-          name: "Ruby3-0_Rails6-0"
+          gemfile: "gemfiles/activerecord_7_2.gemfile"
+          ruby: 3.1.6
+          name: "Ruby3-1_rails7-2"
       - build:
-          gemfile: "gemfiles/activerecord_7_0.gemfile"
-          ruby: 2.7.7
-          name: "Ruby2-7_rails7-0"
+          gemfile: "gemfiles/activerecord_7_1.gemfile"
+          ruby: 3.1.6
+          name: "Ruby3-1_rails7-1"
       - build:
-          gemfile: "gemfiles/activerecord_6_0.gemfile"
-          ruby: 2.7.7
-          name: "Ruby2-7_Rails6-0"
+          gemfile: "gemfiles/activerecord_7_0.gemfile"
+          ruby: 3.1.6
+          name: "Ruby3-1_rails7-0"
       - build:
           gemfile: "gemfiles/faraday_1.gemfile"
-          ruby: 2.7.7
+          ruby: 3.1.6
           name: "Faraday1"
-      - build:
-          gemfile: "gemfiles/faraday_0.gemfile"
-          ruby: 2.7.7
-          name: "Faraday0"
-      - build:
-          gemfile: "gemfiles/activerecord_6_0.gemfile"
-          ruby: 2.6.10
-          name: "Ruby2-6_Rails6-0"
-      - build:
-          gemfile: "gemfiles/activerecord_5_2.gemfile"
-          ruby: 2.7.7
-          name: "Ruby2-7_Rails5-2"
-      - build:
-          gemfile: "gemfiles/activerecord_5_2.gemfile"
-          ruby: 2.6.10
-          name: "Ruby2-6_Rails5-2"
 
   nightly:
     triggers:
@@ -158,54 +146,42 @@ workflows:
                 - master
     jobs:
       - build:
-          gemfile: "gemfiles/activerecord_7_0.gemfile"
-          ruby: 3.2.0
-          name: "Ruby3-2_rails7-0"
+          gemfile: "gemfiles/activerecord_7_2.gemfile"
+          ruby: 3.3.5
+          name: "Ruby3-3_rails7-2"
       - build:
-          gemfile: "gemfiles/activerecord_6_0.gemfile"
-          ruby: 3.2.0
-          name: "Ruby3-2_Rails6-0"
+          gemfile: "gemfiles/activerecord_7_1.gemfile"
+          ruby: 3.3.5
+          name: "Ruby3-3_rails7-1"
       - build:
           gemfile: "gemfiles/activerecord_7_0.gemfile"
-          ruby: 3.1.3
-          name: "Ruby3-1_rails7-0"
+          ruby: 3.3.5
+          name: "Ruby3-3_rails7-0"
       - build:
-          gemfile: "gemfiles/activerecord_6_0.gemfile"
-          ruby: 3.1.3
-          name: "Ruby3-1_Rails6-0"
+          gemfile: "gemfiles/activerecord_7_2.gemfile"
+          ruby: 3.2.5
+          name: "Ruby3-2_rails7-2"
+      - build:
+          gemfile: "gemfiles/activerecord_7_1.gemfile"
+          ruby: 3.2.5
+          name: "Ruby3-2_rails7-1"
       - build:
           gemfile: "gemfiles/activerecord_7_0.gemfile"
-          ruby: 3.0.5
-          name: "Ruby3-0_rails7-0"
+          ruby: 3.2.5
+          name: "Ruby3-2_rails7-0"
       - build:
-          gemfile: "gemfiles/activerecord_6_0.gemfile"
-          ruby: 3.0.5
-          name: "Ruby3-0_Rails6-0"
+          gemfile: "gemfiles/activerecord_7_2.gemfile"
+          ruby: 3.1.6
+          name: "Ruby3-1_rails7-2"
       - build:
-          gemfile: "gemfiles/activerecord_7_0.gemfile"
-          ruby: 2.7.7
-          name: "Ruby2-7_rails7-0"
+          gemfile: "gemfiles/activerecord_7_1.gemfile"
+          ruby: 3.1.6
+          name: "Ruby3-1_rails7-1"
       - build:
-          gemfile: "gemfiles/activerecord_6_0.gemfile"
-          ruby: 2.7.7
-          name: "Ruby2-7_Rails6-0"
+          gemfile: "gemfiles/activerecord_7_0.gemfile"
+          ruby: 3.1.6
+          name: "Ruby3-1_rails7-0"
       - build:
           gemfile: "gemfiles/faraday_1.gemfile"
-          ruby: 2.7.7
+          ruby: 3.1.6
           name: "Faraday1"
-      - build:
-          gemfile: "gemfiles/faraday_0.gemfile"
-          ruby: 2.7.7
-          name: "Faraday0"
-      - build:
-          gemfile: "gemfiles/activerecord_6_0.gemfile"
-          ruby: 2.6.10
-          name: "Ruby2-6_Rails6-0"
-      - build:
-          gemfile: "gemfiles/activerecord_5_2.gemfile"
-          ruby: 2.7.7
-          name: "Ruby2-7_Rails5-2"
-      - build:
-          gemfile: "gemfiles/activerecord_5_2.gemfile"
-          ruby: 2.6.10
-          name: "Ruby2-6_Rails5-2"

data/.lando.yml

@@ -51,7 +51,7 @@ services:
     volumes:
       fedora6:
     services:
-      image: fcrepo/fcrepo:6.4.0
+      image: fcrepo/fcrepo:6.5.1-RC3-tomcat9
       command:
         - "catalina.sh"
         - "run"

data/.tool-versions

@@ -1 +1 @@
-ruby 2.7.5
+ruby 3.3.5

data/Appraisals

@@ -1,23 +1,18 @@
 # frozen_string_literal: true
 
-appraise "activerecord-7-0" do
-  gem "activerecord", "~> 7.0.0"
-end
-
-appraise "activerecord-6-0" do
-  gem "activerecord", "~> 6.0.0"
+appraise "activerecord-7-2" do
+  gem "activerecord", "~> 7.2.0"
 end
 
-appraise "activerecord-5-2" do
-  gem "activerecord", "~> 5.2.0"
+appraise "activerecord-7-1" do
+  gem "activerecord", "~> 7.1.0"
 end
 
-appraise "faraday-0" do
-  gem "faraday", "< 1"
-  gem "activerecord", "~> 6.0.0"
+appraise "activerecord-7-0" do
+  gem "activerecord", "~> 7.0.0"
 end
 
 appraise "faraday-1" do
   gem "faraday", "~> 1.0"
-  gem "activerecord", "~> 6.0.0"
+  gem "activerecord", "~> 7.0.0"
 end

data/CHANGELOG.md

@@ -1,3 +1,20 @@
+# v3.5.0 2024-12-10
+
+* Provides new boolean configuration setting `auto_cast_iso8601_as_datetime`  ([dchandekstark](https://github.com/dchandekstark))
+* Align behavior of server managed properties with Fedora. ([dlpierce](https://github.com/dlpierce))
+* Check for nil id to avoid InvalidURIError ([dlpierce](https://github.com/dlpierce))
+* Support Fedora 6.5 ([dlpierce](https://github.com/dlpierce))
+* Make storage adapters report their protocol string ([dlpierce](https://github.com/dlpierce))
+
+# v3.4.0 2024-10-11
+
+## Changes since last release
+
+* Adds config setting index_tsim_only_threshold ([dchandekstark](https://github.com/dchandekstark))
+* Fix related URIs for pairtree paths in Fedora. ([randalldfloyd](https://github.com/randalldfloyd))
+* Return nil when #id_to_uri is passed empty strings. ([bwatson78](https://github.com/bwatson78))
+* Add support for Ruby 3.3 and Rails 7.2 ([cjcolvar](https://github.com/cjcolvar))
+
 # v3.3.0 2024-07-03
 
 ## Changes since last release

data/Rakefile

@@ -68,10 +68,14 @@ namespace :db do
       ActiveRecord::Migrator.migrate(MIGRATIONS_DIR, version) do |migration|
         scope.blank? || scope == migration.scope
       end
-    else
+    elsif ActiveRecord.version < Gem::Version.new("7.2.0")
       ActiveRecord::Base.connection.migration_context.migrate(version) do |migration|
         scope.blank? || scope == migration.scope
       end
+    else
+      ActiveRecord::MigrationContext.new(ActiveRecord::Migrator.migrations_paths).migrate(version) do |migration|
+        scope.blank? || scope == migration.scope
+      end
     end
     ActiveRecord::Base.clear_cache!
   ensure

data/gemfiles/{activerecord_5_2.gemfile → activerecord_7_1.gemfile}

@@ -3,7 +3,7 @@
 
 source "https://rubygems.org"
 
-gem "activerecord", "~> 5.2.0"
+gem "activerecord", "~> 7.1.0"
 gem "ldp"
 gem "pg"
 gem "rsolr"

data/gemfiles/{activerecord_6_0.gemfile → activerecord_7_2.gemfile}

@@ -3,7 +3,7 @@
 
 source "https://rubygems.org"
 
-gem "activerecord", "~> 6.0.0"
+gem "activerecord", "~> 7.2.0"
 gem "ldp"
 gem "pg"
 gem "rsolr"

data/gemfiles/faraday_1.gemfile

@@ -3,7 +3,7 @@
 
 source "https://rubygems.org"
 
-gem "activerecord", "~> 6.0.0"
+gem "activerecord", "~> 7.0.0"
 gem "ldp"
 gem "pg"
 gem "rsolr"

data/lib/valkyrie/persistence/fedora/metadata_adapter.rb

@@ -64,6 +64,7 @@ module Valkyrie::Persistence::Fedora
     # @param [RDF::URI] id the Valkyrie ID
     # @return [RDF::URI]
     def id_to_uri(id)
+      return if id.to_s.empty?
       prefix = ""
       prefix = "#{pair_path(id)}/" if fedora_version == 4 || (fedora_version >= 6.5 && (pairtree_count * pairtree_length).positive?)
       RDF::URI("#{connection_prefix}/#{prefix}#{CGI.escape(id.to_s)}")

data/lib/valkyrie/persistence/fedora/persister/orm_converter.rb

@@ -105,7 +105,8 @@ module Valkyrie::Persistence::Fedora
           # @param [Property] value
           # @return [Boolean]
           def self.handles?(value)
-            value.statement.object.to_s.start_with?("http://www.w3.org/ns/ldp", "http://fedora.info")
+            value.statement.predicate == RDF.type &&
+              value.statement.object.to_s.start_with?('http://www.w3.org/ns/ldp#', 'http://fedora.info/definitions/v4/repository#')
           end
 
           # Provide the NullApplicator Class for any Property in a deny listed namespace
@@ -562,7 +563,7 @@ module Valkyrie::Persistence::Fedora
           # @return [Array<String>]
           def denylist
             [
-              "http://fedora.info/definitions",
+              "http://fedora.info/definitions/v4/repository#",
               "http://www.iana.org/assignments/relation/last"
             ]
           end

data/lib/valkyrie/persistence/fedora/query_service.rb

@@ -52,7 +52,7 @@ module Valkyrie::Persistence::Fedora
     # @return [Array<RDF::URI>]
     def include_uris
       [
-        [5, 6].include?(adapter.fedora_version) ? "http://fedora.info/definitions/fcrepo#PreferInboundReferences" : ::RDF::Vocab::Fcrepo4.InboundReferences
+        adapter.fedora_version >= 5 ? "http://fedora.info/definitions/fcrepo#PreferInboundReferences" : ::RDF::Vocab::Fcrepo4.InboundReferences
       ]
     end
 
@@ -162,6 +162,7 @@ module Valkyrie::Persistence::Fedora
     # @return [Valkyrie::Resource]
     # @raise [Valkyrie::Persistence::ObjectNotFoundError]
     def resource_from_uri(uri)
+      raise ::Valkyrie::Persistence::ObjectNotFoundError if uri.nil?
       resource = Ldp::Resource.for(connection, uri, connection.get(uri))
       resource_factory.to_resource(object: resource)
     rescue ::Ldp::Gone, ::Ldp::NotFound

data/lib/valkyrie/persistence/shared/json_value_mapper.rb

@@ -139,6 +139,7 @@ module Valkyrie::Persistence::Shared
       # @return [Boolean]
       # rubocop:disable Metrics/CyclomaticComplexity
       def self.handles?(value)
+        return false unless ::Valkyrie.config.auto_cast_iso8601_as_datetime
         return false unless value.is_a?(String)
         return false unless value[4] == "-" && value[10] == "T"
         year = value.to_s[0..3]

data/lib/valkyrie/persistence/solr/model_converter.rb

@@ -422,7 +422,7 @@ module Valkyrie::Persistence::Solr
       # @see https://github.com/samvera-labs/valkyrie/blob/main/solr/config/schema.xml
       # @return [Array<Symbol>]
       def fields
-        if value.value.length > 1000
+        if value.value.length > Valkyrie.config.index_tsim_only_threshold
           [:tsim]
         else
           [:tsim, :ssim, :tesim, :tsi, :ssi, :tesi]

data/lib/valkyrie/specs/shared_specs/queries.rb

@@ -86,8 +86,13 @@ RSpec.shared_examples 'a Valkyrie query provider' do
       expect { query_service.find_by(id: Valkyrie::ID.new("123123123")) }.to raise_error ::Valkyrie::Persistence::ObjectNotFoundError
     end
 
+    it 'returns a Valkyrie::Persistence::ObjectNotFoundError for an empty string' do
+      expect { query_service.find_by(id: '') }.to raise_error ::Valkyrie::Persistence::ObjectNotFoundError
+    end
+
     it 'raises an error if the id is not a Valkyrie::ID or a string' do
       expect { query_service.find_by(id: 123) }.to raise_error ArgumentError
+      expect { query_service.find_by(id: nil) }.to raise_error ArgumentError
     end
   end
 
@@ -117,8 +122,13 @@ RSpec.shared_examples 'a Valkyrie query provider' do
       expect { query_service.find_by_alternate_identifier(alternate_identifier: Valkyrie::ID.new("123123123")) }.to raise_error ::Valkyrie::Persistence::ObjectNotFoundError
     end
 
+    it "raises a Valkyrie::Persistence::ObjectNotFoundError for a non-found alternate identifier" do
+      expect { query_service.find_by_alternate_identifier(alternate_identifier: '') }.to raise_error ::Valkyrie::Persistence::ObjectNotFoundError
+    end
+
     it 'raises an error if the alternate identifier is not a Valkyrie::ID or a string' do
       expect { query_service.find_by_alternate_identifier(alternate_identifier: 123) }.to raise_error ArgumentError
+      expect { query_service.find_by_alternate_identifier(alternate_identifier: nil) }.to raise_error ArgumentError
     end
 
     it 'can have multiple alternate identifiers' do

data/lib/valkyrie/specs/shared_specs/storage_adapter.rb

@@ -12,6 +12,7 @@ RSpec.shared_examples 'a Valkyrie::StorageAdapter' do
     Valkyrie::Specs.send(:remove_const, :CustomResource)
   end
   subject { storage_adapter }
+  it { is_expected.to respond_to(:protocol) }
   it { is_expected.to respond_to(:handles?).with_keywords(:id) }
   it { is_expected.to respond_to(:find_by).with_keywords(:id) }
   it { is_expected.to respond_to(:delete).with_keywords(:id) }
@@ -74,8 +75,9 @@ RSpec.shared_examples 'a Valkyrie::StorageAdapter' do
     resource = Valkyrie::Specs::CustomResource.new(id: "test#{SecureRandom.uuid}")
     sha1 = Digest::SHA1.file(file).to_s
     size = file.size
-    expect(uploaded_file = storage_adapter.upload(file: file, original_filename: 'foo.jpg', resource: resource, fake_upload_argument: true)).to be_kind_of Valkyrie::StorageAdapter::File
+    uploaded_file = storage_adapter.upload(file: file, original_filename: 'foo.jpg', resource: resource, fake_upload_argument: true)
 
+    expect(uploaded_file).to be_kind_of Valkyrie::StorageAdapter::File
     expect(uploaded_file).to respond_to(:checksum).with_keywords(:digests)
     expect(uploaded_file).to respond_to(:valid?).with_keywords(:size, :digests)
     expect(uploaded_file.checksum(digests: [Digest::SHA1.new])).to eq([sha1])
@@ -133,28 +135,42 @@ RSpec.shared_examples 'a Valkyrie::StorageAdapter' do
     # Deleting a version should leave the current versions
     if storage_adapter.supports?(:version_deletion)
       storage_adapter.delete(id: uploaded_file.version_id)
-      expect(storage_adapter.find_versions(id: uploaded_file.id).length).to eq 1
+      remnants = storage_adapter.find_versions(id: uploaded_file.id)
+      expect(remnants.length).to eq 1
+      expect(remnants.first.version_id).to eq new_version.version_id
       expect { storage_adapter.find_by(id: uploaded_file.version_id) }.to raise_error Valkyrie::StorageAdapter::FileNotFound
     end
     current_length = storage_adapter.find_versions(id: new_version.id).length
 
     # Restoring a previous version is just pumping its file into upload_version
     newest_version = storage_adapter.upload_version(file: new_version, id: new_version.id)
+    current_length += 1
     expect(newest_version.version_id).not_to eq new_version.id
     expect(storage_adapter.find_by(id: newest_version.id).version_id).to eq newest_version.version_id
 
     # I can restore a version twice
     newest_version = storage_adapter.upload_version(file: new_version, id: new_version.id)
+    current_length += 1
     expect(newest_version.version_id).not_to eq new_version.id
     expect(storage_adapter.find_by(id: newest_version.id).version_id).to eq newest_version.version_id
-    expect(storage_adapter.find_versions(id: newest_version.id).length).to eq current_length + 2
+    expect(storage_adapter.find_versions(id: newest_version.id).length).to eq current_length
+
+    # Fedora 6.5 may not create versions when the timestamp is the same?
+    # See: https://fedora-repository.atlassian.net/browse/FCREPO-3958
+    sleep 1 if storage_adapter.supports?(:list_deleted_versions)
 
     # NOTE: We originally wanted deleting the current record to push it into the
     # versions history, but FCRepo 4/5/6 doesn't work that way, so we changed to
     # instead make deleting delete everything.
     storage_adapter.delete(id: new_version.id)
     expect { storage_adapter.find_by(id: new_version.id) }.to raise_error Valkyrie::StorageAdapter::FileNotFound
-    expect(storage_adapter.find_versions(id: new_version.id).length).to eq 0
+
+    if storage_adapter.supports?(:list_deleted_versions)
+      expect(storage_adapter.find_versions(id: new_version.id).length).to eq current_length
+    else
+      expect(storage_adapter.find_versions(id: new_version.id).length).to eq 0
+    end
+
   ensure
     f&.close
   end

data/lib/valkyrie/storage/disk.rb

@@ -3,6 +3,8 @@ module Valkyrie::Storage
   # Implements the DataMapper Pattern to store binary data on disk
   class Disk
     attr_reader :base_path, :path_generator, :file_mover
+    PROTOCOL = 'disk://'
+
     def initialize(base_path:, path_generator: BucketedStorage, file_mover: FileUtils.method(:mv))
       @base_path = Pathname.new(base_path.to_s)
       @path_generator = path_generator.new(base_path: base_path)
@@ -18,13 +20,13 @@ module Valkyrie::Storage
       new_path = path_generator.generate(resource: resource, file: file, original_filename: original_filename)
       FileUtils.mkdir_p(new_path.parent)
       file_mover.call(file.path, new_path)
-      find_by(id: Valkyrie::ID.new("disk://#{new_path}"))
+      find_by(id: Valkyrie::ID.new("#{protocol}#{new_path}"))
     end
 
     # @param id [Valkyrie::ID]
     # @return [Boolean] true if this adapter can handle this type of identifer
     def handles?(id:)
-      id.to_s.start_with?("disk://#{base_path}")
+      id.to_s.start_with?("#{protocol}#{base_path}")
     end
 
     # @param feature [Symbol] Feature to test for.
@@ -33,8 +35,13 @@ module Valkyrie::Storage
       false
     end
 
+    # @return [String] identifier prefix
+    def protocol
+      PROTOCOL
+    end
+
     def file_path(id)
-      id.to_s.gsub(/^disk:\/\//, '')
+      id.to_s.gsub(/^#{Regexp.escape(protocol)}/, '')
     end
 
     # Return the file associated with the given identifier

data/lib/valkyrie/storage/fedora.rb

@@ -27,7 +27,7 @@ module Valkyrie::Storage
     # @param id [Valkyrie::ID]
     # @return [Boolean] true if this adapter can handle this type of identifer
     def handles?(id:)
-      id.to_s.start_with?(PROTOCOL)
+      id.to_s.start_with?(protocol)
     end
 
     # @param feature [Symbol] Feature to test for.
@@ -36,9 +36,16 @@ module Valkyrie::Storage
       return true if feature == :versions
       # Fedora 6 auto versions and you can't delete versions.
       return true if feature == :version_deletion && fedora_version < 6
+      # Fedora 6.5+ lists versions for deleted objects
+      return true if feature == :list_deleted_versions && fedora_version >= 6.5
       false
     end
 
+    # @return [String] identifier prefix
+    def protocol
+      PROTOCOL
+    end
+
     # Return the file associated with the given identifier
     # @param id [Valkyrie::ID]
     # @return [Valkyrie::StorageAdapter::StreamFile]
@@ -61,7 +68,7 @@ module Valkyrie::Storage
       # Fedora 6 auto versions, so check to see if there's a version for this
       # initial upload. If not, then mint one (fedora 4/5)
       version_id = current_version_id(id: valkyrie_identifier(uri: identifier)) || mint_version(identifier, latest_version(identifier))
-      perform_find(id: Valkyrie::ID.new(identifier.to_s.sub(/^.+\/\//, PROTOCOL)), version_id: version_id)
+      perform_find(id: Valkyrie::ID.new(identifier.to_s.sub(/^.+\/\//, protocol)), version_id: version_id)
     end
 
     # @param id [Valkyrie::ID] ID of the Valkyrie::StorageAdapter::StreamFile to
@@ -77,7 +84,7 @@ module Valkyrie::Storage
       end
       upload_file(fedora_uri: uri, io: file)
       version_id = mint_version(uri, latest_version(uri))
-      perform_find(id: Valkyrie::ID.new(uri.to_s.sub(/^.+\/\//, PROTOCOL)), version_id: version_id)
+      perform_find(id: Valkyrie::ID.new(uri.to_s.sub(/^.+\/\//, protocol)), version_id: version_id)
     end
 
     # @param id [Valkyrie::ID]
@@ -88,7 +95,9 @@ module Valkyrie::Storage
       version_list.map do |version|
         id = valkyrie_identifier(uri: version["@id"])
         perform_find(id: id, version_id: id)
-      end
+      rescue Valkyrie::StorageAdapter::FileNotFound
+        nil
+      end.compact
     end
 
     # Delete the file in Fedora associated with the given identifier.
@@ -197,12 +206,12 @@ module Valkyrie::Storage
     # Translate the Valkrie ID into a URL for the fedora file
     # @return [RDF::URI]
     def fedora_identifier(id:)
-      identifier = id.to_s.sub(PROTOCOL, "#{connection.http.scheme}://")
+      identifier = id.to_s.sub(protocol, "#{connection.http.scheme}://")
       RDF::URI(identifier)
     end
 
     def valkyrie_identifier(uri:)
-      id = uri.to_s.sub("http://", PROTOCOL)
+      id = uri.to_s.sub("http://", protocol)
       Valkyrie::ID.new(id)
     end
 
@@ -211,7 +220,7 @@ module Valkyrie::Storage
     # @return [IOProxy]
     def response(id:)
       response = connection.http.get(fedora_identifier(id: id))
-      raise Valkyrie::StorageAdapter::FileNotFound unless response.success?
+      raise Valkyrie::StorageAdapter::FileNotFound, "HTTP #{response.status} #{response.body}" unless response.success?
       IOProxy.new(response.body)
     end
 

data/lib/valkyrie/storage/memory.rb

@@ -6,6 +6,8 @@ module Valkyrie::Storage
   #   in cases where you want to preserve real data
   class Memory
     attr_reader :cache
+    PROTOCOL = 'memory://'
+
     def initialize
       @cache = {}
     end
@@ -16,7 +18,7 @@ module Valkyrie::Storage
     # @param _extra_arguments [Hash] additional arguments which may be passed to other adapters
     # @return [Valkyrie::StorageAdapter::StreamFile]
     def upload(file:, original_filename:, resource: nil, **_extra_arguments)
-      identifier = Valkyrie::ID.new("memory://#{resource.id}")
+      identifier = Valkyrie::ID.new("#{protocol}#{resource.id}")
       version_id = Valkyrie::ID.new("#{identifier}##{SecureRandom.uuid}")
       cache[identifier] ||= {}
       cache[identifier][:current] = Valkyrie::StorageAdapter::StreamFile.new(id: identifier, io: file, version_id: version_id)
@@ -67,7 +69,7 @@ module Valkyrie::Storage
     # @param id [Valkyrie::ID]
     # @return [Boolean] true if this adapter can handle this type of identifer
     def handles?(id:)
-      id.to_s.start_with?("memory://")
+      id.to_s.start_with?(protocol)
     end
 
     # @param feature [Symbol] Feature to test for.
@@ -83,6 +85,11 @@ module Valkyrie::Storage
       end
     end
 
+    # @return [String] identifier prefix
+    def protocol
+      PROTOCOL
+    end
+
     def id_and_version(id)
       id, version = id.to_s.split("#")
       [Valkyrie::ID.new(id), version]

data/lib/valkyrie/storage/versioned_disk.rb

@@ -7,6 +7,8 @@ module Valkyrie::Storage
   # with "deletionmarker" in the name of the file.
   class VersionedDisk
     attr_reader :base_path, :path_generator, :file_mover
+    PROTOCOL = 'versiondisk://'
+
     def initialize(base_path:, path_generator: ::Valkyrie::Storage::Disk::BucketedStorage, file_mover: FileUtils.method(:cp))
       @base_path = Pathname.new(base_path.to_s)
       @path_generator = path_generator.new(base_path: base_path)
@@ -26,7 +28,7 @@ module Valkyrie::Storage
       return sleep(0.001) && upload(file: file, original_filename: original_filename, resource: resource, paused: true, **extra_arguments) if !paused && File.exist?(new_path)
       FileUtils.mkdir_p(new_path.parent)
       file_mover.call(file.try(:path) || file.try(:disk_path), new_path)
-      find_by(id: Valkyrie::ID.new("versiondisk://#{new_path}"))
+      find_by(id: Valkyrie::ID.new("#{protocol}#{new_path}"))
     end
 
     def current_timestamp
@@ -50,13 +52,13 @@ module Valkyrie::Storage
       return sleep(0.001) && upload_version(id: id, file: file, paused: true) if !paused && File.exist?(new_path)
       FileUtils.mkdir_p(new_path.parent)
       file_mover.call(file.try(:path) || file.try(:disk_path), new_path)
-      find_by(id: Valkyrie::ID.new("versiondisk://#{new_path}"))
+      find_by(id: Valkyrie::ID.new("#{protocol}#{new_path}"))
     end
 
     # @param id [Valkyrie::ID]
     # @return [Boolean] true if this adapter can handle this type of identifer
     def handles?(id:)
-      id.to_s.start_with?("versiondisk://#{base_path}")
+      id.to_s.start_with?("#{protocol}#{base_path}")
     end
 
     # @param feature [Symbol] Feature to test for.
@@ -66,6 +68,11 @@ module Valkyrie::Storage
       false
     end
 
+    # @return [String] identifier prefix
+    def protocol
+      PROTOCOL
+    end
+
     # Return the file associated with the given identifier
     # @param id [Valkyrie::ID]
     # @return [Valkyrie::StorageAdapter::File]
@@ -95,7 +102,7 @@ module Valkyrie::Storage
     # @return [Array<Valkyrie::StorageAdapter::File>]
     def find_versions(id:)
       version_files(id: id).select { |x| !x.to_s.include?("deletionmarker") }.map do |file|
-        find_by(id: Valkyrie::ID.new("versiondisk://#{file}"))
+        find_by(id: Valkyrie::ID.new("#{protocol}#{file}"))
       end
     end
 
@@ -106,7 +113,7 @@ module Valkyrie::Storage
     end
 
     def file_path(version_id)
-      version_id.to_s.gsub(/^versiondisk:\/\//, '')
+      version_id.to_s.gsub(/^#{Regexp.escape(protocol)}/, '')
     end
 
     # @return VersionId A VersionId value that's resolved a current reference,
@@ -128,6 +135,10 @@ module Valkyrie::Storage
         @id = id
       end
 
+      def protocol
+        PROTOCOL
+      end
+
       def current_reference_id
         self.class.new(Valkyrie::ID.new(string_id.gsub(version, "current")))
       end
@@ -139,13 +150,13 @@ module Valkyrie::Storage
       end
 
       def file_path
-        @file_path ||= string_id.gsub(/^versiondisk:\/\//, '')
+        @file_path ||= string_id.gsub(/^#{Regexp.escape(protocol)}/, '')
       end
 
       def version_files
         root = Pathname.new(file_path)
         root.parent.children.select { |file| file.basename.to_s.end_with?(filename) }.sort.reverse.map do |file|
-          VersionId.new(Valkyrie::ID.new("versiondisk://#{file}"))
+          VersionId.new(Valkyrie::ID.new("#{protocol}#{file}"))
         end
       end
 

data/lib/valkyrie/version.rb

@@ -1,4 +1,4 @@
 # frozen_string_literal: true
 module Valkyrie
-  VERSION = "3.3.0"
+  VERSION = "3.5.0"
 end

data/lib/valkyrie.rb

@@ -97,6 +97,14 @@ module Valkyrie
       Valkyrie::StorageAdapter.find(self[:storage_adapter].to_sym)
     end
 
+    def index_tsim_only_threshold
+      self[:index_tsim_only_threshold].to_i
+    end
+
+    def auto_cast_iso8601_as_datetime
+      self[:auto_cast_iso8601_as_datetime]
+    end
+
     # @api public
     #
     # The returned anonymous method (e.g. responds to #call) has a signature of
@@ -118,7 +126,9 @@ module Valkyrie
 
     def defaults
       {
-        resource_class_resolver: method(:default_resource_class_resolver)
+        resource_class_resolver: method(:default_resource_class_resolver),
+        index_tsim_only_threshold: 1000,
+        auto_cast_iso8601_as_datetime: true
       }
     end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: valkyrie
 version: !ruby/object:Gem::Version
-  version: 3.3.0
+  version: 3.5.0
 platform: ruby
 authors:
 - Trey Pendragon
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2024-07-08 00:00:00.000000000 Z
+date: 2024-12-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: dry-struct
@@ -452,10 +452,9 @@ files:
 - db/migrate/20180212092225_create_updated_at_index.rb
 - db/migrate/20180802220739_add_optimistic_locking_to_orm_resources.rb
 - db/schema.rb
-- gemfiles/activerecord_5_2.gemfile
-- gemfiles/activerecord_6_0.gemfile
 - gemfiles/activerecord_7_0.gemfile
-- gemfiles/faraday_0.gemfile
+- gemfiles/activerecord_7_1.gemfile
+- gemfiles/activerecord_7_2.gemfile
 - gemfiles/faraday_1.gemfile
 - lib/valkyrie.rb
 - lib/valkyrie/adapter_container.rb
@@ -582,7 +581,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.6
+rubygems_version: 3.5.16
 signing_key:
 specification_version: 4
 summary: An ORM using the Data Mapper pattern, specifically built to solve Digital

data/gemfiles/faraday_0.gemfile

@@ -1,12 +0,0 @@
-# frozen_string_literal: true
-# This file was generated by Appraisal
-
-source "https://rubygems.org"
-
-gem "activerecord", "~> 6.0.0"
-gem "ldp"
-gem "pg"
-gem "rsolr"
-gem "faraday", "< 1"
-
-gemspec path: "../"