Fix deprecations related to implicit nullable types

FriendsOfPHP · Mar 19, 2024 · a081c80 · a081c80
1 parent ecadbdc
commit a081c80
Show file tree

Hide file tree

Showing 6 changed files with 7 additions and 7 deletions.
diff --git a/...xyManager/ProxyGenerator/AccessInterceptor/MethodGenerator/SetMethodPrefixInterceptor.php b/...xyManager/ProxyGenerator/AccessInterceptor/MethodGenerator/SetMethodPrefixInterceptor.php
@@ -27,7 +27,7 @@ public function __construct(PropertyGenerator $prefixInterceptor)
 
         $interceptor = new ParameterGenerator('prefixInterceptor');
 
-        $interceptor->setType(Closure::class);
+        $interceptor->setType('?' . Closure::class);
         $interceptor->setDefaultValue(null);
         $this->setParameter(new ParameterGenerator('methodName', 'string'));
         $this->setParameter($interceptor);

diff --git a/...xyManager/ProxyGenerator/AccessInterceptor/MethodGenerator/SetMethodSuffixInterceptor.php b/...xyManager/ProxyGenerator/AccessInterceptor/MethodGenerator/SetMethodSuffixInterceptor.php
@@ -27,7 +27,7 @@ public function __construct(PropertyGenerator $suffixInterceptor)
 
         $interceptor = new ParameterGenerator('suffixInterceptor');
 
-        $interceptor->setType(Closure::class);
+        $interceptor->setType('?' . Closure::class);
         $interceptor->setDefaultValue(null);
         $this->setParameter(new ParameterGenerator('methodName', 'string'));
         $this->setParameter($interceptor);

diff --git a/src/ProxyManager/ProxyGenerator/LazyLoadingGhost/MethodGenerator/SetProxyInitializer.php b/src/ProxyManager/ProxyGenerator/LazyLoadingGhost/MethodGenerator/SetProxyInitializer.php
@@ -22,7 +22,7 @@ public function __construct(PropertyGenerator $initializerProperty)
     {
         parent::__construct(
             'setProxyInitializer',
-            [(new ParameterGenerator('initializer', Closure::class))->setDefaultValue(null)],
+            [(new ParameterGenerator('initializer', '?' . Closure::class))->setDefaultValue(null)],
             self::FLAG_PUBLIC,
             '$this->' . $initializerProperty->getName() . ' = $initializer;'
         );

diff --git a/...roxyManager/ProxyGenerator/LazyLoadingValueHolder/MethodGenerator/SetProxyInitializer.php b/...roxyManager/ProxyGenerator/LazyLoadingValueHolder/MethodGenerator/SetProxyInitializer.php
@@ -27,7 +27,7 @@ public function __construct(PropertyGenerator $initializerProperty)
 
         $initializerParameter = new ParameterGenerator('initializer');
 
-        $initializerParameter->setType(Closure::class);
+        $initializerParameter->setType('?' . Closure::class);
         $initializerParameter->setDefaultValue(null);
         $this->setParameter($initializerParameter);
         $this->setBody('$this->' . $initializerProperty->getName() . ' = $initializer;');

diff --git a/tests/ProxyManagerTestAsset/AccessInterceptorValueHolderMock.php b/tests/ProxyManagerTestAsset/AccessInterceptorValueHolderMock.php
@@ -45,12 +45,12 @@ public static function staticProxyConstructor($instance, $prefixInterceptors, $s
         return $selfInstance;
     }
 
-    public function setMethodPrefixInterceptor(string $methodName, \Closure $prefixInterceptor = null) : void
+    public function setMethodPrefixInterceptor(string $methodName, ?\Closure $prefixInterceptor = null) : void
     {
         // no-op (on purpose)
     }
 
-    public function setMethodSuffixInterceptor(string $methodName, \Closure $suffixInterceptor = null) : void
+    public function setMethodSuffixInterceptor(string $methodName, ?\Closure $suffixInterceptor = null) : void
     {
         // no-op (on purpose)
     }

diff --git a/tests/ProxyManagerTestAsset/LazyLoadingMock.php b/tests/ProxyManagerTestAsset/LazyLoadingMock.php
@@ -35,7 +35,7 @@ public static function staticProxyConstructor($initializer) : self
         return $instance;
     }
 
-    public function setProxyInitializer(\Closure $initializer = null) : void
+    public function setProxyInitializer(?\Closure $initializer = null) : void
     {
         $this->initializer = $initializer;
     }