Fixed merging metadata from IEntityProvider with config [Fixes #299]

Introduced in 4cda9bb IEntityProvider cannot change already configured namespace

[#88]

src/Kdyby/Doctrine/DI/OrmExtension.php

@@ -252,11 +252,11 @@ protected function processEntityManager($name, array $defaults)
 			if ($extension instanceof IEntityProvider) {
 				$metadata = $extension->getEntityMappings();
 				Validators::assert($metadata, 'array');
-				foreach ($metadata as $namespace => $config) {
+				foreach ($metadata as $namespace => $nsConfig) {
 					if (array_key_exists($namespace, $config['metadata'])) {
 						throw new Nette\Utils\AssertionException(sprintf('The namespace %s is already configured, provider cannot change it', $namespace));
 					}
-					$config['metadata'][$namespace] = $config;
+					$config['metadata'][$namespace] = $nsConfig;
 				}
 			}
 

tests/KdybyTests/Doctrine/Extension.phpt

@@ -121,6 +121,23 @@ class ExtensionTest extends Tester\TestCase
 		}, $default->getMetadataFactory()->getAllMetadata());
 
 		Assert::contains('KdybyTests\\Doctrine\\CmsArticle', $entityClasses);
+		Assert::notContains('KdybyTests\Doctrine\Models2\Foo', $entityClasses);
+	}
+
+
+
+	public function testEntityMetadataMergingFromProvider()
+	{
+		$container = $this->createContainer('entity-provider-merging');
+
+		/** @var Kdyby\Doctrine\EntityManager $default */
+		$default = $container->getByType('Kdyby\Doctrine\EntityManager');
+		$entityClasses = array_map(function (ClassMetadata $class) {
+			return $class->getName();
+		}, $default->getMetadataFactory()->getAllMetadata());
+
+		Assert::contains('KdybyTests\Doctrine\CmsArticle', $entityClasses);
+		Assert::contains('KdybyTests\Doctrine\Models2\Foo', $entityClasses);
 	}
 
 

tests/KdybyTests/Doctrine/config/entity-provider-merging.neon

@@ -0,0 +1,10 @@
+kdyby.doctrine:
+	driver: pdo_sqlite
+	memory: true
+	metadata:
+		KdybyTests\Doctrine: annotations(%appDir%/Doctrine/models)
+	targetEntityMappings:
+		KdybyTests\Doctrine\ICmsAddress: KdybyTests\Doctrine\CmsAddress
+
+extensions:
+	- KdybyTests\Doctrine\Models2\Models2Extension

tests/KdybyTests/Doctrine/models2/Models2Extension.php

@@ -0,0 +1,26 @@
+<?php
+
+/**
+ * This file is part of the Kdyby (http://www.kdyby.org)
+ *
+ * Copyright (c) 2008 Filip Procházka ([email protected])
+ *
+ * For the full copyright and license information, please view the file license.txt that was distributed with this source code.
+ */
+
+namespace KdybyTests\Doctrine\Models2;
+
+use Kdyby\Doctrine\DI\IEntityProvider;
+use Nette\DI\CompilerExtension;
+
+class Models2Extension extends CompilerExtension implements IEntityProvider
+{
+
+	public function getEntityMappings()
+	{
+		return [
+			__NAMESPACE__ => __DIR__,
+		];
+	}
+
+}