Disallow override of inherited property #7077

Closed
opened 2026-01-22 15:44:12 +01:00 by admin · 2 comments
Owner

Originally created by @michnovka on GitHub (Dec 11, 2022).

I came across https://github.com/doctrine/orm/pull/8348 and I found in current code we still have

b25561ad96/lib/Doctrine/ORM/Mapping/ClassMetadataInfo.php (L2632-L2635)

I think that this exception should be thrown here, don't you think? @beberlei

Originally created by @michnovka on GitHub (Dec 11, 2022). I came across https://github.com/doctrine/orm/pull/8348 and I found in current code we still have https://github.com/doctrine/orm/blob/b25561ad9623e1436f23241b180b6958d1a7e09c/lib/Doctrine/ORM/Mapping/ClassMetadataInfo.php#L2632-L2635 I think that this exception should be thrown here, don't you think? @beberlei
admin closed this issue 2026-01-22 15:44:13 +01:00
Author
Owner

@mpdude commented on GitHub (Jan 19, 2023):

I think pointing out configurations that are not officially supported and only work by chance is a good thing to do. Hopefully it works towards bringing down the number of bug reports for dubious use cases.

What about creating a PR for this? Are there tests already?

@mpdude commented on GitHub (Jan 19, 2023): I think pointing out configurations that are not officially supported and only work by chance is a good thing to do. Hopefully it works towards bringing down the number of bug reports for dubious use cases. What about creating a PR for this? Are there tests already?
Author
Owner

@mpdude commented on GitHub (Feb 13, 2023):

#10470 solved this, please close

@mpdude commented on GitHub (Feb 13, 2023): #10470 solved this, please close
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: doctrine/archived-orm#7077