[
https://issues.apache.org/jira/browse/LANG-1002?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14596394#comment-14596394
]
Benedikt Ritter commented on LANG-1002:
---------------------------------------
Hi,
without getting into the details here, I think we should:
* provide constants with the correct values
* provide additional constants the differ from the standards constants if they
are useful.
Deprecating incorrect constants to remove them later on is okay to me. Changes
the value of incorrect constants and document this in the RELEASE-NOTES is also
okay to me.
> Several predefined ISO FastDateFormats in DateFormatUtils are incorrect
> -----------------------------------------------------------------------
>
> Key: LANG-1002
> URL: https://issues.apache.org/jira/browse/LANG-1002
> Project: Commons Lang
> Issue Type: Bug
> Components: lang.time.*
> Affects Versions: 3.3.2
> Reporter: Michael Osipov
>
> Formats {{ISO_TIME_FORMAT}}, {{ISO_TIME_TIME_ZONE_FORMAT}} prepend a {{T}}
> but this is not correct. Sole time is never prepended by defintion. {{T}} is
> used only when date *and* time are given.
> The Javadocs of {{ISO_TIME_NO_T_FORMAT}}, {{ISO_TIME_NO_T_FORMAT}} are in
> correct too because they say: "This pattern does not comply with the formal
> ISO8601 specification as the standard requires the 'T' prefix for times."
> You might want to read [Markus Kuhn's
> reference|https://www.cl.cam.ac.uk/~mgk25/iso-time.html#time] on that.
> A solution would be remove the first two and rename the second two by
> dropping the {{NO_T}} in the name.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
