fix(api): remove default payload when it's not on root node (#3928)

Signed-off-by: Benjamin Coenen <benjamin.coenen@corp.ovh.com>
ovh · Feb 12, 2019 · dee8abd · dee8abd
1 parent f7ab12e
commit dee8abd
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/engine/api/workflow/dao_node.go b/engine/api/workflow/dao_node.go
@@ -97,7 +97,9 @@ func insertNode(db gorp.SqlExecutor, store cache.Store, w *sdk.Workflow, n *sdk.
 	if n.Name != w.WorkflowData.Node.Name && n.Context.DefaultPayload != nil {
 		defaultPayloadMap, err := n.Context.DefaultPayloadToMap()
 		if err == nil && len(defaultPayloadMap) > 0 {
-			return sdk.WrapError(sdk.ErrInvalidNodeDefaultPayload, "payload on node %s", n.Name)
+			log.Error("%v", sdk.WrapError(sdk.ErrInvalidNodeDefaultPayload, "payload on node %s", n.Name))
+			// TODO: return error when all migrations are done
+			n.Context.DefaultPayload = nil
 		}
 		if err != nil {
 			n.Context.DefaultPayload = nil