Commons Validator
  1. Commons Validator
  2. VALIDATOR-62

i18n issue, variant not being picked up by Validator

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: Nightly Builds
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None
    • Environment:

      Operating System: other
      Platform: Other

      Description

      In my i18n application using Struts 1.1-b2, I've used the locale object (using
      the variant) provided to pick different validations rules.

      After upgrading to 1.1-b3, using version 1.0 of commons validator, the
      validations that used to work are now throwing null pointer exceptions (when i
      turn off validation for the specific form in struts-config everything works)

      I was reading the DTD for the 1.0 version of the Commons Validator
      http://jakarta.apache.org/commons/dtds/validator_1_0.dtd, and noticed that the
      formset element, no longer takes in the VARIANT as an attribute. I was
      wondering why that was taken out or if it is taken out, or is there a mistake
      in the DTD?

      I know 1.1b-3 still supports the locale w/ variant as all of my
      ApplicationResources.properties files with variants are being picked up, what
      changed in the the Validator? Please take a look.

      Thanks,

      -Jeremy

        Activity

        Hide
        James Turner added a comment -

        It was left out of the DTD by accident, but it turns out that the Validator
        code has never actually looked at the variant field when selecting formsets,
        something I just discovered looking through the code tonight. It's an easy
        enough fix, but it's in commons-validator, so it doesn't belong here anyway.
        I'll re-assigning it to commons and will fix and test it, then get a 1.0.2
        point release so that it will be fixed in Struts 1.1

        Show
        James Turner added a comment - It was left out of the DTD by accident, but it turns out that the Validator code has never actually looked at the variant field when selecting formsets, something I just discovered looking through the code tonight. It's an easy enough fix, but it's in commons-validator, so it doesn't belong here anyway. I'll re-assigning it to commons and will fix and test it, then get a 1.0.2 point release so that it will be fixed in Struts 1.1
        Hide
        James Turner added a comment -

        Ok, fixed in commons-validator nightly build, it will take a bit of time to
        get a new point release built and update the DTD stored on the web site.

        Show
        James Turner added a comment - Ok, fixed in commons-validator nightly build, it will take a bit of time to get a new point release built and update the DTD stored on the web site.

          People

          • Assignee:
            Unassigned
            Reporter:
            Jeremy Tan
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development