Fixing a bug where a repository would not be generated under certain

conditions

Author: weaverryan

src/Doctrine/EntityRegenerator.php

@@ -206,5 +206,7 @@ private function generateRepository(ClassMetadata $metadata)
                 'entity_alias' => strtolower($entityClassName[0]),
             ]
         );
+
+        $this->generator->writeChanges();
     }
 }

tests/Doctrine/fixtures/expected_xml/src/Entity/User.php

@@ -9,6 +9,8 @@ class User
 {
     private $id;
 
+    private $name;
+
     private $avatars;
 
     public function __construct()
@@ -22,6 +24,18 @@ public function getId(): ?int
         return $this->id;
     }
 
+    public function getName(): ?string
+    {
+        return $this->name;
+    }
+
+    public function setName(string $name): self
+    {
+        $this->name = $name;
+
+        return $this;
+    }
+
     /**
      * @return Collection|UserAvatar[]
      */

tests/Doctrine/fixtures/expected_xml/src/Entity/XOther.php

@@ -0,0 +1,13 @@
+<?php
+
+namespace Symfony\Bundle\MakerBundle\Tests\tmp\current_project\src\Entity;
+
+class XOther
+{
+    private $id;
+
+    public function getId(): ?int
+    {
+        return $this->id;
+    }
+}

tests/Doctrine/fixtures/expected_xml/src/Repository/XOtherRepository.php

@@ -0,0 +1,50 @@
+<?php
+
+namespace Symfony\Bundle\MakerBundle\Tests\tmp\current_project\src\Repository;
+
+use Symfony\Bundle\MakerBundle\Tests\tmp\current_project\src\Entity\XOther;
+use Doctrine\Bundle\DoctrineBundle\Repository\ServiceEntityRepository;
+use Symfony\Bridge\Doctrine\RegistryInterface;
+
+/**
+ * @method XOther|null find($id, $lockMode = null, $lockVersion = null)
+ * @method XOther|null findOneBy(array $criteria, array $orderBy = null)
+ * @method XOther[]    findAll()
+ * @method XOther[]    findBy(array $criteria, array $orderBy = null, $limit = null, $offset = null)
+ */
+class XOtherRepository extends ServiceEntityRepository
+{
+    public function __construct(RegistryInterface $registry)
+    {
+        parent::__construct($registry, XOther::class);
+    }
+
+//    /**
+//     * @return XOther[] Returns an array of XOther objects
+//     */
+    /*
+    public function findByExampleField($value)
+    {
+        return $this->createQueryBuilder('x')
+            ->andWhere('x.exampleField = :val')
+            ->setParameter('val', $value)
+            ->orderBy('x.id', 'ASC')
+            ->setMaxResults(10)
+            ->getQuery()
+            ->getResult()
+        ;
+    }
+    */
+
+    /*
+    public function findOneBySomeField($value): ?XOther
+    {
+        return $this->createQueryBuilder('x')
+            ->andWhere('x.exampleField = :val')
+            ->setParameter('val', $value)
+            ->getQuery()
+            ->getOneOrNullResult()
+        ;
+    }
+    */
+}

tests/Doctrine/fixtures/xml_source_project/config/doctrine/User.orm.xml

@@ -7,6 +7,11 @@
         <id name="id" type="integer">
             <generator strategy="AUTO" />
         </id>
+        <field name="name" type="string" length="100">
+            <options>
+                <option name="coment">First Name</option>
+            </options>
+        </field>
         <one-to-many field="avatars" target-entity="UserAvatar" mapped-by="user" />
     </entity>
 </doctrine-mapping>

tests/Doctrine/fixtures/xml_source_project/config/doctrine/XOther.orm.xml

@@ -0,0 +1,11 @@
+<doctrine-mapping xmlns="http://doctrine-project.org/schemas/orm/doctrine-mapping"
+    xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+    xsi:schemaLocation="http://doctrine-project.org/schemas/orm/doctrine-mapping
+        http://doctrine-project.org/schemas/orm/doctrine-mapping.xsd">
+
+    <entity name="Symfony\Bundle\MakerBundle\Tests\tmp\current_project\src\Entity\XOther" repository-class="Symfony\Bundle\MakerBundle\Tests\tmp\current_project\src\Repository\XOtherRepository">
+        <id name="id" type="integer">
+            <generator strategy="AUTO" />
+        </id>
+    </entity>
+</doctrine-mapping>