Doc #79304 [Com]: Doc. for DateTime::createFromFormat/date_create_from_format not consistent

From: Date: Wed, 07 Jul 2021 06:50:01 +0000
Subject: Doc #79304 [Com]: Doc. for DateTime::createFromFormat/date_create_from_format not consistent
References: 1  Groups: php.doc.bugs 
Request: Send a blank email to doc-bugs+get-18921@lists.php.net to get a copy of this message
Edit report at https://bugs.php.net/bug.php?id=79304&edit=1

 ID:                 79304
 Comment by:         derodixeyari at getnada dot com
 Reported by:        patrick dot kuijvenhoven at gmail dot com
 Summary:            Doc. for
                     DateTime::createFromFormat/date_create_from_format
                     not consistent
 Status:             Open
 Type:               Documentation Problem
 Package:            Documentation problem
 PHP Version:        Irrelevant
 Block user comment: N
 Private report:     N

 New Comment:

Hello Guys! If you are here to find out the details of the Books-A-Million Customer Satisfaction
Survey, then you are at the right place here.
https://surveyz.onl/www-tellbam-com-survey/


Previous Comments:
------------------------------------------------------------------------
[2020-02-25 19:02:09] patrick dot kuijvenhoven at gmail dot com

(Sorry, Summary altered by password manager by accident)

------------------------------------------------------------------------
[2020-02-25 19:00:40] 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 00:43:39] 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-24 20:11:00] 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-24 19:57:21] 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) ...".




------------------------------------------------------------------------



--
Edit this bug report at https://bugs.php.net/bug.php?id=79304&edit=1


Thread (13 messages)

« previous php.doc.bugs (#18921) next »