Documented more missing Timezones features#11616

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Closed

javiereguiluz wants to merge4 commits intosymfony:4.3fromjaviereguiluz:fix_11499

Closed

Documented more missing Timezones features#11616

javiereguiluz wants to merge4 commits intosymfony:4.3fromjaviereguiluz:fix_11499

Conversation

Copy link

Member

javiereguiluz commentedMay 28, 2019

Fixes#11499.

@ro0NL I need your help. I don't know:

What's the difference between getRawOffset and getGmtOffset
What's the purpose of the timestamp argument defined by both

Thanks!

Documented more missing Timezones features

4c73827

javiereguiluz added the Intl label

May 28, 2019

javiereguiluz added this to the4.3 milestone

May 28, 2019

carsonbot added the Status: Needs Review label

May 28, 2019

Copy link

Contributor

ro0NL commentedMay 28, 2019

hi@javiereguiluz

the offset represents the timezone compared to GMT, soEurope/Amsterdam is e.g.GMT+01:00 (raw 3600) orGMT+02:00 (raw 7200). The difference is based on time, depending on a active DST transition. Hence the extra argument, to get the value at a certain point in time, defaulting to "now/today".