Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use EnumReflectionProperty class from persistence package #11330

Merged
merged 2 commits into from
Mar 3, 2024

Conversation

greg0ire
Copy link
Member

@greg0ire greg0ire commented Mar 1, 2024

It is meant to remove duplication between the ORM and the ODM.

@greg0ire

This comment was marked as resolved.

@greg0ire

This comment was marked as resolved.

@stof

This comment was marked as resolved.

@greg0ire

This comment was marked as resolved.

@greg0ire

This comment was marked as resolved.

It is meant to remove duplication between the ORM and the ODM.
@greg0ire greg0ire marked this pull request as ready for review March 1, 2024 19:59
@greg0ire greg0ire changed the title Try using EnumReflectionProperty @greg0ire Use class from persistence package Mar 1, 2024
@greg0ire greg0ire changed the title @greg0ire Use class from persistence package Use class from persistence package Mar 1, 2024
UPGRADE.md Outdated Show resolved Hide resolved
Co-authored-by: Steve Todd <[email protected]>
@derrabus derrabus added this to the 3.1.0 milestone Mar 3, 2024
@derrabus derrabus merged commit ba0ea89 into doctrine:3.1.x Mar 3, 2024
64 checks passed
derrabus added a commit that referenced this pull request Mar 3, 2024
* 3.1.x:
  Use class from persistence package  (#11330)
  Refator array_map into simple loop for performance. (#11332)
@derrabus derrabus changed the title Use class from persistence package Use EnumReflectionProperty class from persistence package Mar 3, 2024
@greg0ire greg0ire deleted the persistence-3-3 branch March 3, 2024 21:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants