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 2007/04/20 00:16:03 UTC
svn commit: r530568 -
/incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/ajax/markup/html/navigation/paging/AjaxPagingNavigator.java
Author: ivaynberg
Date: Thu Apr 19 15:16:02 2007
New Revision: 530568
URL: http://svn.apache.org/viewvc?view=rev&rev=530568
Log:
better automatic decision making as to what add to the ajax target
Modified:
incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/ajax/markup/html/navigation/paging/AjaxPagingNavigator.java
Modified: incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/ajax/markup/html/navigation/paging/AjaxPagingNavigator.java
URL: http://svn.apache.org/viewvc/incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/ajax/markup/html/navigation/paging/AjaxPagingNavigator.java?view=diff&rev=530568&r1=530567&r2=530568
==============================================================================
--- incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/ajax/markup/html/navigation/paging/AjaxPagingNavigator.java (original)
+++ incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/ajax/markup/html/navigation/paging/AjaxPagingNavigator.java Thu Apr 19 15:16:02 2007
@@ -25,6 +25,7 @@
import org.apache.wicket.markup.html.navigation.paging.IPagingLabelProvider;
import org.apache.wicket.markup.html.navigation.paging.PagingNavigation;
import org.apache.wicket.markup.html.navigation.paging.PagingNavigator;
+import org.apache.wicket.markup.repeater.AbstractRepeater;
/**
* A Wicket panel component to draw and maintain a complete page navigator,
@@ -146,21 +147,19 @@
// markup container.
Component container = ((Component)pageable);
- if ((pageable instanceof MarkupContainer) && !(pageable instanceof ListView))
+ // no need for a nullcheck as there is bound to be a non-repeater
+ // somewhere higher in the hierarchy
+ while ((container instanceof AbstractRepeater))
{
- container = (MarkupContainer)pageable;
- }
- else
- {
- container = ((Component)pageable).findParent(MarkupContainer.class);
+ container = container.getParent();
}
target.addComponent(container);
-
+
// in case the navigator is not contained by the container, we have
// to add it to the response
if (((MarkupContainer)container).contains(this, true) == false)
{
target.addComponent(this);
- }
+ }
}
}