Attributes added in beforeInstantiate are not passed to afterPersist

src/Factory.php

@@ -28,12 +28,19 @@ abstract class Factory
     /** @var Attributes[] */
     private array $attributes;
 
+    /**
+     * Memoization of normalized parameters
+     *
+     * @internal
+     * @var Parameters|null
+     */
+    protected array|null $normalizedParameters = null;
+
     // keep an empty constructor for BC
     public function __construct()
     {
     }
 
-
     /**
      * @param Attributes $attributes
      */
@@ -144,6 +151,16 @@ final protected static function faker(): Faker\Generator
         return Configuration::instance()->faker;
     }
 
+    /**
+     * Override to adjust default attributes & config.
+     *
+     * @return static
+     */
+    protected function initialize(): static
+    {
+        return $this;
+    }
+
     /**
      * @internal
      *
@@ -170,16 +187,6 @@ final protected function normalizeAttributes(array|callable $attributes = []): a
         );
     }
 
-    /**
-     * Override to adjust default attributes & config.
-     *
-     * @return static
-     */
-    protected function initialize(): static
-    {
-        return $this;
-    }
-
     /**
      * @internal
      *
@@ -189,7 +196,7 @@ protected function initialize(): static
      */
     protected function normalizeParameters(array $parameters): array
     {
-        return array_combine(
+        return $this->normalizedParameters = array_combine(
             array_keys($parameters),
             \array_map($this->normalizeParameter(...), array_keys($parameters), $parameters)
         );

src/Persistence/PersistentObjectFactory.php

@@ -226,7 +226,7 @@ final public function create(callable|array $attributes = []): object
         $this->tempAfterPersist = [];
 
         if ($this->afterPersist) {
-            $attributes = $this->normalizeAttributes($attributes);
+            $attributes = $this->normalizedParameters ?? throw new \LogicException('Factory::$normalizedParameters has not been initialized.');
 
             foreach ($this->afterPersist as $callback) {
                 $callback($object, $attributes);

tests/Integration/ObjectFactoryTest.php

@@ -44,4 +44,20 @@ public function can_create_non_service_factories(): void
 
         $this->assertSame('router-constructor', $object->object->getProp1());
     }
+
+    /**
+     * @test
+     */
+    public function can_create_different_objects_based_on_same_factory(): void
+    {
+        $factory = Object1Factory::new(['prop1' => 'first object']);
+        $object1 = $factory->create();
+        self::assertSame('first object-constructor', $object1->getProp1());
+
+        $object2 = $factory->create(['prop1' => 'second object']);
+        self::assertSame('second object-constructor', $object2->getProp1());
+
+        $object3 = $factory->with(['prop1' => 'third object'])->create();
+        self::assertSame('third object-constructor', $object3->getProp1());
+    }
 }

tests/Integration/Persistence/GenericProxyFactoryTestCase.php

@@ -280,6 +280,27 @@ public function can_use_after_persist_with_attributes(): void
         $this->assertSame($value, $object->getProp1());
     }
 
+    /**
+     * @test
+     */
+    public function can_use_after_persist_with_attributes_added_in_before_instantiate(): void
+    {
+        $value = 'value set with before instantiate';
+        $object = $this->factory()
+            ->instantiateWith(Instantiator::withConstructor()->allowExtra('extra'))
+            ->beforeInstantiate(function (array $attributes) use ($value) {
+                $attributes['extra'] = $value;
+
+                return $attributes;
+            })
+            ->afterPersist(function (GenericModel $object, array $attributes) {
+                $object->setProp1($attributes['extra']);
+            })
+            ->create();
+
+        $this->assertSame($value, $object->getProp1());
+    }
+
     /**
      * @return PersistentProxyObjectFactory<GenericModel>
      */