]> granicus.if.org Git - php/commitdiff
Cleanup of remaining E_STRICT in tests
authorGeorge Peter Banyard <girgias@php.net>
Sun, 14 Jul 2019 01:45:31 +0000 (03:45 +0200)
committerNikita Popov <nikita.ppv@gmail.com>
Tue, 23 Jul 2019 09:27:23 +0000 (11:27 +0200)
28 files changed:
Zend/tests/bug33771.phpt
Zend/tests/bug52160.phpt
Zend/tests/bug62956.phpt
Zend/tests/bug64988.phpt
Zend/tests/bug67436/bug67436_nohandler.phpt
Zend/tests/error_reporting02.phpt
Zend/tests/error_reporting03.phpt
Zend/tests/error_reporting04.phpt
Zend/tests/error_reporting08.phpt
Zend/tests/error_reporting09.phpt
Zend/tests/exception_004.phpt
ext/pdo/pdo_stmt.c
ext/pdo_mysql/tests/pdo_mysql_subclass.phpt
ext/pdo_pgsql/tests/bug72294.phpt
ext/phar/util.c
ext/soap/tests/bugs/bug46427.phpt
ext/standard/tests/general_functions/parse_ini_booleans.data
ext/standard/tests/mail/mail_basic_alt1-win32.phpt
ext/standard/tests/mail/mail_basic_alt2-win32.phpt
ext/standard/tests/mail/mail_basic_alt3-win32.phpt
ext/standard/tests/mail/mail_basic_alt4-win32.phpt
ext/standard/tests/mail/mail_variation_alt1-win32.phpt
ext/standard/tests/mail/mail_variation_alt2-win32.phpt
ext/standard/tests/mail/mail_variation_alt3-win32.phpt
ext/zip/examples/get_set_comments.php
run-tests.php
tests/func/011.phpt
tests/func/bug64523.phpt

index 0f3dd9815f64c17d95d74d86a361cdfd045768a7..6eea49edbe64060eab8e2ce3f3f2a226c37b9e6d 100644 (file)
@@ -3,7 +3,7 @@ Bug #33771 (error_reporting falls to 0 when @ was used inside try/catch block)
 --FILE--
 <?php
 
-error_reporting(E_ALL | E_STRICT);
+error_reporting(E_ALL);
 
 var_dump(error_reporting());
 
@@ -14,7 +14,7 @@ function make_exception()
 
 function make_exception_and_change_err_reporting()
 {
-    error_reporting(E_ALL & ~E_STRICT);
+    error_reporting(E_ALL & ~E_NOTICE);
     throw new Exception();
 }
 
@@ -36,5 +36,5 @@ echo "Done\n";
 --EXPECT--
 int(32767)
 int(32767)
-int(30719)
+int(32759)
 Done
index a0e5a9d7ae23b1954d2a980250bec273f26bc7e5..f0a57000e09da13aa6be99f176647c9a50d742c9 100644 (file)
@@ -1,5 +1,5 @@
 --TEST--
-Bug #52160 (Invalid E_STRICT redefined constructor error)
+Bug #52160 (Invalid E_DEPRECATED redefined constructor error)
 --FILE--
 <?php
 
index c8694d5beaa5d1f098e370390db22114b948977c..fab180b4c3e3c0084578dc462a09afb758b70f55 100644 (file)
@@ -1,5 +1,5 @@
 --TEST--
-Bug #62956: "incompatible" signatures for private methods should not cause E_STRICT
+Bug #62956: "incompatible" signatures for private methods should not cause E_WARNING
 --FILE--
 <?php
 class Base
index 671d69de3b1e18a2d540369e48027e71b1ed4dfd..41d89cc54cb603cb0dafa63dc861087359149184 100644 (file)
@@ -1,5 +1,5 @@
 --TEST--
