From 043273f67c37c11a3fdcbefd492a1a76d5c7d286 Mon Sep 17 00:00:00 2001 From: Valentin Deniaud Date: Tue, 26 Jan 2021 15:34:38 +0100 Subject: [PATCH] wf: remove trigger from jump item attached to global action (#11280) --- tests/admin_pages/test_workflow.py | 1 + wcs/wf/jump.py | 4 +++- 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/tests/admin_pages/test_workflow.py b/tests/admin_pages/test_workflow.py index 2bd50c4e..a0bb5b34 100644 --- a/tests/admin_pages/test_workflow.py +++ b/tests/admin_pages/test_workflow.py @@ -774,6 +774,7 @@ def test_workflows_edit_jump_previous(pub): resp = app.get('/backoffice/workflows/1/global-actions/ac1/items/_jump/') assert 'Previously Marked Status' in [x[2] for x in resp.form['status'].options] + assert 'trigger' not in resp.form.fields jump_global.set_marker_on_status = False workflow.store() diff --git a/wcs/wf/jump.py b/wcs/wf/jump.py index 6cc2ca1d..00dd3c99 100644 --- a/wcs/wf/jump.py +++ b/wcs/wf/jump.py @@ -31,7 +31,7 @@ from ..qommon import errors from ..qommon.publisher import get_publisher_class from ..qommon.cron import CronJob -from wcs.workflows import Workflow, WorkflowStatusJumpItem, register_item_class +from wcs.workflows import Workflow, WorkflowStatusJumpItem, WorkflowGlobalAction, register_item_class from wcs.api import get_user_from_api_query_string, is_url_signed from wcs.conditions import Condition @@ -168,6 +168,8 @@ class JumpWorkflowStatusItem(WorkflowStatusJumpItem): return wf_status[0].name def get_parameters(self): + if hasattr(self, 'parent') and isinstance(self.parent, WorkflowGlobalAction): + return ('status', 'condition', 'by', 'timeout', 'set_marker_on_status') return ('status', 'condition', 'trigger', 'by', 'timeout', 'set_marker_on_status') def add_parameters_widgets(self, form, parameters, prefix='', formdef=None): -- 2.20.1