fix: PHP 8.4 deprecation notice - implicit nullable parameters

michalbundyra · web-flow · commit 1e18e1458125 · 2025-09-22T22:02:09.000+01:00
Fixes #83 Drop `Closure` typehint on MockDisabler constructor (compatibility class for PHPUnit 7-9 only) See also: php-mock/php-mock/pull/61
diff --git a/classes/MockDisablerPHPUnit7.php b/classes/MockDisablerPHPUnit7.php
@@ -35,7 +35,7 @@ class MockDisablerPHPUnit7 extends BaseTestListener
      * @param Deactivatable $deactivatable The function mocks.
      * @param Closure|null $callback       The callback to execute after the test.
      */
-    public function __construct(Deactivatable $deactivatable, Closure $callback = null)
+    public function __construct(Deactivatable $deactivatable, $callback = null)
     {
         $this->deactivatable = $deactivatable;
         $this->callback = $callback;

Original file line number	Diff line number	Diff line change
`@@ -35,7 +35,7 @@ class MockDisablerPHPUnit7 extends BaseTestListener`
`35`	`35`	`* @param Deactivatable $deactivatable The function mocks.`
`36`	`36`	`* @param Closure\|null $callback The callback to execute after the test.`
`37`	`37`	`*/`
`38`		`- public function __construct(Deactivatable $deactivatable, Closure $callback = null)`
	`38`	`+ public function __construct(Deactivatable $deactivatable, $callback = null)`
`39`	`39`	`{`
`40`	`40`	`$this->deactivatable = $deactivatable;`
`41`	`41`	`$this->callback = $callback;`