Merge 4.2

Author: soyuka

features/filter/filter_validation.feature

@@ -16,7 +16,7 @@ Feature: Validate filters based upon filter description
   Scenario: Required filter should throw an error if not set
     When I am on "/filter_validators"
     Then the response status code should be 422
-    And the JSON node "detail" should be equal to 'required: This value should not be blank.\nrequired-allow-empty: The parameter "required-allow-empty" is required.'
+    And the JSON node "detail" should be equal to 'required: This value should not be blank.\nrequired-allow-empty: This value should not be null.'
 
   Scenario: Required filter should not throw an error if set
     When I am on "/array_filter_validators?arrayRequired[]=foo&indexedArrayRequired[foo]=foo"

features/http_cache/headers.feature

@@ -9,4 +9,4 @@ Feature: Default values of HTTP cache headers
     Then the response status code should be 200
     And the header "Etag" should be equal to '"032297ac74d75a50"'
     And the header "Cache-Control" should be equal to "max-age=60, public, s-maxage=3600"
-    And the header "Vary" should be equal to "Accept, Cookie"
+    And the header "Vary" should be equal to "Accept, Cookie, Accept-Language"

src/Doctrine/Orm/Extension/ParameterExtension.php

@@ -15,6 +15,7 @@
 
 use ApiPlatform\Doctrine\Common\Filter\LoggerAwareInterface;
 use ApiPlatform\Doctrine\Common\Filter\ManagerRegistryAwareInterface;
+use ApiPlatform\Doctrine\Common\Filter\PropertyAwareFilterInterface;
 use ApiPlatform\Doctrine\Common\ParameterValueExtractorTrait;
 use ApiPlatform\Doctrine\Orm\Filter\AbstractFilter;
 use ApiPlatform\Doctrine\Orm\Filter\FilterInterface;
@@ -75,19 +76,21 @@ private function applyFilter(QueryBuilder $queryBuilder, QueryNameGeneratorInter
                 $filter->setLogger($this->logger);
             }
 
