[PR #6392] Correct DQL INSTANCE OF to filter all possible child classes #9963

New Issue

admin · 2026-01-22T16:06:00+01:00

admin commented

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

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

State: closed
Merged: Yes

This is the rebased version of #1441, which fixes #4646.
Here is the original text:

There was a bug in the "INSTANCE OF" operator as described in
https://groups.google.com/forum/#!topic/doctrine-user/B8raq8CNMgg

"INSTANCE OF" was not taking into account subclasses.
It was merely translating the class to its discriminator.
This is not correct since the class can have subtypes and those
are, indeed, still instance of the superclass.
Also, classes may not have a discriminator (e.g. abstract classes).

This commit also provides useful tests to avoid regression.

**Original Pull Request:** https://github.com/doctrine/orm/pull/6392 **State:** closed **Merged:** Yes --- This is the rebased version of #1441, which fixes #4646. Here is the original text: > There was a bug in the "INSTANCE OF" operator as described in > https://groups.google.com/forum/#!topic/doctrine-user/B8raq8CNMgg > > "INSTANCE OF" was not taking into account subclasses. > It was merely translating the class to its discriminator. > This is not correct since the class can have subtypes and those > are, indeed, still instance of the superclass. > Also, classes may not have a discriminator (e.g. abstract classes). > > This commit also provides useful tests to avoid regression.

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

admin closed this issue

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

admin referenced this issue

2026-01-22 16:12:36 +01:00

[PR #9963] Flag QueryType as internal #12004

Sign in to join this conversation.