Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:104196
MIME-Version: 1.0
References: <CAL=_i_=KCzMg63OPSGfjnb5m2sjK-5XTuCfokGMBwP-RBFH5Ag@mail.gmail.com>
 <2fed98af-3831-1a1c-def8-04793fcc0c6f@birkholz.biz> <CAF+90c-KE6NawSF+8DBrnugUR3Kgo7=9r53Uzgb5d+jBeNYQWQ@mail.gmail.com>
 <2a566239-d205-72f8-fd78-630b7ff01581@birkholz.biz>
In-Reply-To: <2a566239-d205-72f8-fd78-630b7ff01581@birkholz.biz>
Date: Tue, 5 Feb 2019 16:53:08 +0000
Message-ID: <CALKiJKqX9VrGntOjaJiWS8nDhPStYPQhfpKnz-8p-ssN9SU5iw@mail.gmail.com>
To: Dennis Birkholz <php@dennis.birkholz.biz>
Cc: Nikita Popov <nikita.ppv@gmail.com>, Joe Watkins <krakjoe@php.net>, 
	PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="0000000000004adbfe058128720c"
Subject: Re: [PHP-DEV] RFC Weakrefs
From: rowan.collins@gmail.com (Rowan Collins)

--0000000000004adbfe058128720c
Content-Type: text/plain; charset="UTF-8"

On Tue, 5 Feb 2019 at 16:06, Dennis Birkholz <php@dennis.birkholz.biz>
wrote:

> Maybe for PHP 8 we
> will decide to require all classes to be serializable except classes
> that implement the Unserializable interface or something... Or the magic
> __isSerializable() method that can decide this at runtime for the actual
> state of an object without failing hard when trying it.
>


I agree, it can be very confusing if you have something like a SimpleXML
object buried somewhere, and your serialize() call blows up. (See the
example I've raised a couple of times where Exceptions may or may not be
serializable, because they slurp up objects from everywhere in their
backtrace.)

I was going to suggest this would fit with the discussion Nikita raised
about adding a new serialization mechanism, but thinking about it more, I'm
not sure how the API for this would work. If you don't recursively check
that the properties of an object are also serializable, the call is
useless; but if you do, it's really no different from try {
serialize($foo); } catch {}

Regards,
-- 
Rowan Collins
[IMSoP]

--0000000000004adbfe058128720c--