Repository navigation
Datetime created from negative float timestamp is not monotonic (fraction part is added instead of substracted) #7758
Description
Description
The following code:
foreach ([0.4, 0, -0.4, -1, -1.4] as $ts) {
$date = new DateTime('@' . $ts);
print_r($date);
}Diff of current output vs expected:
DateTime Object
(
[date] => 1970-01-01 00:00:00.400000
[timezone_type] => 1
[timezone] => +00:00
)
DateTime Object
(
[date] => 1970-01-01 00:00:00.000000
[timezone_type] => 1
[timezone] => +00:00
)
DateTime Object
(
- [date] => 1970-01-01 00:00:00.400000
+ [date] => 1969-12-31 23:59:59.600000
[timezone_type] => 1
[timezone] => +00:00
)
DateTime Object
(
[date] => 1969-12-31 23:59:59.000000
[timezone_type] => 1
[timezone] => +00:00
)
DateTime Object
(
- [date] => 1969-12-31 23:59:59.400000
+ [date] => 1969-12-31 23:59:58.600000
[timezone_type] => 1
[timezone] => +00:00
)PHP Version
present since PHP 8.0, should be fixed starting this version
it seems the issue is present only when TS is parsed, getTimestamp / format('u') seems correct across all PHP versions - https://3v4l.org/CjJLf
Activity
damianwadley commentedon Dec 11, 2021
For the explanation: fractional timestamps are currently not supported when parsing, which means PHP handles inputs like
@-1.4as an absolute "@-1" (1970 minus 1s) followed by a relative ".4" (plus 0.4s).mvorisek commentedon Dec 13, 2021
this is how it is parsed currently probably, however if unix timestamp is negative, the fractional part must be decremented as if the whole TS was a standard floating point number
https://3v4l.org/rKOAb shows the fractional part cannot be even used alone (eg. parsed with
@tstoken and the parsing must be fixed)iluuu1994 commentedon Dec 13, 2021
I think it's fair to say that this a feature request as the docs do state that only whole numbers are supported.
Fix negative us in unix timestamp syntax
Fix negative us in unix timestamp syntax
Added test for: Fix negative us in unix timestamp syntax
hormus commentedon Apr 24, 2022
Fixed bug phpGH-7758 (Problems with negative timestamps and fractions)
tidy #66848 [Uid] Fix UuidFactoryTest on PHP < 8.1.7 (nicolas-grekas)