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 18, 2023
1 parent 0679b84 commit 2611231
Showing 1 changed file with 0 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,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 @@ -117,7 +116,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 2611231

Please sign in to comment.