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

Select correlation wrongly tries to load data based on null correlation basis #724

Closed
beikov opened this issue Jan 23, 2019 · 0 comments · Fixed by #721

Comments

@beikov
Copy link
Member

commented Jan 23, 2019

Description

When having a correlation in e.g. a subview and that subview object is null because the association is null, SELECT fetching fails with an exception, saying that a null id is invalid for EntityManager.getReference(). Obviously, we need to skip correlating when the correlation basis is null.

Expected behavior

Actual behavior

Steps to reproduce

Environment

Version: 1.3.0
JPA-Provider: -
DBMS: -
Application Server: -

@beikov beikov added this to the 1.4.0 milestone Jan 23, 2019

@beikov beikov self-assigned this Jan 23, 2019

beikov added a commit to beikov/blaze-persistence that referenced this issue Jan 23, 2019
beikov added a commit to beikov/blaze-persistence that referenced this issue Jan 24, 2019
beikov added a commit to beikov/blaze-persistence that referenced this issue Jan 28, 2019
beikov added a commit to beikov/blaze-persistence that referenced this issue Jan 29, 2019
beikov added a commit to beikov/blaze-persistence that referenced this issue Feb 10, 2019
beikov added a commit to beikov/blaze-persistence that referenced this issue Feb 19, 2019
beikov added a commit to beikov/blaze-persistence that referenced this issue Feb 19, 2019

@beikov beikov closed this in #721 Feb 25, 2019

beikov added a commit to beikov/blaze-persistence that referenced this issue Feb 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.