diogo
/
symfony
Archived
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

[Validator] Optimized RecursiveContextualValidator

This commit is contained in:
Bernhard Schussek 2014-03-11 14:45:21 +01:00
parent eed29d8ad3
commit 50bb84d06b
2 changed files with 19 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
src/Symfony/Component/Validator/NodeVisitor/NodeValidationVisitor.php
View File

@ -103,18 +103,18 @@ class NodeValidationVisitor extends AbstractVisitor
$context->markObjectAsValidatedForGroup($objectHash, $groupHash);
}
// Validate normal group
if (!$group instanceof GroupSequence) {
$this->validateNodeForGroup($node, $group, $context, $objectHash);
if ($group instanceof GroupSequence) {
// Traverse group sequence until a violation is generated
$this->traverseGroupSequence($node, $group, $context);
// Skip the group sequence when validating successor nodes
unset($node->groups[$key]);
continue;
}
// Traverse group sequence until a violation is generated
$this->traverseGroupSequence($node, $group, $context);
// Skip the group sequence when validating successor nodes
unset($node->groups[$key]);
// Validate normal group
$this->validateNodeForGroup($node, $group, $context, $objectHash);
}
return true;

23
src/Symfony/Component/Validator/Validator/RecursiveContextualValidator.php
View File

@ -268,6 +268,9 @@ class RecursiveContextualValidator implements ContextualValidatorInterface
*/
private function traverseClassNode($value, ClassMetadataInterface $metadata = null, $propertyPath, array $groups, $cascadedGroups, $traversalStrategy, ExecutionContextInterface $context)
{
// Replace "Default" group by group sequence, if appropriate
$groups = $this->replaceDefaultGroup($value, $metadata, $groups);
$groups = $this->validateNode($value, $value, $metadata, $propertyPath, $groups, $traversalStrategy, $context);
if (0 === count($groups)) {
@ -566,10 +569,6 @@ class RecursiveContextualValidator implements ContextualValidatorInterface
$context->setMetadata($metadata);
$context->setPropertyPath($propertyPath);
if ($metadata instanceof ClassMetadataInterface) {
$groups = $this->replaceDefaultGroup($value, $metadata, $groups);
}
$objectHash = is_object($object) ? spl_object_hash($object) : null;
// if group (=[<G1,G2>,G3,G4]) contains group sequence (=<G1,G2>)
@ -599,18 +598,18 @@ class RecursiveContextualValidator implements ContextualValidatorInterface
$context->markObjectAsValidatedForGroup($objectHash, $groupHash);
}
// Validate normal group
if (!$group instanceof GroupSequence) {
$this->validateNodeForGroup($value, $objectHash, $metadata, $group, $context);
if ($group instanceof GroupSequence) {
// Traverse group sequence until a violation is generated
$this->stepThroughGroupSequence($value, $object, $metadata, $propertyPath, $traversalStrategy, $group, $context);
// Skip the group sequence when validating successor nodes
unset($groups[$key]);
continue;
}
// Traverse group sequence until a violation is generated
$this->stepThroughGroupSequence($value, $object, $metadata, $propertyPath, $traversalStrategy, $group, $context);
// Skip the group sequence when validating successor nodes
unset($groups[$key]);
// Validate normal group
$this->validateNodeForGroup($value, $objectHash, $metadata, $group, $context);
}
return $groups;