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

[ivaynberg]

Author: ivaynberg
Date: Fri Jun 10 06:42:40 2011
New Revision: 1134210

URL: http://svn.apache.org/viewvc?rev=1134210&view=rev
Log:

Issue: WICKET-3777

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

Modified: 
wicket/trunk/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/repeater/data/table/DataTable.java
URL: 
http://svn.apache.org/viewvc/wicket/trunk/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/repeater/data/table/DataTable.java?rev=1134210&r1=1134209&r2=1134210&view=diff
==============================================================================
--- 
wicket/trunk/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/repeater/data/table/DataTable.java
 (original)
+++ 
wicket/trunk/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/repeater/data/table/DataTable.java
 Fri Jun 10 06:42:40 2011
@@ -424,12 +424,9 @@ public class DataTable<T> extends Panel 
                }
 
                @Override
-               public boolean isVisible()
+               public void onConfigure()
                {
-                       if (!super.isVisible())
-                       {
-                               return false;
-                       }
+                       super.onConfigure();
 
                        toolbars.configure();
 
@@ -448,7 +445,11 @@ public class DataTable<T> extends Panel 
                                        }
                                }
                        });
-                       return visible == Boolean.TRUE;
+                       if (visible == null)
+                       {
+                               visible = false;
+                       }
+                       setVisible(visible);
                }
        }
 }