[PR #626] [CLOSED] default discriminator map - full namespace #8468

New Issue

admin · 2026-01-22T16:00:04+01:00

admin commented

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

📋 Pull Request Information

Original PR: https://github.com/doctrine/orm/pull/626
Author: @Padam87
Created: 3/22/2013
Status: ❌ Closed

Base: master ← Head: autodmap

📝 Commits (1)

cd3003a default discriminator map - full namespace

📊 Changes

3 files changed (+11 additions, -44 deletions)

View changed files

📝 lib/Doctrine/ORM/Mapping/ClassMetadataFactory.php (+5 -20)
📝 lib/Doctrine/ORM/Mapping/MappingException.php (+0 -18)
📝 tests/Doctrine/Tests/ORM/Mapping/ClassMetadataFactoryTest.php (+6 -6)

📄 Description

Using Symfony2, if you have two entities with the same name in different bundles, automatic discriminator map detection will trow MappingException.

If the namespace is included, there should be no more duplicate key issues.

(In this case, setting the map manually is not an option)

_{🔄 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/626 **Author:** [@Padam87](https://github.com/Padam87) **Created:** 3/22/2013 **Status:** ❌ Closed **Base:** `master` ← **Head:** `autodmap` --- ### 📝 Commits (1) - [`cd3003a`](https://github.com/doctrine/orm/commit/cd3003a7852742386b504106de571f893c7e9b32) default discriminator map - full namespace ### 📊 Changes **3 files changed** (+11 additions, -44 deletions) <details> <summary>View changed files</summary> 📝 `lib/Doctrine/ORM/Mapping/ClassMetadataFactory.php` (+5 -20) 📝 `lib/Doctrine/ORM/Mapping/MappingException.php` (+0 -18) 📝 `tests/Doctrine/Tests/ORM/Mapping/ClassMetadataFactoryTest.php` (+6 -6) </details> ### 📄 Description Using Symfony2, if you have two entities with the same name in different bundles, automatic discriminator map detection will trow MappingException. If the namespace is included, there should be no more duplicate key issues. (In this case, setting the map manually is not an option) --- <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:00:04 +01:00

admin referenced this issue

2026-01-22 16:09:23 +01:00

[PR #8468] CS batch 4/an estimated 30 #11001

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