DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=12732>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=12732

[PATCH] Making CommonsMultipartRequestHandler aware of character encoding in request

           Summary: [PATCH] Making CommonsMultipartRequestHandler aware of
                    character encoding in request
           Product: Struts
           Version: Nightly Build
          Platform: Other
        OS/Version: Other
            Status: NEW
          Severity: Normal
          Priority: Other
         Component: Controller
        AssignedTo: [EMAIL PROTECTED]
        ReportedBy: [EMAIL PROTECTED]


problem: when using enctype="multipart/form-data" in a form, charset in the
httprequest is ignored for textfields.

fix: grab the character encoding from the request and encode it.



Index: src/share/org/apache/struts/upload/CommonsMultipartRequestHandler.java
===================================================================
RCS file:
/home/cvspublic/jakarta-struts/src/share/org/apache/struts/upload/CommonsMultipartRequestHandler.java,v
retrieving revision 1.2
diff -u -r1.2 CommonsMultipartRequestHandler.java
--- src/share/org/apache/struts/upload/CommonsMultipartRequestHandler.java      31
Jul 2002 06:43:18 -0000 1.2
+++ src/share/org/apache/struts/upload/CommonsMultipartRequestHandler.java      17
Sep 2002 10:35:50 -0000
@@ -427,7 +427,14 @@
      */
     protected void addTextParameter(HttpServletRequest request, FileItem item) {
         String name = item.getFieldName();
-        String value = item.getString();
+        String value = null;
+        String characterEncoding = request.getCharacterEncoding();
+        try {
+            value = item.getString(characterEncoding);
+        }catch (Exception e) {
+            value = item.getString();
+        }
+        
 
         if (request instanceof MultipartRequestWrapper) {
             MultipartRequestWrapper wrapper = (MultipartRequestWrapper) request;

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

Reply via email to