From 928c32d61626255bf514eadfcdf10288d08a6e09 Mon Sep 17 00:00:00 2001
From: Jan Kramer <jan@deft.nl>
Date: Sat, 7 Dec 2013 16:04:48 +0100
Subject: [PATCH] Update XML schema to reflect addition of embeddables

---
 doctrine-mapping.xsd                           | 18 ++++++++++++++++++
 lib/Doctrine/ORM/Mapping/Driver/XmlDriver.php  |  2 +-
 .../Tests/Models/ValueObjects/Name.php         |  9 +++++++++
 .../Tests/Models/ValueObjects/Person.php       |  9 +++++++++
 ...rine.Tests.Models.ValueObjects.Name.dcm.xml | 10 ++++++++++
 ...ne.Tests.Models.ValueObjects.Person.dcm.xml | 12 ++++++++++++
 6 files changed, 59 insertions(+), 1 deletion(-)
 create mode 100644 tests/Doctrine/Tests/Models/ValueObjects/Name.php
 create mode 100644 tests/Doctrine/Tests/Models/ValueObjects/Person.php
 create mode 100644 tests/Doctrine/Tests/ORM/Mapping/xml/Doctrine.Tests.Models.ValueObjects.Name.dcm.xml
 create mode 100644 tests/Doctrine/Tests/ORM/Mapping/xml/Doctrine.Tests.Models.ValueObjects.Person.dcm.xml

diff --git a/doctrine-mapping.xsd b/doctrine-mapping.xsd
index f9c774d57..b66e027e9 100644
--- a/doctrine-mapping.xsd
+++ b/doctrine-mapping.xsd
@@ -17,6 +17,7 @@
       <xs:sequence>
         <xs:element name="mapped-superclass" type="orm:mapped-superclass" minOccurs="0" maxOccurs="unbounded" />
         <xs:element name="entity" type="orm:entity" minOccurs="0" maxOccurs="unbounded" />
+        <xs:element name="embeddable" type="orm:embeddable" minOccurs="0" maxOccurs="unbounded" />
         <xs:any minOccurs="0" maxOccurs="unbounded" namespace="##other"/>
       </xs:sequence>
       <xs:anyAttribute namespace="##other"/>
@@ -166,6 +167,7 @@
       <xs:element name="sql-result-set-mappings" type="orm:sql-result-set-mappings" minOccurs="0" maxOccurs="unbounded" />
       <xs:element name="id" type="orm:id" minOccurs="0" maxOccurs="unbounded" />
       <xs:element name="field" type="orm:field" minOccurs="0" maxOccurs="unbounded"/>
+      <xs:element name="embedded" type="orm:embedded" minOccurs="0" maxOccurs="unbounded"/>
       <xs:element name="one-to-one" type="orm:one-to-one" minOccurs="0" maxOccurs="unbounded"/>
       <xs:element name="one-to-many" type="orm:one-to-many" minOccurs="0" maxOccurs="unbounded" />
       <xs:element name="many-to-one" type="orm:many-to-one" minOccurs="0" maxOccurs="unbounded" />
@@ -212,6 +214,16 @@
     </xs:complexContent>
   </xs:complexType>
 
+  <xs:complexType name="embeddable">
+    <xs:complexContent>
+      <xs:extension base="orm:entity">
+        <xs:sequence>
+          <xs:any minOccurs="0" maxOccurs="unbounded" namespace="##other"/>
+        </xs:sequence>
+      </xs:extension>
+    </xs:complexContent>
+  </xs:complexType>
+
   <xs:simpleType name="change-tracking-policy">
     <xs:restriction base="xs:token">
       <xs:enumeration value="DEFERRED_IMPLICIT"/>
@@ -274,6 +286,12 @@
     <xs:anyAttribute namespace="##other"/>
   </xs:complexType>
   
+  <xs:complexType name="embedded">
+    <xs:attribute name="name" type="xs:string" use="required" />
+    <xs:attribute name="class" type="xs:string" use="required" />
+    <xs:attribute name="column-prefix" type="xs:string" use="optional" />
+  </xs:complexType>
+  
   <xs:complexType name="discriminator-column">
     <xs:sequence>
       <xs:any minOccurs="0" maxOccurs="unbounded" namespace="##other"/>
