You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by al...@apache.org on 2007/11/05 00:07:57 UTC
svn commit: r591860 -
/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/repeater/AbstractPageableView.java
Author: almaw
Date: Sun Nov 4 15:07:56 2007
New Revision: 591860
URL: http://svn.apache.org/viewvc?rev=591860&view=rev
Log:
Don't do page range check when we set the page to zero, to avoid DataView, etc. constructors calling getPageCount().
Modified:
wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/repeater/AbstractPageableView.java
Modified: wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/repeater/AbstractPageableView.java
URL: http://svn.apache.org/viewvc/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/repeater/AbstractPageableView.java?rev=591860&r1=591859&r2=591860&view=diff
==============================================================================
--- wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/repeater/AbstractPageableView.java (original)
+++ wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/repeater/AbstractPageableView.java Sun Nov 4 15:07:56 2007
@@ -192,7 +192,7 @@
public String toString()
{
return "ItemsPerPageChange[component: " + getPath() + ", itemsPerPage: " +
- old + "]";
+ old + "]";
}
});
}
@@ -259,10 +259,10 @@
*/
public final void setCurrentPage(int page)
{
- if (page < 0 || (page >= getPageCount() && getPageCount() > 0))
+ if (page != 0 && (page < 0 || (page >= getPageCount() && getPageCount() > 0)))
{
throw new IndexOutOfBoundsException("argument [page]=" + page + ", must be 0<=page<" +
- getPageCount());
+ getPageCount());
}
if (currentPage != page)
@@ -283,7 +283,7 @@
public String toString()
{
return "CurrentPageChange[component: " + getPath() + ", currentPage: " +
- old + "]";
+ old + "]";
}
});