From c1adf2e8af005a27de8a6ff0eaa7812424aa2128 Mon Sep 17 00:00:00 2001 From: 6543 <6543@obermui.de> Date: Wed, 13 May 2026 21:23:33 +0200 Subject: [PATCH] saveWorkflowsFromPipelineBuilder: check for pre-loaded workflows (#6474) --- server/pipeline/items.go | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/server/pipeline/items.go b/server/pipeline/items.go index 420c998488..731e1da6fd 100644 --- a/server/pipeline/items.go +++ b/server/pipeline/items.go @@ -210,9 +210,12 @@ func saveWorkflowsFromPipelineBuilder(store store.Store, pipeline *model.Pipelin } } - // the workflows in the pipeline should be empty as only we do populate them, - // but if a pipeline was already loaded form database it might contain things, so we just clean it - pipeline.Workflows = nil + // The workflows in the pipeline should be empty, only we populate them. + // But if a pipeline was already loaded from the database and contains workflows, + // we error out to prevent harm. + if pipeline.Workflows != nil { + return nil, errors.New("cannot save new workflows from pipeline builder: pipeline already has workflows loaded") + } for _, item := range pipelineItems { workflow := &model.Workflow{