From bcb5a8d2a70727a0c0388fbab19bdcc6c9ec9ee5 Mon Sep 17 00:00:00 2001 From: Phillip Webb Date: Thu, 12 Apr 2012 17:48:49 -0700 Subject: [PATCH] Revert use of putAll for JSF spec compliance Issues: SWF-1532 --- .../main/java/org/springframework/faces/mvc/JsfView.java | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/spring-faces/src/main/java/org/springframework/faces/mvc/JsfView.java b/spring-faces/src/main/java/org/springframework/faces/mvc/JsfView.java index 70906ce7..debe4fe7 100644 --- a/spring-faces/src/main/java/org/springframework/faces/mvc/JsfView.java +++ b/spring-faces/src/main/java/org/springframework/faces/mvc/JsfView.java @@ -88,7 +88,11 @@ public class JsfView extends AbstractUrlBasedView { } private void populateRequestMap(FacesContext facesContext, Map model) { - facesContext.getExternalContext().getRequestMap().putAll(model); + Map requestMap = facesContext.getExternalContext().getRequestMap(); + for (Map.Entry entry : model.entrySet()) { + // JSF does not insist that putAll is implemented, hence we use individual put calls + requestMap.put(entry.getKey(), entry.getValue()); + } } private Lifecycle createFacesLifecycle() {