[hibernate-commits] Hibernate SVN: r16954 - validator/trunk/hibernate-validator/src/main/java/org/hibernate/validation/engine.

[hibernate-commits at lists.jboss.org]

Author: hardy.ferentschik
Date: 2009-06-26 04:23:23 -0400 (Fri, 26 Jun 2009)
New Revision: 16954

Modified:
   validator/trunk/hibernate-validator/src/main/java/org/hibernate/validation/engine/ValidatorImpl.java
Log:
Moved the marking of a processed group into the for loop where it belongs.

Modified: validator/trunk/hibernate-validator/src/main/java/org/hibernate/validation/engine/ValidatorImpl.java
===================================================================
--- validator/trunk/hibernate-validator/src/main/java/org/hibernate/validation/engine/ValidatorImpl.java	2009-06-26 08:14:34 UTC (rev 16953)
+++ validator/trunk/hibernate-validator/src/main/java/org/hibernate/validation/engine/ValidatorImpl.java	2009-06-26 08:23:23 UTC (rev 16954)
@@ -206,7 +206,7 @@
 
 		BeanMetaData<U> beanMetaData = getBeanMetaData( localExecutionContext.getCurrentBeanType() );
 		if ( beanMetaData.defaultGroupSequenceIsRedefined() ) {
-		  	groupChain.assertDefaulGroupSequenceIsExpandable(beanMetaData.getDefaultGroupSequence());
+			groupChain.assertDefaulGroupSequenceIsExpandable( beanMetaData.getDefaultGroupSequence() );
 		}
 
 		// process first single groups. For these we can skip some object traversal, by first running all validations on the current bean
@@ -291,12 +291,12 @@
 				boolean tmp = metaConstraint.validateConstraint( globalExecutionContext, localExecutionContext );
 				validationSuccessful = validationSuccessful && tmp;
 			}
+			globalExecutionContext.markProcessed(
+					localExecutionContext.getCurrentBean(),
+					localExecutionContext.getCurrentGroup(),
+					localExecutionContext.getPropertyPath()
+			);
 		}
-		globalExecutionContext.markProcessed(
-				localExecutionContext.getCurrentBean(),
-				localExecutionContext.getCurrentGroup(),
-				localExecutionContext.getPropertyPath()
-		);
 		return validationSuccessful;
 	}