diff --git a/lib/Doctrine/ORM/Mapping/Driver/XmlDriver.php b/lib/Doctrine/ORM/Mapping/Driver/XmlDriver.php
index 3b8bf7e99..95d6ad817 100644
--- a/lib/Doctrine/ORM/Mapping/Driver/XmlDriver.php
+++ b/lib/Doctrine/ORM/Mapping/Driver/XmlDriver.php
@@ -248,7 +248,7 @@ class XmlDriver extends FileDriver
                 $mapping = array(
                     'fieldName' => (string) $embeddedMapping['name'],
                     'class' => (string) $embeddedMapping['class'],
-                    'columnPrefix' => isset($embeddedMapping['class']) ? (string) $embeddedMapping['class'] : null,
+                    'columnPrefix' => isset($embeddedMapping['column-prefix']) ? (string) $embeddedMapping['column-prefix'] : null,
                 );
                 $metadata->mapEmbedded($mapping);
             }
diff --git a/tests/Doctrine/Tests/Models/ValueObjects/Name.php b/tests/Doctrine/Tests/Models/ValueObjects/Name.php
new file mode 100644
index 000000000..1c8360324
--- /dev/null
+++ b/tests/Doctrine/Tests/Models/ValueObjects/Name.php
@@ -0,0 +1,9 @@
+<?php
+
+namespace Doctrine\Tests\Models\ValueObjects;
+
+class Name
+{
+    private $firstName;
+    private $lastName;
+}
diff --git a/tests/Doctrine/Tests/Models/ValueObjects/Person.php b/tests/Doctrine/Tests/Models/ValueObjects/Person.php
new file mode 100644
index 000000000..7aa43918b
--- /dev/null
+++ b/tests/Doctrine/Tests/Models/ValueObjects/Person.php
@@ -0,0 +1,9 @@
+<?php
+
+namespace Doctrine\Tests\Models\ValueObjects;
+
+class Person
+{
+    private $id;
+    private $name;
+}
diff --git a/tests/Doctrine/Tests/ORM/Mapping/xml/Doctrine.Tests.Models.ValueObjects.Name.dcm.xml b/tests/Doctrine/Tests/ORM/Mapping/xml/Doctrine.Tests.Models.ValueObjects.Name.dcm.xml
new file mode 100644
index 000000000..97bb6a90e
--- /dev/null
+++ b/tests/Doctrine/Tests/ORM/Mapping/xml/Doctrine.Tests.Models.ValueObjects.Name.dcm.xml
@@ -0,0 +1,10 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<doctrine-mapping xmlns="http://doctrine-project.org/schemas/orm/doctrine-mapping"
+                  xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+                  xsi:schemaLocation="http://doctrine-project.org/schemas/orm/doctrine-mapping
+                          http://www.doctrine-project.org/schemas/orm/doctrine-mapping.xsd">
+  <embeddable name="Doctrine\Tests\Models\ValueObjects\Name">
+    <field name="firstName"/>
+    <field name="lastName"/>
+  </embeddable>
+</doctrine-mapping>
diff --git a/tests/Doctrine/Tests/ORM/Mapping/xml/Doctrine.Tests.Models.ValueObjects.Person.dcm.xml b/tests/Doctrine/Tests/ORM/Mapping/xml/Doctrine.Tests.Models.ValueObjects.Person.dcm.xml
new file mode 100644
index 000000000..c2480bca7
--- /dev/null
+++ b/tests/Doctrine/Tests/ORM/Mapping/xml/Doctrine.Tests.Models.ValueObjects.Person.dcm.xml
@@ -0,0 +1,12 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<doctrine-mapping xmlns="http://doctrine-project.org/schemas/orm/doctrine-mapping"
+                  xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+                  xsi:schemaLocation="http://doctrine-project.org/schemas/orm/doctrine-mapping
+                          http://www.doctrine-project.org/schemas/orm/doctrine-mapping.xsd">
+  <entity name="Doctrine\Tests\Models\ValueObjects\Person">
+    <id name="id" type="integer" column="id">
+      <generator strategy="AUTO"/>
+    </id>
+    <embedded name="name" class="Doctrine\Tests\Models\ValueObjects\Name" column-prefix="nm_"/>
+  </entity>
+</doctrine-mapping>