Apply suggestions from code review

soyuka · nicolas-grekas · web-flow · commit 3535b28442bc · 2025-08-21T15:14:33.000+02:00
Co-authored-by: Nicolas Grekas &lt;nicolas.grekas@gmail.com&gt;
diff --git a/src/Symfony/Component/ObjectMapper/CHANGELOG.md b/src/Symfony/Component/ObjectMapper/CHANGELOG.md
@@ -5,8 +5,7 @@ CHANGELOG
 ---
 
  * Add `ObjectMapperAwareInterface` to set the owning object mapper instance
- * Add a `MapCollection` transform that calls the Mapper over iterable
-   properties
+ * Add a `MapCollection` transform that calls the Mapper over iterable properties
 
 7.3
 ---
diff --git a/src/Symfony/Component/ObjectMapper/Tests/Fixtures/TransformCollection/TransformCollectionC.php b/src/Symfony/Component/ObjectMapper/Tests/Fixtures/TransformCollection/TransformCollectionC.php
@@ -9,6 +9,7 @@ class TransformCollectionC
 {
     public function __construct(
         #[Map(target: 'baz')]
-        public string $bar
-    ) {}
+        public string $bar,
+    ) {
+    }
 }
diff --git a/src/Symfony/Component/ObjectMapper/Tests/Fixtures/TransformCollection/TransformCollectionD.php b/src/Symfony/Component/ObjectMapper/Tests/Fixtures/TransformCollection/TransformCollectionD.php
@@ -5,6 +5,7 @@
 class TransformCollectionD
 {
     public function __construct(
-        public string $baz
-    ) {}
+        public string $baz,
+    ) {
+    }
 }
diff --git a/src/Symfony/Component/ObjectMapper/Transform/MapCollection.php b/src/Symfony/Component/ObjectMapper/Transform/MapCollection.php
@@ -23,8 +23,9 @@
  */
 class MapCollection implements TransformCallableInterface
 {
-    public function __construct(private ObjectMapperInterface $objectMapper = new ObjectMapper())
-    {
+    public function __construct(
+        private ObjectMapperInterface $objectMapper = new ObjectMapper(),
+    ) {
     }
 
     public function __invoke(mixed $value, object $source, ?object $target): mixed

Original file line number	Diff line number	Diff line change
`@@ -9,6 +9,7 @@ class TransformCollectionC`
`9`	`9`	`{`
`10`	`10`	`public function __construct(`
`11`	`11`	`#[Map(target: 'baz')]`
`12`		`- public string $bar`
`13`		`- ) {}`
	`12`	`+ public string $bar,`
	`13`	`+ ) {`
	`14`	`+ }`
`14`	`15`	`}`
Original file line number	Diff line number	Diff line change
`@@ -5,6 +5,7 @@`
`5`	`5`	`class TransformCollectionD`
`6`	`6`	`{`
`7`	`7`	`public function __construct(`
`8`		`- public string $baz`
`9`		`- ) {}`
	`8`	`+ public string $baz,`
	`9`	`+ ) {`
	`10`	`+ }`
`10`	`11`	`}`
Original file line number	Diff line number	Diff line change
`@@ -23,8 +23,9 @@`
`23`	`23`	`*/`
`24`	`24`	`class MapCollection implements TransformCallableInterface`
`25`	`25`	`{`
`26`		`- public function __construct(private ObjectMapperInterface $objectMapper = new ObjectMapper())`
`27`		`- {`
	`26`	`+ public function __construct(`
	`27`	`+ private ObjectMapperInterface $objectMapper = new ObjectMapper(),`
	`28`	`+ ) {`
`28`	`29`	`}`
`29`	`30`
`30`	`31`	`public function __invoke(mixed $value, object $source, ?object $target): mixed`