Author: ivaynberg Date: Wed Aug 8 14:10:03 2007 New Revision: 564022 URL: http://svn.apache.org/viewvc?view=rev&rev=564022 Log: extra check just for cheeser
Modified: wicket/trunk/jdk-1.4/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/repeater/data/table/DataTable.java Modified: wicket/trunk/jdk-1.4/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/repeater/data/table/DataTable.java URL: http://svn.apache.org/viewvc/wicket/trunk/jdk-1.4/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/repeater/data/table/DataTable.java?view=diff&rev=564022&r1=564021&r2=564022 ============================================================================== --- wicket/trunk/jdk-1.4/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/repeater/data/table/DataTable.java (original) +++ wicket/trunk/jdk-1.4/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/repeater/data/table/DataTable.java Wed Aug 8 14:10:03 2007 @@ -75,14 +75,14 @@ static abstract class CssAttributeBehavior extends AbstractBehavior { protected abstract String getCssClass(); - + /** * @see IBehavior#onComponentTag(Component, ComponentTag) */ public void onComponentTag(Component component, ComponentTag tag) { String className = getCssClass(); - if (!Strings.isEmpty(className)) + if (!Strings.isEmpty(className)) { CharSequence oldClassName = tag.getString("class"); if (Strings.isEmpty(oldClassName)) @@ -128,6 +128,11 @@ { super(id); + if (columns == null || columns.length < 1) + { + throw new IllegalArgumentException("Argument `columns` cannot be null or empty"); + } + this.columns = columns; datagrid = new DataGridView("rows", columns, dataProvider) @@ -140,10 +145,10 @@ final IColumn column = DataTable.this.columns[index]; if (column instanceof IStyledColumn) { - item.add(new DataTable.CssAttributeBehavior() + item.add(new DataTable.CssAttributeBehavior() { private static final long serialVersionUID = 1L; - + protected String getCssClass() { return ((IStyledColumn)column).getCssClass();