Added options attribute export to Annotation, Xml & Yaml

exporters.

lib/Doctrine/ORM/Mapping/Table.php

@@ -48,5 +48,5 @@ final class Table implements Annotation
     /**
      * @var array
      */
-    public $options = array();
+    public $options;
 }

lib/Doctrine/ORM/Tools/EntityGenerator.php

@@ -910,6 +910,10 @@ public function __construct()
             $table[] = 'name="' . $metadata->table['name'] . '"';
         }
 
+        if (isset($metadata->table['options']) && $metadata->table['options']) {
+            $table[] = 'options={' . $this->_exportOptions((array) $metadata->table['options']) . '}';
+        }
+
         if (isset($metadata->table['uniqueConstraints']) && $metadata->table['uniqueConstraints']) {
             $constraints = $this->generateTableConstraints('UniqueConstraint', $metadata->table['uniqueConstraints']);
             $table[] = 'uniqueConstraints={' . $constraints . '}';
@@ -1556,4 +1560,25 @@ public function __construct()
 
         return static::$generatorStrategyMap[$type];
     }
+
+    /**
+     * Exports (nested) option elements.
+     *
+     * @param array $options
+     */
+    private function _exportOptions(array $options)
+    {
+        $optionsStr = array();
+        
+        foreach($options as $name => $option)
+        {
+            if (is_array($option)) {
+                $optionsStr[] = '"' . $name . '"={' . $this->_exportOptions($option) . '}';
+            } else {
+                $optionsStr[] = '"' . $name . '"="' . (string) $option . '"';
+            }            
+        }
+        
+        return implode(',', $optionsStr);
+    }
 }

lib/Doctrine/ORM/Tools/Export/Driver/XmlExporter.php

@@ -45,10 +45,6 @@ class XmlExporter extends AbstractExporter
             "xmlns:xsi=\"http://www.w3.org/2001/XMLSchema-instance\" ".
             "xsi:schemaLocation=\"http://doctrine-project.org/schemas/orm/doctrine-mapping http://doctrine-project.org/schemas/orm/doctrine-mapping.xsd\" />");
 
-        /*$xml->addAttribute('xmlns', 'http://doctrine-project.org/schemas/orm/doctrine-mapping');
-        $xml->addAttribute('xmlns:xsi', 'http://www.w3.org/2001/XMLSchema-instance');
-        $xml->addAttribute('xsi:schemaLocation', 'http://doctrine-project.org/schemas/orm/doctrine-mapping http://doctrine-project.org/schemas/orm/doctrine-mapping.xsd');*/
-
         if ($metadata->isMappedSuperclass) {
             $root = $xml->addChild('mapped-superclass');
         } else {
@@ -73,6 +69,12 @@ class XmlExporter extends AbstractExporter
             $root->addAttribute('inheritance-type', $this->_getInheritanceTypeString($metadata->inheritanceType));
         }
 
+        if (isset($metadata->table['options'])) {
+            $optionsXml = $root->addChild('options');
+
+            $this->_exportOptions($optionsXml, $metadata->table['options']);
+        }
+
         if ($metadata->discriminatorColumn) {
             $discriminatorColumnXml = $root->addChild('discriminator-column');
             $discriminatorColumnXml->addAttribute('name', $metadata->discriminatorColumn['name']);
@@ -106,6 +108,9 @@ class XmlExporter extends AbstractExporter
                 $indexXml = $indexesXml->addChild('index');
                 $indexXml->addAttribute('name', $name);
                 $indexXml->addAttribute('columns', implode(',', $index['columns']));
+                if(isset($index['flags'])) {
+                    $indexXml->addAttribute('flags', implode(',', $index['flags']));
+                }
             }
         }
 
@@ -380,6 +385,26 @@ class XmlExporter extends AbstractExporter
         return $this->_asXml($xml);
     }
 
