Dates Created from Strings with Timezone Offsets Aren't Properly Formatted When Using 'p' #10447

Activity

1. dustinwilson

   added

   Bug

   Status: Needs Triage

   on Jan 25, 2023
2. Girgias

   added

   Extension: date

   and removed

   Status: Needs Triage

   on Jan 25, 2023
3. Girgias

   assigned

   derickr

   on Jan 25, 2023
4. damianwadley

   added

   Status: Verified

   on Jan 25, 2023
5. hormus commented on Jan 26, 2023

   

   hormus

   on Jan 26, 2023

   Last edited by hormus

   More actions

   Currently support p is with timezone type 3 or type 2 "UTC" or explicit "Z" https://php.watch/versions/8.0/date-utc-p-format :)

   <?php
   
   date_default_timezone_set('Europe/Berlin');
   $date = new \DateTime('2023-01-25T00:00:00');
   echo $date->format('p');
   
   ?>

   For missing offset is default offset:
   Expected result: +01:00
   For explicit offset +00:00
   Expected result: +00:00
   Or !localtime
   a6e3ce4#diff-975ac482adc74b487c6ac9d8dde32153e4b8a803ef8ec41e8d6594f0a042db69R715
   Expected result with p and timezone UTC
   $date2 = new DateTime('now', new DateTimeZone('UTC'));
   echo $date2->format('p'); // Z
6. derickr commented on Jan 26, 2023

   

   derickr

   on Jan 26, 2023

   Member

   More actions

   @damianwadley Why did you change this to "Verified" — I see no triage report as to why this issue occurs, and whether this is an issue at all.
7. damianwadley commented on Jan 26, 2023

   

   damianwadley

   on Jan 26, 2023

   Member

   More actions

   I assumed it had been verified because the "needs triage" was removed. Though I did briefly confirm that "p" wasn't outputting just Z which it's documented to do.

   👍React with 👍1Reacted by Christoph M. Becker
8. derickr

   added 2 commits that reference this issue on Jan 31, 2023

   Fixed phpGH-10447: 'p' format specifier does not yield 'Z' for 00:00

   This issue will close once commit a42bf93 is merged into the 'master' branch.

   a42bf93

   Merge remote-tracking branch 'derickr/GH-10447-p-format-specifier' in…

   ...

   This issue will close once commit d17175c is merged into the 'master' branch.

   d17175c
9. derickr

   closed this as completedin a42bf93on Jan 31, 2023