Cleanup: drop deprecated yaml support

Author: f3l1x

.docs/README.md

@@ -11,7 +11,6 @@
   - [Attributes](#attributes)
   - [Annotations](#annotations)
   - [XML](#xml)
-  - [YAML](#yaml)
   - [Helpers](#helpers)
 - [Examples](#examples)
 - [Other](#other)
@@ -188,7 +187,6 @@ Additional metadata provider needs to be registered. We provide bridges for thes
 
 - **attributes** (`Nettrine\ORM\DI\OrmAttributesExtension`)
 - **annotations** (`Nettrine\ORM\DI\OrmAnnotationsExtension`)
-- **yaml** (`Nettrine\ORM\DI\OrmYamlExtension`)
 - **xml** (`Nettrine\ORM\DI\OrmXmlExtension`)
 
 
@@ -343,43 +341,6 @@ nettrine.orm.xml:
 Using **simple** you will enable [`SimplifiedXmlDriver`](https://www.doctrine-project.org/projects/doctrine-orm/en/2.7/reference/xml-mapping.html#simplified-xml-driver).
 
 
-### YAML
-
-Are you using [YAML mapping](https://www.doctrine-project.org/projects/doctrine-orm/en/2.7/reference/yaml-mapping.html) for your entities?
-
-```yaml
-# Doctrine.Tests.ORM.Mapping.User.dcm.yml
-Doctrine\Tests\ORM\Mapping\User:
-  type: entity
-  repositoryClass: Doctrine\Tests\ORM\Mapping\UserRepository
-  table: cms_users
-  schema: schema_name
-  readOnly: true
-  indexes:
-    name_index:
-      columns: [ name ]
-  id:
-    id:
-      type: integer
-      generator:
-        strategy: AUTO
-```
-
-You will also appreciate ORM => YAML bridge, use `OrmYamlExtension`. This is the default configuration:
-
-```neon
-extensions:
-  nettrine.orm: Nettrine\ORM\DI\OrmExtension
-  nettrine.orm.yaml: Nettrine\ORM\DI\OrmYamlExtension
-
-nettrine.orm.yaml:
-  mapping: [
-    namespace: path
-  ]
-  fileExtension: .orm.yml
-```
-
-
 ### Helpers
 
 **MappingHelper**
@@ -400,7 +361,6 @@ class CategoryExtension extends CompilerExtension
         ->addAnnotation('Forum\Modules\Database', __DIR__ . '/../../modules/Forum/Database')
         ->addXml('Gallery1\Modules\Database', __DIR__ . '/../../modules/Gallery1/Database')
         ->addXml('Gallery2\Modules\Database', __DIR__ . '/../../modules/Gallery2/Database', $simple = TRUE)
-        ->addYaml('Users\Modules\Database', __DIR__ . '/../../modules/Users/Database');
   }
 
 }

src/DI/Helpers/MappingHelper.php

@@ -10,7 +10,6 @@
 use Nettrine\ORM\DI\OrmAttributesExtension;
 use Nettrine\ORM\DI\OrmExtension;
 use Nettrine\ORM\DI\OrmXmlExtension;
-use Nettrine\ORM\DI\OrmYamlExtension;
 use Nettrine\ORM\Exception\Logical\InvalidStateException;
 
 class MappingHelper
@@ -84,23 +83,6 @@ public function addXml(string $namespace, string $path, bool $simple = false): s
 		return $this;
 	}
 
-	public function addYaml(string $namespace, string $path): self
-	{
-		if (!is_dir($path)) {
-			throw new InvalidStateException(sprintf('Given mapping path "%s" does not exist', $path));
-		}
-
-		/** @var ServiceDefinition $yamlDriver */
-		$yamlDriver = $this->getService(OrmYamlExtension::DRIVER_TAG, 'YamlDriver');
-		$yamlDriver->addSetup(new Statement('$service->getLocator()->addNamespacePrefixes([? => ?])', [$path, $namespace]));
-
-		/** @var ServiceDefinition $chainDriver */
-		$chainDriver = $this->getService(OrmExtension::MAPPING_DRIVER_TAG, 'MappingDriverChain');
-		$chainDriver->addSetup('addDriver', [$yamlDriver, $namespace]);
-
-		return $this;
-	}
-
 	private function getService(string $tag, string $name): Definition
 	{
 		$builder = $this->extension->getContainerBuilder();

src/DI/OrmYamlExtension.php

@@ -1,14 +1,12 @@
 <?php declare(strict_types = 1);
 
 use Doctrine\ORM\Mapping\Driver\AnnotationDriver;
-use Doctrine\ORM\Mapping\Driver\SimplifiedYamlDriver;
 use Doctrine\ORM\Mapping\Driver\XmlDriver;
 use Doctrine\Persistence\Mapping\Driver\MappingDriverChain;
 use Nette\DI\Compiler;
 use Nettrine\Annotations\DI\AnnotationsExtension;
 use Nettrine\ORM\DI\OrmAnnotationsExtension;
 use Nettrine\ORM\DI\OrmXmlExtension;
-use Nettrine\ORM\DI\OrmYamlExtension;
 use Ninjify\Nunjuck\Toolkit;
 use Tester\Assert;
 use Tests\Fixtures\EntityMappingCompilerExtension;
@@ -25,7 +23,6 @@ Toolkit::test(function (): void {
 			$compiler->addExtension('annotations', new AnnotationsExtension());
 			$compiler->addExtension('nettrine.orm.annotations', new OrmAnnotationsExtension());
 			$compiler->addExtension('nettrine.orm.xml', new OrmXmlExtension());
-			$compiler->addExtension('nettrine.orm.yaml', new OrmYamlExtension());
 			$compiler->addExtension('tests.mapping', new EntityMappingCompilerExtension());
 			$compiler->addConfig(Helpers::neon('
 					nettrine.orm.annotations:
@@ -34,9 +31,6 @@ Toolkit::test(function (): void {
 					nettrine.orm.xml:
 						mapping:
 							App\Model\Entity2: %appDir%
-					nettrine.orm.yaml:
-						mapping:
-							App\Model\Entity3: %appDir%
 				'));
 		})
 		->build();
@@ -57,15 +51,7 @@ Toolkit::test(function (): void {
 		Tests::FIXTURES_PATH,
 	], array_values($xmlDriver->getLocator()->getPaths()));
 
-	/** @var SimplifiedYamlDriver $yamlDriver */
-	$yamlDriver = $container->getService(current(array_keys($container->findByTag(OrmYamlExtension::DRIVER_TAG))));
-
-	Assert::equal([
-		Tests::APP_PATH,
-		Tests::FIXTURES_PATH,
-	], array_values($yamlDriver->getLocator()->getPaths()));
-
 	/** @var MappingDriverChain $chainDriver */
 	$chainDriver = $container->getService('nettrine.orm.mappingDriver');
-	Assert::count(6, $chainDriver->getDrivers());
+	Assert::count(4, $chainDriver->getDrivers());
 });

tests/Cases/DI/OrmYamlExtension.phpt

@@ -13,7 +13,6 @@ public function beforeCompile(): void
 	{
 		MappingHelper::of($this)->addAnnotation('Tests1', Tests::FIXTURES_PATH);
 		MappingHelper::of($this)->addXml('Tests2', Tests::FIXTURES_PATH);
-		MappingHelper::of($this)->addYaml('Tests3', Tests::FIXTURES_PATH);
 	}
 
 }