+    /**
+     * Exports (nested) option elements.
+     *
+     * @param \SimpleXMLElement $parentXml
+     * @param array $options
+     */
+    private function _exportOptions(\SimpleXMLElement $parentXml, array $options)
+    {        
+        foreach ($options as $name => $option) {
+            $optionXml = $parentXml->addChild('option');
+            $optionXml->addAttribute('name', (string) $name);
+            
+            if (is_array($option)) {
+                $this->_exportOptions($optionXml, $option);
+            } else {
+                $optionXml[0] = (string) $option;
+            }
+        }
+    }
+
     /**
      * @param \SimpleXMLElement $simpleXml
      *

lib/Doctrine/ORM/Tools/Export/Driver/YamlExporter.php

@@ -85,6 +85,10 @@ class YamlExporter extends AbstractExporter
             $array['uniqueConstraints'] = $metadata->table['uniqueConstraints'];
         }
 
+        if (isset($metadata->table['options'])) {
+            $array['options'] = $metadata->table['options'];
+        }
+
         $fieldMappings = $metadata->fieldMappings;
 
         $ids = array();

tests/Doctrine/Tests/ORM/Tools/Export/AbstractClassMetadataExporterTest.php

@@ -157,6 +157,8 @@ abstract class AbstractClassMetadataExporterTest extends \Doctrine\Tests\OrmTest
     public function testTableIsExported($class)
     {
         $this->assertEquals('cms_users', $class->table['name']);
+        $this->assertEquals(array('engine' => 'MyISAM', 'foo' => array('bar' => 'baz')),
+            $class->table['options']);
 
         return $class;
     }

tests/Doctrine/Tests/ORM/Tools/Export/annotation/Doctrine.Tests.ORM.Tools.Export.User.php

@@ -5,7 +5,7 @@ namespace Doctrine\Tests\ORM\Tools\Export;
 /**
  * @Entity
  * @HasLifecycleCallbacks
- * @Table(name="cms_users")
+ * @Table(name="cms_users",options={"engine"="MyISAM","foo"={"bar"="baz"}})
  */
 class User
 {

tests/Doctrine/Tests/ORM/Tools/Export/php/Doctrine.Tests.ORM.Tools.Export.User.php

@@ -5,6 +5,7 @@ use Doctrine\ORM\Mapping\ClassMetadataInfo;
 $metadata->setInheritanceType(ClassMetadataInfo::INHERITANCE_TYPE_NONE);
 $metadata->setPrimaryTable(array(
    'name' => 'cms_users',
+   'options' => array('engine' => 'MyISAM', 'foo' => array('bar' => 'baz')),
   ));
 $metadata->setChangeTrackingPolicy(ClassMetadataInfo::CHANGETRACKING_DEFERRED_IMPLICIT);
 $metadata->addLifecycleCallback('doStuffOnPrePersist', 'prePersist');

tests/Doctrine/Tests/ORM/Tools/Export/xml/Doctrine.Tests.ORM.Tools.Export.User.dcm.xml

@@ -6,7 +6,12 @@
                           http://www.doctrine-project.org/schemas/orm/doctrine-mapping.xsd">
 
     <entity name="Doctrine\Tests\ORM\Tools\Export\User" table="cms_users">
-
+        <options>
+            <option name="engine">MyISAM</option>
+            <option name="foo">
+                <option name="bar">baz</option>
+            </option>
+        </options>
         <lifecycle-callbacks>
             <lifecycle-callback type="prePersist" method="doStuffOnPrePersist"/>
             <lifecycle-callback type="prePersist" method="doOtherStuffOnPrePersistToo"/>

tests/Doctrine/Tests/ORM/Tools/Export/yaml/Doctrine.Tests.ORM.Tools.Export.User.dcm.yml

@@ -1,6 +1,9 @@
 Doctrine\Tests\ORM\Tools\Export\User:
   type: entity
   table: cms_users
+  options:
+    engine: MyISAM
+    foo: { bar: baz }
   id:
     id:
       type: integer