Tapestry 5
  1. Tapestry 5
  2. TAP5-979

Form component should be more careful with the validation tracker to ensure that a session is not created unless needed


    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 5.2.0
    • Fix Version/s: 5.2.0
    • Component/s: tapestry-core
    • Labels:


      Currently, the Form component will create a ValidationTracker instance if one does not already exist when rendering or when handling a form submission. Since, typically, it is using the defaultValidator property, this creates a session to store the ValidationTracker.

      A little more work can yield a ValidationTracker that doesn't uπdate the Form.validator property until an error is recorded. If no errors are recorded for the Form during a submission, then the ValidationTracker is discarded anyway ... this change will just ensure that no session is created unless the tracker is being kept around (for a re-render of the page, to present errors to the user).


        Howard M. Lewis Ship 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 -
        Status In Progress [ 3 ] Closed [ 6 ]
        Fix Version/s 5.2.0 [ 12314122 ]
        Resolution Fixed [ 1 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open In Progress In Progress
        4s 1 Howard M. Lewis Ship 12/Jan/10 17:46
        In Progress In Progress Closed Closed
        1h 42m 1 Howard M. Lewis Ship 12/Jan/10 19:29


          • Assignee:
            Howard M. Lewis Ship
            Howard M. Lewis Ship
          • Votes:
            0 Vote for this issue
            0 Start watching this issue


            • Created: