commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Lorenzo Dee (JIRA)" <>
Subject [jira] Updated: (VALIDATOR-281) Indexed property validation stops on first failed element
Date Wed, 18 Nov 2009 05:03:39 GMT


Lorenzo Dee updated VALIDATOR-281:


Here's a simple fix I did. I just modified line 912 ( Instead of returning, I
changed it to break out of the while-loop to continue validating the next element of the indexed
list property.

> Indexed property validation stops on first failed element
> ---------------------------------------------------------
>                 Key: VALIDATOR-281
>                 URL:
>             Project: Commons Validator
>          Issue Type: Improvement
>          Components: Framework
>    Affects Versions: 1.1.4 Release
>            Reporter: Lorenzo Dee
>            Priority: Trivial
>         Attachments:
> I'm using Struts 1.2.9 and its validator framework. The validation of an indexed property
stops on the first element that fails vaildation. For example, the following list of postal
addresses is being validated:
> addr[0].city
> addr[0].state
> addr[0].zip
> addr[1].city
> addr[1].state
> addr[1].zip
> addr[2].city
> addr[2].state
> addr[2].zip
> My validation.xml contains something like:
> <form name="...">
>     <field property="city" indexedListProperty="addresses" depends="...">
>         . . .
>     </field>
> </form>
> When validating the city fields, it would go from addresses[0].city upto addresses[2].city.
But when validation fails at addr[0].city, the rest is not being validated anymore (i.e. addr[1].city
and addr[2].city are not validated). It would be better if addr[1].city and addr[2].city fields
are validated, even if one of them fails.
> Note that this behavior also affects state and zip fields. Say for example, if addr[1].state
fails validation, it will no longer continue to validate addr[2].state.

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

View raw message