Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:109340
To: "'Jakob Givoni'" <jakob@givoni.dk>,
	"'Christoph M. Becker'" <cmbecker69@gmx.de>
Cc: "'PHP internals'" <internals@lists.php.net>
References: <003701d6013c$9afe9750$d0fbc5f0$@gmx.de> <379095fd-ba11-4d84-6cab-89905eeab32a@gmx.de> <CAJmjfvXrtC7b8B2F5fYUy+s5Dvao6VohH=kVfODaJZ_CNJ2jRw@mail.gmail.com>
In-Reply-To: <CAJmjfvXrtC7b8B2F5fYUy+s5Dvao6VohH=kVfODaJZ_CNJ2jRw@mail.gmail.com>
Date: Thu, 26 Mar 2020 17:37:02 +0100
Message-ID: <004501d6038c$c7870df0$569529d0$@gmx.de>
MIME-Version: 1.0
Content-Type: text/plain;
	charset="utf-8"
Content-Transfer-Encoding: quoted-printable
Thread-Index: AQJhA3CUe9vS9hOq0bNeMXk6llj72ADzlMokAW3ZrX6nMfe8AA==
Content-Language: de
Subject: RE: [PHP-DEV] Re: [VOTE] Userspace operator overloading
From: jan.h.boehmer@gmx.de

On Thursday, March 26, 2020 3:50 AM Jakob Givoni <jakob@givoni.dk> =
wrote:

> On Wed, Mar 25, 2020 at 6:28 AM Christoph M. Becker =
<cmbecker69@gmx.de> wrote:
>
>> It seems to me that the RFC is not sufficiently specific enough=20
>> regarding the concatenation of instances of classes which implement=20
>> __toString().
>
> Exactly what I was thinking too. Would be nice with some examples on =
this.

I am not sure if I can (or should) still add this to the RFC, here some =
clarification on this:

The overloaded concat operator has higher priority than the __toString() =
method.
So if Class A overloades the concat operator, then calling $a . $b means =
ClassA::__concat($a, $b);  (Note that both operands are passed in their =
original form)
If you want to concat the string representations, you will have to =
explicitly convert the objects to strings:=20
	$ret =3D (string) $a . (string) $b;

If the concat operator is not overloaded, the behavior is like now, and =
the objects are converted implicitly to strings (so $a . $b actually =
means (string) $a . (string) $b).
Furthermore an notice is triggered, hinting the user that he could =
overload the concat operator. (Maybe here a different message than for =
the other operators would be useful).