fix #1153

Author: abluchet

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

@@ -233,7 +233,9 @@ protected function splitPropertyParts(string $property/*, string $resourceClass*
                     @trigger_error(sprintf('Method %s() will have a second `$resourceClass` argument in version API Platform 3.0. Not defining it is deprecated since API Platform 2.1.', __FUNCTION__), E_USER_DEPRECATED);
                 }
             }
+        }
 
+        if (!isset($resourceClass)) {
             return [
                 'associations' => array_slice($parts, 0, -1),
                 'field' => end($parts),

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

@@ -0,0 +1,39 @@
+<?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\Core\Tests\Bridge\Doctrine\Orm\Filter;
+
+use ApiPlatform\Core\Tests\Fixtures\TestBundle\Doctrine\Orm\Filter\DummyFilter;
+use Doctrine\Common\Persistence\ManagerRegistry;
+use Symfony\Component\HttpFoundation\RequestStack;
+
+/**
+ * Regression test case on issue 1154.
+ *
+ * @author Antoine Bluchet <[email protected]>
+ */
+class AbstractFilterTest extends \PHPUnit_Framework_TestCase
+{
+    public function testSplitPropertiesWithoutResourceClass()
+    {
+        $managerRegistry = $this->prophesize(ManagerRegistry::class);
+        $requestStack = $this->prophesize(RequestStack::class);
+
+        $filter = new DummyFilter($managerRegistry->reveal(), $requestStack->reveal());
+
+        $this->assertEquals($filter->doSplitPropertiesWithoutResourceClass('foo.bar'), [
+            'associations' => ['foo'],
+            'field' => 'bar',
+        ]);
+    }
+}

tests/Fixtures/TestBundle/Doctrine/Orm/Filter/DummyFilter.php

@@ -0,0 +1,35 @@
+<?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\Core\Tests\Fixtures\TestBundle\Doctrine\Orm\Filter;
+
+use ApiPlatform\Core\Bridge\Doctrine\Orm\Filter\AbstractFilter;
+use ApiPlatform\Core\Bridge\Doctrine\Orm\Util\QueryNameGeneratorInterface;
+use Doctrine\ORM\QueryBuilder;
+
+class DummyFilter extends AbstractFilter
+{
+    public function doSplitPropertiesWithoutResourceClass($property)
+    {
+        return $this->splitPropertyParts($property);
+    }
+
+    protected function filterProperty(string $property, $values, QueryBuilder $queryBuilder, QueryNameGeneratorInterface $queryNameGenerator, string $resourceClass, string $operationName = null)
+    {
+    }
+
+    public function getDescription(string $resourceClass): array
+    {
+        return [];
+    }
+}