From 7e4dab17ec964fcbedaf9cd67f50b2f9d466d90a Mon Sep 17 00:00:00 2001
From: Marco Pivetta <ocramius@gmail.com>
Date: Fri, 16 Jan 2015 19:45:16 +0100
Subject: [PATCH] #1133 DDC-3305 - test case with embeddable without referenced
 embeddable class

---
 .../ORM/Mapping/ClassMetadataFactory.php      |  2 +-
 .../ORM/Mapping/ClassMetadataInfo.php         |  8 +++--
 .../ORM/Mapping/ClassMetadataFactoryTest.php  | 32 +++++++++++++++++--
 3 files changed, 36 insertions(+), 6 deletions(-)

diff --git a/lib/Doctrine/ORM/Mapping/ClassMetadataFactory.php b/lib/Doctrine/ORM/Mapping/ClassMetadataFactory.php
index 2056af6f8..24ce1ea2e 100644
--- a/lib/Doctrine/ORM/Mapping/ClassMetadataFactory.php
+++ b/lib/Doctrine/ORM/Mapping/ClassMetadataFactory.php
@@ -170,7 +170,7 @@ class ClassMetadataFactory extends AbstractClassMetadataFactory
                     continue;
                 }
 
-                if (!isset($embeddableClass['class'])) {
+                if (! (isset($embeddableClass['class']) && $embeddableClass['class'])) {
                     throw MappingException::missingEmbeddedClass($property);
                 }
 
diff --git a/lib/Doctrine/ORM/Mapping/ClassMetadataInfo.php b/lib/Doctrine/ORM/Mapping/ClassMetadataInfo.php
index 8d482e84d..769b80f25 100644
--- a/lib/Doctrine/ORM/Mapping/ClassMetadataInfo.php
+++ b/lib/Doctrine/ORM/Mapping/ClassMetadataInfo.php
@@ -3166,11 +3166,15 @@ class ClassMetadataInfo implements ClassMetadata
     }
 
     /**
-     * @param   string $className
-     * @return  string
+     * @param  string|null $className
+     * @return string|null null if the input value is null
      */
     public function fullyQualifiedClassName($className)
     {
+        if (empty($className)) {
+            return $className;
+        }
+
         if ($className !== null && strpos($className, '\\') === false && strlen($this->namespace) > 0) {
             return $this->namespace . '\\' . $className;
         }
diff --git a/tests/Doctrine/Tests/ORM/Mapping/ClassMetadataFactoryTest.php b/tests/Doctrine/Tests/ORM/Mapping/ClassMetadataFactoryTest.php
index 31345fdcc..cd17871ee 100644
--- a/tests/Doctrine/Tests/ORM/Mapping/ClassMetadataFactoryTest.php
+++ b/tests/Doctrine/Tests/ORM/Mapping/ClassMetadataFactoryTest.php
@@ -29,7 +29,7 @@ class ClassMetadataFactoryTest extends \Doctrine\Tests\OrmTestCase
         $cm1 = $this->_createValidClassMetadata();
 
         // SUT
-        $cmf = new \Doctrine\ORM\Mapping\ClassMetadataFactory();
+        $cmf = new ClassMetadataFactory();
         $cmf->setEntityManager($entityManager);
         $cmf->setMetadataFor($cm1->name, $cm1);
 
@@ -375,10 +375,35 @@ class ClassMetadataFactoryTest extends \Doctrine\Tests\OrmTestCase
         // not really the cleanest way to check it, but we won't add a getter to the CMF just for the sake of testing.
         $this->assertAttributeSame($entityManager, 'em', $classMetadataFactory);
     }
+
+    /**
+     * @group DDC-3305
+     */
+    public function testRejectsEmbeddableWithoutValidClassName()
+    {
+        $metadata = $this->_createValidClassMetadata();
+
+        $metadata->mapEmbedded(array(
+            'fieldName'    => 'embedded',
+            'class'        => '',
+            'columnPrefix' => false,
+        ));
+
+        $cmf = $this->_createTestFactory();
+
+        $cmf->setMetadataForClass($metadata->name, $metadata);
+
+        $this->setExpectedException(
+            'Doctrine\ORM\Mapping\MappingException',
+            'The embed mapping \'embedded\' misses the \'class\' attribute.'
+        );
+
+        $cmf->getMetadataFor($metadata->name);
+    }
 }
 
 /* Test subject class with overridden factory method for mocking purposes */
-class ClassMetadataFactoryTestSubject extends \Doctrine\ORM\Mapping\ClassMetadataFactory
+class ClassMetadataFactoryTestSubject extends ClassMetadataFactory
 {
     private $mockMetadata = array();
     private $requestedClasses = array();
@@ -388,7 +413,7 @@ class ClassMetadataFactoryTestSubject extends \Doctrine\ORM\Mapping\ClassMetadat
     {
         $this->requestedClasses[] = $className;
         if ( ! isset($this->mockMetadata[$className])) {
-            throw new InvalidArgumentException("No mock metadata found for class $className.");
+            throw new \InvalidArgumentException("No mock metadata found for class $className.");
         }
         return $this->mockMetadata[$className];
     }
@@ -410,6 +435,7 @@ class TestEntity1
     private $name;
     private $other;
     private $association;
+    private $embedded;
 }
 
 class CustomIdGenerator extends AbstractIdGenerator