Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 488290 - Error boundary event on script task
Summary: Error boundary event on script task
Status: RESOLVED FIXED
Alias: None
Product: BPMN2Modeler
Classification: SOA
Component: Core (show other bugs)
Version: 1.2.3   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: future   Edit
Assignee: Robert Brodt CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-02-23 08:14 EST by Jozef Marko CLA
Modified: 2016-02-23 10:55 EST (History)
0 users

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Jozef Marko CLA 2016-02-23 08:14:03 EST 
It shouldn't be possible to add error boundary event to script task. If is such process validated by jBPM engine, there occurs this error:

java.lang.AssertionError: Error Messages:
"unable to parse xml : Exception class java.lang.IllegalArgumentException : Boundary events are supported only on StateBasedNode, found node: org.jbpm.workflow.core.node.ActionNode [ScriptTask_1]]"

Web designer of BPM Studio doesn't allow adding boundary events to script tasks.
Comment 1 Robert Brodt CLA 2016-02-23 10:35:45 EST 
Jozef, this was already reported here:

https://bugzilla.redhat.com/show_bug.cgi?id=1141619

and I had committed a fix for this bug about 5 weeks ago. Looking back in the Git commit history, I see two commits, 4f7afb9 and eebd706: the first one created the JbpmBoundaryEventFeatureContainer class and added the <featureContainer> entry in plugin.xml which fixed the problem; the second commit removed these changes. I have no idea how or why this happened - I blame it on a git hiccup.

Oddly enough, the exact same thing happened in the master branch with commits b4e13bc and b155e89 and again, no clue how this happened :(

Long story short, I have reapplied these changes to the current HEAD in both mars and master branches.
Comment 2 Robert Brodt CLA 2016-02-23 10:37:49 EST 
Fixed, again...until next time ;)
Comment 3 Jozef Marko CLA 2016-02-23 10:55:43 EST 
Head up Bob, it happens. Thanks for fast fix.