You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by si...@apache.org on 2013/03/13 11:33:21 UTC

svn commit: r1455869 - /commons/proper/fileupload/trunk/src/main/java/org/apache/commons/fileupload/servlet/ServletFileUpload.java

Author: simonetripodi
Date: Wed Mar 13 10:33:20 2013
New Revision: 1455869

URL: http://svn.apache.org/r1455869
Log:
trivial: no needs to duplicate the code to check if an HttpServletRequest is a multipart/* request, just reuse existing FileUploadBase#isMultipartContent method

Modified:
    commons/proper/fileupload/trunk/src/main/java/org/apache/commons/fileupload/servlet/ServletFileUpload.java

Modified: commons/proper/fileupload/trunk/src/main/java/org/apache/commons/fileupload/servlet/ServletFileUpload.java
URL: http://svn.apache.org/viewvc/commons/proper/fileupload/trunk/src/main/java/org/apache/commons/fileupload/servlet/ServletFileUpload.java?rev=1455869&r1=1455868&r2=1455869&view=diff
==============================================================================
--- commons/proper/fileupload/trunk/src/main/java/org/apache/commons/fileupload/servlet/ServletFileUpload.java (original)
+++ commons/proper/fileupload/trunk/src/main/java/org/apache/commons/fileupload/servlet/ServletFileUpload.java Wed Mar 13 10:33:20 2013
@@ -26,6 +26,7 @@ import org.apache.commons.fileupload.Fil
 import org.apache.commons.fileupload.FileItemFactory;
 import org.apache.commons.fileupload.FileItemIterator;
 import org.apache.commons.fileupload.FileUpload;
+import org.apache.commons.fileupload.FileUploadBase;
 import org.apache.commons.fileupload.FileUploadException;
 
 /**
@@ -67,14 +68,7 @@ public class ServletFileUpload extends F
         if (!POST_METHOD.equals(request.getMethod())) {
             return false;
         }
-        String contentType = request.getContentType();
-        if (contentType == null) {
-            return false;
-        }
-        if (contentType.toLowerCase().startsWith(MULTIPART)) {
-            return true;
-        }
-        return false;
+        return FileUploadBase.isMultipartContent(new ServletRequestContext(request));
     }
 
     // ----------------------------------------------------------- Constructors