workflows: log status item on external workflow action error (#45652)
This commit is contained in:
parent
02fd08460c
commit
7f155f3a52
|
@ -4891,6 +4891,7 @@ def test_call_external_workflow_with_evolution_linked_object(pub):
|
||||||
logged_error = LoggedError.select()[0]
|
logged_error = LoggedError.select()[0]
|
||||||
assert logged_error.summary == 'Could not find linked "External Form" object by id %s' % external_formdata.id
|
assert logged_error.summary == 'Could not find linked "External Form" object by id %s' % external_formdata.id
|
||||||
assert logged_error.exception_class == 'KeyError'
|
assert logged_error.exception_class == 'KeyError'
|
||||||
|
assert logged_error.status_item_id == action.id
|
||||||
|
|
||||||
# try remove an unexisting carddef: do nothing
|
# try remove an unexisting carddef: do nothing
|
||||||
unused_carddef = CardDef()
|
unused_carddef = CardDef()
|
||||||
|
|
|
@ -144,7 +144,7 @@ class ExternalWorkflowGlobalAction(WorkflowStatusItem):
|
||||||
# use custom error message depending on target type
|
# use custom error message depending on target type
|
||||||
LoggedError.record(_('Could not find linked "%(object_name)s" object by id %(object_id)s') % {
|
LoggedError.record(_('Could not find linked "%(object_name)s" object by id %(object_id)s') % {
|
||||||
'object_name': objectdef.name, 'object_id': target_id},
|
'object_name': objectdef.name, 'object_id': target_id},
|
||||||
formdata=formdata, exception=e)
|
formdata=formdata, status_item=self, exception=e)
|
||||||
|
|
||||||
def get_parameters(self):
|
def get_parameters(self):
|
||||||
return ('slug', 'trigger_id', 'condition')
|
return ('slug', 'trigger_id', 'condition')
|
||||||
|
@ -156,7 +156,8 @@ class ExternalWorkflowGlobalAction(WorkflowStatusItem):
|
||||||
|
|
||||||
trigger = self.get_trigger(objectdef.workflow)
|
trigger = self.get_trigger(objectdef.workflow)
|
||||||
if not trigger:
|
if not trigger:
|
||||||
LoggedError.record(_('No trigger with id "%s" found in workflow') % self.trigger_id)
|
LoggedError.record(_('No trigger with id "%s" found in workflow') % self.trigger_id,
|
||||||
|
formdata=formdata, status_item=self)
|
||||||
return
|
return
|
||||||
|
|
||||||
class CallerSource:
|
class CallerSource:
|
||||||
|
|
Loading…
Reference in New Issue