[FrameworkBundle][Validator] Move the PSR-11 factory to the component

Author: ogizanagi

CHANGELOG.md

@@ -31,7 +31,6 @@ CHANGELOG
  * Deprecated `Symfony\Bundle\FrameworkBundle\DependencyInjection\Compiler\ConfigCachePass`.
    Use `Symfony\Component\Console\DependencyInjection\ConfigCachePass` instead.
  * Deprecated `PropertyInfoPass`, use `Symfony\Component\PropertyInfo\DependencyInjection\PropertyInfoPass` instead
- * Deprecated extending `ConstraintValidatorFactory`
  * Deprecated `ControllerArgumentValueResolverPass`. Use
    `Symfony\Component\HttpKernel\DependencyInjection\ControllerArgumentValueResolverPass` instead
  * Deprecated `RoutingResolverPass`, use `Symfony\Component\Routing\DependencyInjection\RoutingResolverPass` instead
@@ -47,9 +46,10 @@ CHANGELOG
    `Symfony\Component\Validator\DependencyInjection\AddValidatorInitializersPass` instead
  * Deprecated `AddConstraintValidatorsPass`, use
    `Symfony\Component\Validator\DependencyInjection\AddConstraintValidatorsPass` instead
- * Deprecated `ValidateWorkflowsPass`, use 
+ * Deprecated `ValidateWorkflowsPass`, use
    `Symfony\Component\Workflow\DependencyInjection\ValidateWorkflowsPass` instead
- * Deprecated `ConstraintValidatorFactory::__construct()` second argument.
+ * Deprecated `ConstraintValidatorFactory`, use 
+   `Symfony\Component\Validator\ContainerConstraintValidatorFactory` instead.
 
 3.2.0
 -----

Resources/config/validator.xml

@@ -59,7 +59,7 @@
             </argument>
         </service>
 
-        <service id="validator.validator_factory" class="Symfony\Bundle\FrameworkBundle\Validator\ConstraintValidatorFactory">
+        <service id="validator.validator_factory" class="Symfony\Component\Validator\ContainerConstraintValidatorFactory">
             <argument /> <!-- Constraint validators locator -->
         </service>
 

Tests/Validator/ConstraintValidatorFactoryTest.php

@@ -19,6 +19,9 @@
 use Symfony\Component\Validator\Constraints\Blank as BlankConstraint;
 use Symfony\Component\Validator\ConstraintValidator;
 
+/**
+ * @group legacy
+ */
 class ConstraintValidatorFactoryTest extends TestCase
 {
     public function testGetInstanceCreatesValidator()
@@ -27,7 +30,7 @@ public function testGetInstanceCreatesValidator()
 
         $constraint = $this->getMockBuilder('Symfony\\Component\\Validator\\Constraint')->getMock();
         $constraint
-            ->expects($this->once())
+            ->expects($this->exactly(2))
             ->method('validatedBy')
             ->will($this->returnValue($class));
 
@@ -63,18 +66,14 @@ public function testGetInstanceReturnsService()
 
         $constraint = $this->getMockBuilder(Constraint::class)->getMock();
         $constraint
-            ->expects($this->once())
+            ->expects($this->exactly(2))
             ->method('validatedBy')
             ->will($this->returnValue($service));
 
         $factory = new ConstraintValidatorFactory($container);
         $this->assertSame($validator, $factory->getInstance($constraint));
     }
 
