Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:71394
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.217.181 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <2D2D996C-9455-4DD1-9669-90AFAF591E32@strojny.net>
References: <CAESVnVq3=+_N-c5e1n==c=w9Og82FM24pBqJJt66yS-Swru0TA@mail.gmail.com>
	<2D2D996C-9455-4DD1-9669-90AFAF591E32@strojny.net>
Date: Wed, 22 Jan 2014 06:46:36 +0100
Message-ID: <CAEZPtU7UMfJn3boTi0n-fOtqaA4n-Osw7jzkP3WTWKjfhtozuw@mail.gmail.com>
To: Lars Strojny <lars@strojny.net>
Cc: Sara Golemon <pollita@php.net>, PHP internals <internals@lists.php.net>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
Subject: Re: [PHP-DEV] [RFC] __debug_info()
From: pierre.php@gmail.com (Pierre Joye)

hi,

On Tue, Jan 21, 2014 at 10:37 PM, Lars Strojny <lars@strojny.net> wrote:
> Hi Sara,
>
> On 21 Jan 2014, at 22:26, Sara Golemon <pollita@php.net> wrote:
>
>> Something new to argue about:
>>
>> https://wiki.php.net/rfc/debug-info
>
> Love it, two things: the var_dump() properties will be replaced (not merg=
ed) with the "real" properties in the object (I am asking as that would mak=
e it very very cool for stuff like doctrine, where lazy loaded entities are=
 way too large to dump)? As for the name, I would prefer __debugInfo() as i=
n __callStatic(). I know there is __set_state() but that's something that n=
eeds deprecation and fixing :)

I can only second both statements. Love the idea (very useful!) and it
should indeed follow other "magic" method naming convention.

One additional comment, I am not sure about the manual merge of
properties. Maybe it should create an additional property called
"debugInfo" or something similar and store the result of __debugInfo
in it? It could be more user-friendly and less confusing.

Cheers,
--=20
Pierre

@pierrejoye | http://www.libgd.org