Merge pull request #12482 from greg0ire/fix-el-formatting

Avoid passing arrays to get_class

Author: greg0ire

phpstan-baseline.neon

@@ -4233,12 +4233,6 @@ parameters:
 			count: 1
 			path: src/Tools/Console/Command/MappingDescribeCommand.php
 
-		-
-			message: '#^Parameter \#1 \$entityListeners of method Doctrine\\ORM\\Tools\\Console\\Command\\MappingDescribeCommand\:\:formatEntityListeners\(\) expects list\<object\>, array\<string, list\<array\{class\: class\-string, method\: string\}\>\> given\.$#'
-			identifier: argument.type
-			count: 1
-			path: src/Tools/Console/Command/MappingDescribeCommand.php
-
 		-
 			message: '#^Parameter \#2 \$callback of function array_filter expects \(callable\(class\-string\)\: bool\)\|null, Closure\(mixed\)\: \(0\|1\|false\) given\.$#'
 			identifier: argument.type

src/Tools/Console/Command/MappingDescribeCommand.php

@@ -15,7 +15,6 @@
 use Symfony\Component\Console\Style\SymfonyStyle;
 
 use function array_filter;
-use function array_map;
 use function array_merge;
 use function count;
 use function current;
@@ -271,13 +270,21 @@ private function formatMappings(array $propertyMappings): array
     /**
      * Format the entity listeners
      *
-     * @phpstan-param list<object> $entityListeners
+     * @phpstan-param array<string, list<array{class: class-string, method: string}>> $entityListeners
      *
      * @return string[]
      * @phpstan-return array{0: string, 1: string}
      */
     private function formatEntityListeners(array $entityListeners): array
     {
-        return $this->formatField('Entity listeners', array_map('get_class', $entityListeners));
+        $listeners = [];
+
+        foreach ($entityListeners as $eventName => $eventListeners) {
+            foreach ($eventListeners as $listener) {
+                $listeners[] = sprintf('%s: %s::%s', $eventName, $listener['class'], $listener['method']);
+            }
+        }
+
+        return $this->formatField('Entity listeners', $listeners);
     }
 }

tests/Tests/ORM/Tools/Console/Command/MappingDescribeCommandTest.php

@@ -8,6 +8,7 @@
 use Doctrine\ORM\Tools\Console\Command\MappingDescribeCommand;
 use Doctrine\ORM\Tools\Console\EntityManagerProvider\SingleManagerProvider;
 use Doctrine\Tests\Models\Cache\AttractionInfo;
+use Doctrine\Tests\Models\CMS\CmsAddress;
 use Doctrine\Tests\OrmFunctionalTestCase;
 use Symfony\Component\Console\Application;
 use Symfony\Component\Console\Tester\CommandTester;
@@ -79,4 +80,22 @@ public function testShowSpecificNotFound(): void
             ]
         );
     }
+
+    public function testShowEntityWithEntityListeners(): void
+    {
+        $this->tester->execute(
+            [
+                'command'    => $this->command->getName(),
+                'entityName' => CmsAddress::class,
+            ]
+        );
+
+        $display = $this->tester->getDisplay();
+
+        self::assertStringContainsString(CmsAddress::class, $display);
+        self::assertStringContainsString('Entity listeners', $display);
+        self::assertStringContainsString('CmsAddressListener', $display);
+        self::assertStringContainsString('postPersist', $display);
+        self::assertStringContainsString('prePersist', $display);
+    }
 }