split the header map between request and response
git-svn-id: https://svn.apache.org/repos/asf/tiles/framework/trunk/tiles-request@1215004 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/tiles-request-api/src/main/java/org/apache/tiles/request/Request.java b/tiles-request-api/src/main/java/org/apache/tiles/request/Request.java
index 60e79d5..5a0a6db 100644
--- a/tiles-request-api/src/main/java/org/apache/tiles/request/Request.java
+++ b/tiles-request-api/src/main/java/org/apache/tiles/request/Request.java
@@ -54,6 +54,13 @@
Map<String, String[]> getHeaderValues();
/**
+ * Return an unreadable Map that writes headers to the response.
+ *
+ * @return The header map.
+ */
+ Map<String, String> getResponseHeaders();
+
+ /**
* Returns a context map, given the scope name.
*
* @param scope The name of the scope.
diff --git a/tiles-request-api/src/main/java/org/apache/tiles/request/attribute/Addable.java b/tiles-request-api/src/main/java/org/apache/tiles/request/attribute/Addable.java
new file mode 100644
index 0000000..5e8b9a9
--- /dev/null
+++ b/tiles-request-api/src/main/java/org/apache/tiles/request/attribute/Addable.java
@@ -0,0 +1,38 @@
+/*
+ * $Id$
+ *
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied. See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package org.apache.tiles.request.attribute;
+
+/**
+ * Allows to get and set attributes.
+ *
+ * @version $Rev$ $Date$
+ * @param <V> The type of the value of the attribute.
+ */
+public interface Addable<V> {
+
+ /**
+ * Sets a value for the given key.
+ *
+ * @param key The key of the attribute.
+ * @param value The value of the attribute.
+ */
+ void setValue(String key, V value);
+}
diff --git a/tiles-request-api/src/main/java/org/apache/tiles/request/attribute/HasAddableKeys.java b/tiles-request-api/src/main/java/org/apache/tiles/request/attribute/HasAddableKeys.java
index 16dfcab..115fd2b 100644
--- a/tiles-request-api/src/main/java/org/apache/tiles/request/attribute/HasAddableKeys.java
+++ b/tiles-request-api/src/main/java/org/apache/tiles/request/attribute/HasAddableKeys.java
@@ -26,13 +26,5 @@
* @version $Rev$ $Date$
* @param <V> The type of the value of the attribute.
*/
-public interface HasAddableKeys<V> extends HasKeys<V> {
-
- /**
- * Sets a value for the given key.
- *
- * @param key The key of the attribute.
- * @param value The value of the attribute.
- */
- void setValue(String key, V value);
+public interface HasAddableKeys<V> extends HasKeys<V>, Addable<V> {
}
diff --git a/tiles-request-api/src/main/java/org/apache/tiles/request/collection/AddOnlyMap.java b/tiles-request-api/src/main/java/org/apache/tiles/request/collection/AddOnlyMap.java
new file mode 100644
index 0000000..355b19d
--- /dev/null
+++ b/tiles-request-api/src/main/java/org/apache/tiles/request/collection/AddOnlyMap.java
@@ -0,0 +1,189 @@
+/*
+ * $Id$
+ *
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied. See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package org.apache.tiles.request.collection;
+
+import java.util.Collection;
+import java.util.Collections;
+import java.util.Iterator;
+import java.util.Map;
+import java.util.Set;
+
+import org.apache.tiles.request.attribute.Addable;
+
+/**
+ * Exposes an {@link Addable} object as a put-only (no remove, no get) map.
+ * This map will appear empty to anyone trying to fetch its content.
+ *
+ * @version $Rev$ $Date$
+ * @since 3.0.0
+ * @param <V> The type of the value of the attribute.
+ */
+public class AddOnlyMap<V> implements Map<String, V> {
+ /** The request. */
+ private Addable<V> request;
+
+ /**
+ * Constructor.
+ *
+ * @param request
+ * The request object to use.
+ */
+ public AddOnlyMap(Addable<V> request) {
+ this.request = request;
+ }
+
+ /** {@inheritDoc} */
+ public int size() {
+ return 0;
+ }
+
+ /** {@inheritDoc} */
+ public boolean isEmpty() {
+ return true;
+ }
+
+ /** {@inheritDoc} */
+ public boolean containsKey(Object key) {
+ return false;
+ }
+
+ /** {@inheritDoc} */
+ public boolean containsValue(Object value) {
+ return false;
+ }
+
+ /** {@inheritDoc} */
+ public V get(Object key) {
+ return null;
+ }
+
+ /** {@inheritDoc} */
+ public V put(String key, V value) {
+ request.setValue(key, value);
+ return null;
+ }
+
+ /** {@inheritDoc} */
+ public V remove(Object key) {
+ return null;
+ }
+
+ /** {@inheritDoc} */
+ public void putAll(Map<? extends String, ? extends V> map) {
+ for (Map.Entry<? extends String, ? extends V> entry : map.entrySet()) {
+ request.setValue(entry.getKey(), entry.getValue());
+ }
+ }
+
+ /** {@inheritDoc} */
+ public void clear() {
+ }
+
+ /** {@inheritDoc} */
+ public Set<String> keySet() {
+ return Collections.<String> emptySet();
+ }
+
+ /** {@inheritDoc} */
+ public Collection<V> values() {
+ return Collections.<V> emptySet();
+ }
+
+ /** {@inheritDoc} */
+ public Set<java.util.Map.Entry<String, V>> entrySet() {
+ return new AddOnlyEntrySet();
+ }
+
+ /**
+ * Entry set implementation for {@link AddableParameterMap}.
+ */
+ private class AddOnlyEntrySet implements Set<Map.Entry<String, V>> {
+
+ @Override
+ public boolean add(java.util.Map.Entry<String, V> e) {
+ request.setValue(e.getKey(), e.getValue());
+ return true;
+ }
+
+ @Override
+ public boolean addAll(
+ Collection<? extends java.util.Map.Entry<String, V>> c) {
+ for (Map.Entry<String, V> entry : c) {
+ request.setValue(entry.getKey(), entry.getValue());
+ }
+ return true;
+ }
+
+ @Override
+ public int size() {
+ return 0;
+ }
+
+ @Override
+ public boolean isEmpty() {
+ return false;
+ }
+
+ @Override
+ public boolean contains(Object o) {
+ return false;
+ }
+
+ @Override
+ public Iterator<java.util.Map.Entry<String, V>> iterator() {
+ return Collections.<java.util.Map.Entry<String, V>>emptySet().iterator();
+ }
+
+ @Override
+ public Object[] toArray() {
+ return new java.util.Map.Entry[0];
+ }
+
+ @Override
+ public <T> T[] toArray(T[] a) {
+ return Collections.<java.util.Map.Entry<String, V>>emptySet().toArray(a);
+ }
+
+ @Override
+ public boolean remove(Object o) {
+ return false;
+ }
+
+ @Override
+ public boolean containsAll(Collection<?> c) {
+ return false;
+ }
+
+ @Override
+ public boolean retainAll(Collection<?> c) {
+ return false;
+ }
+
+ @Override
+ public boolean removeAll(Collection<?> c) {
+ return false;
+ }
+
+ @Override
+ public void clear() {
+ }
+ }
+}
diff --git a/tiles-request-api/src/main/java/org/apache/tiles/request/util/DefaultRequestWrapper.java b/tiles-request-api/src/main/java/org/apache/tiles/request/util/DefaultRequestWrapper.java
index 0e1c2e6..c900898 100644
--- a/tiles-request-api/src/main/java/org/apache/tiles/request/util/DefaultRequestWrapper.java
+++ b/tiles-request-api/src/main/java/org/apache/tiles/request/util/DefaultRequestWrapper.java
@@ -71,6 +71,11 @@
}
/** {@inheritDoc} */
+ public Map<String, String> getResponseHeaders() {
+ return context.getResponseHeaders();
+ }
+
+ /** {@inheritDoc} */
public ApplicationContext getApplicationContext() {
return context.getApplicationContext();
}
diff --git a/tiles-request-api/src/main/java/org/apache/tiles/request/util/WebRequestWrapper.java b/tiles-request-api/src/main/java/org/apache/tiles/request/util/WebRequestWrapper.java
index e7651a9..34ae5ca 100644
--- a/tiles-request-api/src/main/java/org/apache/tiles/request/util/WebRequestWrapper.java
+++ b/tiles-request-api/src/main/java/org/apache/tiles/request/util/WebRequestWrapper.java
@@ -71,6 +71,11 @@
}
/** {@inheritDoc} */
+ public Map<String, String> getResponseHeaders() {
+ return context.getResponseHeaders();
+ }
+
+ /** {@inheritDoc} */
public Map<String, Object> getContext(String scope) {
ContextResolver resolver = ApplicationAccess.getContextResolver(context
.getApplicationContext());
diff --git a/tiles-request-api/src/test/java/org/apache/tiles/request/collection/AddOnlyMapTest.java b/tiles-request-api/src/test/java/org/apache/tiles/request/collection/AddOnlyMapTest.java
new file mode 100644
index 0000000..dc0a7e9
--- /dev/null
+++ b/tiles-request-api/src/test/java/org/apache/tiles/request/collection/AddOnlyMapTest.java
@@ -0,0 +1,113 @@
+/*
+ * $Id$
+ *
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied. See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package org.apache.tiles.request.collection;
+
+import static org.easymock.EasyMock.*;
+import static org.easymock.classextension.EasyMock.*;
+import static org.junit.Assert.*;
+
+import java.util.ArrayList;
+import java.util.HashMap;
+import java.util.List;
+import java.util.Map;
+import java.util.Set;
+
+import org.apache.tiles.request.attribute.Addable;
+import org.junit.Before;
+import org.junit.Test;
+
+/**
+ * Tests {@link AddOnlyMap}.
+ *
+ * @version $Rev$ $Date$
+ */
+public class AddOnlyMapTest {
+
+ /**
+ * The object to test.
+ */
+ private AddOnlyMap<Integer> map;
+
+ /**
+ * The extractor to use.
+ */
+ private Addable<Integer> extractor;
+
+ /**
+ * Sets up the test.
+ */
+ @SuppressWarnings("unchecked")
+ @Before
+ public void setUp() {
+ extractor = createMock(Addable.class);
+ map = new AddOnlyMap<Integer>(extractor);
+ }
+
+ /**
+ * Test method for {@link org.apache.tiles.request.collection.AddableParameterMap#entrySet()}.
+ */
+ @Test
+ public void testEntrySet() {
+ Set<Map.Entry<String, Integer>> entrySet = map.entrySet();
+ MapEntry<String, Integer> entry1 = new MapEntry<String, Integer>("one", 13, false);
+ MapEntry<String, Integer> entry2 = new MapEntry<String, Integer>("two", 42, false);
+ List<Map.Entry<String, Integer>> entries = new ArrayList<Map.Entry<String, Integer>>(2);
+ entries.add(entry1);
+ entries.add(entry2);
+
+ extractor.setValue("one", 13);
+ expectLastCall().times(2);
+ extractor.setValue("two", 42);
+ replay(extractor);
+ entrySet.add(entry1);
+ entrySet.addAll(entries);
+ verify(extractor);
+ }
+
+ /**
+ * Test method for {@link AddableParameterMap#put(String, String)}.
+ */
+ @Test
+ public void testPut() {
+ extractor.setValue("one", 42);
+
+ replay(extractor);
+ assertNull(map.put("one", 42));
+ verify(extractor);
+ }
+
+ /**
+ * Test method for {@link org.apache.tiles.request.collection.AddableParameterMap#putAll(java.util.Map)}.
+ */
+ @Test
+ public void testPutAll() {
+ Map<String, Integer> map = new HashMap<String, Integer>();
+ map.put("one", 13);
+ map.put("two", 42);
+
+ extractor.setValue("one", 13);
+ extractor.setValue("two", 42);
+
+ replay(extractor);
+ this.map.putAll(map);
+ verify(extractor);
+ }
+}
diff --git a/tiles-request-api/src/test/java/org/apache/tiles/request/scope/ReflectionContextResolverTest.java b/tiles-request-api/src/test/java/org/apache/tiles/request/scope/ReflectionContextResolverTest.java
index ccffed1..12c801b 100644
--- a/tiles-request-api/src/test/java/org/apache/tiles/request/scope/ReflectionContextResolverTest.java
+++ b/tiles-request-api/src/test/java/org/apache/tiles/request/scope/ReflectionContextResolverTest.java
@@ -260,6 +260,11 @@
}
@Override
+ public Map<String, String> getResponseHeaders() {
+ return null;
+ }
+
+ @Override
public OutputStream getOutputStream() {
return null;
}
diff --git a/tiles-request-api/src/test/java/org/apache/tiles/request/util/DefaultRequestWrapperTest.java b/tiles-request-api/src/test/java/org/apache/tiles/request/util/DefaultRequestWrapperTest.java
index 595f117..ac77b42 100644
--- a/tiles-request-api/src/test/java/org/apache/tiles/request/util/DefaultRequestWrapperTest.java
+++ b/tiles-request-api/src/test/java/org/apache/tiles/request/util/DefaultRequestWrapperTest.java
@@ -100,6 +100,23 @@
}
/**
+ * Test method for {@link org.apache.tiles.request.util.DefaultRequestWrapper#getResponseHeaders()}.
+ */
+ @SuppressWarnings("unchecked")
+ @Test
+ public void testGetResponseHeaders() {
+ Request wrappedRequest = createMockRequest();
+ Map<String, String> header = createMock(Map.class);
+
+ expect(wrappedRequest.getResponseHeaders()).andReturn(header);
+
+ replay(wrappedRequest);
+ RequestWrapper request = createRequestWrapper(wrappedRequest);
+ assertEquals(header, request.getResponseHeaders());
+ verify(wrappedRequest);
+ }
+
+ /**
* Test method for {@link org.apache.tiles.request.util.DefaultRequestWrapper#getHeaderValues()}.
*/
@SuppressWarnings("unchecked")
diff --git a/tiles-request-portlet/src/main/java/org/apache/tiles/request/portlet/PortletRequest.java b/tiles-request-portlet/src/main/java/org/apache/tiles/request/portlet/PortletRequest.java
index 0e1eb9d..68a1305 100644
--- a/tiles-request-portlet/src/main/java/org/apache/tiles/request/portlet/PortletRequest.java
+++ b/tiles-request-portlet/src/main/java/org/apache/tiles/request/portlet/PortletRequest.java
@@ -35,8 +35,9 @@
import org.apache.tiles.request.AbstractClientRequest;
import org.apache.tiles.request.ApplicationContext;
-import org.apache.tiles.request.collection.AddableParameterMap;
+import org.apache.tiles.request.collection.AddOnlyMap;
import org.apache.tiles.request.collection.HeaderValuesMap;
+import org.apache.tiles.request.collection.ReadOnlyEnumerationMap;
import org.apache.tiles.request.collection.ScopeMap;
import org.apache.tiles.request.portlet.delegate.RequestDelegate;
import org.apache.tiles.request.portlet.delegate.ResponseDelegate;
@@ -64,6 +65,13 @@
/**
+ * <p>The lazily instantiated <code>Map</code> of header name-value
+ * combinations (write-only).</p>
+ */
+ private Map<String, String> responseHeaders = null;
+
+
+ /**
* <p>The lazily instantitated <code>Map</code> of header name-values
* combinations (immutable).</p>
*/
@@ -170,12 +178,20 @@
/** {@inheritDoc} */
public Map<String, String> getHeader() {
if ((header == null) && (request != null)) {
- header = new AddableParameterMap(new HeaderExtractor(request, response));
+ header = new ReadOnlyEnumerationMap<String>(new HeaderExtractor(request, null));
}
return (header);
}
/** {@inheritDoc} */
+ public Map<String, String> getResponseHeaders() {
+ if ((responseHeaders == null) && (request != null)) {
+ responseHeaders = new AddOnlyMap<String>(new HeaderExtractor(null, response));
+ }
+ return (responseHeaders);
+ }
+
+ /** {@inheritDoc} */
public Map<String, String[]> getHeaderValues() {
if ((headerValues == null) && (request != null)) {
headerValues = new HeaderValuesMap(new HeaderExtractor(request, response));
diff --git a/tiles-request-portlet/src/test/java/org/apache/tiles/request/portlet/PortletRequestTest.java b/tiles-request-portlet/src/test/java/org/apache/tiles/request/portlet/PortletRequestTest.java
index 293eeab..908e17a 100644
--- a/tiles-request-portlet/src/test/java/org/apache/tiles/request/portlet/PortletRequestTest.java
+++ b/tiles-request-portlet/src/test/java/org/apache/tiles/request/portlet/PortletRequestTest.java
@@ -36,8 +36,9 @@
import javax.servlet.ServletOutputStream;
import org.apache.tiles.request.ApplicationContext;
-import org.apache.tiles.request.collection.AddableParameterMap;
+import org.apache.tiles.request.collection.AddOnlyMap;
import org.apache.tiles.request.collection.HeaderValuesMap;
+import org.apache.tiles.request.collection.ReadOnlyEnumerationMap;
import org.apache.tiles.request.collection.ScopeMap;
import org.apache.tiles.request.portlet.delegate.RequestDelegate;
import org.apache.tiles.request.portlet.delegate.ResponseDelegate;
@@ -235,7 +236,15 @@
*/
@Test
public void testGetHeader() {
- assertTrue(req.getHeader() instanceof AddableParameterMap);
+ assertTrue(req.getHeader() instanceof ReadOnlyEnumerationMap);
+ }
+
+ /**
+ * Test method for {@link org.apache.tiles.request.portlet.PortletRequest#getResponseHeaders()}.
+ */
+ @Test
+ public void testGetResponseHeaders() {
+ assertTrue(req.getResponseHeaders() instanceof AddOnlyMap);
}
/**
diff --git a/tiles-request-servlet/src/main/java/org/apache/tiles/request/servlet/ServletRequest.java b/tiles-request-servlet/src/main/java/org/apache/tiles/request/servlet/ServletRequest.java
index dd08938..3f6cd2f 100644
--- a/tiles-request-servlet/src/main/java/org/apache/tiles/request/servlet/ServletRequest.java
+++ b/tiles-request-servlet/src/main/java/org/apache/tiles/request/servlet/ServletRequest.java
@@ -34,7 +34,7 @@
import org.apache.tiles.request.AbstractClientRequest;
import org.apache.tiles.request.ApplicationContext;
-import org.apache.tiles.request.collection.AddableParameterMap;
+import org.apache.tiles.request.collection.AddOnlyMap;
import org.apache.tiles.request.collection.HeaderValuesMap;
import org.apache.tiles.request.collection.ReadOnlyEnumerationMap;
import org.apache.tiles.request.collection.ScopeMap;
@@ -81,6 +81,12 @@
*/
private Map<String, String> header = null;
+ /**
+ * <p>The lazily instantiated <code>Map</code> of header name-value
+ * combinations (write-only).</p>
+ */
+ private Map<String, String> responseHeaders = null;
+
/**
* <p>The lazily instantitated <code>Map</code> of header name-values
@@ -129,12 +135,21 @@
public Map<String, String> getHeader() {
if ((header == null) && (request != null)) {
- header = new AddableParameterMap(new HeaderExtractor(request, response));
+ header = new ReadOnlyEnumerationMap<String>(new HeaderExtractor(request, null));
}
return (header);
}
+ /** {@inheritDoc} */
+ public Map<String, String> getResponseHeaders() {
+
+ if ((responseHeaders == null) && (response != null)) {
+ responseHeaders = new AddOnlyMap<String>(new HeaderExtractor(null, response));
+ }
+ return (responseHeaders);
+
+ }
/** {@inheritDoc} */
public Map<String, String[]> getHeaderValues() {
diff --git a/tiles-request-servlet/src/test/java/org/apache/tiles/request/servlet/ServletRequestTest.java b/tiles-request-servlet/src/test/java/org/apache/tiles/request/servlet/ServletRequestTest.java
index e19fbb9..93a3d6e 100644
--- a/tiles-request-servlet/src/test/java/org/apache/tiles/request/servlet/ServletRequestTest.java
+++ b/tiles-request-servlet/src/test/java/org/apache/tiles/request/servlet/ServletRequestTest.java
@@ -36,7 +36,7 @@
import javax.servlet.http.HttpServletResponse;
import org.apache.tiles.request.ApplicationContext;
-import org.apache.tiles.request.collection.AddableParameterMap;
+import org.apache.tiles.request.collection.AddOnlyMap;
import org.apache.tiles.request.collection.HeaderValuesMap;
import org.apache.tiles.request.collection.ReadOnlyEnumerationMap;
import org.apache.tiles.request.collection.ScopeMap;
@@ -215,7 +215,15 @@
*/
@Test
public void testGetHeader() {
- assertTrue(req.getHeader() instanceof AddableParameterMap);
+ assertTrue(req.getHeader() instanceof ReadOnlyEnumerationMap);
+ }
+
+ /**
+ * Test method for {@link org.apache.tiles.request.servlet.ServletRequest#getHeader()}.
+ */
+ @Test
+ public void testGetResponseHeaders() {
+ assertTrue(req.getResponseHeaders() instanceof AddOnlyMap);
}
/**