修复 #21531
[spring-projects/spring-boot]通过将时间或日期时间属性设置为 iso-offset,支持使用 MVC 和 WebFlux 进行 iso 偏移时间和日期时间转换
回答
@gaurav-91 感谢您签署贡献者许可协议!
@philwebb:你有机会完成公关吗?如果代码需要一起审查,请告诉我。谢谢
@gaurav-91 恐怕在发布 2.3.1 之前我们有许多错误需要我注意。我可能还需要一周左右的时间才能做到这一点。
@gaurav-91 感谢您的公关。您能否添加一些测试以确保该iso-offset
模式按预期工作。您可以查看“iso”模式的现有测试。
当然会在今天之前添加它。
@mbhave 我已经添加了测试用例,但它们似乎失败了。看起来正在使用 OffsetDateTime 转换器。如果我使用 LocalDate 对象,则在解析时会失败,因为我们正在尝试解析 ISO_Offset 日期。你能看一下我在这里做错了什么吗?谢谢
@mbhave 请您看一下我添加的测试用例,并帮助我理解我做错了什么。谢谢
感谢您的公关,@gaurav-91。
我认为为日期提供 ISO 偏移格式没有意义。日期格式用于配置Spring框架的日期格式DateTimeFormatterRegistrar
,其方法的javadocsetDateFormatter
声明它仅用于格式化LocalDate
. ALocalDate
不包含时区偏移量,因此无法使用基于偏移量的格式对其进行格式化,因为没有要格式化的偏移量信息。您介意稍微修改一下您的提案,以便iso-offset
仅支持spring.mvc.format.date-time
和spring.mvc.format.time
属性吗?
谢谢@wilkinsona。它与我的理解同步,我将开始研究你提到的更改。
怎么样,@gaurav-91?
@wilkinsona我已按照您的建议进行了更改。 spring.mvc.format.time 的测试用例似乎失败了。我已经提交了到目前为止完成的代码。如果可能的话,请您看一下并提供建议。谢谢
非常感谢 @gaurav-91 对 Spring Boot 做出的第一个贡献。提议的更改现已合并到 master 中。如果您感兴趣,我还添加了一个小的抛光提交,主要是为了允许iso-offset
用作值以及isooffset
.
@wilkinsona 我可以提出的任何其他问题。
@gaurav-91 感谢您的提议。我们有一个可能感兴趣的理想贡献标签。