Revision: 630
http://stripes.svn.sourceforge.net/stripes/?rev=630&view=rev
Author: bengunter
Date: 2007-11-17 21:34:48 -0800 (Sat, 17 Nov 2007)
Log Message:
-----------
STS-370: StripesFilter cleans up too early on forwarded request.
Moved the line that clears the Configuration object from the ThreadLocal into
the block that only executes after the initial invocation completes.
Modified Paths:
--------------
trunk/stripes/src/net/sourceforge/stripes/controller/StripesFilter.java
Modified:
trunk/stripes/src/net/sourceforge/stripes/controller/StripesFilter.java
===================================================================
--- trunk/stripes/src/net/sourceforge/stripes/controller/StripesFilter.java
2007-11-17 06:14:55 UTC (rev 629)
+++ trunk/stripes/src/net/sourceforge/stripes/controller/StripesFilter.java
2007-11-18 05:34:48 UTC (rev 630)
@@ -90,7 +90,7 @@
// we'll just use the DefaultConfiguration
if (configurationClassName != null) {
try {
- Class clazz = ReflectUtil.findClass(configurationClassName);
+ Class<?> clazz = ReflectUtil.findClass(configurationClassName);
this.configuration = (Configuration) clazz.newInstance();
}
catch (Exception e) {
@@ -206,10 +206,10 @@
if (initial) {
initialInvocation.set(true);
flashOutbound(httpRequest);
+
+ // Once the request is processed, take the Configuration back
out of thread local
+ StripesFilter.configurationStash.remove();
}
-
- // Once the request is processed, take the Configuration back out
of thread local
- StripesFilter.configurationStash.set(null);
}
}
This was sent by the SourceForge.net collaborative development platform, the
world's largest Open Source development site.
-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Stripes-development mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/stripes-development