ID:               47184
 Updated by:       der...@php.net
 Reported By:      guilhermebla...@php.net
 Status:           Bogus
 Bug Type:         Date/time related
 Operating System: *
 PHP Version:      5.2.8
 New Comment:

tbh, I don't think we should change this either...


Previous Comments:
------------------------------------------------------------------------

[2009-01-22 15:03:20] guilhermebla...@php.net

Ilia,

It's not a case of being short or full format.
It's a matter of consistency.

Even this value is valid: 20090122T130112Z0200

The question here is that both DATE_ISO8601 and date('c') should be
consistent, since they refer to the same format.

I am just forwarding what I've heard not only once, but a couple of
times from different work teams about this mismatch.

Regards,

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

[2009-01-22 14:34:54] il...@php.net

One is basic and the other is extended format, both are valid.

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

[2009-01-21 17:07:06] guilhermebla...@php.net

Description:
------------
Following the documentation, DATE_ISO8601 contains this syntax:
2005-08-15T15:52:01+0000

Also, date('c') documents that: ISO 8601 date (added in PHP 5) 
        2004-02-12T15:19:21+00:00

We should keep consistency when supporting this spec.

I propose the change then to DATE_ISO8601 constant, defined in
ext/date/php_date.c, line 714:

- #define DATE_FORMAT_ISO8601 "Y-m-d\\TH:i:sO"
+ #define DATE_FORMAT_ISO8601 "Y-m-d\\TH:i:sP" 

Reproduce code:
---------------
echo DATE_ISO8601 === date('c') ? 'true' : 'false';

Expected result:
----------------
true

Actual result:
--------------
false


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


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

Reply via email to