-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Improve JavaDoc for config ACCEPT_EMPTY_STRING_AS_NULL_OBJECT
wrt special cases
#4010
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
Conversation
} | ||
|
||
final ObjectMapper DISABLED_MAPPER = newJsonMapper() | ||
.configure(DeserializationFeature.ACCEPT_EMPTY_STRING_AS_NULL_OBJECT, false); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does not need to be set on mapper; could set on ObjectReader
instead?
Hmmh. I think that this behavior should be changed for 2.16, see my notes on #4009. What could be done here would be:
If test was added, it would then be modified as part of #4009 for 2.16. |
Oh yes I just read the comment 👍🏻.
WIll do.
Will remove reference to |
For safe verion control, I will not test my git skills rebasing from |
Sorry about this. I have not figure out how to use rebase properly with Github PRs in cases like this. :-( |
Np! I don't think there is legit clean way to do such thing, speaking from logical perspective 😆 |
Yeah I guess at the point PR is created it is bound to specified branch as... base? |
relates to #4009