handle ctor only

Author: TomasVotruba

rules-tests/DeadCode/Rector/ClassMethod/RemoveEmptyClassMethodRector/Fixture/non_constructor_attribute.php.inc

@@ -0,0 +1,27 @@
+<?php
+
+namespace Rector\Tests\DeadCode\Rector\ClassMethod\RemoveEmptyClassMethodRector\Fixture;
+
+#[\Attribute()]
+final class NonConstructorAttribute
+{
+    /**
+     * @param class-string $className
+     */
+    public function run(string $className)
+    {
+    }
+}
+
+?>
+-----
+<?php
+
+namespace Rector\Tests\DeadCode\Rector\ClassMethod\RemoveEmptyClassMethodRector\Fixture;
+
+#[\Attribute()]
+final class NonConstructorAttribute
+{
+}
+
+?>

rules/DeadCode/Rector/ClassMethod/RemoveEmptyClassMethodRector.php

@@ -171,8 +171,7 @@ private function shouldSkipClassMethod(Class_ $class, ClassMethod $classMethod):
             return false;
         }
 
-        // skip constructor in attributes as might be a marker parameter
-        return $classReflection->isAttributeClass() && $classMethod->getDocComment() instanceof Doc;
+        return $this->isAttributeMarkerConstructor($classMethod, $classReflection);
     }
 
     private function hasDeprecatedAnnotation(ClassMethod $classMethod): bool
@@ -184,4 +183,20 @@ private function hasDeprecatedAnnotation(ClassMethod $classMethod): bool
 
         return $phpDocInfo->hasByType(DeprecatedTagValueNode::class);
     }
+
+    /**
+     * Skip constructor in attributes as might be a marker parameter
+     */
+    private function isAttributeMarkerConstructor(ClassMethod $classMethod, ClassReflection $classReflection): bool
+    {
+        if (! $this->isName($classMethod, MethodName::CONSTRUCT)) {
+            return false;
+        }
+
+        if (! $classReflection->isAttributeClass()) {
+            return false;
+        }
+
+        return $classMethod->getDocComment() instanceof Doc;
+    }
 }