Author: tmjee
Date: Thu Apr 27 08:43:30 2006
New Revision: 397572

URL: http://svn.apache.org/viewcvs?rev=397572&view=rev
Log:
WW-1278


Modified:
    
incubator/webwork2/action/src/main/java/org/apache/struts/action2/dispatcher/FilterDispatcher.java

Modified: 
incubator/webwork2/action/src/main/java/org/apache/struts/action2/dispatcher/FilterDispatcher.java
URL: 
http://svn.apache.org/viewcvs/incubator/webwork2/action/src/main/java/org/apache/struts/action2/dispatcher/FilterDispatcher.java?rev=397572&r1=397571&r2=397572&view=diff
==============================================================================
--- 
incubator/webwork2/action/src/main/java/org/apache/struts/action2/dispatcher/FilterDispatcher.java
 (original)
+++ 
incubator/webwork2/action/src/main/java/org/apache/struts/action2/dispatcher/FilterDispatcher.java
 Thu Apr 27 08:43:30 2006
@@ -163,10 +163,20 @@
     public void doFilter(ServletRequest req, ServletResponse res, FilterChain 
chain) throws IOException, ServletException {
         HttpServletRequest request = (HttpServletRequest) req;
         HttpServletResponse response = (HttpServletResponse) res;
+        ServletContext servletContext = filterConfig.getServletContext();
 
         // prepare the request no matter what - this ensures that the proper 
character encoding
         // is used before invoking the mapper (see WW-9127)
         DispatcherUtils du = DispatcherUtils.getInstance();
+        try {
+               // Wrap request first, just in case it is multipart/form-data 
+               // parameters might not be accessible through before encoding 
(ww-1278)
+            request = du.wrapRequest(request, servletContext);
+        } catch (IOException e) {
+            String message = "Could not wrap servlet request with 
MultipartRequestWrapper!";
+            LOG.error(message, e);
+            throw new ServletException(message, e);
+        }
         du.prepare(request, response);
 
         ActionMapper mapper = ActionMapperFactory.getMapper();
@@ -194,19 +204,11 @@
 
 
         Object o = null;
-        ServletContext servletContext = filterConfig.getServletContext();
         try {
 
             setupContainer(request);
             o = beforeActionInvocation(request, servletContext);
-
-            try {
-                request = du.wrapRequest(request, servletContext);
-            } catch (IOException e) {
-                String message = "Could not wrap servlet request with 
MultipartRequestWrapper!";
-                LOG.error(message, e);
-                throw new ServletException(message, e);
-            }
+            
             du.serviceAction(request, response, servletContext, mapping);
         } finally {
             afterActionInvocation(request, servletContext, o);



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to