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.

Sign up for GitHub

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

Jump to bottom

ConditionalEventDefinition created with out condition leads to invalid BPMN 2.0 XML #526

Closed
1 task done
nikku opened this issue Feb 9, 2017 · 2 comments
Closed
1 task done

ConditionalEventDefinition created with out condition leads to invalid BPMN 2.0 XML #526

nikku opened this issue Feb 9, 2017 · 2 comments
Labels
BPMN bug Something isn't working modeling
Milestone
M16

Comments

@nikku
Copy link
Member

nikku commented Feb 9, 2017 •
edited by philippfromme

bpmn:ConditionalEventDefinition is only valid with an attached condition. Right now, we do not attach it on creation and produce invalid XML (according to the BPMN 2.0 XML schema.

Expected

  • We attach an empty bpmn:FormalExpression element so that the generated BPMN 2.0 XML is schema compliant
@nikku nikku added this to the Backlog milestone Feb 9, 2017
@nikku nikku added BPMN bug Something isn't working modeling labels Feb 9, 2017
@nikku nikku changed the title Creating ConditionalEventDefinition should attach condition ConditionalEventDefinition created with out condition leads to invalid BPMN 2.0 XML Feb 9, 2017
@nikku nikku mentioned this issue Jan 3, 2018
Closed
@nikku
Copy link
Member Author

nikku commented Jan 3, 2018

Will be addressed via bpmn-io/bpmn-js#747.

@nikku nikku modified the milestones: Backlog, M16 Jan 3, 2018
@philippfromme
Copy link
Contributor

Closed via bpmn-io/bpmn-js@1f5c69a.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
BPMN bug Something isn't working modeling
Projects
None yet
Milestone
M16
Development

No branches or pull requests
2 participants
@nikku @philippfromme