From 0a8fa83abc86fddf11804d40f4e204292c587004 Mon Sep 17 00:00:00 2001 From: 6543 <6543@obermui.de> Date: Mon, 20 Apr 2026 13:36:04 +0200 Subject: [PATCH] saveWorkflowsFromPipelineBuilder: check for pre-loaded workflows --- 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 420c998488e..731e1da6fd8 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{