Bug #52221 [Asn]: Misbehaviour of magic_quotes_runtime (get/set))

Edit report at http://bugs.php.net/bug.php?id=52221&edit=1 ID: 52221 Updated by: philip@php.net Reported by: m dot philipp at coreto dot de Summary: Misbehaviour of magic_quotes_runtime (get/set)) Status: Assigned Type: Bug Package: Documentation problem Operating System: Windows Server 2008 R2 PHP Version: 5.3.2 Assigned To: mysql Block user comment: N New Comment: I reopened it after reading "The documentation should note that" but personally I think MQ should work the same everywhere. Previous Comments: ------------------------------------------------------------------------ [2010-08-25 17:06:49] andrey@php.net Automatic comment from SVN on behalf of andrey Revision: http://svn.php.net/viewvc/?view=revision&revision=302776 Log: Fix for bug #52221 Misbehaviour of magic_quotes_runtime (get/set)) ------------------------------------------------------------------------ [2010-08-25 16:32:46] uw@php.net Getting quite unsure about the won't fix after discussion with Johannes. Johannes is correct in arguing that the silent removal of a deprecated feature could break apps unexpectedly, which is bad because magic quotes has some security impact. Maybe adding magic quotes (although they are deprecated) to mysqlnd and throwing a deprecation warning would be best? Phillip, thoughts? You re-opened it. Not sure if you noticed I had changed the category/package: Did you intend to re-open as docs or code issue? ------------------------------------------------------------------------ [2010-08-25 16:28:16] andrey@php.net Doc Problem, not for user mysql. ------------------------------------------------------------------------ [2010-08-25 16:27:44] andrey@php.net It should be documented that mysqlnd is a new development and doesn't implement magic quotes. PDO is a new development too, and doesn't implement magic quotes too. What can be done is adding a php_error_docref(E_WARNING) at MINIT/RINIT which will tell the developer that if mysqlnd is active and magic_quotes is set that it won't work with mysqli. Also a warning can be thrown at every mysqli call. ------------------------------------------------------------------------ [2010-08-25 15:55:47] uw@php.net mysqlnd does not support magic quotes any more. It is a deprecated feature and mysqlnd is new. The documentation should note that. ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at http://bugs.php.net/bug.php?id=52221 -- Edit this bug report at http://bugs.php.net/bug.php?id=52221&edit=1