YAML Parse Failure, Model Continues Processing

I see, where you're coming from. However, this was a deliberate design decision as this yaml parsing is happening all over ECA in this way, not just in cache writing.

I can only see 2 options moving forward, since changing the behavior would break existing models:

• Add a new option to the drop-down to have a choice whether to fail or not for yaml parsing; or a separate checkbox
• Add a condition that validates the yaml before using it in any of the actions

I would actually vote for the second on in any case, and wonder if we then even require the first one.

Changed to feature request since this is operating as intended.

I agree with the second option of validating the YAML before use. This falls in line with programming standards where we want to validate before usage.

This is now implemented with an extra checkbox below the "Use YAML" checkbox when configuring the action. Building this with an extra condition which would be executed before the action would require the YAML data to be provided twice; once for the action and separately also for the condition. This will then not ensure that the same data would be validated or executed.

With the extra checkbox with the action, this really tests the same data, and in ECA 3 with the modern bpmn_ui, that second checkbox only displays on screen when the first one is being enabled.

Please have a look and set to RTBC when this is doing what you need.

According to the issue description it also affects Expirable Key Value Store and Key Value Store. The merge request does not address them, and others either, e.g. Private temporary store: write.

Thanks for noticing this. I don't know what happened, I used text search in my IDE to give me all occurrences of use_yaml and it failed somehow. Anyway, I've now added it to hopefully all instances.

Great suggestions again, I've addressed them all.

Yaml validation works and code looks good:

• If validate yaml is checked and yaml is invalid, processing is stopped
• If validate yaml is checked and yaml is valid, processing continues
• If validate yaml is NOT checked and yaml is invalid, processing is not stopped and a message is logged

The duplicate token replacement is not crucial.

Merged and back ported.