Re: [PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/ext/session/tests/rfc1867_invalid_settings.phpt trunk/ext/session/tests/rfc1867_invalid_settings.phpt
Created bug report after discussion on irc (just to make a note here): https://bugs.php.net/bug.php?id=55583 2011/9/4 Ferenc Kovacs tyr...@php.net: Hi, because I think that the tests is at fault here. I checked some other cases, where we validate INI variables(through PHP_INI_MH), and I think that the code is fine. a similar case which generates double warning: tyrael@sanctuary:~$ php -n -d error_reporting=-1 -d display_startup_errors=1 -d zlib.output_compression=1 -d output_handler=1 -r '' PHP Fatal error: PHP Startup: Cannot use both zlib.output_compression and output_handler together!! in Unknown on line 0 Fatal error: PHP Startup: Cannot use both zlib.output_compression and output_handler together!! in Unknown on line 0 this happens consistently with all current branches and even with 5.2, so I think that it is the intended behavior. so changing the test to expect this seemed the logical choice. On Sun, Sep 4, 2011 at 6:10 PM, Alexey Shein con...@gmail.com wrote: Hello, why did you fix the test this way? Why not to fix the code and remove excessive warning? 2011/9/4 Ferenc Kovacs tyr...@php.net: tyrael Sun, 04 Sep 2011 14:28:52 + Revision: http://svn.php.net/viewvc?view=revisionrevision=316109 Log: passing an invalid session.upload_progress.freq always generates an additional startup error Changed paths: U php/php-src/branches/PHP_5_4/ext/session/tests/rfc1867_invalid_settings.phpt U php/php-src/trunk/ext/session/tests/rfc1867_invalid_settings.phpt Modified: php/php-src/branches/PHP_5_4/ext/session/tests/rfc1867_invalid_settings.phpt === --- php/php-src/branches/PHP_5_4/ext/session/tests/rfc1867_invalid_settings.phpt 2011-09-04 14:21:27 UTC (rev 316108) +++ php/php-src/branches/PHP_5_4/ext/session/tests/rfc1867_invalid_settings.phpt 2011-09-04 14:28:52 UTC (rev 316109) @@ -9,5 +9,7 @@ var_dump(ini_get(session.upload_progress.freq)); ? --EXPECTF-- +PHP Warning: PHP Startup: session.upload_progress.freq must be greater than or equal to zero in %s + Warning: PHP Startup: session.upload_progress.freq must be greater than or equal to zero in %s string(%d) 1% Modified: php/php-src/trunk/ext/session/tests/rfc1867_invalid_settings.phpt === --- php/php-src/trunk/ext/session/tests/rfc1867_invalid_settings.phpt 2011-09-04 14:21:27 UTC (rev 316108) +++ php/php-src/trunk/ext/session/tests/rfc1867_invalid_settings.phpt 2011-09-04 14:28:52 UTC (rev 316109) @@ -9,5 +9,7 @@ var_dump(ini_get(session.upload_progress.freq)); ? --EXPECTF-- +PHP Warning: PHP Startup: session.upload_progress.freq must be greater than or equal to zero in %s + Warning: PHP Startup: session.upload_progress.freq must be greater than or equal to zero in %s string(%d) 1% -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php -- Regards, Shein Alexey -- Regards, Shein Alexey -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
[PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/ext/session/tests/rfc1867_invalid_settings.phpt trunk/ext/session/tests/rfc1867_invalid_settings.phpt
tyrael Sun, 04 Sep 2011 14:28:52 + Revision: http://svn.php.net/viewvc?view=revisionrevision=316109 Log: passing an invalid session.upload_progress.freq always generates an additional startup error Changed paths: U php/php-src/branches/PHP_5_4/ext/session/tests/rfc1867_invalid_settings.phpt U php/php-src/trunk/ext/session/tests/rfc1867_invalid_settings.phpt Modified: php/php-src/branches/PHP_5_4/ext/session/tests/rfc1867_invalid_settings.phpt === --- php/php-src/branches/PHP_5_4/ext/session/tests/rfc1867_invalid_settings.phpt 2011-09-04 14:21:27 UTC (rev 316108) +++ php/php-src/branches/PHP_5_4/ext/session/tests/rfc1867_invalid_settings.phpt 2011-09-04 14:28:52 UTC (rev 316109) @@ -9,5 +9,7 @@ var_dump(ini_get(session.upload_progress.freq)); ? --EXPECTF-- +PHP Warning: PHP Startup: session.upload_progress.freq must be greater than or equal to zero in %s + Warning: PHP Startup: session.upload_progress.freq must be greater than or equal to zero in %s string(%d) 1% Modified: php/php-src/trunk/ext/session/tests/rfc1867_invalid_settings.phpt === --- php/php-src/trunk/ext/session/tests/rfc1867_invalid_settings.phpt 2011-09-04 14:21:27 UTC (rev 316108) +++ php/php-src/trunk/ext/session/tests/rfc1867_invalid_settings.phpt 2011-09-04 14:28:52 UTC (rev 316109) @@ -9,5 +9,7 @@ var_dump(ini_get(session.upload_progress.freq)); ? --EXPECTF-- +PHP Warning: PHP Startup: session.upload_progress.freq must be greater than or equal to zero in %s + Warning: PHP Startup: session.upload_progress.freq must be greater than or equal to zero in %s string(%d) 1% -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
Re: [PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/ext/session/tests/rfc1867_invalid_settings.phpt trunk/ext/session/tests/rfc1867_invalid_settings.phpt
Hello, why did you fix the test this way? Why not to fix the code and remove excessive warning? 2011/9/4 Ferenc Kovacs tyr...@php.net: tyrael Sun, 04 Sep 2011 14:28:52 + Revision: http://svn.php.net/viewvc?view=revisionrevision=316109 Log: passing an invalid session.upload_progress.freq always generates an additional startup error Changed paths: U php/php-src/branches/PHP_5_4/ext/session/tests/rfc1867_invalid_settings.phpt U php/php-src/trunk/ext/session/tests/rfc1867_invalid_settings.phpt Modified: php/php-src/branches/PHP_5_4/ext/session/tests/rfc1867_invalid_settings.phpt === --- php/php-src/branches/PHP_5_4/ext/session/tests/rfc1867_invalid_settings.phpt 2011-09-04 14:21:27 UTC (rev 316108) +++ php/php-src/branches/PHP_5_4/ext/session/tests/rfc1867_invalid_settings.phpt 2011-09-04 14:28:52 UTC (rev 316109) @@ -9,5 +9,7 @@ var_dump(ini_get(session.upload_progress.freq)); ? --EXPECTF-- +PHP Warning: PHP Startup: session.upload_progress.freq must be greater than or equal to zero in %s + Warning: PHP Startup: session.upload_progress.freq must be greater than or equal to zero in %s string(%d) 1% Modified: php/php-src/trunk/ext/session/tests/rfc1867_invalid_settings.phpt === --- php/php-src/trunk/ext/session/tests/rfc1867_invalid_settings.phpt 2011-09-04 14:21:27 UTC (rev 316108) +++ php/php-src/trunk/ext/session/tests/rfc1867_invalid_settings.phpt 2011-09-04 14:28:52 UTC (rev 316109) @@ -9,5 +9,7 @@ var_dump(ini_get(session.upload_progress.freq)); ? --EXPECTF-- +PHP Warning: PHP Startup: session.upload_progress.freq must be greater than or equal to zero in %s + Warning: PHP Startup: session.upload_progress.freq must be greater than or equal to zero in %s string(%d) 1% -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php -- Regards, Shein Alexey -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php