Skip to content

Commit

Permalink
fix(core): serialization inclusion is a single property not a list
Browse files Browse the repository at this point in the history
So setting it two times will override the previous one.
  • Loading branch information
loicmathieu committed Jul 21, 2023
1 parent a0bd3b7 commit 48edf70
Showing 1 changed file with 0 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -99,7 +99,6 @@ public static <T> String log(T Object) {
new IonObjectMapper(new IonFactory())
)
.registerModule(new IonModule())
.setSerializationInclusion(JsonInclude.Include.USE_DEFAULTS)
.setSerializationInclusion(JsonInclude.Include.ALWAYS);

public static ObjectMapper ofIon() {
Expand All @@ -115,7 +114,6 @@ private static ObjectMapper configure(ObjectMapper mapper) {

return mapper
.configure(SerializationFeature.WRITE_DATES_AS_TIMESTAMPS, false)
.setSerializationInclusion(JsonInclude.Include.NON_NULL)
.setSerializationInclusion(JsonInclude.Include.NON_EMPTY)
.registerModule(new JavaTimeModule())
.registerModule(new Jdk8Module())
Expand Down

0 comments on commit 48edf70

Please sign in to comment.