Use isVoid() (#3574)

rectorphp · Apr 8, 2023 · 7611dcb · 7611dcb
1 parent aa73118
commit 7611dcb
Show file tree

Hide file tree

Showing 7 changed files with 8 additions and 8 deletions.
diff --git a/packages/PHPStanStaticTypeMapper/TypeMapper/UnionTypeMapper.php b/packages/PHPStanStaticTypeMapper/TypeMapper/UnionTypeMapper.php
@@ -190,7 +190,7 @@ private function resolveNullableType(NullableType $nullableType): null|NullableT
     private function mapNullabledType(Type $nullabledType, string $typeKind): ?Node
     {
         // void cannot be nullable
-        if ($nullabledType instanceof VoidType) {
+        if ($nullabledType->isVoid()->yes()) {
             return null;
         }
 

diff --git a/packages/VendorLocker/NodeVendorLocker/ClassMethodReturnTypeOverrideGuard.php b/packages/VendorLocker/NodeVendorLocker/ClassMethodReturnTypeOverrideGuard.php
@@ -176,7 +176,7 @@ private function shouldSkipHasChildHasReturnType(array $childrenClassReflections
             }
 
             $childReturnType = $this->returnTypeInferer->inferFunctionLike($method);
-            if ($returnType instanceof VoidType && ! $childReturnType instanceof VoidType) {
+            if ($returnType->isVoid()->yes() && ! $childReturnType->isVoid()->yes()) {
                 return true;
             }
         }

diff --git a/rules/Php72/NodeFactory/AnonymousFunctionFactory.php b/rules/Php72/NodeFactory/AnonymousFunctionFactory.php
@@ -401,7 +401,7 @@ private function resolveStmts(
         FunctionVariantWithPhpDocs $functionVariantWithPhpDocs,
         StaticCall|MethodCall $innerMethodCall
     ): array {
-        if ($functionVariantWithPhpDocs->getReturnType() instanceof VoidType) {
+        if ($functionVariantWithPhpDocs->getReturnType()->isVoid()->yes()) {
             return [new Expression($innerMethodCall)];
         }
 

diff --git a/rules/TypeDeclaration/Rector/ArrowFunction/AddArrowFunctionReturnTypeRector.php b/rules/TypeDeclaration/Rector/ArrowFunction/AddArrowFunctionReturnTypeRector.php
@@ -53,7 +53,7 @@ public function refactor(Node $node): ?Node
 
         $type = $this->getType($node->expr);
 
-        if ($type instanceof VoidType) {
+        if ($type->isVoid()->yes()) {
             return null;
         }
 

diff --git a/rules/TypeDeclaration/Rector/ClassMethod/ReturnTypeFromStrictTypedCallRector.php b/rules/TypeDeclaration/Rector/ClassMethod/ReturnTypeFromStrictTypedCallRector.php
@@ -150,7 +150,7 @@ private function processArrowFunction(ArrowFunction $arrowFunction): ?ArrowFunct
         $resolvedType = $this->nodeTypeResolver->getType($arrowFunction->expr);
 
         // void type is not accepted for arrow functions - https://www.php.net/manual/en/functions.arrow.php#125673
-        if ($resolvedType instanceof VoidType) {
+        if ($resolvedType->isVoid()->yes()) {
             return null;
         }
 
@@ -169,7 +169,7 @@ private function isUnionPossibleReturnsVoid(ClassMethod | Function_ | Closure $n
         $inferReturnType = $this->returnTypeInferer->inferFunctionLike($node);
         if ($inferReturnType instanceof UnionType) {
             foreach ($inferReturnType->getTypes() as $type) {
-                if ($type instanceof VoidType) {
+                if ($type->isVoid()->yes()) {
                     return true;
                 }
             }

diff --git a/rules/TypeDeclaration/TypeInferer/ReturnTypeInferer.php b/rules/TypeDeclaration/TypeInferer/ReturnTypeInferer.php
@@ -122,7 +122,7 @@ private function resolveTypeWithVoidHandling(
         ClassMethod|Function_|Closure|ArrowFunction $functionLike,
         Type $resolvedType
     ): Type {
-        if ($resolvedType instanceof VoidType) {
+        if ($resolvedType->isVoid()->yes()) {
             if ($functionLike instanceof ArrowFunction) {
                 return new MixedType();
             }

diff --git a/...peDeclaration/TypeInferer/ReturnTypeInferer/ReturnedNodesReturnTypeInfererTypeInferer.php b/...peDeclaration/TypeInferer/ReturnTypeInferer/ReturnedNodesReturnTypeInfererTypeInferer.php
@@ -170,7 +170,7 @@ private function correctWithNestedType(Type $resolvedType, Return_ $return, Func
             $correctedType = $this->inferFromReturnedMethodCall($return, $functionLike);
 
             // override only if has some extra value
-            if (! $correctedType instanceof MixedType && ! $correctedType instanceof VoidType) {
+            if (! $correctedType instanceof MixedType && ! $correctedType->isVoid()->yes()) {
                 return $correctedType;
             }
         }