-Bug #64988 (Class loading order affects E_STRICT warning)
+Bug #64988 (Class loading order affects E_WARNING warning)
 --FILE--
 <?php
 abstract class Base1 {
index 74ea394f5d82363481cc4a74f3cff0bf6b45dcf7..a155ccc8428db889af13b8d7852ea04b653fee58 100644 (file)
@@ -1,5 +1,5 @@
 --TEST--
-bug67436: E_STRICT instead of custom error handler
+bug67436: E_WARNING instead of custom error handler
 --FILE--
 <?php
 
index 6ceb8deaa37abc99ffb7ac4aa96376ae18ea91d2..c76c5b54c8ccec07019de5a2d1df4a7d559ccc6f 100644 (file)
@@ -3,13 +3,13 @@ testing @ and error_reporting - 2
 --FILE--
 <?php
 
-error_reporting(E_ALL);
+error_reporting(E_ALL & ~E_DEPRECATED);
 
 function foo($arg) {
 }
 
 function bar() {
-       error_reporting(E_ALL|E_STRICT);
+       error_reporting(E_ALL);
        throw new Exception("test");
 }
 
index 959a29b557f996e5c115fe7305506883109675bf..f344e9557b06386165e8bbfc84682db0f81841c3 100644 (file)
@@ -3,7 +3,7 @@ testing @ and error_reporting - 3
 --FILE--
 <?php
 
-error_reporting(E_ALL);
+error_reporting(E_ALL & ~E_DEPRECATED);
 
 function foo($arg) {
        echo @$nonex_foo;
@@ -16,7 +16,7 @@ function bar() {
 
 function foo1() {
        echo $undef1;
-       error_reporting(E_ALL|E_STRICT);
+       error_reporting(E_ALL);
        echo $undef2;
 }
 
index 295a3a4d406259eda193d806c3f783ec0db26229..5b476cdf7f54393cd0773401dd261b2d10532734 100644 (file)
@@ -3,11 +3,11 @@ testing @ and error_reporting - 4
 --FILE--
 <?php
 
-error_reporting(E_ALL);
+error_reporting(E_ALL & ~E_DEPRECATED);
 
 function foo() {
        echo $undef;
-       error_reporting(E_ALL|E_STRICT);
+       error_reporting(E_ALL);
 }
 
 
index 13d32ca4bc8b841750b612a912784ce9399ee2cb..32cb6b8d16cf97781176be849cf2c07b047ca71b 100644 (file)
@@ -3,7 +3,7 @@ testing @ and error_reporting - 8
 --FILE--
 <?php
 
-error_reporting(E_ALL);
+error_reporting(E_ALL & ~E_DEPRECATED);
 
 function foo1($arg) {
 }
@@ -12,7 +12,7 @@ function foo2($arg) {
 }
 
 function foo3() {
-       error_reporting(E_ALL|E_STRICT);
+       error_reporting(E_ALL);
        echo $undef3;
        throw new Exception("test");
 }
index a3c254d94d6bfd53b5673ad5a5a72f06c535a087..3d723f36c06dc5752cb7a8dd09b1970b77591133 100644 (file)
@@ -3,7 +3,7 @@ testing @ and error_reporting - 9
 --FILE--
 <?php
 
-error_reporting(E_ALL);
+error_reporting(E_ALL & ~E_DEPRECATED);
 
 function bar() {
        echo @$blah;
@@ -12,7 +12,7 @@ function bar() {
 
 function foo() {
        echo @$undef;
-       error_reporting(E_ALL|E_STRICT);
+       error_reporting(E_ALL);
        echo $blah;
        return bar();
 }
index 29288e25acbb50eefd664d5a6bc13b335b350666..d9ffc748a4f210608b0a8f5bc53eec806b215cdd 100644 (file)
@@ -3,8 +3,6 @@ Throwing exception using a class that isn't derived from the Exception base clas
 --FILE--
 <?php
 
-error_reporting(E_ALL|E_STRICT);
-
 class Foo { }
 
 try {
index 1fc114ccf9f8e9abea66052d8f774fb9c6768c46..52d38ca1944a945cf47f01530c0821a56d95d8ec 100644 (file)
@@ -764,7 +764,7 @@ static int make_callable_ex(pdo_stmt_t *stmt, zval *callable, zend_fcall_info *
                return 0;
        }
        if (is_callable_error) {
-               /* Possible E_STRICT error message */
+               /* Possible error message */
                efree(is_callable_error);
        }
 
index 915df7d69f7101b72e3d3b2756ba6e0061b261f4..604a13bdb38cd5ebeaea1e14cc4d714ad25578e5 100644 (file)
@@ -10,10 +10,6 @@ MySQLPDOTest::skip();
 <?php
        require_once(__DIR__ . DIRECTORY_SEPARATOR . 'mysql_pdo_test.inc');
 
-       // No silly strict mode warnings, please!
-       error_reporting(E_ALL^E_STRICT);
-       ini_set('display_errors', false);
-
        try {
 
                class MyPDO extends PDO {
@@ -23,9 +19,9 @@ MySQLPDOTest::skip();
                                return call_user_func_array(array($this, 'parent::__construct'), func_get_args());
                        }
 
-                       public function exec() {
+                       public function exec($statement) {
                                $this->protocol();
-                               return call_user_func_array(array($this, 'parent::exec'), func_get_args());
+                               return parent::exec($statement);
                        }
 
                        public function query() {
index 9ef4a424bb478325f706f32c9330e4baf6c0330b..bd5124ff17d887aa9f62a26fff10cd3e03712d25 100644 (file)
@@ -74,7 +74,7 @@ class PHPUnit_Framework_TestResult
 
         $oldErrorHandler = set_error_handler(
             'handleError',
-            E_ALL | E_STRICT
+            E_ALL
         );
 
         try {
index 59a6cd7c88abab990811265b357f86c5b71c23f2..53982b0f85206fa6e91fd3b425a0cf81c9eebda4 100644 (file)
@@ -934,7 +934,7 @@ int phar_free_alias(phar_archive_data *phar, char *alias, size_t alias_len) /* {
                return FAILURE;
        }
 
-       /* this archive has no open references, so emit an E_STRICT and remove it */
+       /* this archive has no open references, so emit a notice and remove it */
        if (zend_hash_str_del(&(PHAR_G(phar_fname_map)), phar->fname, phar->fname_len) != SUCCESS) {
                return FAILURE;
        }
index a49071e21ad792d8e35d0c8fc45f65a9cdd55d28..38d53518b8dc0a24870b50e3d35b742699ae323b 100644 (file)
@@ -4,7 +4,6 @@ Bug #46427 (SoapClient() stumbles over its "stream_context" parameter)
 <?php require_once('skipif.inc'); ?>
 --FILE--
 <?php
-error_reporting(E_ALL|E_STRICT);
 
 function getSoapClient_1() {
     $ctx = stream_context_create();
index d408577ea4f94ee62dafa5b86035827e8e56d07f..f29e32a92ec826ad9cdf8187f277f1596ebd98c9 100644 (file)
@@ -3,8 +3,8 @@ foo = E_ALL E_NOTICE
 error_reporting = E_ALL
 error_reporting1 = E_COMPILE_ERROR|E_RECOVERABLE_ERROR |E_ERROR|E_CORE_ERROR   
 error_reporting2 = E_ALL&~E_NOTICE                     
-error_reporting3 =  E_ALL & ~E_NOTICE   
-error_reporting4 = E_ALL & ~E_NOTICE | E_STRICT 
+error_reporting3 =  E_ALL & ~E_NOTICE
+error_reporting4 = E_ALL & ~E_NOTICE | E_PARSE
 
 ['true or false']
 bool_true =    true 
index f615972a84730610805ae46787bcc46b0a4dda16..9f06e0985424bd2a6f0add2ab6628c9b8b864a31 100644 (file)
@@ -19,7 +19,6 @@ max_execution_time = 120
  * Alias to functions:
  */
 
-error_reporting(E_ALL & ~E_STRICT);
 ini_set("SMTP", "localhost");
 ini_set("smtp_port", 25);
 ini_set("sendmail_from", "user@example.com");
index 5efe4504fec5af83a1f4ad4b32de37a489f8d640..498554f9472f9704133b361fae828a783f8ffa7d 100644 (file)
@@ -18,8 +18,6 @@ max_execution_time = 120
  * Alias to functions:
  */
 
-error_reporting(E_ALL & ~E_STRICT);
-
 echo "*** Testing mail() : basic functionality ***\n";
 require_once(__DIR__.'/mail_include.inc');
 $subject_prefix = "!**PHPT**!";
index 955c47451ff1db85cfe754ede2360573157f1070..172474b1d0e15522446df7bea42edd6900e9f4e5 100644 (file)
@@ -18,8 +18,6 @@ max_execution_time = 120
  * Alias to functions:
  */
 
-error_reporting(E_ALL & ~E_STRICT);
-
 echo "*** Testing mail() : basic functionality ***\n";
 require_once(__DIR__.'/mail_include.inc');
 $subject_prefix = "!**PHPT**!";
index 4cf33670736f107047398be95e8e0bb089799f2f..9e7cbc5802084ea8f9eb6412b2b511d89a56ad3a 100644 (file)
@@ -18,8 +18,6 @@ max_execution_time = 120
  * Alias to functions:
  */
 
-error_reporting(E_ALL & ~E_STRICT);
-
 echo "*** Testing mail() : basic functionality ***\n";
 require_once(__DIR__.'/mail_include.inc');
 $subject_prefix = "!**PHPT**!";
index 598dea9c3bb5a60215c0f60f47ef6e278ca3f166..bd259667c262016dbbbb2b75bd41da430e9474fb 100644 (file)
@@ -18,7 +18,6 @@ max_execution_time = 120
  * Alias to functions:
  */
 
-error_reporting(E_ALL & ~E_STRICT);
 ini_set("SMTP", "localhost");
 ini_set("smtp_port", 2525);
 ini_set("sendmail_from", "user@example.com");
index 78df2c7ddf1878530192ebe7943a8d5434d0102c..c0698076e1c5163bf9518d824147d2abbe9b57d7 100644 (file)
@@ -18,7 +18,6 @@ max_execution_time = 120
  * Alias to functions:
  */
 
-error_reporting(E_ALL & ~E_STRICT);
 ini_set("SMTP", "localplace");
 ini_set("smtp_port", 25);
 ini_set("sendmail_from", "user@example.com");
index afae8e8729fa8e38080a4a58886e8338c4d41087..fe961d3d166009afe15608db752a165e86d0e57b 100644 (file)
@@ -18,7 +18,6 @@ max_execution_time = 120
  * Alias to functions:
  */
 
-error_reporting(E_ALL & ~E_STRICT);
 ini_set("SMTP", "localhost");
 ini_set("smtp_port", 25);
 
index bec4240246de5dea632860de4121bd3888cdd86d..600b51138d684b4174a8a933c0ab32d5e09d7929 100644 (file)
@@ -1,5 +1,4 @@
 <?php
-error_reporting(E_ALL|E_STRICT);
 
 copy('test_with_comment.zip', 't.zip');
 $z = new ZipArchive;
index 2c11961a053eb34c80a7143737b83415b5d53329..7f4fce37e2368d6fadecf92df5b4b4a4fdc1cd59 100755 (executable)
@@ -238,7 +238,7 @@ NO_PROC_OPEN_ERROR;
                'open_basedir=',
                'disable_functions=',
                'output_buffering=Off',
-               'error_reporting=' . (E_ALL | E_STRICT),
+               'error_reporting=' . E_ALL,
                'display_errors=1',
                'display_startup_errors=1',
                'log_errors=0',
@@ -1624,7 +1624,7 @@ escape:
                                                                'E_USER_ERROR',
                                                                'E_USER_WARNING',
                                                                'E_USER_NOTICE',
-                                                               'E_STRICT',
+                                                               'E_STRICT', // TODO Cleanup when removed from Zend Engine.
                                                                'E_RECOVERABLE_ERROR',
                                                                'E_USER_DEPRECATED'
                                                        ];
index ec93214a785aea844f9f97c5e483d949bafa1d97..bc698c7743a4d4bad47d7a8b72c5a264a524aa5c 100644 (file)
@@ -1,10 +1,10 @@
 --TEST--
 Test bitwise AND, OR, XOR, NOT and logical NOT in INI via error_reporting
 --INI--
-error_reporting = E_ALL & E_NOTICE | E_STRICT ^ E_DEPRECATED & ~E_WARNING | !E_ERROR
+error_reporting = E_ALL & E_NOTICE | E_PARSE ^ E_DEPRECATED & ~E_WARNING | !E_ERROR
 --FILE--
 <?php
 echo ini_get('error_reporting');
 ?>
 --EXPECT--
-10248
+8204
index 88adc439596792508137c1a65ac2fcf553a2eb28..d360f097ff449e0a16e2b83dd0c4253702dc0baa 100644 (file)
@@ -1,10 +1,10 @@
 --TEST--
 Bug #64523: XOR not parsed in INI
 --INI--
-error_reporting = E_ALL ^ E_NOTICE ^ E_STRICT ^ E_DEPRECATED
+error_reporting = E_ALL ^ E_NOTICE ^ E_WARNING ^ E_DEPRECATED
 --FILE--
 <?php
 echo ini_get('error_reporting');
 ?>
 --EXPECT--
-22519
+24565