-
Notifications
You must be signed in to change notification settings - Fork 7.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
DateTimeZone::getName() does not include seconds in offset #11281
Labels
Comments
Seems like the internal function php_timezone_to_string() is not updated indeed. |
nielsdos
added a commit
to nielsdos/php-src
that referenced
this issue
May 19, 2023
…offset If the seconds portion is non-zero, include the seconds in the output.
nielsdos
added a commit
to nielsdos/php-src
that referenced
this issue
May 19, 2023
…offset If the seconds portion is non-zero, include the seconds in the output.
iluuu1994
added a commit
to iluuu1994/php-src
that referenced
this issue
May 22, 2023
iluuu1994
added a commit
to iluuu1994/php-src
that referenced
this issue
May 22, 2023
iluuu1994
added a commit
to iluuu1994/php-src
that referenced
this issue
May 22, 2023
iluuu1994
added a commit
to iluuu1994/php-src
that referenced
this issue
May 22, 2023
nielsdos
added a commit
that referenced
this issue
May 23, 2023
* PHP-8.1: Fix GH-11281: DateTimeZone::getName() does not include seconds in offset
nielsdos
added a commit
that referenced
this issue
May 23, 2023
* PHP-8.2: Fix GH-11281: DateTimeZone::getName() does not include seconds in offset
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
The following code:
Resulted in this output:
But I expected this output instead:
Context: since PHP 8.1.7 (#8589), time-zone offsets with seconds are properly supported:
https://3v4l.org/m5VkU
But it looks like
getName()
has not been updated to support seconds.PHP Version
8.2.6
Operating System
No response
The text was updated successfully, but these errors were encountered: