skip getter

Author: TomasVotruba

rules-tests/Unambiguous/Rector/Expression/FluentSettersToStandaloneCallMethodRector/Fixture/skip_getter_setter.php.inc

@@ -0,0 +1,18 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\Tests\Unambiguous\Rector\Expression\FluentSettersToStandaloneCallMethodRector\Fixture;
+
+use Rector\Tests\Unambiguous\Rector\Expression\FluentSettersToStandaloneCallMethodRector\Source\GetterSetterClass;
+
+final class SkipGetterSetter
+{
+    public function setup()
+    {
+        return (new GetterSetterClass())
+            ->getter()
+            ->setName('John')
+            ->setSurname('Doe');
+    }
+}

rules-tests/Unambiguous/Rector/Expression/FluentSettersToStandaloneCallMethodRector/Source/GetterSetterClass.php

@@ -0,0 +1,11 @@
+<?php
+
+namespace Rector\Tests\Unambiguous\Rector\Expression\FluentSettersToStandaloneCallMethodRector\Source;
+
+final class GetterSetterClass
+{
+    public function getter()
+    {
+        return new SomeSetterClass();
+    }
+}

rules/Unambiguous/Rector/Expression/FluentSettersToStandaloneCallMethodRector.php

@@ -98,6 +98,11 @@ public function refactor(Node $node): ?array
 
         $currentMethodCall = $firstMethodCall;
         while ($currentMethodCall instanceof MethodCall) {
+            // must be exactly one argument
+            if (count($currentMethodCall->getArgs()) !== 1) {
+                return null;
+            }
+
             $methodCalls[] = $currentMethodCall;
             $currentMethodCall = $currentMethodCall->var;
         }