bug symfony#61121 [DependencyInjection] Fix proxying services defined with an abstract class and a factory (nicolas-grekas)

This PR was merged into the 7.3 branch.

Discussion
----------

[DependencyInjection] Fix proxying services defined with an abstract class and a factory

| Q             | A
| ------------- | ---
| Branch?       | 7.3
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Issues        | Fix symfony#61095
| License       | MIT

Commits
-------

0c3da8b [DependencyInjection] Fix proxying services defined with an abstract class and a factory

Author: nicolas-grekas

src/Symfony/Component/DependencyInjection/LazyProxy/PhpDumper/LazyServiceDumper.php

@@ -197,7 +197,7 @@ public function getProxyClass(Definition $definition, bool $asGhostObject, ?\Ref
             return $class->name;
         }
 
-        if (!$definition->hasTag('proxy') && !$class->isInterface()) {
+        if (!$definition->hasTag('proxy') && !$class->isAbstract()) {
             $parent = $class;
             do {
                 $extendsInternalClass = $parent->isInternal();

src/Symfony/Component/DependencyInjection/Tests/Fixtures/AbstractSayClass.php

@@ -0,0 +1,17 @@
+<?php
+
+/*
+ * This file is part of the Symfony package.
+ *
+ * (c) Fabien Potencier <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Symfony\Component\DependencyInjection\Tests\Fixtures;
+
+abstract class AbstractSayClass
+{
+    abstract public function say(): string;
+}

src/Symfony/Component/DependencyInjection/Tests/LazyProxy/Instantiator/LazyServiceInstantiatorTest.php

@@ -0,0 +1,45 @@
+<?php
+
+/*
+ * This file is part of the Symfony package.
+ *
+ * (c) Fabien Potencier <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Symfony\Component\DependencyInjection\Tests\LazyProxy\Instantiator;
+
+use PHPUnit\Framework\TestCase;
+use Symfony\Component\DependencyInjection\Container;
+use Symfony\Component\DependencyInjection\Definition;
+use Symfony\Component\DependencyInjection\LazyProxy\Instantiator\LazyServiceInstantiator;
+use Symfony\Component\DependencyInjection\Tests\Fixtures\AbstractSayClass;
+
+class LazyServiceInstantiatorTest extends TestCase
+{
+    public function testInstantiateAbstractClassProxy()
+    {
+        $instantiator = new LazyServiceInstantiator();
+        $instance = new class extends AbstractSayClass {
+            public int $calls = 0;
+
+            public function say(): string
+            {
+                ++$this->calls;
+
+                return 'Hello from the abstract class!';
+            }
+        };
+
+        $definition = (new Definition(AbstractSayClass::class))
+            ->setLazy(true);
+
+        $proxy = $instantiator->instantiateProxy(new Container(), $definition, 'foo', fn () => $instance);
+
+        $this->assertSame(0, $instance->calls);
+        $this->assertSame('Hello from the abstract class!', $proxy->say());
+        $this->assertSame(1, $instance->calls);
+    }
+}

src/Symfony/Component/DependencyInjection/Tests/LazyProxy/PhpDumper/LazyServiceDumperTest.php

@@ -16,6 +16,7 @@
 use Symfony\Component\DependencyInjection\Definition;
 use Symfony\Component\DependencyInjection\Exception\InvalidArgumentException;
 use Symfony\Component\DependencyInjection\LazyProxy\PhpDumper\LazyServiceDumper;
+use Symfony\Component\DependencyInjection\Tests\Fixtures\AbstractSayClass;
 use Symfony\Component\DependencyInjection\Tests\Fixtures\ReadOnlyClass;
 
 class LazyServiceDumperTest extends TestCase
@@ -40,6 +41,16 @@ public function testFinalClassInterface()
         $this->assertStringContainsString('function get(', $dumper->getProxyCode($definition));
     }
 
+    public function testAbstractClass()
+    {
+        $dumper = new LazyServiceDumper();
+        $definition = (new Definition(AbstractSayClass::class))
+            ->setLazy(true);
+
+        $this->assertTrue($dumper->isProxyCandidate($definition));
+        $this->assertNotSame(AbstractSayClass::class, $dumper->getProxyClass($definition, false));
+    }
+
     public function testInvalidClass()
     {
         $dumper = new LazyServiceDumper();