Send string ids into job (#6975)

* Send string ids into job Serialize was failing on valkyrie ids * Fix work factory to not automatically index
samvera · Nov 15, 2024 · abd5955 · abd5955
1 parent a9c8e9c
commit abd5955
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/lib/freyja/persister.rb b/lib/freyja/persister.rb
@@ -34,7 +34,7 @@ def convert_and_migrate_resource(orm_object)
       new_resource = resource_factory.to_resource(object: orm_object)
       if Hyrax.config.valkyrie_transition?
         MigrateFilesToValkyrieJob.perform_later(new_resource) if new_resource.is_a?(Hyrax::FileSet) && new_resource.file_ids.size == 1 && new_resource.file_ids.first.id.to_s.match('/files/')
-        MigrateResourcesJob.perform_later(ids: new_resource.member_ids) if new_resource.is_a?(Hyrax::Work)
+        MigrateResourcesJob.perform_later(ids: new_resource.member_ids.map(&:to_s)) if new_resource.is_a?(Hyrax::Work)
       end
       new_resource
     end

diff --git a/lib/hyrax/specs/shared_specs/factories/hyrax_work.rb b/lib/hyrax/specs/shared_specs/factories/hyrax_work.rb
@@ -107,7 +107,14 @@
       Hyrax::EmbargoManager.new(resource: work).apply
       Hyrax::LeaseManager.new(resource: work).apply
 
-      work.permission_manager.acl.save
+      if evaluator.with_index
+        work.permission_manager.acl.save
+      else
+        # manually save acl change_set so it does not automatically index the work
+        change_set = work.permission_manager.acl.send(:change_set)
+        change_set.sync
+        Hyrax.persister.save(resource: change_set.resource)
+      end
 
       # This has to happen after permissions for permissions to propagate.
       if evaluator.uploaded_files.present?