fix: constructor with default parameter array does not work with cont…

…ext (#185)

After this PR #177 behavior was fixed when not using constructor
arguments context, but with context it's still uses `??`.

Before:
```PHP
if (\AutoMapper\MapperContext::hasConstructorArgument($context, 'ConstructorWithDefaultValues', 'someOtters')) {
    $values = [];
    foreach ($value['someOtters'] ?? [] as $key => $value) {
        $values[$key] = $value;
    }
    $constructarg = $values ?? \AutoMapper\MapperContext::getConstructorArgument($context, 'ConstructorWithDefaultValues', 'someOtters');
} else {
    $values = [];
    foreach ($value['someOtters'] ?? [] as $key => $value) {
        $values[$key] = $value;
    }
    $constructarg = count($values) > 0 ? $values : array();
}
```
After:
```PHP
if (\AutoMapper\MapperContext::hasConstructorArgument($context, 'ConstructorWithDefaultValues', 'someOtters')) {
    $values = [];
    foreach ($value['someOtters'] ?? [] as $key => $value) {
        $values[$key] = $value;
    }
    $constructarg = count($values) > 0 ? $values : \AutoMapper\MapperContext::getConstructorArgument($context, 'ConstructorWithDefaultValues', 'someOtters');
} else {
    $values = [];
    foreach ($value['someOtters'] ?? [] as $key => $value) {
        $values[$key] = $value;
    }
    $constructarg = count($values) > 0 ? $values : array();
}
```

CHANGELOG.md

@@ -11,6 +11,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Fixed
 - [GH#184](https://github.com/jolicode/automapper/pull/184) Fix error when mapping from stdClass to constructor with nullable/optional arguments
+- [GH#185](https://github.com/jolicode/automapper/pull/185) Fix constructor with default parameter array does not work with constructor_arguments context
 
 ## [9.1.2] - 2024-09-03
 ### Fixed

src/Generator/CreateTargetStatementsGenerator.php

@@ -206,11 +206,15 @@ private function constructorArgument(GeneratorMetadata $metadata, PropertyMetada
         }
 
         if ($defaultValueExpr instanceof Expr\Array_) {
-            // $constructarg_3 = count($values) > 0 ? $values : array();
-            $argumentAssignedValue = new Expr\Ternary(new Expr\BinaryOp\Greater(new Expr\FuncCall(new Name('count'), [new Arg($output)]), create_scalar_int(0)), $output, $defaultValueExpr);
+            // $constructarg = count($values) > 0 ? $values : {expression};
+            $argumentAssignClosure = static fn (Expr $expr) => new Expr\Assign($constructVar, new Expr\Ternary(
+                new Expr\BinaryOp\Greater(new Expr\FuncCall(new Name('count'), [new Arg($output)]), create_scalar_int(0)),
+                $output,
+                $expr,
+            ));
         } else {
-            // $constructarg_0 = $values ?? array();
-            $argumentAssignedValue = new Expr\BinaryOp\Coalesce($output, $defaultValueExpr);
+            // $constructarg = $values ?? {expression};
+            $argumentAssignClosure = static fn (Expr $expr) => new Expr\Assign($constructVar, new Expr\BinaryOp\Coalesce($output, $expr));
         }
 
         return [
@@ -221,15 +225,15 @@ private function constructorArgument(GeneratorMetadata $metadata, PropertyMetada
             ]), [
                 'stmts' => [
                     ...$propStatements,
-                    new Stmt\Expression(new Expr\Assign($constructVar, new Expr\BinaryOp\Coalesce($output, new Expr\StaticCall(new Name\FullyQualified(MapperContext::class), 'getConstructorArgument', [
+                    new Stmt\Expression($argumentAssignClosure(new Expr\StaticCall(new Name\FullyQualified(MapperContext::class), 'getConstructorArgument', [
                         new Arg($variableRegistry->getContext()),
                         new Arg(new Scalar\String_($metadata->mapperMetadata->target)),
                         new Arg(new Scalar\String_($propertyMetadata->target->property)),
-                    ])))),
+                    ]))),
                 ],
                 'else' => new Stmt\Else_([
                     ...$propStatements,
-                    new Stmt\Expression(new Expr\Assign($constructVar, $argumentAssignedValue)),
+                    new Stmt\Expression($argumentAssignClosure($defaultValueExpr)),
                 ]),
             ]),
             new Arg($constructVar, name: new Identifier($parameter->getName())),

tests/AutoMapperTest.php

@@ -469,6 +469,19 @@ public function testConstructor(): void
         self::assertTrue($userDto->getConstructor());
     }
 
+    public function testConstructorArrayArgumentFromContext(): void
+    {
+        $data = ['baz' => 'baz'];
+        /** @var ConstructorWithDefaultValues $userDto */
+        $object = $this->autoMapper->map($data, ConstructorWithDefaultValues::class, [MapperContext::CONSTRUCTOR_ARGUMENTS => [
+            ConstructorWithDefaultValues::class => ['someOtters' => [1]],
+        ]]);
+
+        self::assertInstanceOf(ConstructorWithDefaultValues::class, $object);
+        self::assertSame('baz', $object->baz);
+        self::assertSame([1], $object->someOtters);
+    }
+
     public function testConstructorNotAllowed(): void
     {
         $this->buildAutoMapper(mapPrivatePropertiesAndMethod: true, constructorStrategy: ConstructorStrategy::NEVER, classPrefix: 'NotAllowedMapper_');