Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:108465
MIME-Version: 1.0
References: <CAOWwgp=STMdOnp9NHn35-mm7TdZD3KKk5PwSGKXeqTyaDBoYHA@mail.gmail.com>
 <CAF+90c_piDMr7joL=aSZXBBn-4eTQqM65P7_RKz2GhidDaj7zA@mail.gmail.com> <CAOWwgp=jMnafLs3RzqnUNdjJ0=UCtJO7SvWoxthrpmPyQAWhLw@mail.gmail.com>
In-Reply-To: <CAOWwgp=jMnafLs3RzqnUNdjJ0=UCtJO7SvWoxthrpmPyQAWhLw@mail.gmail.com>
Date: Tue, 11 Feb 2020 12:58:29 +0100
Message-ID: <CAF+90c-0bm=O2m5Csyp6LxP6SM6twBe8kWzdhVDjK-1ibExAkg@mail.gmail.com>
To: Nicolas Grekas <nicolas.grekas+php@gmail.com>
Cc: PHP Internals List <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="000000000000fc7267059e4b93d1"
Subject: Re: [PHP-DEV] [RFC] Adding a "Stringable" interface to PHP 8
From: nikita.ppv@gmail.com (Nikita Popov)

--000000000000fc7267059e4b93d1
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Tue, Feb 11, 2020 at 9:57 AM Nicolas Grekas <nicolas.grekas+php@gmail.co=
m>
wrote:

> Le lun. 27 janv. 2020 =C3=A0 10:19, Nikita Popov <nikita.ppv@gmail.com> a
> =C3=A9crit :
>
> > On Wed, Jan 22, 2020 at 4:47 PM Nicolas Grekas <
> > nicolas.grekas+php@gmail.com> wrote:
> >
> >> Hello everyone,
> >>
> >> as announced last week, I'm officially opening a discussion for adding=
 a
> >> "Stringable" interface to PHP 8.
> >>
> >> The RFC and its rationale are presented here as required:
> >> https://wiki.php.net/rfc/stringable
> >>
> >> The patch is found on GitHub, where some discussions happened already,
> you
> >> might be interested in having a look:
> >> https://github.com/php/php-src/pull/5083
> >>
> >> TL;DR, I think we need an interface to allow passing objects with a
> >> __toString() method in a type-safe way, using the string|Stringable
> type.
> >> Here is the stub of the proposal:
> >> interface Stringable { public function __toString(): string; }
> >>
> >> I invite everyone to review the RFC before commenting here as I tried =
to
> >> sum up considerations that some already had before and that you might
> have
> >> too.
> >>
> >> Cheers,
> >> Nicolas
> >>
> >
> > I'm wondering whether it could make sense to automatically implement th=
is
> > interface for all classes that define __toString(). Explicitly
> implementing
> > the interface is useful for the transition period (where old PHP versio=
ns
> > also need to be supported), but when considering only PHP 8 support, I
> > think it may make sense to implicitly implement the interface. If the
> > __toString() functionality also works fine without implementing the
> > interface, and there's probably only going to be relatively few consume=
rs
> > of the Stringable type, it will likely be quite common that the interfa=
ce
> > will not get implemented...
> >
>
>
> This proposal has been implemented by Nikita himself on the linked PR, se=
e
> 2nd commit:
> https://github.com/php/php-src/pull/5083
>
> I updated the RFC accordingly:
> https://wiki.php.net/rfc/stringable
>
> I think the RFC is ready for entering the voting stage.
> Any other comments before?
>

Just so someone has mentioned it... is "Stringable" really the best name
for this interface? Reddit really didn't like it ;) Some possible
alternatives: ToString, HasToString, CastsToString.

Something to keep in mind is that there has been a recent proposal for
"Arrayable" as well, and that had very different semantics (not about
__toArray() at all).

Regards,
Nikita

--000000000000fc7267059e4b93d1--