Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:109345
MIME-Version: 1.0
References: <003701d6013c$9afe9750$d0fbc5f0$@gmx.de> <379095fd-ba11-4d84-6cab-89905eeab32a@gmx.de>
 <CAJmjfvXrtC7b8B2F5fYUy+s5Dvao6VohH=kVfODaJZ_CNJ2jRw@mail.gmail.com> <004501d6038c$c7870df0$569529d0$@gmx.de>
In-Reply-To: <004501d6038c$c7870df0$569529d0$@gmx.de>
Date: Thu, 26 Mar 2020 18:18:58 +0100
Message-ID: <CAC5V6gi6wxCuhjR5qsTF3WkankP-usC-h=hw_wJ-+8rV-_PJxQ@mail.gmail.com>
To: jan.h.boehmer@gmx.de
Cc: Jakob Givoni <jakob@givoni.dk>, "Christoph M. Becker" <cmbecker69@gmx.de>, 
	PHP internals <internals@lists.php.net>
Content-Type: text/plain; charset="UTF-8"
Subject: Re: [PHP-DEV] Re: [VOTE] Userspace operator overloading
From: guilliam.xavier@gmail.com (Guilliam Xavier)

On Thu, Mar 26, 2020 at 5:37 PM <jan.h.boehmer@gmx.de> wrote:
>
> 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:
>         $ret = (string) $a . (string) $b;

This first part seems legit to me.

> 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).

I fear that "hint" notice could break Symfony apps... Couldn't you
just not trigger it in this case?

-- 
Guilliam Xavier