From 5c479d803c462a6455e9f3848c8229eff90801bc Mon Sep 17 00:00:00 2001
From: Bernhard Schussek <bschussek@gmail.com>
Date: Tue, 11 Mar 2014 11:36:07 +0100
Subject: [PATCH] [Validator] Simplified validateNodeForGroup

---
 .../RecursiveContextualValidator.php          | 53 ++++++++-----------
 1 file changed, 22 insertions(+), 31 deletions(-)

diff --git a/src/Symfony/Component/Validator/Validator/RecursiveContextualValidator.php b/src/Symfony/Component/Validator/Validator/RecursiveContextualValidator.php
index acd79165c0..ff6c6e740d 100644
--- a/src/Symfony/Component/Validator/Validator/RecursiveContextualValidator.php
+++ b/src/Symfony/Component/Validator/Validator/RecursiveContextualValidator.php
@@ -646,43 +646,34 @@ class RecursiveContextualValidator implements ContextualValidatorInterface
      */
     private function validateNodeForGroup($value, $objectHash, MetadataInterface $metadata = null, $group, ExecutionContextInterface $context)
     {
-        try {
-            $context->setGroup($group);
+        $context->setGroup($group);
 
-            foreach ($metadata->findConstraints($group) as $constraint) {
-                // Prevent duplicate validation of constraints, in the case
-                // that constraints belong to multiple validated groups
-                if (null !== $objectHash) {
-                    $constraintHash = spl_object_hash($constraint);
+        foreach ($metadata->findConstraints($group) as $constraint) {
+            // Prevent duplicate validation of constraints, in the case
+            // that constraints belong to multiple validated groups
+            if (null !== $objectHash) {
+                $constraintHash = spl_object_hash($constraint);
 
-                    if ($metadata instanceof ClassMetadataInterface) {
-                        if ($context->isClassConstraintValidated($objectHash, $constraintHash)) {
-                            continue;
-                        }
-
-                        $context->markClassConstraintAsValidated($objectHash, $constraintHash);
-                    } elseif ($metadata instanceof PropertyMetadataInterface) {
-                        $propertyName = $metadata->getPropertyName();
-
-                        if ($context->isPropertyConstraintValidated($objectHash, $propertyName, $constraintHash)) {
-                            continue;
-                        }
-
-                        $context->markPropertyConstraintAsValidated($objectHash, $propertyName, $constraintHash);
+                if ($metadata instanceof ClassMetadataInterface) {
+                    if ($context->isClassConstraintValidated($objectHash, $constraintHash)) {
+                        continue;
                     }
-                }
 
-                $validator = $this->validatorFactory->getInstance($constraint);
-                $validator->initialize($context);
-                $validator->validate($value, $constraint);
+                    $context->markClassConstraintAsValidated($objectHash, $constraintHash);
+                } elseif ($metadata instanceof PropertyMetadataInterface) {
+                    $propertyName = $metadata->getPropertyName();
+
+                    if ($context->isPropertyConstraintValidated($objectHash, $propertyName, $constraintHash)) {
+                        continue;
+                    }
+
+                    $context->markPropertyConstraintAsValidated($objectHash, $propertyName, $constraintHash);
+                }
             }
 
-            $context->setGroup(null);
-        } catch (\Exception $e) {
-            // Should be put into a finally block once we switch to PHP 5.5
-            $context->setGroup(null);
-
-            throw $e;
+            $validator = $this->validatorFactory->getInstance($constraint);
+            $validator->initialize($context);
+            $validator->validate($value, $constraint);
         }
     }