Fix deprecations in symfony/validator, including the fix in #2417

Fixes #2416

Author: tjveldhuizen

Controller/Annotations/AbstractScalarParam.php

@@ -41,19 +41,19 @@ public function getConstraints()
         if ($this->requirements instanceof Constraint) {
             $constraints[] = $this->requirements;
         } elseif (is_scalar($this->requirements)) {
-            $constraints[] = new Regex([
-                'pattern' => '#^(?:'.$this->requirements.')$#xsu',
-                'message' => sprintf(
+            $constraints[] = new Regex(
+                '#^(?:'.$this->requirements.')$#xsu',
+                sprintf(
                     'Parameter \'%s\' value, does not match requirements \'%s\'',
                     $this->getName(),
                     $this->requirements
                 ),
-            ]);
+            );
         } elseif (is_array($this->requirements) && isset($this->requirements['rule']) && $this->requirements['error_message']) {
-            $constraints[] = new Regex([
-                'pattern' => '#^(?:'.$this->requirements['rule'].')$#xsu',
-                'message' => $this->requirements['error_message'],
-            ]);
+            $constraints[] = new Regex(
+                '#^(?:'.$this->requirements['rule'].')$#xsu',
+                $this->requirements['error_message'],
+            );
         } elseif (is_array($this->requirements)) {
             foreach ($this->requirements as $index => $requirement) {
                 if ($requirement instanceof Constraint) {
@@ -76,7 +76,7 @@ public function getConstraints()
         // value of the map
         if ($this->map) {
             $constraints = [
-                new All(['constraints' => $constraints]),
+                new All($constraints),
             ];
             if (false === $this->nullable) {
                 $constraints[] = new NotNull();

Controller/Annotations/FileParam.php

@@ -79,15 +79,18 @@ public function getConstraints()
 
         $options = is_array($this->requirements) ? $this->requirements : [];
         if ($this->image) {
-            $constraints[] = new Image($options);
+            $constraint = new Image();
         } else {
-            $constraints[] = new File($options);
+            $constraint = new File();
+        }
+        foreach ($options as $name => $value) {
+            $constraint->$name = $value;
         }
 
         // If the user wants to map the value
         if ($this->map) {
             $constraints = [
-                new All(['constraints' => $constraints]),
+                new All($constraints),
             ];
         }
 

Tests/Controller/Annotations/AbstractScalarParamTest.php

@@ -80,10 +80,10 @@ public function testScalarRequirements()
         $this->param->requirements = 'foo %bar% %%';
         $this->assertEquals([
             new NotNull(),
-            new Regex([
-                'pattern' => '#^(?:foo %bar% %%)$#xsu',
-                'message' => "Parameter 'bar' value, does not match requirements 'foo %bar% %%'",
-            ]),
+            new Regex(
+                '#^(?:foo %bar% %%)$#xsu',
+                "Parameter 'bar' value, does not match requirements 'foo %bar% %%'",
+            ),
         ], $this->param->getConstraints());
     }
 
@@ -95,10 +95,10 @@ public function testArrayRequirements()
         ];
         $this->assertEquals([
             new NotNull(),
-            new Regex([
-                'pattern' => '#^(?:foo)$#xsu',
-                'message' => 'bar',
-            ]),
+            new Regex(
+                '#^(?:foo)$#xsu',
+                'bar',
+            ),
         ], $this->param->getConstraints());
     }
 
@@ -133,8 +133,6 @@ public function testArrayWithNoConstraintsDoesNotCreateInvalidConstraint()
     {
         $this->param->nullable = true;
         $this->param->map = true;
-        $this->assertEquals([new All([
-            'constraints' => [],
-        ])], $this->param->getConstraints());
+        $this->assertEquals([new All([])], $this->param->getConstraints());
     }
 }

Tests/Controller/Annotations/FileParamTest.php

@@ -75,40 +75,40 @@ public function testComplexRequirements()
     public function testFileRequirements()
     {
         $this->param->nullable = true;
-        $this->param->requirements = $requirements = ['mimeTypes' => 'application/json'];
+        $this->param->requirements = ['mimeTypes' => 'application/json'];
         $this->assertEquals([
-            new File($requirements),
+            new File(null, null, null, 'application/json'),
         ], $this->param->getConstraints());
     }
 
     public function testImageRequirements()
     {
         $this->param->image = true;
-        $this->param->requirements = $requirements = ['mimeTypes' => 'image/gif'];
+        $this->param->requirements = ['mimeTypes' => 'image/gif'];
         $this->assertEquals([
             new NotNull(),
-            new Image($requirements),
+            new Image(null, null, null, 'image/gif'),
         ], $this->param->getConstraints());
     }
 
     public function testImageConstraintsTransformWhenParamIsAnArray()
     {
         $this->param->image = true;
         $this->param->map = true;
-        $this->param->requirements = $requirements = ['mimeTypes' => 'image/gif'];
+        $this->param->requirements = ['mimeTypes' => 'image/gif'];
         $this->assertEquals([new All([
             new NotNull(),
-            new Image($requirements),
+            new Image(null, null, null, 'image/gif'),
         ])], $this->param->getConstraints());
     }
 
     public function testFileConstraintsWhenParamIsAnArray()
     {
         $this->param->map = true;
-        $this->param->requirements = $requirements = ['mimeTypes' => 'application/pdf'];
+        $this->param->requirements = ['mimeTypes' => 'application/pdf'];
         $this->assertEquals([new All([
             new NotNull(),
-            new File($requirements),
+            new File(null, null, null, 'application/pdf'),
         ])], $this->param->getConstraints());
     }
 }