feat(state): review validation for required parameters (#6441)

Author: deguif

composer.json

@@ -34,7 +34,6 @@
         "symfony/property-info": "^6.4 || ^7.0",
         "symfony/serializer": "^6.4 || ^7.0",
         "symfony/translation-contracts": "^3.3",
-        "symfony/type-info": "^7.1",
         "symfony/web-link": "^6.4 || ^7.0",
         "willdurand/negotiation": "^3.0"
     },

src/Metadata/Resource/Factory/ParameterResourceMetadataCollectionFactory.php

@@ -156,10 +156,14 @@ private function addSchemaValidation(Parameter $parameter, ?array $schema = null
     {
         $assertions = [];
 
-        if ($required) {
+        if ($required && false !== ($allowEmptyValue = $openApi?->getAllowEmptyValue())) {
             $assertions[] = new NotNull(message: sprintf('The parameter "%s" is required.', $parameter->getKey()));
         }
 
+        if (false === ($allowEmptyValue ?? $openApi?->getAllowEmptyValue())) {
+            $assertions[] = new NotBlank(allowNull: !$required);
+        }
+
         if (isset($schema['exclusiveMinimum'])) {
             $assertions[] = new GreaterThan(value: $schema['exclusiveMinimum']);
         }
@@ -200,10 +204,6 @@ private function addSchemaValidation(Parameter $parameter, ?array $schema = null
             $assertions[] = new Choice(choices: $schema['enum']);
         }
 
-        if (false === $openApi?->getAllowEmptyValue()) {
-            $assertions[] = new NotBlank(allowNull: !$required);
-        }
-
         if (!$assertions) {
             return $parameter;
         }

tests/Fixtures/app/AppKernel.php

@@ -20,8 +20,6 @@
 use Doctrine\Bundle\DoctrineBundle\DoctrineBundle;
 use Doctrine\Bundle\MongoDBBundle\Command\TailCursorDoctrineODMCommand;
 use Doctrine\Bundle\MongoDBBundle\DoctrineMongoDBBundle;
-use Doctrine\Common\Inflector\Inflector;
-use Doctrine\Inflector\InflectorFactory;
 use FriendsOfBehat\SymfonyExtension\Bundle\FriendsOfBehatSymfonyExtensionBundle;
 use Symfony\Bundle\FrameworkBundle\FrameworkBundle;
 use Symfony\Bundle\FrameworkBundle\Kernel\MicroKernelTrait;