[PR #10442] Add embedded class mapping array shape #12362

Closed
opened 2026-01-22 16:13:49 +01:00 by admin · 0 comments
Owner

Original Pull Request: https://github.com/doctrine/orm/pull/10442

State: closed
Merged: Yes


This should be replaced with a DTO in the next major. To be able to have something usable, I had to move the validation of the DTO (checking that it has a "class" attribute) to mapEmbedded, which happens earlier than doLoadMetadata(). It is unclear to me why it was not put here in the first place in https://github.com/doctrine/orm/pull/1133

Cc @mpdude

**Original Pull Request:** https://github.com/doctrine/orm/pull/10442 **State:** closed **Merged:** Yes --- This should be replaced with a DTO in the next major. To be able to have something usable, I had to move the validation of the DTO (checking that it has a "class" attribute) to mapEmbedded, which happens earlier than doLoadMetadata(). It is unclear to me why it was not put here in the first place in https://github.com/doctrine/orm/pull/1133 Cc @mpdude
admin added the pull-request label 2026-01-22 16:13:49 +01:00
admin closed this issue 2026-01-22 16:13:49 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: doctrine/archived-orm#12362