Simplify ExistsFilter declaration

Author: antograssiot

src/Bridge/Doctrine/Orm/Filter/ExistsFilter.php

@@ -46,31 +46,8 @@ class ExistsFilter extends AbstractContextAwareFilter implements ExistsFilterInt
      * @param RequestStack|null $requestStack No prefix to prevent autowiring of this deprecated property
      * @param mixed|null        $properties
      */
-    public function __construct(ManagerRegistry $managerRegistry, $requestStack = null, LoggerInterface $logger = null, /* string $existsParameterName = self::QUERY_PARAMETER_KEY, array*/ $properties = null)
+    public function __construct(ManagerRegistry $managerRegistry, $requestStack = null, LoggerInterface $logger = null, array $properties = null, string $existsParameterName = self::QUERY_PARAMETER_KEY)
     {
-        $existsParameterName = self::QUERY_PARAMETER_KEY;
-
-        if (($funcNumArgs = \func_num_args()) > 3) {
-            $fourthArgument = func_get_arg(3);
-            if (4 <= $funcNumArgs && (null === $fourthArgument || \is_array($fourthArgument))) {
-                @trigger_error(sprintf('Passing the "$properties" argument as 4th argument of "%s" is deprecated since API Platform 2.5 and will not be possible anymore in API Platform 3. Pass the new "$existsParameterName" argument as 4th argument and the old "$properties" argument as 5th argument instead.', __CLASS__), E_USER_DEPRECATED);
-                $properties = $fourthArgument;
-            } elseif (\is_string($fourthArgument)) {
-                $existsParameterName = $fourthArgument;
-            } else {
-                throw new InvalidArgumentException(sprintf('The "$existsParameterName" argument of "%s" is expected to be a string.', __CLASS__));
-            }
-        }
-
-        if ($funcNumArgs > 4) {
-            $fifthArgument = func_get_arg(4);
-            if (null === $fifthArgument || \is_array($fifthArgument)) {
-                $properties = $fifthArgument;
-            } else {
-                throw new InvalidArgumentException(sprintf('The "$properties" argument of "%s" is expected to be an array or null.', __CLASS__));
-            }
-        }
-
         parent::__construct($managerRegistry, $requestStack, $logger, $properties);
 
         $this->existsParameterName = $existsParameterName;

src/Bridge/Symfony/Bundle/Resources/config/doctrine_orm.xml

@@ -99,7 +99,7 @@
             <argument type="service" id="doctrine" />
             <argument>null</argument>
             <argument type="service" id="logger" on-invalid="ignore" />
-            <argument>%api_platform.collection.exists_parameter_name%</argument>
+            <argument key="$existsParameterName">%api_platform.collection.exists_parameter_name%</argument>
         </service>
         <service id="ApiPlatform\Core\Bridge\Doctrine\Orm\Filter\ExistsFilter" alias="api_platform.doctrine.orm.exists_filter" />
 

tests/Bridge/Doctrine/Orm/Filter/ExistsFilterTest.php

@@ -95,10 +95,10 @@ public function testGetDescriptionDefaultFields()
     public function provideApplyTestData(): array
     {
         $existsFilterFactory = function (ManagerRegistry $managerRegistry, array $properties = null, RequestStack $requestStack = null): ExistsFilter {
-            return new ExistsFilter($managerRegistry, $requestStack, null, 'exists', $properties);
+            return new ExistsFilter($managerRegistry, $requestStack, null, $properties, 'exists');
         };
         $customExistsFilterFactory = function (ManagerRegistry $managerRegistry, array $properties = null, RequestStack $requestStack = null): ExistsFilter {
-            return new ExistsFilter($managerRegistry, $requestStack, null, 'customExists', $properties);
+            return new ExistsFilter($managerRegistry, $requestStack, null, $properties, 'customExists');
         };
 
         return array_merge_recursive(
@@ -245,7 +245,7 @@ public function testLegacyExistsAfterSyntax()
             sprintf('SELECT o FROM %s o WHERE o.description IS NOT NULL', Dummy::class),
             null,
             function (ManagerRegistry $managerRegistry, array $properties = null, RequestStack $requestStack = null): ExistsFilter {
-                return new ExistsFilter($managerRegistry, $requestStack, null, 'exists', $properties);
+                return new ExistsFilter($managerRegistry, $requestStack, null, $properties, 'exists');
             },
         ];
 
@@ -273,7 +273,7 @@ public function testLegacyRequest()
             sprintf('SELECT o FROM %s o WHERE o.description IS NOT NULL', Dummy::class),
             null,
             function (ManagerRegistry $managerRegistry, array $properties = null, RequestStack $requestStack = null): ExistsFilter {
-                return new ExistsFilter($managerRegistry, $requestStack, null, 'exists', $properties);
+                return new ExistsFilter($managerRegistry, $requestStack, null, $properties, 'exists');
             },
         ];
 
@@ -282,6 +282,6 @@ function (ManagerRegistry $managerRegistry, array $properties = null, RequestSta
 
     protected function buildFilter(?array $properties = null)
     {
-        return new $this->filterClass($this->managerRegistry, null, null, 'exists', $properties);
+        return new $this->filterClass($this->managerRegistry, null, null, $properties, 'exists');
     }
 }