[PR #957] [MERGED] makes doctrine less dependent upon the symfony yaml component #8957

New Issue

admin · 2026-01-22T16:02:29+01:00

admin commented

2026-01-22 16:02:29 +01:00

📋 Pull Request Information

Original PR: https://github.com/doctrine/orm/pull/957
Author: @thealjey
Created: 2/20/2014
Status: ✅ Merged
Merged: 3/23/2014
Merged by: @beberlei

Base: master ← Head: master

📝 Commits (1)

c062013 makes doctrine less dependent upon the symfony yamp component

📊 Changes

1 file changed (+17 additions, -1 deletions)

View changed files

📝 lib/Doctrine/ORM/Tools/Export/Driver/YamlExporter.php (+17 -1)

📄 Description

This is the last place in the framework where the class is just too tightly coupled with the symfony yaml component, that the yaml parser alone cannot be overridden without overriding an entire class.
I simply brought it in accordance with "Doctrine\ORM\Mapping\Driver\YamlDriver" where a separate method called loadMappingFile exists to parse the mapping file.

_{🔄 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/957 **Author:** [@thealjey](https://github.com/thealjey) **Created:** 2/20/2014 **Status:** ✅ Merged **Merged:** 3/23/2014 **Merged by:** [@beberlei](https://github.com/beberlei) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (1) - [`c062013`](https://github.com/doctrine/orm/commit/c06201303d8775eb3e8d89326c16640e04a8ae84) makes doctrine less dependent upon the symfony yamp component ### 📊 Changes **1 file changed** (+17 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `lib/Doctrine/ORM/Tools/Export/Driver/YamlExporter.php` (+17 -1) </details> ### 📄 Description This is the last place in the framework where the class is just too tightly coupled with the symfony yaml component, that the yaml parser alone cannot be overridden without overriding an entire class. I simply brought it in accordance with "Doctrine\ORM\Mapping\Driver\YamlDriver" where a separate method called loadMappingFile exists to parse the mapping file. --- <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 16:02:29 +01:00

admin referenced this issue

2026-01-22 16:10:16 +01:00

[PR #8957] Only wire ConnectionHelper if it's available #11285

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#8957