fix: apply last reviews made in th latest pr

Continues the work at api-platform#7079 and before at api-platform#6865

Author: vinceAmstoutz

src/Doctrine/Odm/Filter/IriFilter.php

@@ -0,0 +1,62 @@
+<?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\Doctrine\Odm\Filter;
+
+use ApiPlatform\Metadata\OpenApiParameterFilterInterface;
+use ApiPlatform\Metadata\Operation;
+use ApiPlatform\Metadata\Parameter;
+use ApiPlatform\Metadata\ParameterProviderFilterInterface;
+use ApiPlatform\OpenApi\Model\Parameter as OpenApiParameter;
+use ApiPlatform\State\Provider\IriConverterParameterProvider;
+use Doctrine\ODM\MongoDB\Aggregation\Builder;
+
+class IriFilter implements FilterInterface, OpenApiParameterFilterInterface, ParameterProviderFilterInterface
+{
+    public function apply(Builder $aggregationBuilder, string $resourceClass, ?Operation $operation = null, array &$context = []): void
+    {
+        if (!$parameter = $context['parameter'] ?? null) {
+            return;
+        }
+
+        \assert($parameter instanceof Parameter);
+
+        $value = $parameter->getValue();
+        if (!\is_array($value)) {
+            $value = [$value];
+        }
+
+        // TODO: do something for nested properties?
+        $matchField = $parameter->getProperty();
+
+        $aggregationBuilder
+            ->match()
+            ->field($matchField)
+            ->in($value);
+    }
+
+    public static function getParameterProvider(): string
+    {
+        return IriConverterParameterProvider::class;
+    }
+
+    public function getOpenApiParameters(Parameter $parameter): OpenApiParameter|array|null
+    {
+        return new OpenApiParameter(name: $parameter->getKey().'[]', in: 'query', style: 'deepObject', explode: true);
+    }
+
+    public function getDescription(string $resourceClass): array
+    {
+        return [];
+    }
+}

src/Doctrine/Orm/Filter/ExactSearchFilter.php

@@ -30,14 +30,14 @@
 use GraphQL\Type\Definition\EnumType;
 use GraphQL\Type\Definition\ObjectType;
 use GraphQL\Type\Definition\Type as GraphQLType;
+use PHPUnit\Framework\Attributes\DataProvider;
+use PHPUnit\Framework\Attributes\IgnoreDeprecations;
 use PHPUnit\Framework\TestCase;
 use Prophecy\Argument;
 use Prophecy\PhpUnit\ProphecyTrait;
 use Prophecy\Prophecy\ObjectProphecy;
 use Symfony\Component\PropertyInfo\Type as LegacyType;
 use Symfony\Component\TypeInfo\Type;
