diff --git a/src/Symfony/Component/ObjectMapper/ObjectMapper.php b/src/Symfony/Component/ObjectMapper/ObjectMapper.php index 204e6af608a7c..28eda13513119 100644 --- a/src/Symfony/Component/ObjectMapper/ObjectMapper.php +++ b/src/Symfony/Component/ObjectMapper/ObjectMapper.php @@ -156,6 +156,14 @@ public function map(object $source, object|string|null $target = null): object } } + if ($mappingToObject && $ctorArguments) { + foreach ($ctorArguments as $property => $value) { + if ($targetRefl->hasProperty($property) && $targetRefl->getProperty($property)->isPublic()) { + $mapToProperties[$property] = $value; + } + } + } + foreach ($mapToProperties as $property => $value) { $this->propertyAccessor ? $this->propertyAccessor->setValue($mappedTarget, $property, $value) : ($mappedTarget->{$property} = $value); } diff --git a/src/Symfony/Component/ObjectMapper/Tests/Fixtures/PromotedConstructor/Source.php b/src/Symfony/Component/ObjectMapper/Tests/Fixtures/PromotedConstructor/Source.php new file mode 100644 index 0000000000000..2612a092bcf1a --- /dev/null +++ b/src/Symfony/Component/ObjectMapper/Tests/Fixtures/PromotedConstructor/Source.php @@ -0,0 +1,21 @@ + + * + * For the full copyright and license information, please view the LICENSE + * file that was distributed with this source code. + */ + +namespace Symfony\Component\ObjectMapper\Tests\Fixtures\PromotedConstructor; + +class Source +{ + public function __construct( + public int $id, + public string $name, + ) { + } +} diff --git a/src/Symfony/Component/ObjectMapper/Tests/Fixtures/PromotedConstructor/Target.php b/src/Symfony/Component/ObjectMapper/Tests/Fixtures/PromotedConstructor/Target.php new file mode 100644 index 0000000000000..5833e1d08ca20 --- /dev/null +++ b/src/Symfony/Component/ObjectMapper/Tests/Fixtures/PromotedConstructor/Target.php @@ -0,0 +1,21 @@ + + * + * For the full copyright and license information, please view the LICENSE + * file that was distributed with this source code. + */ + +namespace Symfony\Component\ObjectMapper\Tests\Fixtures\PromotedConstructor; + +class Target +{ + public function __construct( + public int $id, + public string $name, + ) { + } +} diff --git a/src/Symfony/Component/ObjectMapper/Tests/ObjectMapperTest.php b/src/Symfony/Component/ObjectMapper/Tests/ObjectMapperTest.php index e278381c0d6a8..81c1d56b17f6b 100644 --- a/src/Symfony/Component/ObjectMapper/Tests/ObjectMapperTest.php +++ b/src/Symfony/Component/ObjectMapper/Tests/ObjectMapperTest.php @@ -20,6 +20,7 @@ use Symfony\Component\ObjectMapper\Metadata\ObjectMapperMetadataFactoryInterface; use Symfony\Component\ObjectMapper\Metadata\ReflectionObjectMapperMetadataFactory; use Symfony\Component\ObjectMapper\ObjectMapper; +use Symfony\Component\ObjectMapper\ObjectMapperInterface; use Symfony\Component\ObjectMapper\Tests\Fixtures\A; use Symfony\Component\ObjectMapper\Tests\Fixtures\B; use Symfony\Component\ObjectMapper\Tests\Fixtures\C; @@ -51,6 +52,8 @@ use Symfony\Component\ObjectMapper\Tests\Fixtures\MultipleTargetProperty\C as MultipleTargetPropertyC; use Symfony\Component\ObjectMapper\Tests\Fixtures\MultipleTargets\A as MultipleTargetsA; use Symfony\Component\ObjectMapper\Tests\Fixtures\MultipleTargets\C as MultipleTargetsC; +use Symfony\Component\ObjectMapper\Tests\Fixtures\PromotedConstructor\Source as PromotedConstructorSource; +use Symfony\Component\ObjectMapper\Tests\Fixtures\PromotedConstructor\Target as PromotedConstructorTarget; use Symfony\Component\ObjectMapper\Tests\Fixtures\Recursion\AB; use Symfony\Component\ObjectMapper\Tests\Fixtures\Recursion\Dto; use Symfony\Component\ObjectMapper\Tests\Fixtures\ServiceLocator\A as ServiceLocatorA; @@ -345,4 +348,24 @@ public function testDefaultValueStdClassWithPropertyInfo() $this->assertSame('abc', $b->id); $this->assertNull($b->optional); } + + /** + * @dataProvider objectMapperProvider + */ + public function testUpdateObjectWithConstructorPromotedProperties(ObjectMapperInterface $mapper) + { + $a = new PromotedConstructorSource(1, 'foo'); + $b = new PromotedConstructorTarget(1, 'bar'); + $v = $mapper->map($a, $b); + $this->assertSame($v->name, 'foo'); + } + + /** + * @return iterable + */ + public static function objectMapperProvider(): iterable + { + yield [new ObjectMapper()]; + yield [new ObjectMapper(new ReflectionObjectMapperMetadataFactory(), PropertyAccess::createPropertyAccessor())]; + } } pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy