8000 Documented more missing Timezones features by javiereguiluz · Pull Request #11616 · symfony/symfony-docs · GitHub
[go: up one dir, main page]
Skip to content

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 4 commits into from
Closed

Documented more missing Timezones features #11616

wants to merge 4 commits into from

Conversation

javiereguiluz
Copy link
Member

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!

@javiereguiluz javiereguiluz added this to the 4.3 milestone May 28, 2019
@ro0NL
Copy link
Contributor
ro0NL commented May 28, 2019

hi @javiereguiluz

the offset represents the timezone compared to GMT, so Europe/Amsterdam is e.g. GMT+01:00 (raw 3600) or GMT+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".

@javiereguiluz
Copy link
Member Author

Roland, thanks a lot for your help. Everything is more clear now and I could complete the docs.

ro0NL
ro0NL reviewed May 29, 2019
View reviewed changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ro0NL ro0NL ro0NL approved these changes

Assignees
No one assigned
Labels
Intl Status: Reviewed
Projects
None yet
Milestone
4.3
Development

Successfully merging this pull request may close these issues.
3 participants
@javiereguiluz @ro0NL @carsonbot
0