Skip to content
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

DateInterval: weeks and days can be combined since 8.0 #633

Closed
gharlan opened this issue May 27, 2021 · 1 comment
Closed

DateInterval: weeks and days can be combined since 8.0 #633

gharlan opened this issue May 27, 2021 · 1 comment
Labels
bug Documentation contains incorrect information good first issue Good for newcomers

Comments

@gharlan
Copy link

gharlan commented May 27, 2021

From manual page: https://php.net/dateinterval.construct


W | weeks. These get converted into days, so can not be combined with D.

The "so can not be combined with D" part is no longer true since php 8.0: https://3v4l.org/RNqK8
derickr/timelib#85

@kamil-tekiela
Copy link
Member

Thanks for the issue. We also encourage making pull requests. There's a lot of things pending to be updated in the manual and we appreciate any help.

@cmb69 cmb69 added bug Documentation contains incorrect information good first issue Good for newcomers labels May 27, 2021
saundefined added a commit to saundefined/doc-en that referenced this issue May 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Documentation contains incorrect information good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

3 participants