Author: knopp
Date: Wed May  7 02:34:53 2008
New Revision: 654054

URL: http://svn.apache.org/viewvc?rev=654054&view=rev
Log:
WICKET-1475
added check for single column being used in multiple tree tables 

Modified:
    
wicket/trunk/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/tree/table/AbstractColumn.java

Modified: 
wicket/trunk/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/tree/table/AbstractColumn.java
URL: 
http://svn.apache.org/viewvc/wicket/trunk/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/tree/table/AbstractColumn.java?rev=654054&r1=654053&r2=654054&view=diff
==============================================================================
--- 
wicket/trunk/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/tree/table/AbstractColumn.java
 (original)
+++ 
wicket/trunk/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/tree/table/AbstractColumn.java
 Wed May  7 02:34:53 2008
@@ -25,28 +25,28 @@
 
 /**
  * Convenience class for building tree columns.
- * 
+ *
  * @author Matej Knopp
  */
 public abstract class AbstractColumn implements IColumn
 {
        /**
-        * 
+        *
         */
        private static final long serialVersionUID = 1L;
 
-       private String header;
+       private final String header;
 
-       private ColumnLocation location;
+       private final ColumnLocation location;
 
        private TreeTable treeTable = null;
 
        /**
         * Creates the tree column.
-        * 
+        *
         * @param location
         *            Specifies how the column should be aligned and what his 
size should be
-        * 
+        *
         * @param header
         *            Header caption
         */
@@ -93,13 +93,17 @@
         */
        public void setTreeTable(TreeTable treeTable)
        {
+               if (this.treeTable != null && this.treeTable != treeTable)
+               {
+                       throw new IllegalStateException("You can't add single 
IColumn to multiple tree tables.");
+               }
                this.treeTable = treeTable;
        }
 
        /**
         * Returns the tree table that this columns belongs to. If you call 
this method from constructor
         * it will return null, as the column is constructed before the tree is.
-        * 
+        *
         * @return The tree table this column belongs to
         */
        protected TreeTable getTreeTable()


Reply via email to