DDC-2082: [GH-481] check for false as a return value from get_parent_class(), not null #2622

Closed
opened 2026-01-22 13:58:35 +01:00 by admin · 4 comments
Owner

Originally created by @doctrinebot on GitHub (Oct 16, 2012).

Originally assigned to: @beberlei on GitHub.

Jira issue originally created by user @beberlei:

This issue is created automatically through a Github pull request on behalf of beejeebus:

Url: https://github.com/doctrine/doctrine2/pull/481

Message:

MappingException::identifierRequired() and MappingException::classIsNotAValidEntityOrMappedSuperClass() both check for null, when they should check for false from get_parent_class().

Originally created by @doctrinebot on GitHub (Oct 16, 2012). Originally assigned to: @beberlei on GitHub. Jira issue originally created by user @beberlei: This issue is created automatically through a Github pull request on behalf of beejeebus: Url: https://github.com/doctrine/doctrine2/pull/481 Message: MappingException::identifierRequired() and MappingException::classIsNotAValidEntityOrMappedSuperClass() both check for null, when they should check for false from get_parent_class().
admin added the Bug label 2026-01-22 13:58:35 +01:00
admin closed this issue 2026-01-22 13:58:36 +01:00
Author
Owner

@doctrinebot commented on GitHub (Oct 22, 2012):

Comment created by @beberlei:

A related Github Pull-Request [GH-481] was closed
https://github.com/doctrine/doctrine2/pull/481

@doctrinebot commented on GitHub (Oct 22, 2012): Comment created by @beberlei: A related Github Pull-Request [GH-481] was closed https://github.com/doctrine/doctrine2/pull/481
Author
Owner

@doctrinebot commented on GitHub (Oct 24, 2012):

Comment created by @FabioBatSilva:

Merged : 9d0b254407

@doctrinebot commented on GitHub (Oct 24, 2012): Comment created by @FabioBatSilva: Merged : https://github.com/doctrine/doctrine2/commit/9d0b254407e6f8827e0f00aa4ed0fcefc6111aad
Author
Owner

@doctrinebot commented on GitHub (Oct 24, 2012):

Issue was closed with resolution "Fixed"

@doctrinebot commented on GitHub (Oct 24, 2012): Issue was closed with resolution "Fixed"
Author
Owner

@doctrinebot commented on GitHub (Dec 30, 2013):

Comment created by @doctrinebot:

A related Github Pull-Request [GH-481] was closed:
https://github.com/doctrine/dbal/pull/481

@doctrinebot commented on GitHub (Dec 30, 2013): Comment created by @doctrinebot: A related Github Pull-Request [GH-481] was closed: https://github.com/doctrine/dbal/pull/481
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: doctrine/archived-orm#2622