From 9ed535394798f2c15ffbb08602cce898a03f171c Mon Sep 17 00:00:00 2001
From: Keith Donald <kdonald@vmware.com>
Date: Fri, 31 Jul 2009 04:10:55 +0000
Subject: [PATCH] unified map view of request parameters, including multiparts

---
 .../request/NativeWebRequestParameterMap.java       | 13 +++++--------
 1 file changed, 5 insertions(+), 8 deletions(-)

diff --git a/org.springframework.web/src/main/java/org/springframework/web/context/request/NativeWebRequestParameterMap.java b/org.springframework.web/src/main/java/org/springframework/web/context/request/NativeWebRequestParameterMap.java
index 523b8f69c05..8789c29d3e7 100644
--- a/org.springframework.web/src/main/java/org/springframework/web/context/request/NativeWebRequestParameterMap.java
+++ b/org.springframework.web/src/main/java/org/springframework/web/context/request/NativeWebRequestParameterMap.java
@@ -18,7 +18,9 @@ package org.springframework.web.context.request;
 import java.util.Iterator;
 
 import org.springframework.util.Assert;
+import org.springframework.util.CompositeIterator;
 import org.springframework.util.StringKeyedMapAdapter;
+import org.springframework.web.multipart.MultipartRequest;
 
 /**
  * Map backed by a Web request parameter map for accessing request parameters.
@@ -42,7 +44,6 @@ public class NativeWebRequestParameterMap extends StringKeyedMapAdapter<Object>
 	}
 
 	protected Object getAttribute(String key) {
-		/* TODO - MultipartRequest is NOT accessible b/c its in web.servlet
 		if (request instanceof MultipartRequest) {
 			MultipartRequest multipartRequest = (MultipartRequest) request;
 			Object data = multipartRequest.getFileMap().get(key);
@@ -50,7 +51,6 @@ public class NativeWebRequestParameterMap extends StringKeyedMapAdapter<Object>
 				return data;
 			}
 		}
-		*/
 		String[] parameters = request.getParameterValues(key);
 		if (parameters == null) {
 			return null;
@@ -70,17 +70,14 @@ public class NativeWebRequestParameterMap extends StringKeyedMapAdapter<Object>
 	}
 
 	protected Iterator<String> getAttributeNames() {
-		return request.getParameterNames();
-		/* TODO - MultipartRequest is NOT accessible b/c its in web.servlet
 		if (request instanceof MultipartRequest) {
 			MultipartRequest multipartRequest = (MultipartRequest) request;
-			CompositeIterator iterator = new CompositeIterator();
+			CompositeIterator<String> iterator = new CompositeIterator<String>();
 			iterator.add(multipartRequest.getFileMap().keySet().iterator());
-			iterator.add(CollectionUtils.toIterator(request.getParameterNames()));
+			iterator.add(request.getParameterNames());
 			return iterator;
 		} else {
-			return CollectionUtils.toIterator(request.getParameterNames());
+			return request.getParameterNames();			
 		}
-		*/
 	}
 }
\ No newline at end of file