feat(metadata/doctrine): Use Type of TypeInfo instead of PropertyInfo

composer.json

@@ -114,7 +114,8 @@
         "symfony/property-info": "^6.4 || ^7.1",
         "symfony/serializer": "^6.4 || ^7.0",
         "symfony/translation-contracts": "^3.3",
-        "symfony/web-link": "^6.4 || ^7.1",
+        "symfony/type-info": "^7.3-dev",
+        "symfony/web-link": "^6.4 || ^7.0",
         "willdurand/negotiation": "^3.1"
     },
     "require-dev": {
@@ -164,7 +165,7 @@
         "symfony/console": "^6.4 || ^7.0",
         "symfony/css-selector": "^6.4 || ^7.0",
         "symfony/dependency-injection": "^6.4 || ^7.0",
-        "symfony/doctrine-bridge": "^6.4.2 || ^7.0.2",
+        "symfony/doctrine-bridge": "^6.4.2 || ^7.1",
         "symfony/dom-crawler": "^6.4 || ^7.0",
         "symfony/error-handler": "^6.4 || ^7.0",
         "symfony/event-dispatcher": "^6.4 || ^7.0",
@@ -208,5 +209,11 @@
         "symfony/web-profiler-bundle": "To use the data collector.",
         "webonyx/graphql-php": "To support GraphQL."
     },
-    "type": "library"
+    "type": "library",
+    "repositories": [
+        {
+            "type": "vcs",
+            "url": "https://github.com/symfony/type-info"
+        }
+    ]
 }

docs/composer.json

@@ -25,6 +25,7 @@
         "symfony/property-info": "^7.0",
         "symfony/runtime": "^7.0",
         "symfony/security-bundle": "^7.0",
+        "symfony/type-info": "^7.3-dev",
         "symfony/serializer": "^7.0",
         "symfony/validator": "^7.0",
         "symfony/yaml": "^7.0",
@@ -50,5 +51,11 @@
             "name": "api-platform/api-platform",
             "url": "https://github.com/api-platform/api-platform"
         }
-    }
+    },
+    "repositories": [
+        {
+            "type": "vcs",
+            "url": "https://github.com/symfony/type-info"
+        }
+    ]
 }

docs/guides/create-a-custom-doctrine-filter.php

@@ -26,7 +26,6 @@
     use ApiPlatform\Doctrine\Orm\Util\QueryNameGeneratorInterface;
     use ApiPlatform\Metadata\Operation;
     use Doctrine\ORM\QueryBuilder;
-    use Symfony\Component\PropertyInfo\Type;
 
     final class RegexpFilter extends AbstractFilter
     {
@@ -67,7 +66,7 @@
             foreach ($this->properties as $property => $strategy) {
                 $description["regexp_$property"] = [
                     'property' => $property,
-                    'type' => Type::BUILTIN_TYPE_STRING,
+                    'type' => 'string',
                     'required' => false,
                     'description' => 'Filter using a regex. This will appear in the OpenAPI documentation!',
                     'openapi' => [

src/Doctrine/Common/composer.json

@@ -34,7 +34,8 @@
         "doctrine/mongodb-odm": "^2.10",
         "doctrine/orm": "^2.17 || ^3.0",
         "phpspec/prophecy-phpunit": "^2.2",
-        "phpunit/phpunit": "^11.2"
+        "phpunit/phpunit": "^11.2",
+        "symfony/type-info": "^7.3-dev"
     },
     "conflict": {
         "doctrine/persistence": "<1.3"
@@ -73,5 +74,11 @@
     },
     "scripts": {
         "test": "./vendor/bin/phpunit"
-    }
+    },
+    "repositories": [
+        {
+            "type": "vcs",
+            "url": "https://github.com/symfony/type-info"
+        }
+    ]
 }

src/Doctrine/Odm/PropertyInfo/DoctrineExtractor.php

@@ -13,7 +13,6 @@
 
 namespace ApiPlatform\Doctrine\Odm\PropertyInfo;
 
-use ApiPlatform\Metadata\Util\PropertyInfoToTypeInfoHelper;
 use Doctrine\Common\Collections\Collection;
 use Doctrine\ODM\MongoDB\Mapping\ClassMetadata as MongoDbClassMetadata;
 use Doctrine\ODM\MongoDB\Types\Type as MongoDbType;
@@ -25,6 +24,7 @@
 use Symfony\Component\PropertyInfo\PropertyTypeExtractorInterface;
 use Symfony\Component\PropertyInfo\Type as LegacyType;
 use Symfony\Component\TypeInfo\Type;
+use Symfony\Component\TypeInfo\TypeIdentifier;
 
 /**
  * Extracts data using Doctrine MongoDB ODM metadata.
@@ -52,13 +52,71 @@
         return $metadata->getFieldNames();
     }
 
+    public function getType(string $class, string $property, array $context = []): ?Type
+    {
+        if (null === $metadata = $this->getMetadata($class)) {
+            return null;
+        }
+
+        if ($metadata->hasAssociation($property)) {
+            /** @var class-string|null */
+            $class = $metadata->getAssociationTargetClass($property);
+
+            if (null === $class) {
+                return null;
+            }
+
+            if ($metadata->isSingleValuedAssociation($property)) {
+                $nullable = $metadata instanceof MongoDbClassMetadata && $metadata->isNullable($property);
+
+                return $nullable ? Type::nullable(Type::object($class)) : Type::object($class);
+            }
+
+            return Type::collection(Type::object(Collection::class), Type::object($class), Type::int());
+        }
+
+        if (!$metadata->hasField($property)) {
+            return null;
+        }
+
+        $typeOfField = $metadata->getTypeOfField($property);
+
+        if (!$typeIdentifier = $this->getTypeIdentifier($typeOfField)) {
+            return null;
+        }
+
+        $nullable = $metadata instanceof MongoDbClassMetadata && $metadata->isNullable($property);
+        $enumType = null;
+
+        if (null !== $enumClass = $metadata instanceof MongoDbClassMetadata ? $metadata->getFieldMapping($property)['enumType'] ?? null : null) {
+            $enumType = $nullable ? Type::nullable(Type::enum($enumClass)) : Type::enum($enumClass);
+        }
+
+        $builtinType = $nullable ? Type::nullable(Type::builtin($typeIdentifier)) : Type::builtin($typeIdentifier);
+
+        $type = match ($typeOfField) {
+            MongoDbType::DATE => Type::object(\DateTime::class),
+            MongoDbType::DATE_IMMUTABLE => Type::object(\DateTimeImmutable::class),
+            MongoDbType::HASH => Type::array(),
+            MongoDbType::COLLECTION => Type::list(),
+            MongoDbType::INT, MongoDbType::INTEGER, MongoDbType::STRING => $enumType ? $enumType : $builtinType,
+            default => $builtinType,
+        };
+
+        return $nullable ? Type::nullable($type) : $type;
+    }
+
     /**
      * {@inheritdoc}
      *
+     * // deprecated since 4.2 use "getType" instead
+     *
      * @return LegacyType[]|null
      */
