combine schema files into one AppData

This fixes #173. In case the schema.xml files are referenced across multiple bundles, the \AppData needs to be extended by each bundle, before reading data from the respective \Database.
propelorm · Jan 23, 2016 · da1f027 · da1f027
1 parent 4db28b7
commit da1f027
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/Command/FormGenerateCommand.php b/Command/FormGenerateCommand.php
@@ -52,8 +52,11 @@ protected function configure()
     protected function execute(InputInterface $input, OutputInterface $output)
     {
         if ($schemas = $this->getSchemasFromBundle($this->bundle)) {
+            $schemas = $this->getFinalSchemas($this->getContainer()->get('kernel'));
+
+            $transformer = new \XmlToAppData(null, null, 'UTF-8');
             foreach ($schemas as $fileName => $array) {
-                foreach ($this->getDatabasesFromSchema($array[1]) as $database) {
+                foreach ($this->getDatabasesFromSchema($array[1], $transformer) as $database) {
                     $this->createFormTypeFromDatabase($this->bundle, $database, $input->getArgument('models'), $output, $input->getOption('force'));
                 }
             }

diff --git a/Command/GeneratorAwareCommand.php b/Command/GeneratorAwareCommand.php
@@ -38,9 +38,12 @@ protected function loadPropelGenerator()
         set_include_path(sprintf('%s/generator/lib', $propelPath) . PATH_SEPARATOR . get_include_path());
     }
 
-    protected function getDatabasesFromSchema(\SplFileInfo $file)
+    protected function getDatabasesFromSchema(\SplFileInfo $file, \XmlToAppData $transformer = null)
     {
-        $transformer = new \XmlToAppData(null, null, 'UTF-8');
+        if (null === $transformer) {
+            $transformer = new \XmlToAppData(null, null, 'UTF-8');
+        }
+
         $config      = new \QuickGeneratorConfig();
 
         if (file_exists($propelIni = $this->getContainer()->getParameter('kernel.root_dir') . '/config/propel.ini')) {