Tapestry
  1. Tapestry
  2. TAPESTRY-2354

Use "flash" persistence on Form's ValidationTracker

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 5.0.11
    • Fix Version/s: 5.0.12
    • Component/s: None
    • Labels:
      None

      Description

      Currently, a Form's ValidationTracker inherits the default persistence strategy. The result is that by default the session becomes littered with ValidationTrackerImpl objects as you move from page to page.

      In response to this problem there have been several workarounds proposed in the mailing lists. They all involve making the ValidationTracker use "flash" persistence.

      Consequently, I'm proposing that we make "flash" the default for Form's ValidationTracker so we can all forget about it. After all, it would be very unusual to want a form's validation errors to be retained after the form has been displayed, regardless of what persistence strategy makes sense for any other component of the form.

      If there remains a situation in which it makes sense to override it, then perhaps that could be possible through a new parameter of the Form component.

        Activity

        Geoff Callender created issue -
        Howard M. Lewis Ship made changes -
        Field Original Value New Value
        Assignee Howard M. Lewis Ship [ hlship ]
        Howard M. Lewis Ship made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Howard M. Lewis Ship made changes -
        Fix Version/s 5.0.12 [ 12313048 ]
        Status In Progress [ 3 ] Closed [ 6 ]
        Resolution Fixed [ 1 ]
        Mark Thomas made changes -
        Workflow jira [ 12428921 ] Default workflow, editable Closed status [ 12568459 ]
        Mark Thomas made changes -
        Workflow Default workflow, editable Closed status [ 12568459 ] jira [ 12591510 ]

          People

          • Assignee:
            Howard M. Lewis Ship
            Reporter:
            Geoff Callender
          • Votes:
            4 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development