You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by kn...@apache.org on 2007/10/18 23:05:10 UTC

svn commit: r586135 - /wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/BookmarkablePageLink.java

Author: knopp
Date: Thu Oct 18 14:05:09 2007
New Revision: 586135

URL: http://svn.apache.org/viewvc?rev=586135&view=rev
Log:
WICKET-1084

Modified:
    wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/BookmarkablePageLink.java

Modified: wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/BookmarkablePageLink.java
URL: http://svn.apache.org/viewvc/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/BookmarkablePageLink.java?rev=586135&r1=586134&r2=586135&view=diff
==============================================================================
--- wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/BookmarkablePageLink.java (original)
+++ wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/BookmarkablePageLink.java Thu Oct 18 14:05:09 2007
@@ -44,7 +44,7 @@
 	private String pageMapName = null;
 
 	/** The parameters to pass to the class constructor when instantiated. */
-	protected final MiniMap parameters;
+	protected MiniMap parameters;
 
 	/**
 	 * Constructor.
@@ -63,12 +63,7 @@
 	{
 		if (parameters != null)
 		{
-			MiniMap map = new MiniMap(parameters.keySet().size());
-			for (Iterator i = parameters.entrySet().iterator(); i.hasNext();)
-			{
-				Entry entry = (Entry)i.next();
-				map.put(entry.getKey(), entry.getValue());
-			}
+			MiniMap map = new MiniMap(parameters, parameters.keySet().size());
 			return map;
 		}
 		else
@@ -92,6 +87,13 @@
 		return result;
 	}
 
+	private void setParameterImpl(String key, Object value)
+	{
+		PageParameters parameters = getPageParameters();
+		parameters.put(key, value);
+		this.parameters = pageParametersToMiniMap(parameters);
+	}
+
 	/**
 	 * Constructor.
 	 * 
@@ -201,7 +203,7 @@
 	 */
 	public BookmarkablePageLink setParameter(final String property, final int value)
 	{
-		parameters.put(property, Integer.toString(value));
+		setParameterImpl(property, Integer.toString(value));
 		return this;
 	}
 
@@ -216,7 +218,7 @@
 	 */
 	public BookmarkablePageLink setParameter(final String property, final long value)
 	{
-		parameters.put(property, Long.toString(value));
+		setParameterImpl(property, Long.toString(value));
 		return this;
 	}
 
@@ -231,7 +233,7 @@
 	 */
 	public BookmarkablePageLink setParameter(final String property, final String value)
 	{
-		parameters.put(property, value);
+		setParameterImpl(property, value);
 		return this;
 	}