On Tue, Jul 19, 2011 at 8:53 AM, Reindl Harald <h.rei...@thelounge.net> wrote: > > > Am 19.07.2011 01:12, schrieb Ferenc Kovacs: >>> since this is default OFF there is no warning if not configured >>> empty php.ini implicits = Off >> >> sadly no: >> >> tyrael@thor:~/checkouts$ php -n -r 'echo PHP_VERSION."\n";echo >> ini_get("magic_quotes_gpc")."\n";' >> 5.3.6-6~dotdeb.1 >> 1 >> >> tyrael@chronos:~/checkouts/php/src/php/php-src/branches/PHP_5_4$ >> ./sapi/cli/php -n -r 'echo PHP_VERSION."\n";echo >> ini_get("magic_quotes_gpc")."\n";' >> 5.4.0alpha2-dev >> 1 > > i expect if someone is writing the code to deprecate a feature > he well also set it's default to 0 - what now is set does > not interest > >
magic_quotes_gpc is deprecated as of 5.3 http://php.net/manual/en/migration53.deprecated.php only the default values was left to 1 I also expected this to be changed to 0, but it seems it isn't a clear cut as Stas expressed in the 5.4 vote thread that the defaults shouldn't be used anyway. but my whole point for this was that "since this is default OFF there is no warning if not configured empty php.ini implicits = Off" isn't true. if you omit a configuration option from your php.ini not Off, but the default will be used, which is currently On for the magic_quotes_gpc, and it isn't 100% sure that we will change the default, of course I think we should set it to the same values as we use for the php.ini-production. sor I think that the original idea "Raise E_DEPRECATED when running the CGI or any web server SAPI and a php.ini file does not contain “magic_quotes_gpc = Off" would be a bad idea, I agree with Ángel: PHP should not fail if one configuration line is missing from the file. -- Ferenc Kovács @Tyr43l - http://tyrael.hu -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php