Re: Unserialize is broken
| From: | Herman Radtke | Date: | Sun, 28 Feb 2010 19:03:06 +0000 |
| Subject: | Re: Unserialize is broken | ||
| References: | 1 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-47062@lists.php.net to get a copy of this message | ||
> Imo unserialize should check, when applying public or protected values,
> if either exists on the object, and apply it to the one that exists.
> Sure it's gonna cost some performance, but at least changing the
> prototype of your class while stuff is running isn't going to kill your
> code anymore.
This seems like a corner case and one that a conversion script should
handle. Considering that serialize and unserialize are called for
every single web-request, degrading the performance of unserialize is
not something that should be done lightly.
--
Herman Radtke
hermanradtke@gmail.com | http://hermanradtke.com