You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@bval.apache.org by mb...@apache.org on 2011/03/08 20:45:09 UTC

svn commit: r1079504 - /incubator/bval/sandbox/lang3-work/bval-jsr303/src/main/java/org/apache/bval/jsr303/ClassValidator.java

Author: mbenson
Date: Tue Mar  8 19:45:09 2011
New Revision: 1079504

URL: http://svn.apache.org/viewvc?rev=1079504&view=rev
Log:
fix the bug whereby the code supporting CascadingPropertyValidator causes, when validating multiple groups, the validated property's siblings to be erroneously validated as well; see r1079503

Modified:
    incubator/bval/sandbox/lang3-work/bval-jsr303/src/main/java/org/apache/bval/jsr303/ClassValidator.java

Modified: incubator/bval/sandbox/lang3-work/bval-jsr303/src/main/java/org/apache/bval/jsr303/ClassValidator.java
URL: http://svn.apache.org/viewvc/incubator/bval/sandbox/lang3-work/bval-jsr303/src/main/java/org/apache/bval/jsr303/ClassValidator.java?rev=1079504&r1=1079503&r2=1079504&view=diff
==============================================================================
--- incubator/bval/sandbox/lang3-work/bval-jsr303/src/main/java/org/apache/bval/jsr303/ClassValidator.java (original)
+++ incubator/bval/sandbox/lang3-work/bval-jsr303/src/main/java/org/apache/bval/jsr303/ClassValidator.java Tue Mar  8 19:45:09 2011
@@ -757,6 +757,7 @@ public class ClassValidator implements C
                     validateBeanNet(context);
                     if (prop != null) {
                         context.moveUp(bean, prop.getParentMetaBean());
+                        context.setMetaProperty(prop);
                         if (fixed) {
                             context.setFixedValue(value);
                         }
@@ -784,6 +785,7 @@ public class ClassValidator implements C
                         validateBeanNet(context);
                         if (prop != null) {
                             context.moveUp(bean, prop.getParentMetaBean());
+                            context.setMetaProperty(prop);
                             if (fixed) {
                                 context.setFixedValue(value);
                             }