svn commit: r762811 - /wicket/trunk/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/repeater/data/table/NavigationToolbar.java

[jdonnerstag]

Author: jdonnerstag
Date: Tue Apr  7 15:10:17 2009
New Revision: 762811

URL: http://svn.apache.org/viewvc?rev=762811&view=rev
Log:
fixed WICKET-2175 onBeforeRender() set visibility of NavigationToolbar prevents 
it from showing again

Modified:
    
wicket/trunk/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/repeater/data/table/NavigationToolbar.java

Modified: 
wicket/trunk/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/repeater/data/table/NavigationToolbar.java
URL: 
http://svn.apache.org/viewvc/wicket/trunk/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/repeater/data/table/NavigationToolbar.java?rev=762811&r1=762810&r2=762811&view=diff
==============================================================================
--- 
wicket/trunk/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/repeater/data/table/NavigationToolbar.java
 (original)
+++ 
wicket/trunk/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/repeater/data/table/NavigationToolbar.java
 Tue Apr  7 15:10:17 2009
@@ -27,7 +27,6 @@
  * about which rows are being displayed and their total number in the data 
table.
  * 
  * @author Igor Vaynberg (ivaynberg)
- * 
  */
 public class NavigationToolbar extends AbstractToolbar
 {
@@ -55,7 +54,6 @@
                span.add(newNavigatorLabel("navigatorLabel", table));
        }
 
-
        /**
         * Factory method used to create the paging navigator that will be used 
by the datatable
         * 
@@ -85,12 +83,19 @@
                return new NavigatorLabel(navigatorId, table);
        }
 
+       /**
+        * @see org.apache.wicket.Component#isVisible()
+        */
+       @Override
+       public boolean isVisible()
+       {
+               return table.getPageCount() > 1;
+       }
 
        /** {...@inheritdoc} */
        @Override
        protected void onBeforeRender()
        {
-               setVisible(table.getPageCount() > 1);
                super.onBeforeRender();
        }
 }