Merge pull request #33 from Sinjonathan/main

Ehyiah · web-flow · commit 8c21743c6e0e · 2024-06-23T19:57:13.000+02:00
Fix wrong usage of property accessor
diff --git a/src/Transformer/CollectionTransformer.php b/src/Transformer/CollectionTransformer.php
@@ -56,7 +56,8 @@ public function transform(mixed $data, array $options, object $targetObject, obj
         foreach ($data as $datum) {
             $target = null;
             if ($propertyAccessor->isReadable($datum, 'id')) {
-                $target = null !== $propertyAccessor->getValue($datum, 'id') ? $this->em->getRepository($fcqn)->find($datum->id) : $target;
+                $id = $propertyAccessor->getValue($datum, 'id');
+                $target = null !== $id ? $this->em->getRepository($fcqn)->find($id) : $target;
             }
 
             $target = $this->mappingService->mapToTarget($datum, $target);

Original file line number	Diff line number	Diff line change
`@@ -56,7 +56,8 @@ public function transform(mixed $data, array $options, object $targetObject, obj`
`56`	`56`	`foreach ($data as $datum) {`
`57`	`57`	`$target = null;`
`58`	`58`	`if ($propertyAccessor->isReadable($datum, 'id')) {`
`59`		`- $target = null !== $propertyAccessor->getValue($datum, 'id') ? $this->em->getRepository($fcqn)->find($datum->id) : $target;`
	`59`	`+ $id = $propertyAccessor->getValue($datum, 'id');`
	`60`	`+ $target = null !== $id ? $this->em->getRepository($fcqn)->find($id) : $target;`
`60`	`61`	`}`
`61`	`62`
`62`	`63`	`$target = $this->mappingService->mapToTarget($datum, $target);`