Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Define called element statically or as expression #3792

Closed
saig0 opened this issue Feb 7, 2020 · 0 comments · Fixed by #3859
Closed

Define called element statically or as expression #3792

saig0 opened this issue Feb 7, 2020 · 0 comments · Fixed by #3859
Assignees
Labels
kind/feature Categorizes an issue or PR as a feature, i.e. new behavior scope/broker Marks an issue or PR to appear in the broker section of the changelog

Comments

@saig0
Copy link
Member

saig0 commented Feb 7, 2020

Is your feature request related to a problem? Please describe.
For a call activity, I can define the processId of the called element statically as a string or dynamically as an expression.

Describe the solution you'd like

  • the processId of the called element is defined in one attribute in the BPMN XML
  • the attribute contains either a static value (e.g. processId) or an expression (e.g. ${processId})
  • the expression is a valid FEEL expression
  • the expression is evaluated when the element is activated
  • the expression can access the variable context of the element
  • if the evaluation fails (e.g. no variable with the given name) or the result is not a string then an incident is raised
  • if the expression is not valid the deployment is rejected

Describe alternatives you've considered

  • different attributes for a static value and an expression -> more effort when applying to more attributes since it is not a unify type/structure
  • additional attribute to mark a value as an expression -> not possible if an element contains multiple attributes (e.g. task definition)
  • additional attribute/element to override a static value with an expression -> too complex

Additional context

  • the described solution for the call activity is also a blueprint for the other attributes
  • the expression support for the processId was already implemented and will be changed to match the requirements
@saig0 saig0 added Status: Ready kind/feature Categorizes an issue or PR as a feature, i.e. new behavior scope/broker Marks an issue or PR to appear in the broker section of the changelog labels Feb 7, 2020
@saig0 saig0 added this to the Advanced expression supported milestone Feb 7, 2020
@saig0 saig0 self-assigned this Feb 7, 2020
@zeebe-bors zeebe-bors bot closed this as completed in 3d265f0 Feb 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Categorizes an issue or PR as a feature, i.e. new behavior scope/broker Marks an issue or PR to appear in the broker section of the changelog
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants