Uploaded image for project: 'Tapestry'
  1. Tapestry
  2. TAPESTRY-1174

Review validation error messages.

    XMLWordPrintableJSON

Details

    • Task
    • Status: Resolved
    • Minor
    • Resolution: Fixed
    • 4.1.1
    • 4.1.1
    • None
    • None

    Description

      The validation error messages have changed between T4.0.x and 4.1.x. The change was made in an attempt to provide a better hint to the user about what made their input invalid. There is some dispute on the user list as to whether the changes made overly complicate things or actually make things easier for the user.

      Moreover, there's dispute about whether changing the messages is considered a backwards-incompatible change. Developers may have come to rely on these messages and not expect the change. In my own case, it broke integration tests I had that checked for the presence of a particular error message.

      There's also dispute over whether the detailed messages are more appropriate for developers or for end users. If this is all the issue comes down to, adding a "debug" property might be a nice compromise for switching between the two (although, maintaining two sets of messages may be cumbersome).

      At the very least, the changes should be reviewed and some general consensus reached about how to move forward.

      Attachments

        Activity

          People

            jkuhnert Jesse Kuhnert
            nirvdrum Kevin Menard
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: