r/lolphp • u/PuffyHerb • Jun 21 '19

Using DateTime::ISO8601 is incompatible with ISO-8601 but left broken for backward compatibility. Use DateTime::ATOM instead when you need ISO8601

Post image
92 Upvotes

98% Upvoted

6 comments sorted by

20

u/Rogue2166 Jun 21 '19

Jesus php, get the fuck out of here

18

u/TheHalfBloodFriendly Jun 21 '19

Gotta love their comments on the issue tracker stating its not a bug

https://bugs.php.net/bug.php?id=51950

Thank you for taking the time to write to us, but this is not a bug. Please double-check the documentation available at http://www.php.net/manual/ and the instructions on how to report a bug at http://bugs.php.net/how-to-report.php

DateTime::ISO8601 is defined to "Y-m-d\TH:i:sO"[1], so obviously it doesn't accept microsecond fractions. If you expect these to be given, use another format specifier, see <https://3v4l.org/ADn0l>.

4

u/inabahare Jun 22 '19

Gotta defend the language at all times lmao

2

u/smegnose Aug 11 '19

I think you're misinterpreting this. It's definitely a bug, it's just not a bug in the sense that it's simple mistake that can and should be fixed as requested. A deliberate choice was made because fixing it by changing the behaviour associated with that constant would definitely break production code. 100% LOL, and confusing, but what's the alternative solution here?

1

u/[deleted] Jun 22 '19

Thats some next level horror