Re: Adieu a la magie (remove magic_quotes)
| From: | Pierre | Date: | Tue, 07 Mar 2006 15:22:08 +0000 |
| Subject: | Re: Adieu a la magie (remove magic_quotes) | ||
| References: | 1 2 3 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-22203@lists.php.net to get a copy of this message | ||
On 3/7/06, Pierre <pierre.php@gmail.com> wrote:
>
> On 3/7/06, Zeev Suraski <zeev@zend.com> wrote:
>
> > The point is that breakage is aggregated, not binary. The more stuff
> > we break, the more difficult it is to port, and frankly, it's quite
> > likely that a non OO app could migrate fairly cleanly even to PHP 6
> > with unicode disabled (perhaps with minor
> > fixes). get_magic_quotes_gpc() is designed for apps to do something
> > differently depending on the value of magic_quotes_gpc. It's fine
> > that it's always off in PHP 6, but there's no reason not to keep this
> > function (to always return false) so that you don't have to fix God
> > knows how many lines of code to remove it.
>
> Fair enough, as I said in my last post in the magic_quotes thread, it
> is easy to keep this function. I will modify the patch to keep the
> related functions (returning always false). Is is ok?
The patch is now updated (same link). The functions are restored but
deprecated using the new ZEND_DEP_* macros.
set_magic_quotes_runtime raise a E_CORE_ERROR as described in the PDM.
The other always return false.
No more objection?
--Pierre