php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #81560 DateTime-Add($interval) endless loop for dst
Submitted: 2021-10-28 10:37 UTC Modified: 2022-05-13 15:45 UTC
Votes:1
Avg. Score:5.0 ± 0.0
Reproduced:1 of 1 (100.0%)
Same Version:1 (100.0%)
Same OS:1 (100.0%)
From: mihailovb at gmail dot com Assigned: derick (profile)
Status: Wont fix Package: Date/time related
PHP Version: 7.4.25 OS: CentOS7 (and others)
Private report: No CVE-ID: None
View Add Comment Developer Edit
Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know!
Just going to say 'Me too!'? Don't clutter the database with that please — but make sure to vote on the bug!
Your email address:
MUST BE VALID
Solve the problem:
36 - 1 = ?
Subscribe to this entry?

 
 [2021-10-28 10:37 UTC] mihailovb at gmail dot com
Description:
------------
PHP 7.4.25 (CentOS), 7.4.3(Ubuntu20.04), 7.2.34(Ubuntu20.04)

It goes into an endless loop when trying to add a 15M interval only when working with "negative" timezones. Doesn't happen for European "pozitive" timezones

Test script:
---------------
<?php
$timezoneDST = new DateTimeZone('America/New_York');
$date = new DateTime("2020-11-01", $timezoneDST);
$dateInterval = new DateInterval('PT15M');
$counter = 0;
while($counter < 500){
    $date->add($dateInterval);
    $counter++;
}
var_dump($date);

Expected result:
----------------
class DateTime#1 (3) {
  public $date =>
  string(26) "2020-11-06 04:00:00.000000"
  public $timezone_type =>
  int(3)
  public $timezone =>
  string(16) "America/New_York"
}

Actual result:
--------------
class DateTime#2 (3) {
  public $date =>
  string(26) "2020-11-01 01:00:00.000000"
  public $timezone_type =>
  int(3)
  public $timezone =>
  string(16) "America/New_York"
}

Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2021-10-28 11:23 UTC] cmb@php.net
-Status: Open +Status: Verified -Assigned To: +Assigned To: derick
 [2021-10-28 11:23 UTC] cmb@php.net
This is apparently fixed as of PHP 8.1.0[1]; I'm not sure whether
the fix will be backported.

[1] <https://3v4l.org/1df2R>
 [2021-12-03 08:41 UTC] mihailovb at gmail dot com
Ah, I hope it will be backported, cause after looking into the project and the libraries that are being used, not sure if I'd be able to upgrade to 8.1.
On the bright side, we'd have something less than an year to try and do it.
 [2022-05-13 15:45 UTC] derick@php.net
-Status: Verified +Status: Wont fix
 [2022-05-13 15:45 UTC] derick@php.net
Sorry, we can't back port this because that would require merging in an older version of timelib, which also introduces new features and subtly changes behaviour that we can't have in patch releases.
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved.
Last updated: Sat May 11 02:01:29 2024 UTC