fix: only one extends should be generated (#357)

Author: alanpoulain

src/Model/Class_.php

@@ -311,16 +311,13 @@ public function toNetteFile(array $config, Inflector $inflector, ?PhpFile $file
             $class->setImplements($implements);
         }
 
-        if ($this->parent()) {
+        if ($this->parent() && !$class->getExtends()) {
             $parentExtend = $this->resolveName($namespace, $this->parent());
             if ($this->isParentEnum()) {
                 $parentExtend = MyCLabsEnum::class;
             }
-            $extends = (array) $class->getExtends();
-            if (!\in_array($parentExtend, $extends, true)) {
-                $extends[] = $parentExtend;
-            }
-            $class->setExtends($extends);
+
+            $class->setExtends($parentExtend);
         }
 
         $netteConstants = $class->getConstants();

tests/e2e/customized/App/Entity/Thing.php

@@ -5,23 +5,16 @@
 namespace App\Entity;
 
 use ApiPlatform\Core\Annotation\ApiProperty;
-use ApiPlatform\Core\Annotation\ApiResource;
 use Doctrine\ORM\Mapping as ORM;
 
 /**
  * The most generic type of item.
  *
  * @see https://schema.org/Thing
  */
-#[ORM\Entity]
-#[ApiResource(iri: 'https://schema.org/Thing')]
-class Thing
+#[ORM\MappedSuperclass]
+abstract class Thing
 {
-    #[ORM\Id]
-    #[ORM\GeneratedValue(strategy: 'AUTO')]
-    #[ORM\Column(type: 'integer')]
-    private ?int $id = null;
-
     /**
      * The name of the item.
      *
@@ -31,11 +24,6 @@ class Thing
     #[ApiProperty(iri: 'https://schema.org/name')]
     private ?string $name = null;
 
-    public function getId(): ?int
-    {
-        return $this->id;
-    }
-
     public function setName(?string $name): void
     {
         $this->name = $name;

tests/e2e/original/App/Entity/Person.php

@@ -24,7 +24,7 @@
     collectionOperations: ['get' => ['route_name' => 'get_person_collection']],
 )]
 #[UniqueEntity('email')]
-class Person
+class Person extends Thing
 {
     #[ORM\Id]
     #[ORM\GeneratedValue(strategy: 'AUTO')]

tests/e2e/original/App/Entity/Thing.php

@@ -5,23 +5,16 @@
 namespace App\Entity;
 
 use ApiPlatform\Core\Annotation\ApiProperty;
-use ApiPlatform\Core\Annotation\ApiResource;
 use Doctrine\ORM\Mapping as ORM;
 
 /**
  * The most generic type of item.
  *
  * @see https://schema.org/Thing
  */
-#[ORM\Entity]
-#[ApiResource(iri: 'https://schema.org/Thing')]
-class Thing
+#[ORM\MappedSuperclass]
+abstract class Thing
 {
-    #[ORM\Id]
-    #[ORM\GeneratedValue(strategy: 'AUTO')]
-    #[ORM\Column(type: 'integer')]
-    private ?int $id = null;
-
     /**
      * The name of the item.
      *
@@ -31,11 +24,6 @@ class Thing
     #[ApiProperty(iri: 'https://schema.org/name')]
     private ?string $name = null;
 
-    public function getId(): ?int
-    {
-        return $this->id;
-    }
-
     public function setName(?string $name): void
     {
         $this->name = $name;

tests/e2e/schema.yml

@@ -7,6 +7,7 @@ types:
         properties:
             name: ~
     Person:
+        parent: ~
         operations:
             item:
                 get: