Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:108452
MIME-Version: 1.0
References: <CAOWwgp=STMdOnp9NHn35-mm7TdZD3KKk5PwSGKXeqTyaDBoYHA@mail.gmail.com>
 <CAF+90c_piDMr7joL=aSZXBBn-4eTQqM65P7_RKz2GhidDaj7zA@mail.gmail.com>
In-Reply-To: <CAF+90c_piDMr7joL=aSZXBBn-4eTQqM65P7_RKz2GhidDaj7zA@mail.gmail.com>
Date: Tue, 11 Feb 2020 09:56:39 +0100
Message-ID: <CAOWwgp=jMnafLs3RzqnUNdjJ0=UCtJO7SvWoxthrpmPyQAWhLw@mail.gmail.com>
To: PHP Internals List <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="0000000000005f511a059e490943"
Subject: Re: [PHP-DEV] [RFC] Adding a "Stringable" interface to PHP 8
From: nicolas.grekas+php@gmail.com (Nicolas Grekas)

--0000000000005f511a059e490943
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

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, y=
ou
>> 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 ha=
ve
>> too.
>>
>> Cheers,
>> Nicolas
>>
>
> I'm wondering whether it could make sense to automatically implement this
> interface for all classes that define __toString(). Explicitly implementi=
ng
> the interface is useful for the transition period (where old PHP versions
> 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 consumers
> of the Stringable type, it will likely be quite common that the interface
> will not get implemented...
>


This proposal has been implemented by Nikita himself on the linked PR, see
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?

Cheers,
Nicolas

--0000000000005f511a059e490943--