| Summary: | Adhoc subprocess - completion condition does not work when it contains escaped characters | ||
|---|---|---|---|
| Product: | [JBoss] JBoss Enterprise BRMS Platform 5 | Reporter: | Radovan Synek <rsynek> |
| Component: | jBPM 5 | Assignee: | Kris Verlaenen <kverlaen> |
| Status: | VERIFIED --- | QA Contact: | Radovan Synek <rsynek> |
| Severity: | medium | Docs Contact: | |
| Priority: | unspecified | ||
| Version: | BRMS 5.3.0.GA | CC: | atangrin, jlocker, lpetrovi |
| Target Milestone: | --- | ||
| Target Release: | BRMS 5.3.0.GA | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | Bug Fix | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | Type: | --- | |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
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. 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. 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. Update status to ON_QA. Please verify them against ER6. |
Adhoc subprocess does not end when completion condition has following value: <![CDATA[getActivityInstanceAttribute("numberOfActiveInstances") == 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.