You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by iv...@apache.org on 2011/01/25 21:28:39 UTC

svn commit: r1063442 - /wicket/trunk/wicket-core/src/main/java/org/apache/wicket/request/mapper/AbstractBookmarkableMapper.java

Author: ivaynberg
Date: Tue Jan 25 20:28:39 2011
New Revision: 1063442

URL: http://svn.apache.org/viewvc?rev=1063442&view=rev
Log:
added todo

Modified:
    wicket/trunk/wicket-core/src/main/java/org/apache/wicket/request/mapper/AbstractBookmarkableMapper.java

Modified: wicket/trunk/wicket-core/src/main/java/org/apache/wicket/request/mapper/AbstractBookmarkableMapper.java
URL: http://svn.apache.org/viewvc/wicket/trunk/wicket-core/src/main/java/org/apache/wicket/request/mapper/AbstractBookmarkableMapper.java?rev=1063442&r1=1063441&r2=1063442&view=diff
==============================================================================
--- wicket/trunk/wicket-core/src/main/java/org/apache/wicket/request/mapper/AbstractBookmarkableMapper.java (original)
+++ wicket/trunk/wicket-core/src/main/java/org/apache/wicket/request/mapper/AbstractBookmarkableMapper.java Tue Jan 25 20:28:39 2011
@@ -250,6 +250,8 @@ public abstract class AbstractBookmarkab
 	 */
 	public Url mapHandler(IRequestHandler requestHandler)
 	{
+		// TODO see if we can refactor this to remove dependency on instanceof checks below and
+		// eliminate the need for IRequestHandlerDelegate
 		if (requestHandler instanceof IRequestHandlerDelegate)
 		{
 			requestHandler = ((IRequestHandlerDelegate)requestHandler).getDelegateHandler();