[PR #7263] Add quoted column keys while hydrating column information. #10399

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

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

State: closed
Merged: No


We can't hydrate column information while using a quote strategy because all or the most internal doctrine mappings use the quoted column alias name.

While hydrating a result row data we know we've only got the unquoted column alias and there is no way to quote an existing column alias.
So we need to split the column alias back to the real column name and the alias counter.
This is easy because we know it's get generated in the format '<column_name>_' by the basic entity persister.

With this information we can use the current quote strategy to generate a quoted column alias.

This PR should solve at least the following issues:

**Original Pull Request:** https://github.com/doctrine/orm/pull/7263 **State:** closed **Merged:** No --- We can't hydrate column information while using a quote strategy because all or the most internal doctrine mappings use the quoted column alias name. While hydrating a result row data we know we've only got the unquoted column alias and there is no way to quote an existing column alias. So we need to split the column alias back to the real column name and the alias counter. This is easy because we know it's get generated in the format '<column_name>_<counter>' by the basic entity persister. With this information we can use the current quote strategy to generate a quoted column alias. This PR should solve at least the following issues: * https://github.com/doctrine/doctrine2/issues/7262 * https://github.com/doctrine/doctrine2/issues/5654
admin added the pull-request label 2026-01-22 16:07:24 +01:00
admin closed this issue 2026-01-22 16:07:24 +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#10399