From 99c8f2c9fc6f335863f134a5b8b4055d5079cfe1 Mon Sep 17 00:00:00 2001 From: John Chilton Date: Fri, 31 Mar 2017 14:48:14 -0400 Subject: [PATCH] Fix setting UUID on steps when copying steps from another workflow. It seems like the client is correctly setting the UUID to None, but the backend is treating the None as a UUID because it was (incorrectly IMO) making a distinction between ``uuid`` being absent and being set to ``None``. Fixes #3845. --- lib/galaxy/managers/workflows.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/galaxy/managers/workflows.py b/lib/galaxy/managers/workflows.py index bf99eb8be2f..63e1bb033a0 100644 --- a/lib/galaxy/managers/workflows.py +++ b/lib/galaxy/managers/workflows.py @@ -881,7 +881,7 @@ class WorkflowContentsManager(UsesAnnotations): step = model.WorkflowStep() # TODO: Consider handling position inside module. step.position = step_dict['position'] - if "uuid" in step_dict and step_dict['uuid'] != "None": + if step_dict.get("uuid", None) and step_dict['uuid'] != "None": step.uuid = step_dict["uuid"] if "label" in step_dict: step.label = step_dict["label"]