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

Coercion of null Strings to "" incorrect for 3.0 #359

Closed
cowtowncoder opened this issue Sep 8, 2019 · 5 comments
Closed

Coercion of null Strings to "" incorrect for 3.0 #359

cowtowncoder opened this issue Sep 8, 2019 · 5 comments
Labels
3.x Issue planned for 3.x (and not 2.x)

Comments

@cowtowncoder
Copy link
Member

With rewrite of XML module for 3.0, something is wrong with coercion if null values -- so after #354 resolved this for 2.x, there is now one unit test that only fails for 3.0.
This is due to combination of null->Empty coercion in XmlMapper, line ~85:

            _configOverrides.findOrCreateOverride(String.class)
                .setNullHandling(JsonSetter.Value.forValueNulls(Nulls.AS_EMPTY));

but removing that line will fail a bunch of other tests that do expect empty String from input to be taken as empty String, not null.

@cowtowncoder
Copy link
Member Author

Was fixed.

@cowtowncoder
Copy link
Member Author

No, not fixed; test was simply moved under failing. Re-opening.

@cowtowncoder cowtowncoder reopened this Nov 13, 2020
@mensinda
Copy link
Contributor

mensinda commented May 3, 2022

What is the status of this issue?

@cowtowncoder
Copy link
Member Author

@mensinda If there are no updates on the issue, typically nothing has happened.
Note that the issue for upcoming (future) Jackson 3.x, not currently released 2.x versions.

@mensinda
Copy link
Contributor

mensinda commented May 5, 2022

@cowtowncoder Oh, sorry for the noise. I did not check this manually and did not notice the major version bump...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3.x Issue planned for 3.x (and not 2.x)
Projects
None yet
Development

No branches or pull requests

2 participants