[TAP5-2462] Parent component should be able to reset a Grid regardless of its internal state

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 5.4
• Fix Version/s: 5.4
• Component/s: tapestry-core
• Assignee: Kalle Korhonen
• Reporter: Kalle Korhonen
• Created: 06/Mar/15 06:53
• Updated: 06/Mar/15 17:15
• Resolved: 06/Mar/15 07:01

Description

Calling Grid.reset() from the parent component may result in NullPointerException:
Caused by: java.lang.NullPointerException
at org.apache.tapestry5.corelib.components.Grid.setCurrentPage(Grid.java:600)
at org.apache.tapestry5.corelib.components.Grid.reset(Grid.java:636)
at org.tynamo.examples.simple.pages.List.resetGrid(List.java:83)
at org.tynamo.examples.simple.pages.List.setupRender(List.java)

The issue is that setCurrentPage(1) on Grid.reset() is really unnecessary or at least should be after sortModel.clear(); (lines 636-637)

Issue Links

• relates to: TAP5-2437 NullPointerException when trying to access (read or write) Grid's currentPage