DateTimeZone->getTransitions can return first transition twice
Open
@derickr is already working on this.
Since Mar 15, 2025.
Bug
Extension: date
Status: Needs Triage
- Dominant language
- C
- Stars
- 40.4k
- Forks
- 8.1k
- Avg merge
- 2d 13h
- Merged PRs (30d)
- 96
Description
Description
If timestamp_begin is the exact timestamp of an existing transition the transition will be included twice in the resulting array.
The following code:
<?php
$tzBln = new DateTimeZone('Europe/Berlin');
var_dump($tzBln->getTransitions(922582799, 941331599)); // works
var_dump($tzBln->getTransitions(922582800, 941331600)); // first transition reported twice
var_dump($tzBln->getTransitions(922582801, 941331601)); // works
Resulted in this output:
array(2) {
[0]=>
array(5) {
["ts"]=>
int(922582799)
["time"]=>
string(25) "1999-03-28T00:59:59+00:00"
["offset"]=>
int(3600)
["isdst"]=>
bool(false)
["abbr"]=>
string(3) "CET"
}
[1]=>
array(5) {
["ts"]=>
int(922582800)
["time"]=>
string(25) "1999-03-28T01:00:00+00:00"
["offset"]=>
int(7200)
["isdst"]=>
bool(true)
["abbr"]=>
string(4) "CEST"
}
}
array(3) {
[0]=>
array(5) {
["ts"]=>
int(922582800)
["time"]=>
string(25) "1999-03-28T01:00:00+00:00"
["offset"]=>
int(7200)
["isdst"]=>
bool(true)
["abbr"]=>
string(4) "CEST"
}
[1]=>
array(5) {
["ts"]=>
int(922582800)
["time"]=>
string(25) "1999-03-28T01:00:00+00:00"
["offset"]=>
int(7200)
["isdst"]=>
bool(true)
["abbr"]=>
string(4) "CEST"
}
[2]=>
array(5) {
["ts"]=>
int(941331600)
["time"]=>
string(25) "1999-10-31T01:00:00+00:00"
["offset"]=>
int(3600)
["isdst"]=>
bool(false)
["abbr"]=>
string(3) "CET"
}
}
array(2) {
[0]=>
array(5) {
["ts"]=>
int(922582801)
["time"]=>
string(25) "1999-03-28T01:00:01+00:00"
["offset"]=>
int(7200)
["isdst"]=>
bool(true)
["abbr"]=>
string(4) "CEST"
}
[1]=>
array(5) {
["ts"]=>
int(941331600)
["time"]=>
string(25) "1999-10-31T01:00:00+00:00"
["offset"]=>
int(3600)
["isdst"]=>
bool(false)
["abbr"]=>
string(3) "CET"
}
}
But I expected this output instead:
array(2) {
[0]=>
array(5) {
["ts"]=>
int(922582799)
["time"]=>
string(25) "1999-03-28T00:59:59+00:00"
["offset"]=>
int(3600)
["isdst"]=>
bool(false)
["abbr"]=>
string(3) "CET"
}
[1]=>
array(5) {
["ts"]=>
int(922582800)
["time"]=>
string(25) "1999-03-28T01:00:00+00:00"
["offset"]=>
int(7200)
["isdst"]=>
bool(true)
["abbr"]=>
string(4) "CEST"
}
}
array(2) {
[0]=>
array(5) {
["ts"]=>
int(922582800)
["time"]=>
string(25) "1999-03-28T01:00:00+00:00"
["offset"]=>
int(7200)
["isdst"]=>
bool(true)
["abbr"]=>
string(4) "CEST"
}
[1]=>
array(5) {
["ts"]=>
int(941331600)
["time"]=>
string(25) "1999-10-31T01:00:00+00:00"
["offset"]=>
int(3600)
["isdst"]=>
bool(false)
["abbr"]=>
string(3) "CET"
}
}
array(2) {
[0]=>
array(5) {
["ts"]=>
int(922582801)
["time"]=>
string(25) "1999-03-28T01:00:01+00:00"
["offset"]=>
int(7200)
["isdst"]=>
bool(true)
["abbr"]=>
string(4) "CEST"
}
[1]=>
array(5) {
["ts"]=>
int(941331600)
["time"]=>
string(25) "1999-10-31T01:00:00+00:00"
["offset"]=>
int(3600)
["isdst"]=>
bool(false)
["abbr"]=>
string(3) "CET"
}
}
PHP Version
PHP 8.x
Operating System
No response
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.