php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login

go to bug id or search bugs for

Doc Bug #79304 Doc. for DateTime::createFromFormat/date_create_from_format not consistent
Submitted: 2020-02-24 19:57 UTC Modified: 2020-02-25 19:02 UTC
Votes:1
Avg. Score:1.0 ± 0.0
Reproduced:1 of 1 (100.0%)
Same Version:1 (100.0%)
Same OS:1 (100.0%)
From: patrick dot kuijvenhoven at gmail dot com Assigned:
Status: Open Package: Documentation problem
PHP Version: Irrelevant OS:
Private report: No CVE-ID: None
View Add Comment Developer Edit
Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know!
Just going to say 'Me too!'? Don't clutter the database with that please — but make sure to vote on the bug!
Your email address:
MUST BE VALID
Solve the problem:
31 + 35 = ?
Subscribe to this entry?

 
 [2020-02-24 19:57 UTC] patrick dot kuijvenhoven at gmail dot com 
Description:
------------
Reference https://www.php.net/manual/en/datetime.createfromformat.php

Please notice where X and Y are present in each column in the following table:

+---------------------+------------------------------+-----------------+
| format              | description                  | example         |
+---------------------+------------------------------+-----------------+
| X and Y             | ...  with (X) or without (Y) | 01 to 31 (X) or |
| (alphabetical sort) |  leading zeros ...           |  1 to 31 (Y)    |
+---------------------+------------------------------+-----------------+

This style works for "d and j", "D and l", "m and n".

The time variants "g and h" and "G and H" do not follow this style. 

This can be fixed by replacing "with or without" to "without or with" in the "description" column for these formats. So, suggested to rewrite as:

+--------------+-------------------------+--------------------------------+
| format       | description             | example                        |
+--------------+-------------------------+--------------------------------+
| g and h      | ... without or with ... | 01 thought 12 or 01 through 12 |
| G and H      | ... without or with ... | 0 thought 23 or 00 through 23  | 
+--------------+-------------------------+--------------------------------+

Another option could be to explicitly mention the format in the description, thus like "... without (G) or with (H) ...".

Patches

createfromformat.patch (last revision 2020-02-24 20:11 UTC by patrick dot kuijvenhoven at gmail dot com)

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2020-02-24 20:11 UTC] patrick dot kuijvenhoven at gmail dot com 
The following patch has been added/updated:

Patch Name: createfromformat.patch
Revision:   1582575060
URL:        https://bugs.php.net/patch-display.php?bug=79304&patch=createfromformat.patch&revision=1582575060
 [2020-02-25 00:43 UTC] derick@php.net 
This makes it read really weirdly, so I would say "Won't Fix" - if at all, change the examples around.
 [2020-02-25 19:00 UTC] patrick dot kuijvenhoven at gmail dot com
-Summary: Doc. for DateTime::createFromFormat/date_create_from_format not consistent +Summary: patrick.kuijvenhoven@gmail.com
 [2020-02-25 19:00 UTC] patrick dot kuijvenhoven at gmail dot com 
I'm not a native English speaker, so I'm not a good judge concerning readability.

In case you prefer keeping the "with or without" and switching the examples around, then, for consistency the "g and h" and "G and H" should by switched as well (to "h and g" and "H and G" to be precise).

Let me know your thoughts. I'd be happy to supply a new patch.
 [2020-02-25 19:02 UTC] patrick dot kuijvenhoven at gmail dot com
-Summary: patrick.kuijvenhoven@gmail.com +Summary: Doc. for DateTime::createFromFormat/date_create_from_format not consistent
 [2020-02-25 19:02 UTC] patrick dot kuijvenhoven at gmail dot com 
(Sorry, Summary altered by password manager by accident)
 [2022-12-01 05:40 UTC] Janet22coffect at gmail dot com 
Glad you like it. We have been using this for several weeks now and it seems to be going nicely. (https://www.pfmlogin.net/)github.com
 [2022-12-19 10:02 UTC] Gordon896Herzog at gmail dot com 
There's a bug report that exactly describes your problem :)

https://bugs.php.net/bug.php?id=51950

Since 2016-08-07, the bug report has been marked as "not a bug". You need to use strtotime or new DateTime instead.

The constants that have been defined apply to both formatting and parsing in the same way, which forces your ways.
(https://www.mckessonconnect.net/)github.com
 [2023-03-17 07:11 UTC] pfmlogin1 at gmail dot com 
Hey there! I just wish to give you a huge thumbs up for your excellent info you have right here on this post. (https://www.pfmlogin.top/)github.com
 [2023-06-16 09:46 UTC] kevonaol1 at aol dot com 
(https://variety.com/wp-content/uploads/2018/09/rexfeatures_9064813mz.jpg?w=1000&h=563&crop=1&resize=1000%2C563)github.com

(https://static01.nyt.com/images/2013/03/31/arts/31PIVEN1_SPAN/31PIVEN1-superJumbo.jpg?quality=75&auto=webp)github.com

(https://www.telegraph.co.uk/content/dam/comedy/2021/10/16/TELEMMGLPICT000145291306_trans_NvBQzQNjv4BqqVzuuqpFlyLIwiB6NTmJwSX5rhseiWKOo9p9OQ-ymek.jpeg?imwidth=960)github.com

(https://images.squarespace-cdn.com/content/v1/5b61f75d506fbecf8655789d/c14cde80-aa2d-453a-bc3e-4964c9149e0d/jeremy-piven.jpg?format=1000w)github.com

(https://static.wikia.nocookie.net/disney/images/d/d7/Jeremy_Piven.jpg/revision/latest?cb=20180726001612)github.com

(https://www.amazon.com/prime-video/actor/Jeremy-Piven/amzn1.dv.gti.785e61d6-1dd8-4777-88c4-b33c24097594/)github.com
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved. 		Last updated: Fri Apr 19 00:01:29 2024 UTC