Spring/spring-webflow
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

SWF-923 Global Transitions don't fire listeners

Browse Source 
Applied fix suggested in jira.  Global transitions now execute on the request context instead of directly on the transition.  This allows the flow execution listener to hook in like a normal transition.
This commit is contained in:
Scott Andrews
2008-10-27 18:38:04 +00:00
parent 25427a1e09
commit c664c7e73c
1 changed files with 10 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
spring-webflow/src/main/java/org/springframework/webflow/engine/Flow.java
View File

@@ -511,10 +511,11 @@ public class Flow extends AnnotatedObject implements FlowDefinition {
/**
* Start a new session for this flow in its start state. This boils down to the following:
* <ol>
* <li>Create (setup) all registered flow variables ({@link #addVariable(FlowVariable)}) in flow scope.</li> <li>Map
* provided input data into the flow. Typically data will be mapped into flow scope using the registered input
* mapper ({@link #setInputMapper(Mapper)}).</li> <li>Execute all registered start actions (
* {@link #getStartActionList()}).</li> <li>Enter the configured start state ({@link #setStartState(State)})</li>
* <li>Create (setup) all registered flow variables ({@link #addVariable(FlowVariable)}) in flow scope.</li>
* <li>Map provided input data into the flow. Typically data will be mapped into flow scope using the registered
* input mapper ({@link #setInputMapper(Mapper)}).</li>
* <li>Execute all registered start actions ( {@link #getStartActionList()}).</li>
* <li>Enter the configured start state ({@link #setStartState(State)})</li>
* </ol>
* @param context the flow execution control context
* @param input eligible input into the session
@@ -555,7 +556,8 @@ public class Flow extends AnnotatedObject implements FlowDefinition {
// try the flow level transition set for a match
Transition transition = globalTransitionSet.getTransition(context);
if (transition != null) {
return transition.execute(currentState, context);
return context.execute(transition);
// return transition.execute(currentState, context);
} else {
// no matching global transition => let the original exception
// propagate
@@ -568,9 +570,9 @@ public class Flow extends AnnotatedObject implements FlowDefinition {
* Inform this flow definition that an execution session of itself has ended. As a result, the flow will do the
* following:
* <ol>
* <li>Execute all registered end actions ({@link #getEndActionList()}).</li> <li>Map data available in the flow
* execution control context into provided output map using a registered output mapper (
* {@link #setOutputMapper(Mapper)}).</li>
* <li>Execute all registered end actions ({@link #getEndActionList()}).</li>
* <li>Map data available in the flow execution control context into provided output map using a registered output
* mapper ( {@link #setOutputMapper(Mapper)}).</li>
* </ol>
* @param context the flow execution control context
* @param outcome the logical flow outcome that will be returned by the session, generally the id of the terminating