Edit report at https://bugs.php.net/bug.php?id=72243&edit=1
ID: 72243
Updated by: [email protected]
Reported by: Rican7 at gmail dot com
Summary: DateTimes fail to compare correctly when a
DateTimeZone is set
-Status: Open
+Status: Closed
Type: Bug
Package: Date/time related
PHP Version: 7.0.6
-Assigned To:
+Assigned To: heiglandreas
Block user comment: N
Private report: N
Previous Comments:
------------------------------------------------------------------------
[2017-03-29 19:50:45] php-bugs at allenjb dot me dot uk
Refreshing the broken 3v4l (https://3v4l.org/WkjAA ) for current versions shows this issues appears to be fixed as of 7.0.17 and 7.1.3
------------------------------------------------------------------------
[2016-05-19 18:16:35] Rican7 at gmail dot com
May be related to bug #68474
------------------------------------------------------------------------
[2016-05-19 18:15:37] Rican7 at gmail dot com
These 3v4l scripts show the bug in action:
- Broken (TZ abbreviation): https://3v4l.org/WkjAA
- Working (IANA TZ): https://3v4l.org/MDqHr
------------------------------------------------------------------------
[2016-05-19 18:13:59] Rican7 at gmail dot com
Description:
------------
DateTime instances are supposed to be natively "comparable" (using `==`, `>`, `<`, etc).
Unfortunately, a colleague and I just noticed that if a Time Zone is updated on a compared instance, than the comparison fails... but only if the time zone isn't an IANA identifier. Really strange. What's worse, is that it seems to "magically" start to work again if a (seemingly idempotent) `getTimestamp()` call is made on the instance that had the new time-zone applied. Is this due to a lazy calculation of the internal timestamp that isn't getting validated when a new time zone is applied?
Test script:
---------------
<?php
$datetime = new DateTime(null, new DateTimeZone('UTC'));
$datetime_2 = clone $datetime;
$datetime_2->add(new DateInterval('PT2S'));
var_dump($datetime < $datetime_2);
var_dump($datetime->getTimestamp() < $datetime_2->getTimestamp());
$datetime_2->setTimezone(new DateTimeZone('EST'));
var_dump($datetime < $datetime_2);
var_dump($datetime->getTimestamp() < $datetime_2->getTimestamp());
// Now that `getTimestamp()` has been called, it'll "magically" work...
var_dump($datetime < $datetime_2);
Expected result:
----------------
bool(true)
bool(true)
bool(true)
bool(true)
bool(true)
Actual result:
--------------
bool(true)
bool(true)
bool(false)
bool(true)
bool(true)
------------------------------------------------------------------------
--
Edit this bug report at https://bugs.php.net/bug.php?id=72243&edit=1
lmpx.com only provides a reader for public news (NNTP) servers. It is not
affiliated with the servers or forums shown here and is not responsible for
the content of articles, which is written by their respective authors.