[jboss-user] [jBPM] - connection with constraint and jbpm.enable.multi.con

[rigazilla]

rigazilla [https://community.jboss.org/people/rigazilla] created the discussion

"connection with constraint and jbpm.enable.multi.con"

To view the discussion, visit: https://community.jboss.org/message/809283#809283

--------------------------------------------------------------
Hi all,

I had some troubles with the configuration in the subject above.
Shortly: with jbpm.enable.multi.con=true each node at the end of a connection with constraint is triggered two times.

I suspect this is a bug (my quick patch is below), can someone check if I'm correct?

Thanks,
Vittorio

diff --git a/jbpm-flow/src/main/java/org/jbpm/workflow/instance/impl/NodeInstanceImpl.java b/jbpm-flow/src/main/java/org/jbpm/workflow/instance/impl/NodeInstanceImpl.ja
index 274c291..25b53c7 100644
--- a/jbpm-flow/src/main/java/org/jbpm/workflow/instance/impl/NodeInstanceImpl.java
+++ b/jbpm-flow/src/main/java/org/jbpm/workflow/instance/impl/NodeInstanceImpl.java
@@ -201,7 +201,8 @@ public abstract class NodeInstanceImpl implements org.jbpm.workflow.instance.Nod
                 }
                 if ( !found ) {
                     throw new IllegalArgumentException( "Uncontrolled flow node could not find at least one valid outgoing connection " + getNode().getName() );
-                }    
+                }
+                return;
                } else {
                        connections = node.getOutgoingConnections(type);
                }
--------------------------------------------------------------

Reply to this message by going to Community
[https://community.jboss.org/message/809283#809283]

Start a new discussion in jBPM at Community
[https://community.jboss.org/choose-container!input.jspa?contentType=1&containerType=14&container=2034]

[Maciej Swiderski]

Maciej Swiderski [https://community.jboss.org/people/swiderski.maciej] created the discussion

"Re: connection with constraint and jbpm.enable.multi.con"

To view the discussion, visit: https://community.jboss.org/message/809508#809508

--------------------------------------------------------------
Vittorio, could you provide a process and test case for this? Indeed it could be a bug but would like to verify this before pushing the patch.

Thanks
--------------------------------------------------------------

Reply to this message by going to Community
[https://community.jboss.org/message/809508#809508]

Start a new discussion in jBPM at Community
[https://community.jboss.org/choose-container!input.jspa?contentType=1&containerType=14&container=2034]

[Maciej Swiderski]

Maciej Swiderski [https://community.jboss.org/people/swiderski.maciej] created the discussion

"Re: connection with constraint and jbpm.enable.multi.con"

To view the discussion, visit: https://community.jboss.org/message/810074#810074

--------------------------------------------------------------
thanks a lot Vittorio, you were right and it was bug indeed (JBPM-3991), fixed it and pushed to master.

Cheers
Maciej
--------------------------------------------------------------

Reply to this message by going to Community
[https://community.jboss.org/message/810074#810074]

Start a new discussion in jBPM at Community
[https://community.jboss.org/choose-container!input.jspa?contentType=1&containerType=14&container=2034]