Uploaded image for project: 'Tapestry 5'
  1. Tapestry 5
  2. TAP5-2462

Parent component should be able to reset a Grid regardless of its internal state

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 5.4
    • Fix Version/s: 5.4
    • Component/s: tapestry-core
    • Labels:
      None

      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)

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                kaosko Kalle Korhonen
                Reporter:
                kaosko Kalle Korhonen
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: