Shikhar13 / codenameone

Automatically exported from code.google.com/p/codenameone
0 stars 0 forks source link

RFE: MultipartRequest - add an optional 'filename' parameter to addData() #185

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
At the moment, the filename sent for the content is the same as the parameter 
name.  Would be nice to have an option to pass the original filename for 
services that use it.

Original issue reported on code.google.com by 1815...@coolman.ca on 5 May 2012 at 6:46

GoogleCodeExporter commented 9 years ago
Added this a setFilename option

Original comment by shai.almog on 5 May 2012 at 7:36

GoogleCodeExporter commented 9 years ago
Thanks, that worked... just one little adjustment needed :)

Index: src/com/codename1/io/MultipartRequest.java
===================================================================
--- src/com/codename1/io/MultipartRequest.java  (revision 255)
+++ src/com/codename1/io/MultipartRequest.java  (working copy)
@@ -99,7 +99,7 @@
      */
     public void addArgument(String name, String value) {
         args.put(name, value);
-        if(!filenames.contains(name)) {
+        if(!filenames.containsKey(name)) {
             filenames.put(name, name);
         }
     }

Original comment by 1815...@coolman.ca on 5 May 2012 at 3:09

GoogleCodeExporter commented 9 years ago
UGHHHHH I just hate that API! I make that mistake every time and it takes me 
AGES to track it down.

Original comment by shai.almog on 5 May 2012 at 3:13