bpmn-io / bpmn-js-element-templates

The element template extension for bpmn-js
MIT License
5 stars 3 forks source link

fix(Conditions): evaluate all chained conditions #65

Closed marstamm closed 9 months ago

marstamm commented 9 months ago

closes #49 also fixes https://github.com/camunda/camunda-modeler/issues/4124

Root cause here: https://github.com/camunda/camunda-modeler/issues/4124#issuecomment-1940942140

We will now re-apply conditions and the template until no further changes in conditions are present. This applies all chained conditions.

We need to rename context.oldTemplate to avoid naming conflicts with other behaviors that overwrite the context entry. This will cause an infinite loop, as we need the "Before state with conditions" for a proper check.

nikku commented 9 months ago

@marstamm Let's get this integrate end-to-end so folks can try it out. It seems like this affects a bunch of people already.