-    /**
-     * @group legacy
-     * @expectedDeprecation Passing an array of validators or validator aliases as the second argument of "Symfony\Bundle\FrameworkBundle\Validator\ConstraintValidatorFactory::__construct" is deprecated since 3.3 and will be removed in 4.0. Use the service locator instead.
-     */
     public function testGetInstanceReturnsServiceWithAlias()
     {
         $service = 'validator_constraint_service';
@@ -106,7 +105,7 @@ public function testGetInstanceInvalidValidatorClass()
     {
         $constraint = $this->getMockBuilder('Symfony\\Component\\Validator\\Constraint')->getMock();
         $constraint
-            ->expects($this->once())
+            ->expects($this->exactly(2))
             ->method('validatedBy')
             ->will($this->returnValue('Fully\\Qualified\\ConstraintValidator\\Class\\Name'));
 

Validator/ConstraintValidatorFactory.php

@@ -13,10 +13,12 @@
 
 use Psr\Container\ContainerInterface;
 use Symfony\Component\Validator\Constraint;
-use Symfony\Component\Validator\ConstraintValidatorFactoryInterface;
 use Symfony\Component\Validator\ConstraintValidatorInterface;
-use Symfony\Component\Validator\Exception\ValidatorException;
+use Symfony\Component\Validator\ContainerConstraintValidatorFactory;
 use Symfony\Component\Validator\Exception\UnexpectedTypeException;
+use Symfony\Component\Validator\Exception\ValidatorException;
+
+@trigger_error(sprintf('The %s class is deprecated since version 3.3 and will be removed in 4.0. Use %s instead.', ConstraintValidatorFactory::class, ContainerConstraintValidatorFactory::class), E_USER_DEPRECATED);
 
 /**
  * Uses a service container to create constraint validators.
@@ -38,24 +40,19 @@
  *
  * @author Kris Wallsmith <[email protected]>
  *
- * @final since version 3.3
+ * @deprecated since version 3.3
  */
-class ConstraintValidatorFactory implements ConstraintValidatorFactoryInterface
+class ConstraintValidatorFactory extends ContainerConstraintValidatorFactory
 {
     protected $container;
     protected $validators;
 
-    public function __construct(ContainerInterface $container, array $validators = null)
+    public function __construct(ContainerInterface $container, array $validators = array())
     {
-        $this->container = $container;
-
-        if (null !== $validators) {
-            @trigger_error(sprintf('Passing an array of validators or validator aliases as the second argument of "%s" is deprecated since 3.3 and will be removed in 4.0. Use the service locator instead.', __METHOD__), E_USER_DEPRECATED);
-        } else {
-            $validators = array();
-        }
+        parent::__construct($container);
 
         $this->validators = $validators;
+        $this->container = $container;
     }
 
     /**
@@ -73,17 +70,10 @@ public function getInstance(Constraint $constraint)
         $name = $constraint->validatedBy();
 
         if (!isset($this->validators[$name])) {
-            if ($this->container->has($name)) {
-                $this->validators[$name] = $this->container->get($name);
-            } else {
-                if (!class_exists($name)) {
-                    throw new ValidatorException(sprintf('Constraint validator "%s" does not exist or it is not enabled. Check the "validatedBy" method in your constraint class "%s".', $name, get_class($constraint)));
-                }
+            return parent::getInstance($constraint);
+        }
 
-                $this->validators[$name] = new $name();
-            }
-        } elseif (is_string($this->validators[$name])) {
-            // To be removed in 4.0
+        if (is_string($this->validators[$name])) {
             $this->validators[$name] = $this->container->get($this->validators[$name]);
         }
 

composer.json

@@ -49,7 +49,7 @@
         "symfony/serializer": "~3.3",
         "symfony/translation": "~3.2",
         "symfony/templating": "~2.8|~3.0",
-        "symfony/validator": "~3.3",
+        "symfony/validator": "~3.3-rc2",
         "symfony/workflow": "~3.3",
         "symfony/yaml": "~3.2",
         "symfony/property-info": "~3.3",
@@ -69,7 +69,7 @@
         "symfony/property-info": "<3.3",
         "symfony/serializer": "<3.3",
         "symfony/translation": "<3.2",
-        "symfony/validator": "<3.3",
+        "symfony/validator": "<3.3-rc2",
         "symfony/workflow": "<3.3"
     },
     "suggest": {