[WICKET-808] renderIterator of ListView and Loop should use the number of children rather than the current size of the model

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.3.0-beta2
• Fix Version/s: 1.3.0-beta3
• Assignee: Eelco Hillenius
• Reporter: Eelco Hillenius
• Created: 01/Aug/07 21:37
• Updated: 02/Aug/07 18:14
• Resolved: 02/Aug/07 18:14

Description

Currently, when you provide null elements in the model (renderIterator), you'll get a NPE exception further down the line (in ListView#renderItem for instance). We should either throw an meaningful exception earlier, or we should make repeaters more lenient and just print out a warning message. I opt for the latter. This should be clear enough when developing (you probably would wonder where your data is to start with) and a lot better than unexpectedly getting NPEs in a production systems in case you missed it.