[MYFACES-174] UIData.isAllChildrenAndFacetsValid() resets the rowindex to -1. x:columns component not working

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.1.0
• Fix Version/s: 1.0.9m9
• Assignee: Matthias Wessendorf
• Reporter: Mathias Broekelmann
• Created: 07/Apr/05 18:58
• Updated: 17/Apr/06 20:48
• Resolved: 13/Apr/05 23:18

Description

UIData.isAllChildrenAndFacetsValid() resets the rowindex to -1 which prevents x:columns component to work right. The example application for the crosstable doesn´t work when a new column is added or the values are modified.

Here is a fix which will set the rowindex to it´s previous value:

Index: UIData.java
===================================================================
RCS file: /home/cvspublic/incubator-myfaces/src/jsfapi/javax/faces/component/UIData.java,v
retrieving revision 1.30
diff -u -r1.30 UIData.java
— UIData.java 24 Mar 2005 16:46:02 -0000 1.30
+++ UIData.java 7 Apr 2005 09:47:28 -0000
@@ -627,6 +627,7 @@

{ last = first + rows; }

+ int setRowIndex = getRowIndex();
try

{ for (int rowIndex = first; rowIndex < last; rowIndex++) @@ -643,7 +644,7 @@ }

finally

{ - setRowIndex(-1); + setRowIndex(setRowIndex); }

return true;
}