Regression fix for #4741 `Include.NON_DEFAULT` setting used on POJO, empty values are not included in json if default is null

FasterXML / jackson-databind

General data-binding package for Jackson (2.x): works on streaming API (core) implementation(s)

Apache License 2.0

3.53k stars 1.39k forks source link

Closed JooHyukKim closed 1 month ago

JooHyukKim commented 1 month ago

demonstrates "possible" solution that resolves #4741

cowtowncoder commented 1 month ago

@JooHyukKim I figured out a simpler check which I think works. But wanted to ask WDYT before merging. All tests pass which is good.

JooHyukKim commented 1 month ago

I like the simpler check 🙏🏼 LGTM! @cowtowncoder

cowtowncoder commented 1 month ago

@JooHyukKim Thank you for taking a lead on this! I'll go ahead and merge soon.