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; }