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

Filter for and set null values #2330

Closed
recunius opened this issue Apr 3, 2023 · 6 comments
Closed

Filter for and set null values #2330

recunius opened this issue Apr 3, 2023 · 6 comments
Labels
Milestone

Comments

@recunius
Copy link

recunius commented Apr 3, 2023

Hi,

I'm wondering if I am missing an obvious workaround here.

What is the correct way to filter for null values? The value parameter for filter methods are not marked with @Nullable and the package is marked as @NonNullApi in package-info.java

How can Morphia be used with Kotlin when the the variable is defined as nullable with, for instance Any??

A similar problem exists at least with the Update operations. They also do not allow for null values. So how can you set a value to null in an update?

Thanks,
Bob

** Please complete the following information: **
Server Version: 4.2.22
Driver Version: 4.2.3
Morphia Version: 2.2.10

@evanchooly evanchooly added bug and removed question labels Apr 9, 2023
@evanchooly evanchooly added this to the 2.3.1 milestone Apr 9, 2023
@evanchooly
Copy link
Member

I just got back from a conference but i'm hoping to collect the last 2.3.1 issues and do a release this next week.

@Hovhannisyan-K
Copy link

will this be ready by Friday? @evanchooly

evanchooly added a commit that referenced this issue Apr 18, 2023
@evanchooly
Copy link
Member

2.3.1 went out this week, btw.

@recunius
Copy link
Author

recunius commented Apr 21, 2023 via email

@evanchooly
Copy link
Member

No problem. I did a quick audit and i think I've corrected all the places that should allow for null values but if I missed some please do let me know. Annotating at the package level is a very broad brush but it's the easiest route to get things done.

@Hovhannisyan-K
Copy link

thank you so much we will check if we still missing nullable params!

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

No branches or pull requests

3 participants