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

Reference ClassMetadaInfo less #9691

Merged
merged 3 commits into from
Apr 28, 2022
Merged

Conversation

greg0ire
Copy link
Member

In #6886, I proposed an upgrade path from ClassMetadataInfo to ClassMetadata. The PR has gathered a lot of discussion, and cobwebs. This PR aims to take care of a small part of it: documentation. It's a great place to start since it allows to understand why ClassMetadataInfo existed in the first place.

See #314 and #249

Although properties and methods are currently located in
ClassMetadataInfo, it is better to refer to ClassMetadata as the former
is deprecated in favor of the latter.
This documentation must be very old because this is no longer valid as
of e9e36dc . The interface and abstract
file driver have since then been moved to doctrine/common, and the to
doctrine/persistence.
ClassMetadataInfo used to be useful during entity generation, because it
allowed the entity not to exist. We no longer do entity generation, and
even if we did, the reflection methods have been moved to
ClassMetadataInfo as of 76e4f5a .
@derrabus derrabus added this to the 2.12.2 milestone Apr 28, 2022
@greg0ire greg0ire merged commit 38d1124 into doctrine:2.12.x Apr 28, 2022
@greg0ire greg0ire deleted the reference-cmi-less branch April 28, 2022 05:58
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.

2 participants