jackson-modules-java8
jackson-modules-java8 copied to clipboard
ZonedDateTime documentation seems to be incorrect
It says: https://github.com/FasterXML/jackson-modules-java8/tree/master/datetime
For string representation default formats for deserialization are:
| Java type | Format |
|---|---|
ZonedDateTime |
DateTimeFormatter.ISO_ZONED_DATE_TIME |
But in the code I see: https://github.com/FasterXML/jackson-modules-java8/blob/1534c557d4ea997b7a80a9bcba4c98cd558e4493/datetime/src/main/java/com/fasterxml/jackson/datatype/jsr310/ser/ZonedDateTimeSerializer.java#L22-L23
Yes, and the comment could be improved too. A little ambiguous though perhaps trying to explain why ISO_OFFSET_DATE_TIME is used and not ISO_ZONED_DATE_TIME.
@cowtowncoder please confirm As per the issue description documentation seems to be incorrect. Is that correct? or implementation needs to be updated to match the documentation?
@mydata Generally documentation should match implementation unless indicated otherwise. So I'd start with that.