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

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=13526

validator-rules.xml shouldn't have "required" as a depends of all the tests

           Summary: validator-rules.xml shouldn't have "required" as a
                    depends of all the tests
           Product: Struts
           Version: Nightly Build
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: Normal
          Priority: Other
         Component: Validator Framework
        AssignedTo: [EMAIL PROTECTED]
        ReportedBy: [EMAIL PROTECTED]


Having the default validator-rules.xml file give all of the default rules 
a "required" depends= removes a lot of the power of Validator by not allowing 
optional fields.  If someone wants a required credit card field, they should 
use depends="required,credit_card" on the field.  As it is defined now, you 
would need to go into validator-rules.xml and remove all the depends= clauses 
to be able to have an optional field.

Also, from what I can see, the JavaScript for these validations treats a blank 
entry as valid, so the JavaScript is inconsistent with having a 
depends="required" on them.

Patch attached to remove the depends=

--
To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>

Reply via email to