You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by jb...@apache.org on 2007/03/28 17:26:17 UTC
svn commit: r523364 -
/incubator/wicket/branches/wicket-1.x/jdk-1.4/wicket/src/main/java/wicket/markup/repeater/DefaultItemReuseStrategy.java
Author: jbq
Date: Wed Mar 28 08:26:07 2007
New Revision: 523364
URL: http://svn.apache.org/viewvc?view=rev&rev=523364
Log:
Throw a meaningful exception if next item is not an IModel
Modified:
incubator/wicket/branches/wicket-1.x/jdk-1.4/wicket/src/main/java/wicket/markup/repeater/DefaultItemReuseStrategy.java
Modified: incubator/wicket/branches/wicket-1.x/jdk-1.4/wicket/src/main/java/wicket/markup/repeater/DefaultItemReuseStrategy.java
URL: http://svn.apache.org/viewvc/incubator/wicket/branches/wicket-1.x/jdk-1.4/wicket/src/main/java/wicket/markup/repeater/DefaultItemReuseStrategy.java?view=diff&rev=523364&r1=523363&r2=523364
==============================================================================
--- incubator/wicket/branches/wicket-1.x/jdk-1.4/wicket/src/main/java/wicket/markup/repeater/DefaultItemReuseStrategy.java (original)
+++ incubator/wicket/branches/wicket-1.x/jdk-1.4/wicket/src/main/java/wicket/markup/repeater/DefaultItemReuseStrategy.java Wed Mar 28 08:26:07 2007
@@ -18,6 +18,7 @@
import java.util.Iterator;
+import wicket.WicketRuntimeException;
import wicket.model.IModel;
/**
@@ -66,7 +67,10 @@
public Object next()
{
- final IModel model = (IModel)newModels.next();
+ Object next = newModels.next();
+ if (! (next instanceof IModel))
+ throw new WicketRuntimeException("Expecting an instance of " + IModel.class.getName() + ", got " + next.getClass().getName());
+ final IModel model = (IModel)next;
Item item = factory.newItem(index, model);
index++;