DO NOT REPLY [Bug 36899] New: - validation breaks on multiple validation.xml (eg. with Struts 1.3)

[bu...@apache.org]

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG�
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=36899>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND�
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=36899

           Summary: validation breaks on multiple validation.xml (eg. with
                    Struts 1.3)
           Product: Commons
           Version: 1.2 Final
          Platform: All
        OS/Version: other
            Status: NEW
          Severity: regression
          Priority: P2
         Component: Validator
        AssignedTo: commons-dev@jakarta.apache.org
        ReportedBy: wgehner@infonoia.com
                CC: niall.pemberton@blueyonder.co.uk


While bringing an app to a recent build (0909) of struts 1.3 and
commons-validator-1.2.0, I notice the following related to ValidatorPlugIn:

When using multiple validation files, as in:

<plug-in clasname="org.apache.struts.validator.ValidatorPlugIn">
<set-property property="pathnames"  
value="/WEB-INF/lib/validator-rules.xml,/WEB-INF/moduleA/validationA.xml,/WEB-INF/moduleB/validationB.xml"/>
</plug-in>

I get in the tomcat console:

"WARN Overriding FormSet definition. Duplicate for locale: fr"

Result is that validation rules on moduleA are not working.

I suspect  the message is because both validationA.xml and validationB.xml have
a <formset language="fr">

We used to be able to have one validationX.xml per module without problems.
(Having the bundle attribute since struts 1.2x made that even better, so there
could be one resource-file per module).

I see that now commons-validator 1.2 seems to change this. I fear that without a
fix I would have to go back to validations for 400 screens in one single
validationABCDEF...xml.

It looks like commons-validation 1.2 implementation builds a formset key based
on language, country and variant, but doesn't seem to merge formsets any more.

If my analysis is correct, my suggestion would be
- to either have commons-validator continue to merge formsets for  backward
compatibility, and/or,
- add a namespace or id parameter as in <formset id="moduleA" language="fr">, if
necessary to keep the "physical map" feature.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org