Skip to content

Commit b267192

Browse files
[Runtime][ErrorHandler] Don't mess with ini_set('assert.warning')
1 parent c1b9be3 commit b267192

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

Debug.php

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,6 @@ public static function enable(): ErrorHandler
3131

3232
@ini_set('zend.assertions', 1);
3333
ini_set('assert.active', 1);
34-
ini_set('assert.warning', 0);
3534
ini_set('assert.exception', 1);
3635

3736
DebugClassLoader::enable();

Tests/ErrorHandlerTest.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -665,6 +665,7 @@ public function testAssertQuietEval()
665665
$this->markTestSkipped('zend.assertions is forcibly disabled');
666666
}
667667

668+
set_error_handler(function () {});
668669
$ini = [
669670
ini_set('zend.assertions', 1),
670671
ini_set('assert.active', 1),
@@ -673,6 +674,7 @@ public function testAssertQuietEval()
673674
ini_set('assert.callback', null),
674675
ini_set('assert.exception', 0),
675676
];
677+
restore_error_handler();
676678

677679
$logger = new BufferingLogger();
678680
$handler = new ErrorHandler($logger);

0 commit comments

Comments
 (0)