-    public function getTypes(string $class, string $property, array $context = []): ?array
+    public function getTypes($class, $property, array $context = []): ?array
     {
+        trigger_deprecation('api-platform/core', '4.2', 'The "%s()" method is deprecated, use "%s::getType()" instead.', __METHOD__, self::class);
+
         if (null === $metadata = $this->getMetadata($class)) {
             return null;
         }
@@ -115,7 +173,7 @@
                     }
             }
 
-            $builtinType = $this->getPhpType($typeOfField);
+            $builtinType = $this->getNativeTypeLegacy($typeOfField);
 
             return $builtinType ? [new LegacyType($builtinType, $nullable)] : null;
         }
@@ -156,15 +214,23 @@
         }
     }
 
-    public function getType(string $class, string $property, array $context = []): ?Type
+    /**
+     * Gets the corresponding built-in PHP type identifier.
+     */
+    private function getTypeIdentifier(string $doctrineType): ?TypeIdentifier
     {
-        return PropertyInfoToTypeInfoHelper::convertLegacyTypesToType($this->getTypes($class, $property, $context));
+        return match ($doctrineType) {
+            MongoDbType::INTEGER, MongoDbType::INT, MongoDbType::INTID, MongoDbType::KEY => TypeIdentifier::INT,
+            MongoDbType::FLOAT => TypeIdentifier::FLOAT,
+            MongoDbType::STRING, MongoDbType::ID, MongoDbType::OBJECTID, MongoDbType::TIMESTAMP, MongoDbType::BINDATA, MongoDbType::BINDATABYTEARRAY, MongoDbType::BINDATACUSTOM, MongoDbType::BINDATAFUNC, MongoDbType::BINDATAMD5, MongoDbType::BINDATAUUID, MongoDbType::BINDATAUUIDRFC4122 => TypeIdentifier::STRING,
+            MongoDbType::BOOLEAN, MongoDbType::BOOL => TypeIdentifier::BOOL,
+            MongoDbType::DATE, MongoDbType::DATE_IMMUTABLE => TypeIdentifier::OBJECT,
+            MongoDbType::HASH, MongoDbType::COLLECTION => TypeIdentifier::ARRAY,
+            default => null,
+        };
     }
 
-    /**
-     * Gets the corresponding built-in PHP type.
-     */
-    private function getPhpType(string $doctrineType): ?string
+    private function getNativeTypeLegacy(string $doctrineType): ?string
     {
         return match ($doctrineType) {
             MongoDbType::INTEGER, MongoDbType::INT, MongoDbType::INTID, MongoDbType::KEY => LegacyType::BUILTIN_TYPE_INT,