[Nette] Skip constructor on magic template call

rules/nette/src/Rector/ClassMethod/TemplateMagicAssignToExplicitVariableArrayRector.php

@@ -6,12 +6,15 @@
 
 use PhpParser\Node;
 use PhpParser\Node\Expr\MethodCall;
+use PhpParser\Node\Stmt\Class_;
 use PhpParser\Node\Stmt\ClassMethod;
 use PhpParser\Node\Stmt\Expression;
 use Rector\Core\Rector\AbstractRector;
+use Rector\Core\ValueObject\MethodName;
 use Rector\Nette\NodeFactory\ActionRenderFactory;
 use Rector\Nette\TemplatePropertyAssignCollector;
 use Rector\Nette\ValueObject\MagicTemplatePropertyCalls;
+use Rector\NodeTypeResolver\Node\AttributeKey;
 use Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample;
 use Symplify\RuleDocGenerator\ValueObject\RuleDefinition;
 
@@ -86,11 +89,7 @@ public function getNodeTypes(): array
      */
     public function refactor(Node $node): ?Node
     {
-        if (! $this->isObjectType($node, 'Nette\Application\UI\Control')) {
-            return null;
-        }
-
-        if (! $node->isPublic()) {
+        if ($this->shouldSkip($node)) {
             return null;
         }
 
@@ -116,4 +115,22 @@ private function createRenderMethodCall(
 
         return $this->actionRenderFactory->createThisRenderMethodCall($magicTemplatePropertyCalls);
     }
+
+    private function shouldSkip(ClassMethod $classMethod): bool
+    {
+        $classLike = $classMethod->getAttribute(AttributeKey::CLASS_NODE);
+        if (! $classLike instanceof Class_) {
+            return true;
+        }
+
+        if (! $this->isObjectType($classLike, 'Nette\Application\UI\Control')) {
+            return true;
+        }
+
+        if ($this->isName($classMethod, MethodName::CONSTRUCT)) {
+            return true;
+        }
+
+        return ! $classMethod->isPublic();
+    }
 }

rules/nette/tests/Rector/ClassMethod/TemplateMagicAssignToExplicitVariableArrayRector/Fixture/skip_constructor.php.inc

@@ -0,0 +1,13 @@
+<?php
+
+namespace Rector\Nette\Tests\Rector\ClassMethod\TemplateMagicAssignToExplicitVariableArrayRector\Fixture;
+
+use Nette\Application\UI\Presenter;
+
+class SkipConstructor extends Presenter
+{
+    public function __construct($logger)
+    {
+         $this->logger = $logger;
+    }
+}