[PR #11775] Changed confusing negative wording on 'Mapped Superclasses' doc #13259

New Issue

admin · 2026-01-22T16:16:40+01:00

admin commented

2026-01-22 16:16:40 +01:00

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

State: closed
Merged: Yes

As I was reading more about Inheritance Mapping, I've come across this turn of phrase that's a bit confusing, especially if english is not your first language. Or maybe it's just me, in which case I'd understand if you refused this PR.
Otherwise, I've simply used a slightly clearer negation that makes it explicit (perhaps only in my opinion) that Mapped Superclasses don't need an ID column.

**Original Pull Request:** https://github.com/doctrine/orm/pull/11775 **State:** closed **Merged:** Yes --- As I was reading more about [Inheritance Mapping](https://www.doctrine-project.org/projects/doctrine-orm/en/3.3/reference/inheritance-mapping.html), I've come across this turn of phrase that's a bit confusing, especially if english is not your first language. Or maybe it's just me, in which case I'd understand if you refused this PR. Otherwise, I've simply used a slightly clearer negation that makes it explicit (perhaps only in my opinion) that Mapped Superclasses don't need an ID column.

admin added the pull-request label 2026-01-22 16:16:40 +01:00

Sign in to join this conversation.

Branches Tags

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: doctrine/archived-orm#13259