Uploaded image for project: 'Commons Validator'
  1. Commons Validator
  2. VALIDATOR-219

New display method for client validation error

    XMLWordPrintableJSON

Details

    • New Feature
    • Status: Closed
    • Minor
    • Resolution: Won't Fix
    • None
    • None
    • JavaScript
    • None

    Description

      Hi,

      I would like to suggest to make an enhancement to the client side validation process so that the client validation error message is displayed the same way the client side validation messages are displayed. Personally I would prefer making this the default behavour and eliminating the JavaScript alert(). Another option might be to change the value passed through to the client attribute of the s:commonsValidator tag to multiple options like "alert" and "message".

      When message would be chosen, the JavaScript will populate the <div> representing the h:message tag.

      I think this will result in a more elegant notification to the enduser.

      example:
      --------------------------
      <h:inputText id="email" value="#

      {contact.email}

      ">
      <s:commonsValidator type="required" message="#

      {labels['err.requiredField']}

      " server="true" client="message"/>
      </h:inputText>
      <h:message for="email" styleClass="error"/>
      --------------------------

      We currently have a fairly elaborate system in place where empty JavaScript Array's are generated for each validator type. Whenever a value needs to be validated, the input object will be added to the specific validator Array. On submit each array will be checked for their correct values , messages are dysplayed acoringly and the request is stopped.

      How do you think we should implement this feature, and what part of this process belongs in Shale, and what part belongs in the commons validator. I also posted this RFE in the shale issue tracker (SHALE-373).

      Thank you,
      Joost

      Attachments

        Activity

          People

            Unassigned Unassigned
            joostschouten Joost Schouten
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: