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.

Sign up for GitHub

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

Jump to bottom

Fix LINQ projection of nullable enum with list #2703

Merged
merged 2 commits into from
Mar 20, 2021
Merged

Fix LINQ projection of nullable enum with list #2703

merged 2 commits into from
Mar 20, 2021

Conversation

gliljas
Copy link
Member

@gliljas gliljas commented Mar 15, 2021

See #2702

@gliljas
Copy link
Member Author

gliljas commented Mar 16, 2021

I may need some assistance in getting to the root of this problem.

@bahusoid
Copy link
Member

Looks related to #2425
So my guess it's caused by 7fd1a9a

@maca88
Copy link
Contributor

maca88 commented Mar 16, 2021

So my guess it's caused by 7fd1a9a

That is correct, I made a fix for it.

@fredericDelaporte fredericDelaporte added this to the 5.3.7 milestone Mar 16, 2021
@bahusoid bahusoid changed the title Failing test for GH-2702 Fix for GH-2702 Mar 17, 2021
@fredericDelaporte fredericDelaporte changed the title Fix for GH-2702 Fix LINQ projection of nullable enum with list Mar 20, 2021
@fredericDelaporte fredericDelaporte merged commit 4d8ce4e into nhibernate:5.3.x Mar 20, 2021
@fredericDelaporte fredericDelaporte mentioned this pull request Mar 20, 2021
Closed
bahusoid pushed a commit to bahusoid/nhibernate-core that referenced this pull request Aug 10, 2021
@gliljas @maca88 @bahusoid
Fix LINQ projection of nullable enum with list (nhibernate#2703)
208280c 
Co-authored-by: maca88 <bostjan.markezic@siol.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bahusoid bahusoid bahusoid approved these changes

Assignees
No one assigned
Labels
r: Fixed t: Fix
Projects
None yet
Milestone
5.3.7
Development

Successfully merging this pull request may close these issues.
4 participants
@gliljas @bahusoid @maca88 @fredericDelaporte