Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:62629
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.210.170 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <CAArmra3oN6NMn-CPn95piNQeKN1LJ7nq3baj1B0z6pCCPZX0VQ@mail.gmail.com>
References: <CAArmra3oN6NMn-CPn95piNQeKN1LJ7nq3baj1B0z6pCCPZX0VQ@mail.gmail.com>
Date: Sat, 1 Sep 2012 12:36:35 +0200
Message-ID: <CAEZPtU7sPwJxtQyYWP5NvU7nVpZxCaCkVcAF1u1Byd4+cj4MjA@mail.gmail.com>
To: Will Fitch <willfitch@php.net>
Cc: internals@lists.php.net
Content-Type: text/plain; charset=ISO-8859-1
Subject: Re: [PHP-DEV] RFC for Adding __toString to DateTime
From: pierre.php@gmail.com (Pierre Joye)

hi Will,

On Sat, Sep 1, 2012 at 12:14 PM, Will Fitch <willfitch@php.net> wrote:
> I would like to officially introduce an RFC with a patch to implement
> __toString to DateTime.  This is a commonly requested feature that goes
> unanswered mostly because of the inability to agree on a default pattern.
>
> In short, the patch uses the ISO-8601 format as the default pattern.  The
> pattern may be changed via setDefaultPattern and date_default_pattern_set,
> as explained in the RFC.
>
> The link to the RFC and patch are here:
> https://wiki.php.net/rfc/datetime_tostring.
>
> While I know this isn't as interesting as many of the current proposals
> going on, I'm hoping you can give this RFC a look.

It would be better to name the format method setDefaultFormat instead
of setDefaultPattern, to be consistent with the rest of the API.

(not saying I like the idea or not, only a 1st comment :)

Cheers,
-- 
Pierre

@pierrejoye