Skip to content

Commit 1b6635c

Browse files
committed
Fix
1 parent d72cb14 commit 1b6635c

3 files changed

Lines changed: 4 additions & 2 deletions

File tree

composer.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,8 @@
9090
"classmap": [
9191
"stubs",
9292
"rules-tests/CodingStyle/Rector/Namespace_/ImportFullyQualifiedNamesRector/Source",
93-
"rules-tests/Renaming/Rector/Name/RenameClassRector/Source"
93+
"rules-tests/Renaming/Rector/Name/RenameClassRector/Source",
94+
"rules-tests/TypeDeclaration/Rector/ClassMethod/KnownMagicClassMethodTypeRector/Source"
9495
],
9596
"files": [
9697
"tests/debug_functions.php",

rules/TypeDeclaration/Rector/ClassMethod/KnownMagicClassMethodTypeRector.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ public function refactor(Node $node): ?Node
8080
}
8181

8282
if ($this->parentClassMethodTypeOverrideGuard->hasParentClassMethod($classMethod)) {
83-
return null;
83+
continue;
8484
}
8585

8686
if ($this->isNames($classMethod, [MethodName::CALL, MethodName::CALL_STATIC])) {

src/VendorLocker/ParentClassMethodTypeOverrideGuard.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@ public function hasParentClassMethod(ClassMethod|MethodReflection $classMethod):
3434

3535
return $parentClassMethod instanceof MethodReflection;
3636
} catch (UnresolvableClassException) {
37+
dump('here');
3738
// we don't know all involved parents,
3839
// marking as parent exists which usually means the method is guarded against overrides.
3940
return true;

0 commit comments

Comments
 (0)