Hello everyone,
I would like to propose the deprecation of the constants DATE_ISO8601,
DateTimeInterface::ISO8601 and DateTimeInterface::RFC7231, DATE_RFC7231.
Arguments for deprecating DATE_ISO8601:
- outdated format, standard has been changed while ago
- incompatible with format ISO8601_EXPANDED
- misleading behavior for developers [1]
https://stackoverflow.com/questions/38252867/what-is-the-right-iso8601-format
[2]
https://stackoverflow.com/questions/8003446/php-validate-iso-8601-date-string - doesn't support microseconds [3] https://bugs.php.net/bug.php?id=51950
- many misunderstandings in the userland [4]
https://api.drupal.org/api/drupal/includes%21common.inc/function/date_iso8601/7.x
[5] https://carbon.nesbot.com/docs/#api-commonformats
In my opinion the question is not whether this constant should be
deprecated, but when. I know that the problem was discussed in the past,
but I hope enough time has passed already to touch this topic again
although of the BC nature. [6] https://externals.io/message/113657#114885
[7]
https://www.reddit.com/r/PHP/comments/hnd438/why_isnt_date_iso8601_deprecated/
Arguments for deprecating DATE_RFC7231:
- error prone nature, the format never really supported timezone. I'm not
sure how it appeared in the code, but tests clearly lack cases for this
format. I'm not sure if it was missed on purpose, but with tests it would
be obvious that the format shouldn't ever appear in the core. [8]
https://www.php.net/manual/en/class.datetimeinterface.php [9]
https://github.com/php/doc-en/pull/2296
It's my first proposition in the PHP Core and I'm not sure if I should
suggest it through RFC or normal discussion like this. If I should propose
it in RFC then I can prepare it along with PR.
Kind regards,
Jorg
Hello everyone,
I would like to propose the deprecation of the constants DATE_ISO8601,
DateTimeInterface::ISO8601 and DateTimeInterface::RFC7231, DATE_RFC7231.Arguments for deprecating DATE_ISO8601:
- outdated format, standard has been changed while ago
- incompatible with format ISO8601_EXPANDED
- misleading behavior for developers [1]
https://stackoverflow.com/questions/38252867/what-is-the-right-iso8601-format
[2]
https://stackoverflow.com/questions/8003446/php-validate-iso-8601-date-string- doesn't support microseconds [3]
https://bugs.php.net/bug.php?id=51950- many misunderstandings in the userland [4]
https://api.drupal.org/api/drupal/includes%21common.inc/function/date_iso8601/7.x
[5] https://carbon.nesbot.com/docs/#api-commonformatsIn my opinion the question is not whether this constant should be
deprecated, but when. I know that the problem was discussed in the past,
but I hope enough time has passed already to touch this topic again
although of the BC nature. [6] https://externals.io/message/113657#114885
[7]https://www.reddit.com/r/PHP/comments/hnd438/why_isnt_date_iso8601_deprecated/
Arguments for deprecating DATE_RFC7231:
- error prone nature, the format never really supported timezone. I'm not
sure how it appeared in the code, but tests clearly lack cases for this
format. I'm not sure if it was missed on purpose, but with tests it would
be obvious that the format shouldn't ever appear in the core. [8]
https://www.php.net/manual/en/class.datetimeinterface.php [9]
https://github.com/php/doc-en/pull/2296It's my first proposition in the PHP Core and I'm not sure if I should
suggest it through RFC or normal discussion like this. If I should propose
it in RFC then I can prepare it along with PR.Kind regards,
Jorg
How common is it in the wild? I am pretty sure I've used it myself without really thinking many times, which I suspect is common. Knowing how much code would be impacted would determine how we'd go about deprecating it and on what timeline.
--Larry Garfield
How common is it in the wild? I am pretty sure I've used it myself
without really thinking many times, which I suspect is common. Knowing how
much code would be impacted would determine how we'd go about deprecating
it and on what timeline.
I'm not sure how would be possible to find out. I suppose most of the
frameworks (and thus popular PHP libraries) avoid this constant as showed
in Drupal and Carbon. The scanning of popular libraries will not show the
scale of the problem (however I will do it tomorrow) Helpful fact is that
those constants are just constants so it's easily replaceable with
strings 'Y-m-d\TH:i:sO'
and 'D, d M Y H:i:s \G\M\T'.
outdated format, standard has been changed while ago
What do you mean? Do you mean that the constant was legal at some point?
Was there ever a version of ISO8601 allowing the mixing of "extended
format" and "basic format" ? The earliest version I've read, iso8604:2004,
does not allow it (more on that here:
https://externals.io/email/114883/source - can't find the proper link right
now, am on phone). Haven't read the older 1988/1999/2000 versions, and it's
possible one of those allowed it, but I doubt it. The only thing I can say
for sure is that it's been illegal since the 2004 version
How common is it in the wild? I am pretty sure I've used it myself
without really thinking many times, which I suspect is common. Knowing how
much code would be impacted would determine how we'd go about deprecating
it and on what timeline.I'm not sure how would be possible to find out. I suppose most of the
frameworks (and thus popular PHP libraries) avoid this constant as showed
in Drupal and Carbon. The scanning of popular libraries will not show the
scale of the problem (however I will do it tomorrow) Helpful fact is that
those constants are just constants so it's easily replaceable with
strings 'Y-m-d\TH:i:sO'
and 'D, d M Y H:i:s \G\M\T'.
What do you mean? Do you mean that the constant was legal at some point?
Sorry, but I don't have this knowledge. It's invalid since 2004, but I
suppose this format was implemented before then. Undoubtedly the format is
invalid now.
The usage in 1000 most popular libraries:
DATE_ISO8601 (4 packages, 1 in comments)
DateTimeInterface::ISO8601 (18 packages)
DateTimeInterface::RFC7231 (1 package - Jetbrains/phpstorm-stubs)
DATE_RFC7231 (1 package - Jetbrains/phpstorm-stubs)
Most of the packages already inform about deprecation. PHP stubs by
Jetbrans warn about deprecation, although it's not officially.
I would like to propose the deprecation of the constants DATE_ISO8601,
DateTimeInterface::ISO8601 and DateTimeInterface::RFC7231, DATE_RFC7231.
…
In my opinion the question is not whether this constant should be
deprecated, but when. I know that the problem was discussed in the past,
but I hope enough time has passed already to touch this topic again
although of the BC nature. [6] https://externals.io/message/113657#114885
[7]
In my opinion, deprecating this does not do anything besides annoying
users.
https://www.reddit.com/r/PHP/comments/hnd438/why_isnt_date_iso8601_deprecated/
Arguments for deprecating DATE_RFC7231:
- error prone nature, the format never really supported timezone. I'm not
sure how it appeared in the code, but tests clearly lack cases for this
format. I'm not sure if it was missed on purpose, but with tests it would
be obvious that the format shouldn't ever appear in the core. [8]
https://www.php.net/manual/en/class.datetimeinterface.php [9]
https://github.com/php/doc-en/pull/2296
This should instead be fixed of being deprecated. It is a recent
addition after all.
cheers,
Derick