Move worker call outside transaction

lib/project_copier.rb

@@ -23,7 +23,7 @@ def copy
     # Should this all be wrapped in a transaction?
     # to ensure we rollback an sub resource creations,
     # e.g. inband primary lang strings for the associated resources....
-    Project.transaction(requires_new: true) do
+    copied_project = Project.transaction(requires_new: true) do
       copied_project = copy_project
 
       # save the project and create the project versions for use in translation strings
@@ -36,12 +36,14 @@ def copy
       # to keep the translations system working with these copied resources
       setup_associated_primary_language_translations(copied_project)
 
-      # Creates Talk roles via background worker
-      TalkAdminCreateWorker.perform_async(copied_project.id)
-
-      # return the newly copied project
       copied_project
     end
+
+    # Creates Talk roles via background worker
+    TalkAdminCreateWorker.perform_async(copied_project.id)
+
+    # return the newly copied project
+    copied_project
   end
 
   private