fix: add unpersisted object to relation

Author: nikophil

src/Persistence/PersistenceManager.php

@@ -190,6 +190,7 @@ public function refresh(object &$object, bool $force = false): object
 
     public function isPersisted(object $object): bool
     {
+        // prevents doctrine to use its cache and think the object is persisted
         if ($this->strategyFor($object::class)->isScheduledForInsert($object)) {
             return false;
         }

src/Persistence/PersistentObjectFactory.php

@@ -355,12 +355,29 @@ protected function normalizeObject(object $object): object
 
         $configuration = Configuration::instance();
 
-        if (!$configuration->isPersistenceAvailable() || !$configuration->persistence()->hasPersistenceFor($object)) {
+        if (!$configuration->isPersistenceAvailable()) {
             return $object;
         }
 
+        $persistenceManager = $configuration->persistence();
+
+        if ($object instanceof Proxy) {
+            $proxy = $object;
+            $proxy->_disableAutoRefresh();
+            $object = $proxy->_real();
+            $proxy->_enableAutoRefresh();
+        }
+
+        if (!$persistenceManager->hasPersistenceFor($object)) {
+            return $object;
+        }
+
+        if (!$persistenceManager->isPersisted($object)) {
+            $persistenceManager->scheduleForInsert($object);
+        }
+
         try {
-            return $configuration->persistence()->refresh($object, true);
+            return $persistenceManager->refresh($object, force: true);
         } catch (RefreshObjectFailed|VarExportLogicException) {
             return $object;
         }

tests/Integration/ORM/EntityRelationship/EntityFactoryRelationshipTestCase.php

@@ -31,6 +31,7 @@
 use Zenstruck\Foundry\Tests\Fixture\Entity\Contact;
 use Zenstruck\Foundry\Tests\Fixture\Entity\Tag;
 
+use function Zenstruck\Foundry\Persistence\refresh;
 use function Zenstruck\Foundry\Persistence\unproxy;
 
 /**
@@ -361,6 +362,36 @@ public function ensure_one_to_many_relations_are_not_pre_persisted(): void
         }
     }
 
+    /** @test */
+    #[Test]
+    #[DataProvider('provideCascadeRelationshipsCombinations')]
+    #[UsingRelationships(Category::class, ['contacts'])]
+    public function it_can_add_multiple_items_to_one_to_many(): void
+    {
+        $category = static::categoryFactory()->create();
+        self::assertCount(0, $category->getContacts());
+
+        static::contactFactory()->create(['category' => $category]);
+        static::contactFactory()->create(['category' => $category]);
+
+        self::assertCount(2, $category->getContacts());
+    }
+
+    /** @test */
+    #[Test]
+    #[DataProvider('provideCascadeRelationshipsCombinations')]
+    #[UsingRelationships(Category::class, ['contacts'])]
+    public function it_can_add_multiple_not_persisted_item_to_one_to_many(): void
+    {
+        $category = static::categoryFactory()->withoutPersisting()->create();
+        self::assertCount(0, $category->getContacts());
+
+        static::contactFactory()->create(['category' => $category]);
+        static::contactFactory()->create(['category' => $category]);
+
+        self::assertCount(2, $category->getContacts());
+    }
+
     /** @return PersistentObjectFactory<Contact> */
     protected static function contactFactoryWithoutCategory(): PersistentObjectFactory
     {