Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a functional test covering #8127 #10388

Closed
wants to merge 2 commits into from

Conversation

mpdude
Copy link
Contributor

@mpdude mpdude commented Jan 12, 2023

Before merging, let's wait what conclusions #10411 or #10389 make.

This PR adds a functional test covering the use case reported in #8127.

It uses joined table inheritance across a hierarchy of three entity classes, with the root and middle class even being abstract entity classes. All three classes contain a mapped field.

As long as all three entity classes are declared in the Discriminator Map, the leaf (concrete entity) class can be queried through either the root or leaf class name, and all fields will be initialized correctly.

When the middle class is not included in the DM, its fields will not be queried and the entity is not hydrated correctly.

When the middle class is not an @Entity class, but an abstract @MappedSuperclass, it need not be included in the DM for things to work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant