Bug 807677 - Adhoc subprocess - completion condition does not work when it contains escaped characters
Summary: Adhoc subprocess - completion condition does not work when it contains escape...
Keywords:
Status: VERIFIED
Alias: None
Product: JBoss Enterprise BRMS Platform 5
Classification: JBoss
Component: jBPM 5
Version: BRMS 5.3.0.GA
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: ---
: BRMS 5.3.0.GA
Assignee: Kris Verlaenen
QA Contact: Radovan Synek
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-03-28 13:29 UTC by Radovan Synek
Modified: 2021-04-09 12:48 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed:
Type: ---

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

Description Radovan Synek 2012-03-28 13:29:06 UTC 
Adhoc subprocess does not end when completion condition has following value:
<![CDATA[getActivityInstanceAttribute(&quot;numberOfActiveInstances&quot;) == 0]]>
which is legal output from Guvnor BPMN designer (it does XML escaping when you save or export process definition).
On the other hand, the process ends as expected when the condition is:
<![CDATA[getActivityInstanceAttribute("numberOfActiveInstances") == 0]]>

Reproducer is within commit https://github.com/rsynek/jbpm/commit/32fc4cda86d947e1e2c05aa82d2b7e9e36ecbcd8
in pull request https://github.com/droolsjbpm/jbpm/pull/56.
Comment 1 Jiri Locker 2012-04-10 13:38:10 UTC 
I don't think it is a legal output. Since the condition is enclosed by CDATA section, it should not escape XML special characters. To me it looks more like a designer bug.
Comment 2 Tihomir Surdilovic 2012-04-11 14:06:30 UTC 
This issue seems to already have been fixed but we are testing with an older war. What gets generated with latest build from designer master or 2.1.x branch for the issue is not present.
Comment 3 Lukáš Petrovický 2012-04-12 12:10:06 UTC 
I see that this has 5.3.0-. However, since it's already fixed, it should have 5.3.0+. This is just a formality, everyone please approve.
Comment 4 Ryan Zhang 2012-04-23 07:35:48 UTC 
Update status to ON_QA. Please verify them against ER6.
Note You need to log in before you can comment on or make changes to this bug.