Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:98541
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.128.176 as permitted sender)
MIME-Version: 1.0
References: <CAGyGae2yg-NnPmcJEJcPdn3XQTUyS_=Ymg1nrdTDTig9r2rdfw@mail.gmail.com>
 <CAObuZdsJK3eUMb1EzumdOeue1+YoOSVy-7tdYCV4+WmnjjfhOg@mail.gmail.com>
In-Reply-To: <CAObuZdsJK3eUMb1EzumdOeue1+YoOSVy-7tdYCV4+WmnjjfhOg@mail.gmail.com>
Date: Wed, 15 Mar 2017 18:31:50 +0000
Message-ID: <CAGyGae2Hm6N3odayWX6Ashh1GZxiPM6rZK7n+cYxvLB24BwE9Q@mail.gmail.com>
To: Ryan Pallas <derokorian@gmail.com>
Cc: "internals@lists.php.net" <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=001a1149244004a883054ac929b9
Subject: Re: [PHP-DEV] Add __toArray() method to objects that would be called
 on cast to array
From: bburnichon@gmail.com (=?UTF-8?Q?Beno=C3=AEt_Burnichon?=)

--001a1149244004a883054ac929b9
Content-Type: text/plain; charset=UTF-8

> I like the idea kind of, but would this remove the ability to cast to
> array all classes not implementing __toArray, as is the case with
> __toString? This would be a HUGE BC if so:
>
> $ php -r 'class Foo {public $foo = "foobar";} var_dump((array) (new Foo));'
> array(1) {
>   ["foo"]=>
>   string(6) "foobar"
> }
> $ php -r 'class Foo {public $foo = "foobar";} var_dump((string) (new
> Foo));'
> PHP Recoverable fatal error:  Object of class Foo could not be converted
> to string in Command line code on line 1
> $ php -v
> PHP 7.1.2 (cli) (built: Feb 27 2017 00:02:44) ( ZTS )
>

No. For me, classes not implementing __toArray() should keep current
behavior. That was what I had in mind when I wrote that __toArray() should
have a default standard implementation.

Same restrictions could be applied to this magic method:
http://us3.php.net/manual/en/language.oop5.magic.php#language.oop5.magic.tostring

By this, I mean, no exceptions should be thrown in this method and return
value MUST be an array.

--001a1149244004a883054ac929b9--