Skip to content

Deprecate ClassMetadata::$reflFields#12083

Merged
greg0ire merged 1 commit intodoctrine:3.5.xfrom
greg0ire:depr-reflFields
Jul 27, 2025
Merged

Deprecate ClassMetadata::$reflFields#12083
greg0ire merged 1 commit intodoctrine:3.5.xfrom
greg0ire:depr-reflFields

Conversation

@greg0ire
Copy link
Copy Markdown
Member

It is replaced with property accessors since
#11659

Fixes #12082

*
* @throws BadMethodCallException If the class has a composite identifier.
*/
public function getSingleIdReflectionProperty(): ReflectionProperty|null
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let me know if you think an equivalent method should be introduced.

Copy link
Copy Markdown

@ro0NL ro0NL left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Im curious if this could be considered a patch release, to send out better public API deprecations, for what is already triggering confusing internal deprecations.

@greg0ire
Copy link
Copy Markdown
Member Author

@ro0NL I think you're right, let me retarget.

It is replaced with property accessors since
doctrine#11659
@greg0ire greg0ire changed the base branch from 3.6.x to 3.5.x July 24, 2025 07:55
Use the new `Doctrine\ORM\Mapping\PropertyAccessors\PropertyAccessor` API and access
through `Doctrine\ORM\Mapping\ClassMetadata::$propertyAccessors` instead.

Companion accessor methods are deprecated as well.
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe even target 3.4 then ;)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's too late for that, the branch does not exist anymore (and even if it did, it's no longer maintained).

@greg0ire greg0ire added this to the 3.5.1 milestone Jul 27, 2025
@greg0ire greg0ire merged commit 0f32569 into doctrine:3.5.x Jul 27, 2025
85 checks passed
@greg0ire greg0ire deleted the depr-reflFields branch July 27, 2025 21:19
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.

Access to ClassMetadata::$reflFields is deprecated

4 participants