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:10:37 UTC
svn commit: r124150 - /cocoon/branches/BRANCH_2_1_X/src/blocks/forms/java/org/apache/cocoon/forms/formmodel/Repeater.java
Author: sylvain
Date: Tue Jan 4 12:10:34 2005
New Revision: 124150
URL: http://svn.apache.org/viewcvs?view=rev&rev=124150
Log:
fix bug that leads to an ArrayIndexOutOfBoundsException instead of returning null when id is not a number
Modified:
cocoon/branches/BRANCH_2_1_X/src/blocks/forms/java/org/apache/cocoon/forms/formmodel/Repeater.java
Modified: cocoon/branches/BRANCH_2_1_X/src/blocks/forms/java/org/apache/cocoon/forms/formmodel/Repeater.java
Url: http://svn.apache.org/viewcvs/cocoon/branches/BRANCH_2_1_X/src/blocks/forms/java/org/apache/cocoon/forms/formmodel/Repeater.java?view=diff&rev=124150&p1=cocoon/branches/BRANCH_2_1_X/src/blocks/forms/java/org/apache/cocoon/forms/formmodel/Repeater.java&r1=124149&p2=cocoon/branches/BRANCH_2_1_X/src/blocks/forms/java/org/apache/cocoon/forms/formmodel/Repeater.java&r2=124150
==============================================================================
--- cocoon/branches/BRANCH_2_1_X/src/blocks/forms/java/org/apache/cocoon/forms/formmodel/Repeater.java (original)
+++ cocoon/branches/BRANCH_2_1_X/src/blocks/forms/java/org/apache/cocoon/forms/formmodel/Repeater.java Tue Jan 4 12:10:34 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);
}