Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:98545
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.220.175 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <CAGyGae2yg-NnPmcJEJcPdn3XQTUyS_=Ymg1nrdTDTig9r2rdfw@mail.gmail.com>
References: <CAGyGae2yg-NnPmcJEJcPdn3XQTUyS_=Ymg1nrdTDTig9r2rdfw@mail.gmail.com>
Date: Wed, 15 Mar 2017 21:41:15 +0100
Message-ID: <CADyq6sKASo_nV9SvvG6LXKCEOY-H=ofuQqbSwAA08TMb1T_Rmg@mail.gmail.com>
To: =?UTF-8?Q?Beno=C3=AEt_Burnichon?= <bburnichon@gmail.com>
Cc: PHP Internals List <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=001a114c568c61135d054acaf7b6
Subject: Re: [PHP-DEV] Add __toArray() method to objects that would be called
 on cast to array
From: ocramius@gmail.com (Marco Pivetta)

--001a114c568c61135d054acaf7b6
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

This is a BC break due to the fact that the `(array)` cast is used to
extract property information from private properties in library code.

On 15 Mar 2017 6:50 p.m., "Beno=C3=AEt Burnichon" <bburnichon@gmail.com> wr=
ote:

> Hi all,
>
> Looking at code of PHPUnit, I stumbled upon an inconsistent array
> conversion:
>
> ------
> /**
>  * @param ArrayAccess|array $other
>  */
> function evaluate($other)
> {
>   // type cast $other as an array to allow
>   //support in standard array functions.
>   if ($other instanceof ArrayAccess) {
>     $data =3D (array) $data;
>   }
>
>   $patched =3D array_replace_recursive($other, $this->subset);
>
>   // ...
> }
> -----
>
> This would work only for `ArrayAccess` implementations extending
> `ArrayObject` as shown by https://3v4l.org/ti4aY
>
> Looking at the manual
> http://php.net/manual/en/language.types.array.php#
> language.types.array.casting
> ,
> it seems `ArrayObject` class does not comply to array casting because
> integer public properties could also be retrieved. Some tests showed that
> regular class always have string keys even when a `$key =3D 0; $this->{$k=
ey}
> =3D 'avalue';` is called. In this case, `var_export((array)$object);` ret=
urns
> `array('0' =3D> 'avalue')` (Notice the quote around key 0 -
> https://3v4l.org/6QW70)
>
> What do you think of adding an optional `__toArray()` method to classes
> which would default to current behavior but would allow specifying
> behavior. The way of internal `__toString()` method and could explain
> inconsistency of the `ArrayObject` class?
>
> Regards,
>
> Beno=C3=AEt Burnichon
>

--001a114c568c61135d054acaf7b6--