[TRINIDAD-2158] NodeIterator iterates the colection more no of times than the size of the collection.

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.0.1-core
• Fix Version/s: 2.0.1-core
• Component/s: Components
• Assignee: Andrew Robinson
• Reporter: Paresh Kumar Acharya
• Created: 09/Nov/11 11:58
• Updated: 21/Feb/12 20:21
• Resolved: 05/Dec/11 16:23

Description

The class org.apache.myfaces.trinidad.model.RowKeySetTreeImpl contains two Iterator Implementations PathIterator and NodeIterator.
When the NodeIterator is used interanally to iterate the selected Tree or TreeTable nodes(rows), the iterator iterates more no of times than the total number of the slected nodes(rows). The iterator.next() returns the same node(row) more than once.

This issue occurs because of the follwoing 2 reasons.
1) The currentIterator instance is not updated by taking the next iterator from the iteratorStack when one node completeley iterator.
2) Even when a node doesn't have any child nodes, iterator instances are created and pushed to the iteratorStack.