Merge 227668f into bde504b

sul-dlss · May 10, 2019 · 88f0943 · 88f0943
2 parents bde504b + 227668f
commit 88f0943
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/app/services/registration_service.rb b/app/services/registration_service.rb
@@ -114,14 +114,16 @@ def register_object(params = {})
       build_desc_metadata_from_label(new_item, label) if metadata_source == 'label'
       RefreshMetadataAction.run(new_item) if seed_desc_metadata
 
-      workflow_priority = params[:workflow_priority] ? params[:workflow_priority].to_i : 0
-      initiate_workflow(workflows: Array(params[:initiate_workflow]), item: new_item, priority: workflow_priority)
-
       new_item.class.ancestors.select { |x| x.respond_to?(:to_class_uri) && x != ActiveFedora::Base }.each do |parent_class|
         new_item.add_relationship(:has_model, parent_class.to_class_uri)
       end
 
       new_item.save
+
+      # Once we save, then it's safe to start any workflows. Otherwise there could be a race condition
+      workflow_priority = params[:workflow_priority] ? params[:workflow_priority].to_i : 0
+      initiate_workflow(workflows: Array(params[:initiate_workflow]), item: new_item, priority: workflow_priority)
+
       new_item
     end