-use PHPUnit\Framework\Attributes\DataProvider;
-use PHPUnit\Framework\Attributes\IgnoreDeprecations;
 
 /**
  * @author Alan Poulain <[email protected]>

src/Doctrine/Orm/Filter/PartialSearchFilter.php

@@ -45,8 +45,8 @@
 use Symfony\Component\Serializer\NameConverter\NameConverterInterface;
 use Symfony\Component\TypeInfo\Type;
 use Symfony\Component\TypeInfo\Type\CollectionType;
-use Symfony\Component\TypeInfo\TypeIdentifier;
 use Symfony\Component\TypeInfo\Type\ObjectType;
+use Symfony\Component\TypeInfo\TypeIdentifier;
 
 /**
  * Builds the GraphQL fields.

src/GraphQl/Tests/Type/TypeConverterTest.php

@@ -21,7 +21,7 @@
 class Chicken
 {
     #[ODM\Id]
-    private string $id;
+    private ?string $id = null;
 
     #[ODM\Field(type: 'string')]
     private string $name;

src/GraphQl/Type/FieldsBuilder.php

@@ -13,16 +13,18 @@
 
 namespace ApiPlatform\Tests\Fixtures\TestBundle\Document;
 
-use ApiPlatform\Doctrine\Orm\Filter\IriFilter;
+use ApiPlatform\Doctrine\Odm\Filter\IriFilter;
 use ApiPlatform\Metadata\GetCollection;
 use ApiPlatform\Metadata\QueryParameter;
-use ApiPlatform\Tests\Fixtures\TestBundle\Entity\Chicken;
 use Doctrine\Common\Collections\ArrayCollection;
 use Doctrine\Common\Collections\Collection;
 use Doctrine\ODM\MongoDB\Mapping\Annotations as ODM;
 
 #[ODM\Document]
-#[GetCollection(normalizationContext: ['hydra_prefix' => false], parameters: ['chickens' => new QueryParameter(filter: new IriFilter())])]
+#[GetCollection(
+    normalizationContext: ['hydra_prefix' => false],
+    parameters: ['chickens' => new QueryParameter(filter: new IriFilter())])
+]
 class ChickenCoop
 {
     #[ODM\Id]

tests/Fixtures/TestBundle/Document/Chicken.php

@@ -22,16 +22,22 @@
 use Symfony\Component\Serializer\Annotation\Groups;
 
 #[ApiResource]
-#[GetCollection()]
+#[GetCollection]
 #[Get]
 #[Post]
-#[ApiResource(uriTemplate: '/employees/{employeeId}/rooms/{roomId}/company/{companyId}', uriVariables: ['employeeId' => ['from_class' => Employee::class, 'from_property' => 'company']])]
+#[ApiResource(
+    uriTemplate: '/employees/{employeeId}/rooms/{roomId}/company/{companyId}',
+    uriVariables: ['employeeId' => ['from_class' => Employee::class, 'from_property' => 'company']]
+)]
 #[Get]
-#[ApiResource(uriTemplate: '/employees/{employeeId}/company', uriVariables: ['employeeId' => ['from_class' => Employee::class, 'from_property' => 'company']])]
+#[ApiResource(
+    uriTemplate: '/employees/{employeeId}/company',
+    uriVariables: ['employeeId' => ['from_class' => Employee::class, 'from_property' => 'company']]
+)]
 #[ODM\Document]
 class Company
 {
-    #[ODM\Id(strategy: 'INCREMENT', type: 'int')]
+    #[ODM\Id(type: 'int', strategy: 'INCREMENT')]
     private ?int $id = null;
 
     #[ODM\Field]

tests/Fixtures/TestBundle/Document/ChickenCoop.php

@@ -17,7 +17,7 @@
 use Doctrine\ORM\Mapping as ORM;
 
 #[ORM\Entity]
-#[Get()]
+#[Get]
 class Chicken
 {
     #[ORM\Id]

tests/Fixtures/TestBundle/Document/Company.php

@@ -21,15 +21,18 @@
 use Doctrine\ORM\Mapping as ORM;
 
 #[ORM\Entity]
-#[GetCollection(normalizationContext: ['hydra_prefix' => false], parameters: ['chickens' => new QueryParameter(filter: new IriFilter())])]
+#[GetCollection(
+    normalizationContext: ['hydra_prefix' => false],
+    parameters: ['chickens' => new QueryParameter(filter: new IriFilter())]
+)]
 class ChickenCoop
 {
     #[ORM\Id]
     #[ORM\GeneratedValue]
     #[ORM\Column(type: 'integer')]
     private ?int $id = null;
 
-    #[ORM\OneToMany(mappedBy: 'chickenCoop', targetEntity: Chicken::class, cascade: ['persist'])]
+    #[ORM\OneToMany(targetEntity: Chicken::class, mappedBy: 'chickenCoop', cascade: ['persist'])]
     private Collection $chickens;
 
     public function __construct()

tests/Fixtures/TestBundle/Entity/Chicken.php

@@ -22,7 +22,7 @@
 use Symfony\Component\Serializer\Annotation\Groups;
 
 #[ApiResource]
-#[GetCollection()]
+#[GetCollection]
 #[Get]
 #[Post]
 #[ApiResource(