Merge pull request #6708 from samvera/odds-and-ends

🐛 Odds and Ends

Author: dlpierce

app/indexers/concerns/hyrax/permission_indexer.rb

@@ -13,7 +13,7 @@ module PermissionIndexer
     def to_solr
       super.tap do |index_document|
         config      = Hydra.config.permissions
-        permissions = resource.permission_manager
+        permissions = resource.permission_manager || Hyrax::PermissionManager.new(resource: resource)
 
         index_document[config.edit.group] = permissions.edit_groups.to_a
         index_document[config.edit.individual] = permissions.edit_users.to_a

lib/hyrax/configuration.rb

@@ -1024,7 +1024,7 @@ def translate_uri_to_id
     attr_writer :translate_id_to_uri
     def translate_id_to_uri
       @translate_id_to_uri ||= lambda do |id|
-        "#{ActiveFedora.fedora.host}#{ActiveFedora.fedora.base_path}/#{::Noid::Rails.treeify(id)}"
+        "#{ActiveFedora.fedora.host}#{ActiveFedora.fedora.base_path}/#{::Noid::Rails.treeify(id.to_s)}"
       end
     end
 

lib/wings/valkyrie/query_service.rb

@@ -142,7 +142,7 @@ def find_references_by(resource:, property:, model: nil)
         return find_many_by_ids(ids: Array(resource.send(property))) unless model
 
         results = resource.public_send(property).map do |reference|
-          resource_factory.to_resource(object: ::ActiveFedora::Base.find(reference))
+          resource_factory.to_resource(object: ::ActiveFedora::Base.find(reference.to_s))
         end
 
         results.select { |r| r.class.name == model.name }