Go back to nested to avoid hitting the else branch

Author: mbabker

src/Revisionable/Document/Repository/RevisionRepository.php

@@ -124,13 +124,15 @@ protected function fillDocument(object $document, array $data): void
             $mapping = $documentMeta->getFieldMapping($field);
 
             // Fill the embedded document
-            if ($documentWrapper->isEmbeddedAssociation($field) && null !== $value) {
-                assert(class_exists($mapping['targetDocument']));
-
-                $embeddedMetadata = $this->getDocumentManager()->getClassMetadata($mapping['targetDocument']);
-                $document = $embeddedMetadata->newInstance();
-                $this->fillDocument($document, $value);
-                $value = $document;
+            if ($documentWrapper->isEmbeddedAssociation($field)) {
+                if (null !== $value) {
+                    assert(class_exists($mapping['targetDocument']));
+
+                    $embeddedMetadata = $this->getDocumentManager()->getClassMetadata($mapping['targetDocument']);
+                    $document = $embeddedMetadata->newInstance();
+                    $this->fillDocument($document, $value);
+                    $value = $document;
+                }
             } elseif ($documentMeta->isSingleValuedAssociation($field)) {
                 assert(class_exists($mapping['targetDocument']));