Fix PHP 8.4 deprecation.

Author: kagg-design

composer.json

@@ -11,7 +11,7 @@
   "minimum-stability": "stable",
   "require": {
     "php": ">=7.1",
-    "phpunit/phpunit": "7.5 - 11.4",
+    "phpunit/phpunit": "7.5 - 9.6",
     "antecedent/patchwork": "^2.2.0",
     "lucatume/args": "^1.0.1"
   },

src/functions.php

@@ -14,7 +14,7 @@
  * @see \Patchwork\configure()
  *
  */
-function init(array $options = null) {
+function init(?array $options = null) {
     FunctionMocker::init($options);
 }
 

src/tad/FunctionMocker/Call/Logger/LoggerInterface.php

@@ -5,5 +5,5 @@
 interface LoggerInterface
 {
 
-    public function called(array $args = null);
+    public function called(?array $args = null);
 }

src/tad/FunctionMocker/Call/Logger/SpyCallLogger.php

@@ -22,12 +22,12 @@ public function __construct()
             : '\\PHPUnit\\Framework\\Constraint\\Constraint';
     }
 
-    public function called(array $args = null)
+    public function called(?array $args = null)
     {
         $this->calls[] = CallTrace::fromArguments($args);
     }
 
-    public function getCallTimes(array $args = null)
+    public function getCallTimes(?array $args = null)
     {
         $calls = $this->calls;
         if ($args) {

src/tad/FunctionMocker/Call/Verifier/InstanceMethodCallVerifier.php

@@ -43,7 +43,7 @@ private function realWasCalledTimes( $times ) {
 	 *
 	 * @return array
 	 */
-	protected function getCallTimesWithArgs( $methodName, array $args = null ) {
+	protected function getCallTimesWithArgs( $methodName, ?array $args = null ) {
 		$invocations = $this->getInvocations();
 		$callTimes   = 0;
 		array_map( function ( $invocation ) use ( &$callTimes, $args, $methodName ) {

src/tad/FunctionMocker/CallTrace.php

@@ -7,7 +7,7 @@ class CallTrace
 
     protected $args;
 
-    public static function fromArguments(array $args = null)
+    public static function fromArguments(?array $args = null)
     {
         $instance = new self;
         $instance->args = $args ? $args : array();

src/tad/FunctionMocker/FunctionMocker.php

@@ -54,7 +54,7 @@ public static function setUp() {
 	 *
 	 * @see \Patchwork\configure()
 	 */
-    public static function init(array $options = null, $forceReinit = false) {
+    public static function init(?array $options = null, $forceReinit = false) {
         if (!$forceReinit && self::$didInit) {
             return;
         }
@@ -300,7 +300,7 @@ private static function getIndexedReplacements($return) {
      *
      * @return mixed
      */
-    public static function callOriginal(array $args = null) {
+    public static function callOriginal(?array $args = null) {
         return \Patchwork\relay($args);
     }
 
@@ -367,7 +367,7 @@ public static function forge($class) {
 	 *
 	 * @throws \RuntimeException If the Patchwork configuration file or the checksum file could not be written.
 	 */
-	public static function writePatchworkConfig(array $options = null, $destinationFolder) {
+	public static function writePatchworkConfig(?array $options = null, $destinationFolder) {
 		$options = self::getPatchworkConfiguration($options, $destinationFolder);
 
 		$configFileContents = json_encode($options);

src/tad/FunctionMocker/Method/Verifier.php

@@ -27,7 +27,7 @@ public function __construct()
     {
     }
 
-    public function __call($name, array $args = null)
+    public function __call($name, ?array $args = null)
     {
         return InstanceMethodCallVerifier::from($this->returnValue, $this->callLogger);
     }

src/tad/FunctionMocker/MockWrapper.php

@@ -34,7 +34,7 @@ class MockWrapper
 	 */
 	private $methodCode;
 
-	public function __construct($originalClassName = '', ClassTemplateInterface $classTemplate = null, MethodCodeInterface $methodCode = null)
+	public function __construct($originalClassName = '', ?ClassTemplateInterface $classTemplate = null, ?MethodCodeInterface $methodCode = null)
     {
         $this->originalClassName = $originalClassName;
         $this->classTemplate = $classTemplate ?: new ClassTemplate();
@@ -71,7 +71,7 @@ public function wrap($mockObject, $invokedRecorder, ReplacementRequest $request)
 	 * @throws \Exception
 	 *
 	 */
-    protected function getWrappedInstance($object, ExtenderInterface $extender, $invokedRecorder = null, ReplacementRequest $request = null)
+    protected function getWrappedInstance($object, ExtenderInterface $extender, $invokedRecorder = null, ?ReplacementRequest $request = null)
     {
         $mockClassName = get_class($object);
         $extendClassName = sprintf('%s_%s', uniqid('Extended_'), $mockClassName);

src/tad/FunctionMocker/PHPUnitFrameworkAssertWrapper.php

@@ -32,7 +32,7 @@ public static function getTestCase()
      * @param array $args
      * @return mixed
      */
-    public static function __callStatic($name, array $args = null)
+    public static function __callStatic($name, ?array $args = null)
     {
         return call_user_func_array([self::getTestCase(), $name], $args);
     }