bzinchenko / bpmnview

Viewing and printing business process diagrams in a popular BPMN format
MIT License
92 stars 31 forks source link

Adding subprocess as triggeredByEvent="true" #10

Closed hakanseysane closed 6 years ago

hakanseysane commented 6 years ago

Hello,

how do I add a subprocess as triggeredByEvent attribute is true?

bzinchenko commented 6 years ago

You can read rules for this on page 416 of official BPMN specification by OMG.

hakanseysane commented 6 years ago

Thank you sir.

I read rules from official BPMN specification but I don't understand how to add a subprocess as triggeredByEvent is true or isExpanded is true.

bzinchenko commented 6 years ago

According to specification, we ve made it as follows:

if ((activityType == ActivityType.Process || activityType == ActivityType.EventSubProcess) && (marker & ActivityMarker.Collapsed) != 0) subProc.triggeredByEvent.Value = false;

We hope it clarifies. If you see it contradicting to specification, please let us know.

bzinchenko commented 6 years ago

isExpanded is a property of shape, not of a process. Please inspect shape properties for relevant field.