You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by jd...@apache.org on 2009/04/12 23:35:48 UTC

svn commit: r764332 - /wicket/trunk/wicket/src/main/java/org/apache/wicket/markup/html/list/PageableListView.java

Author: jdonnerstag
Date: Sun Apr 12 21:35:47 2009
New Revision: 764332

URL: http://svn.apache.org/viewvc?rev=764332&view=rev
Log:
fixed WICKET-2181 Bounds error in PageableListView#getCurrentPage()
Issue: WICKET-2181

Modified:
    wicket/trunk/wicket/src/main/java/org/apache/wicket/markup/html/list/PageableListView.java

Modified: wicket/trunk/wicket/src/main/java/org/apache/wicket/markup/html/list/PageableListView.java
URL: http://svn.apache.org/viewvc/wicket/trunk/wicket/src/main/java/org/apache/wicket/markup/html/list/PageableListView.java?rev=764332&r1=764331&r2=764332&view=diff
==============================================================================
--- wicket/trunk/wicket/src/main/java/org/apache/wicket/markup/html/list/PageableListView.java (original)
+++ wicket/trunk/wicket/src/main/java/org/apache/wicket/markup/html/list/PageableListView.java Sun Apr 12 21:35:47 2009
@@ -34,9 +34,6 @@
  */
 public abstract class PageableListView<T> extends ListView<T> implements IPageable
 {
-	/**
-	 * 
-	 */
 	private static final long serialVersionUID = 1L;
 
 	/** The page to show. */
@@ -87,7 +84,7 @@
 	public final int getCurrentPage()
 	{
 		// If first cell is out of range, bring page back into range
-		while ((currentPage * rowsPerPage) > getList().size())
+		while ((currentPage * rowsPerPage) >= getList().size())
 		{
 			currentPage--;
 		}