2:17 p.m.
Bug in GroupActivity.execute(ExecutionImpl execution)
-----------------------------------------------------
Key: JBPM-2487
URL: https://jira.jboss.org/jira/browse/JBPM-2487
Project: jBPM
Issue Type: Bug
Security Level: Public (Everyone can see)
Components: Runtime Engine
Affects Versions: jBPM 4.0
Reporter: Michael Wohlfart
just browsing through the codebase and trying to understand the magic,
I came across this in line 56 of GroupActivity.java:
ExecutionImpl concurrentRoot = null;
if (Execution.STATE_ACTIVE_ROOT.equals(execution.getState())) {
concurrentRoot = execution;
} else if (Execution.STATE_ACTIVE_ROOT.equals(execution.getState())) {
concurrentRoot = execution.getParent();
} else {
throw new JbpmException("illegal state");
}
I assume getState() doesn't have any side effects, so I guess this is a bug
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
6:25 a.m.
New subject: [JBoss JIRA] Commented: (JBPM-2487) Bug in GroupActivity.execute(ExecutionImpl execution)
[
https://jira.jboss.org/jira/browse/JBPM-2487?page=com.atlassian.jira.plug...
]
Ronald van Kuijk commented on JBPM-2487:
----------------------------------------
Looks like it. GroupActivity is under development and therefor might indeed contain
'issues', but thanks for reporting
Bug in GroupActivity.execute(ExecutionImpl execution)
-----------------------------------------------------
Key: JBPM-2487
URL: https://jira.jboss.org/jira/browse/JBPM-2487
Project: jBPM
Issue Type: Bug
Security Level: Public(Everyone can see)
Components: Runtime Engine
Affects Versions: jBPM 4.0
Reporter: Michael Wohlfart
just browsing through the codebase and trying to understand the magic,
I came across this in line 56 of GroupActivity.java:
ExecutionImpl concurrentRoot = null;
if (Execution.STATE_ACTIVE_ROOT.equals(execution.getState())) {
concurrentRoot = execution;
} else if (Execution.STATE_ACTIVE_ROOT.equals(execution.getState())) {
concurrentRoot = execution.getParent();
} else {
throw new JbpmException("illegal state");
}
I assume getState() doesn't have any side effects, so I guess this is a bug
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
4:18 a.m.
New subject: [JBoss JIRA] Updated: (JBPM-2487) Bug in GroupActivity.execute(ExecutionImpl execution)
[
https://jira.jboss.org/jira/browse/JBPM-2487?page=com.atlassian.jira.plug...
]
Tom Baeyens updated JBPM-2487:
------------------------------
Priority: Optional (was: Major)
Bug in GroupActivity.execute(ExecutionImpl execution)
-----------------------------------------------------
Key: JBPM-2487
URL: https://jira.jboss.org/jira/browse/JBPM-2487
Project: jBPM
Issue Type: Bug
Security Level: Public(Everyone can see)
Components: Runtime Engine
Affects Versions: jBPM 4.0
Reporter: Michael Wohlfart
Assignee: Tom Baeyens
Priority: Optional
Fix For: jBPM 4.2
just browsing through the codebase and trying to understand the magic,
I came across this in line 56 of GroupActivity.java:
ExecutionImpl concurrentRoot = null;
if (Execution.STATE_ACTIVE_ROOT.equals(execution.getState())) {
concurrentRoot = execution;
} else if (Execution.STATE_ACTIVE_ROOT.equals(execution.getState())) {
concurrentRoot = execution.getParent();
} else {
throw new JbpmException("illegal state");
}
I assume getState() doesn't have any side effects, so I guess this is a bug
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
4:40 a.m.
New subject: [JBoss JIRA] Updated: (JBPM-2487) Bug in GroupActivity.execute(ExecutionImpl execution)
[
https://jira.jboss.org/jira/browse/JBPM-2487?page=com.atlassian.jira.plug...
]
Tom Baeyens updated JBPM-2487:
------------------------------
Original Estimate: 4 hours
Remaining Estimate: 4 hours
Bug in GroupActivity.execute(ExecutionImpl execution)
-----------------------------------------------------
Key: JBPM-2487
URL: https://jira.jboss.org/jira/browse/JBPM-2487
Project: jBPM
Issue Type: Bug
Security Level: Public(Everyone can see)
Components: Runtime Engine
Affects Versions: jBPM 4.0
Reporter: Michael Wohlfart
Assignee: Tom Baeyens
Priority: Minor
Fix For: jBPM 4.2
Original Estimate: 4 hours
Remaining Estimate: 4 hours
just browsing through the codebase and trying to understand the magic,
I came across this in line 56 of GroupActivity.java:
ExecutionImpl concurrentRoot = null;
if (Execution.STATE_ACTIVE_ROOT.equals(execution.getState())) {
concurrentRoot = execution;
} else if (Execution.STATE_ACTIVE_ROOT.equals(execution.getState())) {
concurrentRoot = execution.getParent();
} else {
throw new JbpmException("illegal state");
}
I assume getState() doesn't have any side effects, so I guess this is a bug
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
4:40 a.m.
New subject: [JBoss JIRA] Updated: (JBPM-2487) Bug in GroupActivity.execute(ExecutionImpl execution)
[
https://jira.jboss.org/jira/browse/JBPM-2487?page=com.atlassian.jira.plug...
]
Tom Baeyens updated JBPM-2487:
------------------------------
Priority: Minor (was: Optional)
Bug in GroupActivity.execute(ExecutionImpl execution)
-----------------------------------------------------
Key: JBPM-2487
URL: https://jira.jboss.org/jira/browse/JBPM-2487
Project: jBPM
Issue Type: Bug
Security Level: Public(Everyone can see)
Components: Runtime Engine
Affects Versions: jBPM 4.0
Reporter: Michael Wohlfart
Assignee: Tom Baeyens
Priority: Minor
Fix For: jBPM 4.2
just browsing through the codebase and trying to understand the magic,
I came across this in line 56 of GroupActivity.java:
ExecutionImpl concurrentRoot = null;
if (Execution.STATE_ACTIVE_ROOT.equals(execution.getState())) {
concurrentRoot = execution;
} else if (Execution.STATE_ACTIVE_ROOT.equals(execution.getState())) {
concurrentRoot = execution.getParent();
} else {
throw new JbpmException("illegal state");
}
I assume getState() doesn't have any side effects, so I guess this is a bug
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
11:59 a.m.
New subject: [JBoss JIRA] Updated: (JBPM-2487) Bug in GroupActivity.execute(ExecutionImpl execution)
[
https://jira.jboss.org/jira/browse/JBPM-2487?page=com.atlassian.jira.plug...
]
Scott Peterson updated JBPM-2487:
---------------------------------
Attachment: process.jpdl.xml
Unit test jpdl
Bug in GroupActivity.execute(ExecutionImpl execution)
-----------------------------------------------------
Key: JBPM-2487
URL: https://jira.jboss.org/jira/browse/JBPM-2487
Project: jBPM
Issue Type: Bug
Security Level: Public(Everyone can see)
Components: Runtime Engine
Affects Versions: jBPM 4.0
Reporter: Michael Wohlfart
Assignee: Tom Baeyens
Priority: Minor
Fix For: jBPM 4.2
Attachments: ForkGroupReverseTest.java, process.jpdl.xml
Original Estimate: 4 hours
Remaining Estimate: 4 hours
just browsing through the codebase and trying to understand the magic,
I came across this in line 56 of GroupActivity.java:
ExecutionImpl concurrentRoot = null;
if (Execution.STATE_ACTIVE_ROOT.equals(execution.getState())) {
concurrentRoot = execution;
} else if (Execution.STATE_ACTIVE_ROOT.equals(execution.getState())) {
concurrentRoot = execution.getParent();
} else {
throw new JbpmException("illegal state");
}
I assume getState() doesn't have any side effects, so I guess this is a bug
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
11:59 a.m.
New subject: [JBoss JIRA] Updated: (JBPM-2487) Bug in GroupActivity.execute(ExecutionImpl execution)
[
https://jira.jboss.org/jira/browse/JBPM-2487?page=com.atlassian.jira.plug...
]
Scott Peterson updated JBPM-2487:
---------------------------------
Attachment: ForkGroupReverseTest.java
Unit test
Bug in GroupActivity.execute(ExecutionImpl execution)
-----------------------------------------------------
Key: JBPM-2487
URL: https://jira.jboss.org/jira/browse/JBPM-2487
Project: jBPM
Issue Type: Bug
Security Level: Public(Everyone can see)
Components: Runtime Engine
Affects Versions: jBPM 4.0
Reporter: Michael Wohlfart
Assignee: Tom Baeyens
Priority: Minor
Fix For: jBPM 4.2
Attachments: ForkGroupReverseTest.java, process.jpdl.xml
Original Estimate: 4 hours
Remaining Estimate: 4 hours
just browsing through the codebase and trying to understand the magic,
I came across this in line 56 of GroupActivity.java:
ExecutionImpl concurrentRoot = null;
if (Execution.STATE_ACTIVE_ROOT.equals(execution.getState())) {
concurrentRoot = execution;
} else if (Execution.STATE_ACTIVE_ROOT.equals(execution.getState())) {
concurrentRoot = execution.getParent();
} else {
throw new JbpmException("illegal state");
}
I assume getState() doesn't have any side effects, so I guess this is a bug
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
11:59 a.m.
New subject: [JBoss JIRA] Commented: (JBPM-2487) Bug in GroupActivity.execute(ExecutionImpl execution)
[
https://jira.jboss.org/jira/browse/JBPM-2487?page=com.atlassian.jira.plug...
]
Scott Peterson commented on JBPM-2487:
--------------------------------------
I believe this bug is causing an "illegal state exception" in the attached unit
test.
Bug in GroupActivity.execute(ExecutionImpl execution)
-----------------------------------------------------
Key: JBPM-2487
URL: https://jira.jboss.org/jira/browse/JBPM-2487
Project: jBPM
Issue Type: Bug
Security Level: Public(Everyone can see)
Components: Runtime Engine
Affects Versions: jBPM 4.0
Reporter: Michael Wohlfart
Assignee: Tom Baeyens
Priority: Minor
Fix For: jBPM 4.2
Attachments: ForkGroupReverseTest.java, process.jpdl.xml
Original Estimate: 4 hours
Remaining Estimate: 4 hours
just browsing through the codebase and trying to understand the magic,
I came across this in line 56 of GroupActivity.java:
ExecutionImpl concurrentRoot = null;
if (Execution.STATE_ACTIVE_ROOT.equals(execution.getState())) {
concurrentRoot = execution;
} else if (Execution.STATE_ACTIVE_ROOT.equals(execution.getState())) {
concurrentRoot = execution.getParent();
} else {
throw new JbpmException("illegal state");
}
I assume getState() doesn't have any side effects, so I guess this is a bug
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
12:07 p.m.
New subject: [JBoss JIRA] Commented: (JBPM-2487) Bug in GroupActivity.execute(ExecutionImpl execution)
[
https://jira.jboss.org/jira/browse/JBPM-2487?page=com.atlassian.jira.plug...
]
Scott Peterson commented on JBPM-2487:
--------------------------------------
Sorry, I forgot to attach the stack trace:
12:06:13,262 SEV | [BaseJbpmTestCase] TEST THROWS EXCEPTION: illegal state
org.jbpm.api.JbpmException: illegal state
at org.jbpm.jpdl.internal.activity.GroupActivity.execute(GroupActivity.java:63)
at org.jbpm.jpdl.internal.activity.GroupActivity.execute(GroupActivity.java:45)
at org.jbpm.pvm.internal.model.op.ExecuteActivity.perform(ExecuteActivity.java:60)
at
org.jbpm.pvm.internal.model.ExecutionImpl.performAtomicOperationSync(ExecutionImpl.java:637)
at org.jbpm.pvm.internal.model.ExecutionImpl.fire(ExecutionImpl.java:547)
at org.jbpm.pvm.internal.model.ExecutionImpl.take(ExecutionImpl.java:453)
at org.jbpm.jpdl.internal.activity.ForkActivity.execute(ForkActivity.java:87)
at org.jbpm.jpdl.internal.activity.ForkActivity.execute(ForkActivity.java:43)
at org.jbpm.pvm.internal.model.op.ExecuteActivity.perform(ExecuteActivity.java:60)
at
org.jbpm.pvm.internal.model.ExecutionImpl.performAtomicOperationSync(ExecutionImpl.java:637)
at
org.jbpm.pvm.internal.model.ExecutionImpl.performAtomicOperation(ExecutionImpl.java:597)
at org.jbpm.pvm.internal.model.ExecutionImpl.start(ExecutionImpl.java:201)
at
org.jbpm.pvm.internal.cmd.StartProcessInstanceInLatestCmd.execute(StartProcessInstanceInLatestCmd.java:65)
at
org.jbpm.pvm.internal.cmd.StartProcessInstanceInLatestCmd.execute(StartProcessInstanceInLatestCmd.java:38)
at
org.jbpm.pvm.internal.svc.DefaultCommandService.execute(DefaultCommandService.java:42)
at
org.jbpm.pvm.internal.tx.StandardTransactionInterceptor.execute(StandardTransactionInterceptor.java:54)
at
org.jbpm.pvm.internal.svc.EnvironmentInterceptor.execute(EnvironmentInterceptor.java:46)
at org.jbpm.pvm.internal.svc.RetryInterceptor.execute(RetryInterceptor.java:55)
at
org.jbpm.pvm.internal.svc.ExecutionServiceImpl.startProcessInstanceByKey(ExecutionServiceImpl.java:66)
at reverse.forkgroup.ForkGroupReverseTest.testGroupReverse(ForkGroupReverseTest.java:29)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:79)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:618)
at junit.framework.TestCase.runTest(TestCase.java:154)
at org.jbpm.test.BaseJbpmTestCase.runTest(BaseJbpmTestCase.java:80)
at junit.framework.TestCase.runBare(TestCase.java:127)
at junit.framework.TestResult$1.protect(TestResult.java:106)
at junit.framework.TestResult.runProtected(TestResult.java:124)
at junit.framework.TestResult.run(TestResult.java:109)
at junit.framework.TestCase.run(TestCase.java:118)
at junit.framework.TestSuite.runTest(TestSuite.java:208)
at junit.framework.TestSuite.run(TestSuite.java:203)
at
org.eclipse.jdt.internal.junit.runner.junit3.JUnit3TestReference.run(JUnit3TestReference.java:128)
at org.eclipse.jdt.internal.junit.runner.TestExecution.run(TestExecution.java:38)
at
org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.runTests(RemoteTestRunner.java:460)
at
org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.runTests(RemoteTestRunner.java:673)
at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.run(RemoteTestRunner.java:386)
at
org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.main(RemoteTestRunner.java:196)
Bug in GroupActivity.execute(ExecutionImpl execution)
-----------------------------------------------------
Key: JBPM-2487
URL: https://jira.jboss.org/jira/browse/JBPM-2487
Project: jBPM
Issue Type: Bug
Security Level: Public(Everyone can see)
Components: Runtime Engine
Affects Versions: jBPM 4.0
Reporter: Michael Wohlfart
Assignee: Tom Baeyens
Priority: Minor
Fix For: jBPM 4.2
Attachments: ForkGroupReverseTest.java, process.jpdl.xml
Original Estimate: 4 hours
Remaining Estimate: 4 hours
just browsing through the codebase and trying to understand the magic,
I came across this in line 56 of GroupActivity.java:
ExecutionImpl concurrentRoot = null;
if (Execution.STATE_ACTIVE_ROOT.equals(execution.getState())) {
concurrentRoot = execution;
} else if (Execution.STATE_ACTIVE_ROOT.equals(execution.getState())) {
concurrentRoot = execution.getParent();
} else {
throw new JbpmException("illegal state");
}
I assume getState() doesn't have any side effects, so I guess this is a bug
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
9:34 a.m.
New subject: [JBoss JIRA] Commented: (JBPM-2487) Bug in GroupActivity.execute(ExecutionImpl execution)
[
https://jira.jboss.org/jira/browse/JBPM-2487?page=com.atlassian.jira.plug...
]
Scott Peterson commented on JBPM-2487:
--------------------------------------
I got this working by adding start/end tags to the group. I suspect the real problem was
that I didn't understand how groups work -- without a <start> tag inside a
group, all states without an incoming transition become active at once.
Working jpdl:
<?xml version="1.0" encoding="UTF-8"?>
<process name="GroupReverse" xmlns="http://jbpm.org/4.0/jpdl">
<start>
<transition to="first"/>
</start>
<state name="first">
<transition to="second-group"/>
</state>
<group name="second-group">
<start>
<transition to="second"/>
</start>
<state name="second">
<transition name="end" to="second-group-end"/>
<transition name="third" to="third"/>
</state>
<state name="third">
<transition name="end" to="second-group-end"/>
<transition name="second" to="second"/>
</state>
<end name="second-group-end">
<transition to="end"/>
</end>
</group>
<end name="end"/>
</process>
Bug in GroupActivity.execute(ExecutionImpl execution)
-----------------------------------------------------
Key: JBPM-2487
URL: https://jira.jboss.org/jira/browse/JBPM-2487
Project: jBPM
Issue Type: Bug
Security Level: Public(Everyone can see)
Components: Runtime Engine
Affects Versions: jBPM 4.0
Reporter: Michael Wohlfart
Assignee: Tom Baeyens
Priority: Minor
Fix For: jBPM 4.2
Attachments: ForkGroupReverseTest.java, process.jpdl.xml
Original Estimate: 4 hours
Remaining Estimate: 4 hours
just browsing through the codebase and trying to understand the magic,
I came across this in line 56 of GroupActivity.java:
ExecutionImpl concurrentRoot = null;
if (Execution.STATE_ACTIVE_ROOT.equals(execution.getState())) {
concurrentRoot = execution;
} else if (Execution.STATE_ACTIVE_ROOT.equals(execution.getState())) {
concurrentRoot = execution.getParent();
} else {
throw new JbpmException("illegal state");
}
I assume getState() doesn't have any side effects, so I guess this is a bug
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
4:22 a.m.
New subject: [JBoss JIRA] Commented: (JBPM-2487) Bug in GroupActivity.execute(ExecutionImpl execution)
[
https://jira.jboss.org/jira/browse/JBPM-2487?page=com.atlassian.jira.plug...
]
Ronald van Kuijk commented on JBPM-2487:
----------------------------------------
do you use 4.0 or 4.1? There have been many changes in the group afaik. Best even to try
the latest from svn
Bug in GroupActivity.execute(ExecutionImpl execution)
-----------------------------------------------------
Key: JBPM-2487
URL: https://jira.jboss.org/jira/browse/JBPM-2487
Project: jBPM
Issue Type: Bug
Security Level: Public(Everyone can see)
Components: Runtime Engine
Affects Versions: jBPM 4.0
Reporter: Michael Wohlfart
Assignee: Tom Baeyens
Priority: Minor
Fix For: jBPM 4.2
Attachments: ForkGroupReverseTest.java, process.jpdl.xml
Original Estimate: 4 hours
Remaining Estimate: 4 hours
just browsing through the codebase and trying to understand the magic,
I came across this in line 56 of GroupActivity.java:
ExecutionImpl concurrentRoot = null;
if (Execution.STATE_ACTIVE_ROOT.equals(execution.getState())) {
concurrentRoot = execution;
} else if (Execution.STATE_ACTIVE_ROOT.equals(execution.getState())) {
concurrentRoot = execution.getParent();
} else {
throw new JbpmException("illegal state");
}
I assume getState() doesn't have any side effects, so I guess this is a bug
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
7:48 a.m.
New subject: [JBoss JIRA] Work started: (JBPM-2487) Bug in GroupActivity.execute(ExecutionImpl execution)
[
https://jira.jboss.org/jira/browse/JBPM-2487?page=com.atlassian.jira.plug...
]
Work on JBPM-2487 started by Tom Baeyens.
Bug in GroupActivity.execute(ExecutionImpl execution)
-----------------------------------------------------
Key: JBPM-2487
URL: https://jira.jboss.org/jira/browse/JBPM-2487
Project: jBPM
Issue Type: Bug
Security Level: Public(Everyone can see)
Components: Runtime Engine
Affects Versions: jBPM 4.0
Reporter: Michael Wohlfart
Assignee: Tom Baeyens
Priority: Minor
Fix For: jBPM 4.2
Attachments: ForkGroupReverseTest.java, process.jpdl.xml
Original Estimate: 4 hours
Remaining Estimate: 4 hours
just browsing through the codebase and trying to understand the magic,
I came across this in line 56 of GroupActivity.java:
ExecutionImpl concurrentRoot = null;
if (Execution.STATE_ACTIVE_ROOT.equals(execution.getState())) {
concurrentRoot = execution;
} else if (Execution.STATE_ACTIVE_ROOT.equals(execution.getState())) {
concurrentRoot = execution.getParent();
} else {
throw new JbpmException("illegal state");
}
I assume getState() doesn't have any side effects, so I guess this is a bug
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
8:27 a.m.
New subject: [JBoss JIRA] Resolved: (JBPM-2487) Bug in GroupActivity.execute(ExecutionImpl execution)
[
https://jira.jboss.org/jira/browse/JBPM-2487?page=com.atlassian.jira.plug...
]
Tom Baeyens resolved JBPM-2487.
-------------------------------
Resolution: Done
Bug in GroupActivity.execute(ExecutionImpl execution)
-----------------------------------------------------
Key: JBPM-2487
URL: https://jira.jboss.org/jira/browse/JBPM-2487
Project: jBPM
Issue Type: Bug
Security Level: Public(Everyone can see)
Components: Runtime Engine
Affects Versions: jBPM 4.0
Reporter: Michael Wohlfart
Assignee: Tom Baeyens
Priority: Minor
Fix For: jBPM 4.2
Attachments: ForkGroupReverseTest.java, process.jpdl.xml
Original Estimate: 4 hours
Remaining Estimate: 4 hours
just browsing through the codebase and trying to understand the magic,
I came across this in line 56 of GroupActivity.java:
ExecutionImpl concurrentRoot = null;
if (Execution.STATE_ACTIVE_ROOT.equals(execution.getState())) {
concurrentRoot = execution;
} else if (Execution.STATE_ACTIVE_ROOT.equals(execution.getState())) {
concurrentRoot = execution.getParent();
} else {
throw new JbpmException("illegal state");
}
I assume getState() doesn't have any side effects, so I guess this is a bug
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
8:27 a.m.
New subject: [JBoss JIRA] Commented: (JBPM-2487) Bug in GroupActivity.execute(ExecutionImpl execution)
[
https://jira.jboss.org/jira/browse/JBPM-2487?page=com.atlassian.jira.plug...
]
Tom Baeyens commented on JBPM-2487:
-----------------------------------
i couldn't reproduce the problem with the given process and test. that is strange.
but the code of the activity indeed is incorrect and i fixed it.
i created 4 tests based on your process in the test suite.
Bug in GroupActivity.execute(ExecutionImpl execution)
-----------------------------------------------------
Key: JBPM-2487
URL: https://jira.jboss.org/jira/browse/JBPM-2487
Project: jBPM
Issue Type: Bug
Security Level: Public(Everyone can see)
Components: Runtime Engine
Affects Versions: jBPM 4.0
Reporter: Michael Wohlfart
Assignee: Tom Baeyens
Priority: Minor
Fix For: jBPM 4.2
Attachments: ForkGroupReverseTest.java, process.jpdl.xml
Original Estimate: 4 hours
Remaining Estimate: 4 hours
just browsing through the codebase and trying to understand the magic,
I came across this in line 56 of GroupActivity.java:
ExecutionImpl concurrentRoot = null;
if (Execution.STATE_ACTIVE_ROOT.equals(execution.getState())) {
concurrentRoot = execution;
} else if (Execution.STATE_ACTIVE_ROOT.equals(execution.getState())) {
concurrentRoot = execution.getParent();
} else {
throw new JbpmException("illegal state");
}
I assume getState() doesn't have any side effects, so I guess this is a bug
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
5354
days inactive
5419
days old
13 comments
4 participants
participants (4)
-
Michael Wohlfart (JIRA) -
Ronald van Kuijk (JIRA)
-
Scott Peterson (JIRA)
-
Tom Baeyens (JIRA)