You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@cocoon.apache.org by sy...@apache.org on 2005/01/04 21:13:31 UTC
svn commit: r124151 - /cocoon/trunk/src/blocks/forms/java/org/apache/cocoon/forms/formmodel/Repeater.java
Author: sylvain
Date: Tue Jan 4 12:13:29 2005
New Revision: 124151
URL: http://svn.apache.org/viewcvs?view=rev&rev=124151
Log:
fix bug that leads to an ArrayIndexOutOfBoundsException instead of returning null when id is not a number
Modified:
cocoon/trunk/src/blocks/forms/java/org/apache/cocoon/forms/formmodel/Repeater.java
Modified: cocoon/trunk/src/blocks/forms/java/org/apache/cocoon/forms/formmodel/Repeater.java
Url: http://svn.apache.org/viewcvs/cocoon/trunk/src/blocks/forms/java/org/apache/cocoon/forms/formmodel/Repeater.java?view=diff&rev=124151&p1=cocoon/trunk/src/blocks/forms/java/org/apache/cocoon/forms/formmodel/Repeater.java&r1=124150&p2=cocoon/trunk/src/blocks/forms/java/org/apache/cocoon/forms/formmodel/Repeater.java&r2=124151
==============================================================================
--- cocoon/trunk/src/blocks/forms/java/org/apache/cocoon/forms/formmodel/Repeater.java (original)
+++ cocoon/trunk/src/blocks/forms/java/org/apache/cocoon/forms/formmodel/Repeater.java Tue Jan 4 12:13:29 2005
@@ -97,7 +97,7 @@
rowIndex = Integer.parseInt(id);
} catch (NumberFormatException nfe) {
}
- if (rowIndex < 0 && rowIndex > getSize())
+ if (rowIndex < 0 || rowIndex >= getSize())
return null;
return getRow(rowIndex);
}