[PR #479] [CLOSED] [WIP][Mapping] Ported some of the yaml driver to use Symfony config #8258

New Issue

admin · 2026-01-22T15:59:06+01:00

admin commented

2026-01-22 15:59:06 +01:00

📋 Pull Request Information

Original PR: https://github.com/doctrine/orm/pull/479
Author: @kimhemsoe
Created: 10/14/2012
Status: ❌ Closed

Base: master ← Head: sf_config_driver

📝 Commits (4)

36518d7 Changed Yamldriver to use symfony config
d4254cc Fixed travis build.
9864c71 Removed Extension interface and config submodule. Some renames of classes.
0913081 Removed the rest of git submodule

📊 Changes

7 files changed (+648 additions, -26 deletions)

View changed files

📝 .travis.yml (+1 -1)
📝 composer.json (+7 -1)
➕ lib/Doctrine/ORM/Mapping/Driver/Configuration/YamlExtension.php (+58 -0)
➕ lib/Doctrine/ORM/Mapping/Driver/Configuration/YamlMappingConfiguration.php (+478 -0)
📝 lib/Doctrine/ORM/Mapping/Driver/YamlDriver.php (+50 -19)
📝 tests/Doctrine/Tests/ORM/Mapping/YamlMappingDriverTest.php (+52 -5)
➕ tests/Doctrine/Tests/ORM/Mapping/yaml/Doctrine.Tests.ORM.Mapping.Article.dcm.yml (+2 -0)

📄 Description

Looking for some input. How much validation and normailization should i push to the configuration ? Should i use default values so we can remove alot of lines from the driver ?

Is the way im allowing to extend the configuration good enough for Gedmo and others (untested)?

and.. did i catch all the settings ?

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/doctrine/orm/pull/479 **Author:** [@kimhemsoe](https://github.com/kimhemsoe) **Created:** 10/14/2012 **Status:** ❌ Closed **Base:** `master` ← **Head:** `sf_config_driver` --- ### 📝 Commits (4) - [`36518d7`](https://github.com/doctrine/orm/commit/36518d7afd0104904a6dd648b1b07d73891bf779) Changed Yamldriver to use symfony config - [`d4254cc`](https://github.com/doctrine/orm/commit/d4254ccab0877113d11bc7cc8d250c62916cbc90) Fixed travis build. - [`9864c71`](https://github.com/doctrine/orm/commit/9864c719b8f34d5145225678b7529677b126a186) Removed Extension interface and config submodule. Some renames of classes. - [`0913081`](https://github.com/doctrine/orm/commit/0913081cf6022d3fb4815dd693a0cf862165ae00) Removed the rest of git submodule ### 📊 Changes **7 files changed** (+648 additions, -26 deletions) <details> <summary>View changed files</summary> 📝 `.travis.yml` (+1 -1) 📝 `composer.json` (+7 -1) ➕ `lib/Doctrine/ORM/Mapping/Driver/Configuration/YamlExtension.php` (+58 -0) ➕ `lib/Doctrine/ORM/Mapping/Driver/Configuration/YamlMappingConfiguration.php` (+478 -0) 📝 `lib/Doctrine/ORM/Mapping/Driver/YamlDriver.php` (+50 -19) 📝 `tests/Doctrine/Tests/ORM/Mapping/YamlMappingDriverTest.php` (+52 -5) ➕ `tests/Doctrine/Tests/ORM/Mapping/yaml/Doctrine.Tests.ORM.Mapping.Article.dcm.yml` (+2 -0) </details> ### 📄 Description Looking for some input. How much validation and normailization should i push to the configuration ? Should i use default values so we can remove alot of lines from the driver ? Is the way im allowing to extend the configuration good enough for Gedmo and others (untested)? and.. did i catch all the settings ? --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

admin added the pull-request label 2026-01-22 15:59:06 +01:00

Sign in to join this conversation.

Branches Tags

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: doctrine/archived-orm#8258