Bug 1004311

Summary: Multiple connections feature can be switched on be setting jbpm.enable.multi.con=false
Product: [Retired] JBoss BPMS Platform 6 Reporter: Radovan Synek <rsynek>
Component: jBPM CoreAssignee: Maciej Swiderski <mswiders>
Status: CLOSED CURRENTRELEASE QA Contact: Radovan Synek <rsynek>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 6.0.0   
Target Milestone: ER4   
Target Release: 6.0.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-08-06 20:13:24 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Radovan Synek 2013-09-04 11:33:41 UTC
Multiple connections feature is controlled by system property 'jbpm.enable.multi.con', Looking at the source code (e.g. [1] and [2]), this property is being questioned only for existence, not for actual value: 

System.getProperty("jbpm.enable.multi.con") == null

=> multiple connections feature is switched on if the property is defined no matter what value it has, even if the value is "false", which is a non-standard behaviour.

expected behaviour:
jbpm.enable.multi.con=false => multiple connections disabled
jbpm.enable.multi.con=true => multiple connections enabled
jbpm.enable.multi.con undefined => default behaviour - mult. conn. disabled

Version-Release number of selected component (if applicable):
BPMS 6.0.0.ER2

[1] https://github.com/droolsjbpm/jbpm/blob/master/jbpm-flow/src/main/java/org/jbpm/workflow/core/node/ActionNode.java
[2] https://github.com/droolsjbpm/jbpm/blob/master/jbpm-flow/src/main/java/org/jbpm/workflow/core/node/EndNode.java

Comment 1 Maciej Swiderski 2013-09-05 14:48:33 UTC
fixed so only value 'true' will enable multi connection support. Commit on 6.0.x:
https://github.com/droolsjbpm/jbpm/commit/57637e2f77eef30a9ca6e08ad6ae9fce1efdd882

Comment 5 Radovan Synek 2013-10-17 08:03:56 UTC
Verified on BPMS-6.0.0.ER4