Revision: 520
          http://svn.sourceforge.net/stripes/?rev=520&view=rev
Author:   tfenne
Date:     2007-04-03 16:09:32 -0700 (Tue, 03 Apr 2007)

Log Message:
-----------
Merge to trunk: fixed a bug where the FlashRequest wrapping would have problems 
with a mime/multipart request.

Modified Paths:
--------------
    trunk/stripes/src/net/sourceforge/stripes/controller/FlashRequest.java
    trunk/stripes/src/net/sourceforge/stripes/controller/FlashScope.java

Modified: trunk/stripes/src/net/sourceforge/stripes/controller/FlashRequest.java
===================================================================
--- trunk/stripes/src/net/sourceforge/stripes/controller/FlashRequest.java      
2007-04-03 23:08:12 UTC (rev 519)
+++ trunk/stripes/src/net/sourceforge/stripes/controller/FlashRequest.java      
2007-04-03 23:09:32 UTC (rev 520)
@@ -70,15 +70,25 @@
     private int remotePort;
     private int serverPort;
     
-    public static StripesRequestWrapper wrapRequest(HttpServletRequest 
request) {
-        try {
-            return new StripesRequestWrapper(new FlashRequest(request));
-        }
-        catch (StripesServletException e) {
-            throw new StripesRuntimeException(e);
-        }
+       /**
+        * Finds the StripesRequestWrapper for the supplied request and swaps 
out the underlying
+        * request for an instance of FlashRequest.
+        *
+        * @param request the current HttpServletRequest
+        * @return the StripesRequestWrapper for this request with the "live" 
request replaced
+        */
+    public static StripesRequestWrapper replaceRequest(HttpServletRequest 
request) {
+        StripesRequestWrapper wrapper = 
StripesRequestWrapper.findStripesWrapper(request);
+        wrapper.setRequest(new FlashRequest((HttpServletRequest) 
wrapper.getRequest()));
+        return wrapper;
     }
 
+    /**
+     * Creates a new FlashRequest by copying all appropriate attributes from 
the prototype
+     * request supplied.
+     *
+     * @param prototype the HttpServletRequest to create a disconnected copy of
+     */
     @SuppressWarnings({ "unchecked", "deprecation" })
     public FlashRequest(HttpServletRequest prototype) {
         // copy properties

Modified: trunk/stripes/src/net/sourceforge/stripes/controller/FlashScope.java
===================================================================
--- trunk/stripes/src/net/sourceforge/stripes/controller/FlashScope.java        
2007-04-03 23:08:12 UTC (rev 519)
+++ trunk/stripes/src/net/sourceforge/stripes/controller/FlashScope.java        
2007-04-03 23:09:32 UTC (rev 520)
@@ -140,7 +140,7 @@
 
         // Replace the request and response objects for the request cycle that 
is ending
         // with objects that are safe to use on the ensuing request.
-        HttpServletRequest flashRequest = FlashRequest.wrapRequest(request);
+        HttpServletRequest flashRequest = FlashRequest.replaceRequest(request);
         HttpServletResponse flashResponse = (HttpServletResponse) 
Proxy.newProxyInstance(
                 getClass().getClassLoader(),
                 new Class<?>[] { HttpServletResponse.class },


This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Stripes-development mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/stripes-development

Reply via email to