Revision: 557
http://svn.sourceforge.net/stripes/?rev=557&view=rev
Author: mongus
Date: 2007-05-26 18:07:31 -0700 (Sat, 26 May 2007)
Log Message:
-----------
applied patches in STS-353 to prevent unnecessary HttpSession creation
Modified Paths:
--------------
trunk/stripes/src/net/sourceforge/stripes/controller/DispatcherServlet.java
trunk/stripes/src/net/sourceforge/stripes/integration/spring/SpringHelper.java
trunk/stripes/src/net/sourceforge/stripes/tag/layout/LayoutRenderTag.java
Modified:
trunk/stripes/src/net/sourceforge/stripes/controller/DispatcherServlet.java
===================================================================
--- trunk/stripes/src/net/sourceforge/stripes/controller/DispatcherServlet.java
2007-05-26 15:56:40 UTC (rev 556)
+++ trunk/stripes/src/net/sourceforge/stripes/controller/DispatcherServlet.java
2007-05-27 01:07:31 UTC (rev 557)
@@ -124,7 +124,7 @@
abc.getRequest(), // req
abc.getResponse(), // res
null, // error page url
-
true, // need session
+
(request.getSession(false) != null), // needsSession - don't force a session
creation if one doesn't already exist
abc.getResponse().getBufferSize(),
true); // autoflush
DispatcherHelper.setPageContext(pageContext);
Modified:
trunk/stripes/src/net/sourceforge/stripes/integration/spring/SpringHelper.java
===================================================================
---
trunk/stripes/src/net/sourceforge/stripes/integration/spring/SpringHelper.java
2007-05-26 15:56:40 UTC (rev 556)
+++
trunk/stripes/src/net/sourceforge/stripes/integration/spring/SpringHelper.java
2007-05-27 01:07:31 UTC (rev 557)
@@ -15,6 +15,7 @@
package net.sourceforge.stripes.integration.spring;
import net.sourceforge.stripes.action.ActionBeanContext;
+import net.sourceforge.stripes.controller.StripesFilter;
import net.sourceforge.stripes.exception.StripesRuntimeException;
import net.sourceforge.stripes.util.Log;
import net.sourceforge.stripes.util.ReflectUtil;
@@ -77,7 +78,7 @@
* @param context the ActionBeanContext represented by the current request
*/
public static void injectBeans(Object bean, ActionBeanContext context) {
- injectBeans(bean,
context.getRequest().getSession().getServletContext());
+ injectBeans(bean,
StripesFilter.getConfiguration().getServletContext());
}
/**
Modified:
trunk/stripes/src/net/sourceforge/stripes/tag/layout/LayoutRenderTag.java
===================================================================
--- trunk/stripes/src/net/sourceforge/stripes/tag/layout/LayoutRenderTag.java
2007-05-26 15:56:40 UTC (rev 556)
+++ trunk/stripes/src/net/sourceforge/stripes/tag/layout/LayoutRenderTag.java
2007-05-27 01:07:31 UTC (rev 557)
@@ -14,6 +14,7 @@
*/
package net.sourceforge.stripes.tag.layout;
+import net.sourceforge.stripes.controller.StripesFilter;
import net.sourceforge.stripes.exception.StripesJspException;
import net.sourceforge.stripes.tag.StripesTagSupport;
@@ -109,7 +110,7 @@
// Check that the page named is actually there, because some
containers will
// just quietly ignore includes of non-existent pages!
- URL target =
request.getSession().getServletContext().getResource(this.name);
+ URL target =
StripesFilter.getConfiguration().getServletContext().getResource(this.name);
if (target == null) {
throw new StripesJspException(
"Attempt made to render a layout that does not exist. The
layout name " +
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 DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
Stripes-development mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/stripes-development
