[spring-projects/spring-boot]通过将时间或日期时间属性设置为 iso-offset,支持使用 MVC 和 WebFlux 进行 iso 偏移时间和日期时间转换

2024-04-12 969 views
0

修复 #21531

回答

6

@philwebb:你有机会完成公关吗?如果代码需要一起审查,请告诉我。谢谢

8

@gaurav-91 恐怕在发布 2.3.1 之前我们有许多错误需要我注意。我可能还需要一周左右的时间才能做到这一点。

7

@gaurav-91 感谢您的公关。您能否添加一些测试以确保该iso-offset模式按预期工作。您可以查看“iso”模式的现有测试。

0

当然会在今天之前添加它。

4

@mbhave 我已经添加了测试用例,但它们似乎失败了。看起来正在使用 OffsetDateTime 转换器。如果我使用 LocalDate 对象,则在解析时会失败,因为我们正在尝试解析 ISO_Offset 日期。你能看一下我在这里做错了什么吗?谢谢

8

@mbhave 请您看一下我添加的测试用例,并帮助我理解我做错了什么。谢谢

5

感谢您的公关,@gaurav-91。

我认为为日期提供 ISO 偏移格式没有意义。日期格式用于配置Spring框架的日期格式DateTimeFormatterRegistrar,其方法的javadocsetDateFormatter声明它仅用于格式化LocalDate. ALocalDate不包含时区偏移量,因此无法使用基于偏移量的格式对其进行格式化,因为没有要格式化的偏移量信息。您介意稍微修改一下您的提案,以便iso-offset仅支持spring.mvc.format.date-timespring.mvc.format.time属性吗?

3

谢谢@wilkinsona。它与我的理解同步,我将开始研究你提到的更改。

8

怎么样,@gaurav-91?

9

@wilkinsona我已按照您的建议进行了更改。 spring.mvc.format.time 的测试用例似乎失败了。我已经提交了到目前为止完成的代码。如果可能的话,请您看一下并提供建议。谢谢

0

非常感谢 @gaurav-91 对 Spring Boot 做出的第一个贡献。提议的更改现已合并到 master 中。如果您感兴趣,我还添加了一个小的抛光提交,主要是为了允许iso-offset用作值以及isooffset.

6

@wilkinsona 我可以提出的任何其他问题。