Handle mapping without mappedBy

See #6219 (comment)
sonata-project · Aug 22, 2020 · 8d3cbdb · 8d3cbdb
1 parent f46719b
commit 8d3cbdb
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/src/Manipulator/ObjectManipulator.php b/src/Manipulator/ObjectManipulator.php
@@ -53,7 +53,7 @@ public static function setObject(
         object $object,
         FieldDescriptionInterface $parentFieldDescription
     ): object {
-        $mappedBy = $parentFieldDescription->getAssociationMapping()['mappedBy'];
+        $mappedBy = $parentFieldDescription->getAssociationMapping()['mappedBy'] ?? null;
         if (null === $mappedBy) {
             return $instance;
         }

diff --git a/tests/Manipulator/ObjectManipulatorTest.php b/tests/Manipulator/ObjectManipulatorTest.php
@@ -88,14 +88,22 @@ public function testSetObject(): void
         ObjectManipulator::setObject($instance, $object, $fieldDescription);
     }
 
-    public function testSetObjectWithoutMappedBy(): void
+    public function testSetObjectWithNullMapped(): void
     {
         $fieldDescription = $this->createMock(FieldDescriptionInterface::class);
         $fieldDescription->expects($this->once())->method('getAssociationMapping')->willReturn(['mappedBy' => null]);
 
         ObjectManipulator::setObject(new \stdClass(), new \stdClass(), $fieldDescription);
     }
 
+    public function testSetObjectWithoutMappedBy(): void
+    {
+        $fieldDescription = $this->createMock(FieldDescriptionInterface::class);
+        $fieldDescription->expects($this->once())->method('getAssociationMapping')->willReturn([]);
+
+        ObjectManipulator::setObject(new \stdClass(), new \stdClass(), $fieldDescription);
+    }
+
     public function testSetObjectWithParentAssociation(): void
     {
         $fieldDescription = $this->createMock(FieldDescriptionInterface::class);