-            if ($filter instanceof AbstractFilter && !$filter->getProperties()) {
+            if ($filter instanceof PropertyAwareFilterInterface) {
+                $properties = [];
                 $propertyKey = $parameter->getProperty() ?? $parameter->getKey();
-
-                if (str_contains($propertyKey, ':property')) {
-                    $extraProperties = $parameter->getExtraProperties()['_properties'] ?? [];
-                    foreach (array_keys($extraProperties) as $property) {
-                        $properties[$property] = $parameter->getFilterContext();
+                if ($filter instanceof AbstractFilter) {
+                    $properties = $filter->getProperties() ?? [];
+
+                    if (str_contains($propertyKey, ':property')) {
+                        $extraProperties = $parameter->getExtraProperties()['_properties'] ?? [];
+                        foreach (array_keys($extraProperties) as $property) {
+                            $properties[$property] = $parameter->getFilterContext();
+                        }
                     }
-                } else {
-                    $properties = [$propertyKey => $parameter->getFilterContext()];
                 }
 
-                $filter->setProperties($properties ?? []);
+                $filter->setProperties($properties + [$propertyKey => $parameter->getFilterContext()]);
             }
 
             $filter->apply($queryBuilder, $queryNameGenerator, $resourceClass, $operation,

src/GraphQl/Type/FieldsBuilder.php

@@ -31,6 +31,7 @@
 use ApiPlatform\Metadata\Util\PropertyInfoToTypeInfoHelper;
 use ApiPlatform\Metadata\Util\TypeHelper;
 use ApiPlatform\State\Pagination\Pagination;
+use ApiPlatform\State\Util\StateOptionsTrait;
 use GraphQL\Type\Definition\InputObjectType;
 use GraphQL\Type\Definition\ListOfType;
 use GraphQL\Type\Definition\NonNull;
@@ -55,6 +56,8 @@
  */
 final class FieldsBuilder implements FieldsBuilderEnumInterface
 {
+    use StateOptionsTrait;
+
     private readonly ContextAwareTypeBuilderInterface $typeBuilder;
 
     public function __construct(private readonly PropertyNameCollectionFactoryInterface $propertyNameCollectionFactory, private readonly PropertyMetadataFactoryInterface $propertyMetadataFactory, private readonly ResourceMetadataCollectionFactoryInterface $resourceMetadataCollectionFactory, private readonly ResourceClassResolverInterface $resourceClassResolver, private readonly TypesContainerInterface $typesContainer, ContextAwareTypeBuilderInterface $typeBuilder, private readonly TypeConverterInterface $typeConverter, private readonly ResolverFactoryInterface $resolverFactory, private readonly ContainerInterface $filterLocator, private readonly Pagination $pagination, private readonly ?NameConverterInterface $nameConverter, private readonly string $nestingSeparator, private readonly ?InflectorInterface $inflector = new Inflector())
@@ -85,7 +88,7 @@ public function getItemQueryFields(string $resourceClass, Operation $operation,
             return [];
         }
 
-        $fieldName = lcfirst('item_query' === $operation->getName() ? $operation->getShortName() : $operation->getName().$operation->getShortName());
+        $fieldName = lcfirst('item_query' === $operation->getName() ? ($operation->getShortName() ?? $operation->getName()) : $operation->getName().$operation->getShortName());
 
         if ($fieldConfiguration = $this->getResourceFieldConfiguration(null, $operation->getDescription(), $operation->getDeprecationReason(), Type::nullable(Type::object($resourceClass)), $resourceClass, false, $operation)) {
             $args = $this->resolveResourceArgs($configuration['args'] ?? [], $operation);
@@ -606,7 +609,8 @@ private function getFilterArgs(array $args, ?string $resourceClass, string $root
                 continue;
             }
 
-            foreach ($this->filterLocator->get($filterId)->getDescription($resourceClass) as $key => $description) {
+            $entityClass = $this->getStateOptionsClass($resourceOperation, $resourceOperation->getClass());
+            foreach ($this->filterLocator->get($filterId)->getDescription($entityClass) as $key => $description) {
                 $filterType = \in_array($description['type'], TypeIdentifier::values(), true) ? Type::builtin($description['type']) : Type::object($description['type']);
                 if (!($description['required'] ?? false)) {
                     $filterType = Type::nullable($filterType);

src/Hydra/JsonSchema/SchemaFactory.php

@@ -170,14 +170,7 @@ public function buildSchema(string $className, string $format = 'jsonld', string
 
             $definitions[self::COLLECTION_BASE_SCHEMA_NAME] = [
                 'type' => 'object',
-                'required' => [
-                    $hydraPrefix.'member',
-                ],
                 'properties' => [
-                    $hydraPrefix.'member' => [
-                        'type' => 'array',
-                        'items' => ['type' => 'object'],
-                    ],
                     $hydraPrefix.'totalItems' => [
                         'type' => 'integer',
                         'minimum' => 0,
@@ -249,6 +242,9 @@ public function buildSchema(string $className, string $format = 'jsonld', string
             ['$ref' => $prefix.self::COLLECTION_BASE_SCHEMA_NAME],
             [
                 'type' => 'object',
+                'required' => [
+                    $hydraPrefix.'member',
+                ],
                 'properties' => [
                     $hydraPrefix.'member' => [
                         'type' => 'array',

src/Hydra/Tests/JsonSchema/SchemaFactoryTest.php

@@ -132,7 +132,6 @@ public function testSchemaTypeBuildSchema(): void
         $this->assertNull($resultSchema->getRootDefinitionKey());
         $hydraCollectionSchema = $resultSchema['definitions']['HydraCollectionBaseSchema'];
         $properties = $hydraCollectionSchema['properties'];
-        $this->assertTrue(isset($properties['hydra:member']));
         $this->assertArrayHasKey('hydra:totalItems', $properties);
         $this->assertArrayHasKey('hydra:view', $properties);
         $this->assertArrayHasKey('hydra:search', $properties);

src/JsonSchema/Metadata/Property/Factory/SchemaPropertyMetadataFactory.php

@@ -280,6 +280,7 @@ private function getJsonSchemaFromType(Type $type, ?bool $readableLink = null):
                 TypeIdentifier::OBJECT => ['type' => 'object'],
                 TypeIdentifier::RESOURCE => ['type' => 'string'],
                 TypeIdentifier::CALLABLE => ['type' => 'string'],
+                TypeIdentifier::MIXED => ['type' => 'string'],
                 default => ['type' => 'null'],
             };
 

src/JsonSchema/Tests/Fixtures/DummyWithMixed.php

@@ -0,0 +1,23 @@
+<?php
+
+/*
+ * This file is part of the API Platform project.
+ *
+ * (c) Kévin Dunglas <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+declare(strict_types=1);
+
+namespace ApiPlatform\JsonSchema\Tests\Fixtures;
+
+use ApiPlatform\Metadata\ApiResource;
+
+#[ApiResource]
+class DummyWithMixed
+{
+    public mixed $mixedProperty;
+    public array $mixedArrayProperty;
+}

src/JsonSchema/Tests/Metadata/Property/Factory/SchemaPropertyMetadataFactoryTest.php

@@ -16,6 +16,7 @@
 use ApiPlatform\JsonSchema\Metadata\Property\Factory\SchemaPropertyMetadataFactory;
 use ApiPlatform\JsonSchema\Tests\Fixtures\DummyWithCustomOpenApiContext;
 use ApiPlatform\JsonSchema\Tests\Fixtures\DummyWithEnum;
+use ApiPlatform\JsonSchema\Tests\Fixtures\DummyWithMixed;
 use ApiPlatform\JsonSchema\Tests\Fixtures\DummyWithUnionTypeProperty;
 use ApiPlatform\JsonSchema\Tests\Fixtures\Enum\IntEnumAsIdentifier;
 use ApiPlatform\Metadata\ApiProperty;
@@ -167,4 +168,37 @@ public function testUnionTypeAnyOfIsArray(): void
 
         $this->assertEquals($expectedSchema, $apiProperty->getSchema());
     }
+
+    public function testMixed(): void
+    {
+        if (!method_exists(PropertyInfoExtractor::class, 'getType')) { // @phpstan-ignore-line symfony/property-info 6.4 is still allowed and this may be true
+            $this->markTestSkipped('This test only supports type-info component');
+        }
+
+        $resourceClassResolver = $this->createMock(ResourceClassResolverInterface::class);
+        $apiProperty = new ApiProperty(nativeType: Type::mixed());
+        $decorated = $this->createMock(PropertyMetadataFactoryInterface::class);
+        $decorated->expects($this->once())->method('create')->with(DummyWithMixed::class, 'mixedProperty')->willReturn($apiProperty);
+
+        $schemaPropertyMetadataFactory = new SchemaPropertyMetadataFactory($resourceClassResolver, $decorated);
+        $apiProperty = $schemaPropertyMetadataFactory->create(DummyWithMixed::class, 'mixedProperty');
+
+        $this->assertEquals([
+            'type' => ['string', 'null'],
+        ], $apiProperty->getSchema());
+
+        $apiProperty = new ApiProperty(nativeType: Type::array(Type::mixed()));
+        $decorated = $this->createMock(PropertyMetadataFactoryInterface::class);
+        $decorated->expects($this->once())->method('create')->with(DummyWithMixed::class, 'mixedArrayProperty')->willReturn($apiProperty);
+
+        $schemaPropertyMetadataFactory = new SchemaPropertyMetadataFactory($resourceClassResolver, $decorated);
+        $apiProperty = $schemaPropertyMetadataFactory->create(DummyWithMixed::class, 'mixedArrayProperty');
+
+        $this->assertEquals([
+            'type' => 'array',
+            'items' => [
+                'type' => ['string', 'null'],
+            ],
+        ], $apiProperty->getSchema());
+    }
 }

src/Laravel/ApiPlatformProvider.php

@@ -89,6 +89,8 @@
 use ApiPlatform\Laravel\Eloquent\PropertyAccess\PropertyAccessor as EloquentPropertyAccessor;
 use ApiPlatform\Laravel\Eloquent\PropertyInfo\EloquentExtractor;
 use ApiPlatform\Laravel\Eloquent\Serializer\EloquentNameConverter;
+use ApiPlatform\Laravel\Eloquent\Serializer\Mapping\Loader\AttributeLoader as EloquentAttributeLoader;
+use ApiPlatform\Laravel\Eloquent\Serializer\Mapping\Loader\RelationMetadataLoader;
 use ApiPlatform\Laravel\Eloquent\Serializer\SerializerContextBuilder as EloquentSerializerContextBuilder;
 use ApiPlatform\Laravel\GraphQl\Controller\EntrypointController as GraphQlEntrypointController;
 use ApiPlatform\Laravel\GraphQl\Controller\GraphiQlController;
@@ -174,7 +176,6 @@
 use Symfony\Component\Serializer\Mapping\Factory\ClassMetadataFactoryInterface;
 use Symfony\Component\Serializer\Mapping\Loader\AttributeLoader;
 use Symfony\Component\Serializer\Mapping\Loader\LoaderChain;
-use Symfony\Component\Serializer\Mapping\Loader\LoaderInterface;
 use Symfony\Component\Serializer\NameConverter\MetadataAwareNameConverter;
 use Symfony\Component\Serializer\NameConverter\NameConverterInterface;
 use Symfony\Component\Serializer\NameConverter\SnakeCaseToCamelCaseNameConverter;
@@ -216,7 +217,6 @@ public function register(): void
             return new ModelMetadata();
         });
 
-        $this->app->bind(LoaderInterface::class, AttributeLoader::class);
         $this->app->bind(ClassMetadataFactoryInterface::class, ClassMetadataFactory::class);
         $this->app->singleton(ClassMetadataFactory::class, function (Application $app) {
             /** @var ConfigRepository */
@@ -232,8 +232,8 @@ public function register(): void
                         $app->make(PropertyNameCollectionFactoryInterface::class),
                         $nameConverter
                     ),
-                    new AttributeLoader(),
-                    // new RelationMetadataLoader($app->make(ModelMetadata::class)),
+                    new EloquentAttributeLoader(new AttributeLoader()),
+                    new RelationMetadataLoader($app->make(ModelMetadata::class)),
                 ])
             );
         });
@@ -330,8 +330,8 @@ public function register(): void
             );
         });
 
-        $this->app->bind(PropertyAccessorInterface::class, function () {
-            return new EloquentPropertyAccessor();
+        $this->app->bind(PropertyAccessorInterface::class, function (Application $app) {
+            return new EloquentPropertyAccessor(null, $app->make(ModelMetadata::class));
         });
 
         $this->app->bind(NameConverterInterface::class, function (Application $app) {
@@ -467,7 +467,7 @@ public function register(): void
             $config = $app['config'];
             $defaultContext = $config->get('api-platform.serializer', []);
 
-            return new ObjectNormalizer(defaultContext: $defaultContext);
+            return new ObjectNormalizer($app->make(ClassMetadataFactoryInterface::class), defaultContext: $defaultContext);
         });
 
         $this->app->singleton(DateTimeNormalizer::class, function (Application $app) {